Re: tlv32aic3x, rx-51, regulator

[Jarkko Nikula <jarkko.nikula@bitmer.com>]

Hi

On 05/01/12 12:02, Peter Meerwald wrote:
> Hello,
>
> the tlv320aic3x codec driver requests a number of supplies/regulators
> (IOVDD, DVDD, AVDD, DRVDD), these seem mandatory for the codec driver to
> operate and somehow tied to the rx51 platform
>
They are the supply voltages for the aix3x codecs and not tied to rx51 only.

> I want to use the tlv320aic3x codec driver with some other board where I
> don't care about supplies/regulators -- how is this going to work? do I
> need to have dummy supplies just to keep tlv320aic3x happy?
>
> shouldn't the ai3x_supply_names be something that is optional and passed
> in via platform_data?
>
Simplest option in case you don't have *any* controllable regulator or 
if the kernel is not touching them is to not enable CONFIG_REGULATOR.

IMHO optional platform data for CONFIG_REGULATOR=y case doesn't win much 
over defining fixed voltage regulators. Either it would be duplicate 
information or needs modification to a machine driver anyway. And the 
regulator framework can be useful in debugging too, like following the 
use_counts via sysfs and so on.

-- 
Jarkko