Re: [PATCH V5 1/2] regulator: Add document for MT6323 regulator

From: John Crispin
Date: Tue Feb 09 2016 - 07:45:38 EST




On 09/02/2016 13:41, Mark Brown wrote:
> On Sun, Feb 07, 2016 at 08:40:36PM +0100, John Crispin wrote:
>
>> +Required properties:
>> +- compatible: "mediatek,mt6323-regulator"
>
> I'm really not happy with MFD subfunctions like this which add no
> information over the parent device name appearing directly in the
> binding, it results in us putting Linux specifics about how we split the
> device up into the driver. It's not like this could realistically be
> used separately to the parent device. Just have the MFD create the
> device directly or if the individual regulators are reusable IPs which
> can share a driver then describe them as such in the DT.
>

Hi Mark,

How would you like it to be. i am failing to understand that from your
comment.

John