Re: [PATCH v4 1/2] dt-bindings: iio: adc: Add MAX1241 device tree bindings in documentation

[Jonathan Cameron <jic23@kernel.org>]

On Sun, 22 Mar 2020 18:06:04 +0200
Alexandru Lazar <alazar@startmail.com> wrote:

> > You'd be surprised how often this gets added to drivers precisely because
> > people will put it on a controllable supply.  It may well not have it's own
> > supply but it may share one with a bunch of other external chips and
> > all of them need to use the regulator framework controls to make sure it's
> > only disabled when they are all suspended etc.  
> 
> I figured it might be something like this :-). I've added the vdd-supply
> binding in v5.
> 
> If this isn't something that can be easily handled in the core, do you
> think we can document it somewhere as a convention/common idiom?
> (Assuming it's not already documented, of course). It seems like it's
> something that all IIO devices would need. I can do the writing part.

Hmm. We could do with a sort of 'things you'd normally find in a driver'
document.  We don't have such a document, but interesting to think about
what would be in it...  Perhaps a 'best practice' document would
be a better way of putting it.  I don't really want to see a huge
number of patches adding regulators to drivers that don't have them already
for example.  Clearly no one needed them yet :)

If you want to take a stab at such a document that would be great.

Jonathan


> 
> Thanks,
> Alex