Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1330204

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

Show all headers | View raw

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

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH V5 1/2] regulator: Add document for MT6323 regulator John Crispin <blogic@openwrt.org> - 2016-02-07 20:50 +0100
  Re: [PATCH V5 1/2] regulator: Add document for MT6323 regulator Rob Herring <robh@kernel.org> - 2016-02-08 22:30 +0100
  Re: [PATCH V5 1/2] regulator: Add document for MT6323 regulator John Crispin <blogic@openwrt.org> - 2016-02-09 13:50 +0100
    Re: [PATCH V5 1/2] regulator: Add document for MT6323 regulator Mark Brown <broonie@kernel.org> - 2016-02-09 16:20 +0100
  Re: [PATCH V5 1/2] regulator: Add document for MT6323 regulator Mark Brown <broonie@kernel.org> - 2016-02-09 13:50 +0100

csiph-web