Re: [PATCH v2 09/19] ASoC: tlv320aic31xx: Remove platform data

From: Andrew F. Davis
Date: Tue Dec 05 2017 - 16:20:42 EST


On 12/01/2017 07:26 AM, Mark Brown wrote:
> On Wed, Nov 29, 2017 at 03:32:50PM -0600, Andrew F. Davis wrote:
>> Platform data is not used by anyone (at least in upstream) so
>> drop this data and switch to using fwnode(DT/ACPI) only.
>
> The advantage being...? Not all architectures use DT or ACPI so it's
> not clear that this is a step forwards in itself.
>

Simplifies the code in several places, and you don't need to use DT or
ACPI, it probes just fine anyway you normally add an I2C device.

All we are dropping here is the platform_data way of specifying mic-bias
voltage, which if you are wanting to do that in an out-of-tree board
file, then I'm sure you can locally modify this driver to use your
wanted voltage setting by default.