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

Groups > linux.kernel > #1274643

Re: [PATCH v3 1/2] ASoC: da7218: Add bindings documentation for DA7218 audio codec

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

On Fri, Nov 20, 2015 at 09:53:51AM -0600, Rob Herring wrote:
> On Thu, Nov 19, 2015 at 01:59:34PM +0000, Adam Thomson wrote:

> > +- VDD-supply: VDD power supply for the device
> > +- VDDMIC-supply: VDDMIC power supply for the device
> > +- VDDIO-supply: VDDIO power supply for the device

> These should be lower case (that is documented in ePAPR BTW :)).

Does that actually make a meaningful difference to anything?  Things end
up this way because the strings we use to request the supplies end up in
the property names, we've also got a few other things like representing
microvolts as uV which just look very wrong when written all lowercase.
We could work around it for the supply names by force lowercasing the
name if we don't find the property with the passed name (to avoid
breaking existing users) though.

There's a bunch of stuff in ePAPR that we're ignoring, and a bunch of
other stuff that has no real value like the class naming stuff.

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

Thread

[PATCH v3 1/2] ASoC: da7218: Add bindings documentation for DA7218 audio  codec Adam Thomson <Adam.Thomson.Opensource@diasemi.com> - 2015-11-19 15:10 +0100
  Re: [PATCH v3 1/2] ASoC: da7218: Add bindings documentation for  DA7218 audio codec Rob Herring <robh@kernel.org> - 2015-11-20 17:00 +0100
    Re: [PATCH v3 1/2] ASoC: da7218: Add bindings documentation for  DA7218 audio codec Mark Brown <broonie@kernel.org> - 2015-11-21 14:00 +0100
    RE: [PATCH v3 1/2] ASoC: da7218: Add bindings documentation for  DA7218 audio codec "Opensource [Adam Thomson]" <Adam.Thomson.Opensource@diasemi.com> - 2015-11-23 13:00 +0100

csiph-web