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

Path	csiph.com!news.mixmin.net!weretis.net!feeder4.news.weretis.net!feeder1.news.weretis.net!news.mb-net.net!open-news-network.org!aioe.org!bofh.it!news.nic.it!robomod
From	Mark Brown <broonie@kernel.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH v3 1/2] ASoC: da7218: Add bindings documentation for DA7218 audio codec
Date	Sat, 21 Nov 2015 14:00:01 +0100
Message-ID	<qxfVL-5nt-5@gated-at.bofh.it> (permalink)
References	<qwy4q-1Ur-5@gated-at.bofh.it> <qwy4q-1Ur-3@gated-at.bofh.it> <qwWgp-IS-11@gated-at.bofh.it>
X-Original-To	Rob Herring <robh@kernel.org>
MIME-Version	1.0
Content-Type	multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="76DTJ5CE0DCVQemd"
Content-Disposition	inline
X-Cookie	Your domestic life may be harmonious.
User-Agent	Mutt/1.5.24 (2015-08-30)
X-Sa-Exim-Connect-IP	92.40.248.95
X-Sa-Exim-Mail-From	broonie@sirena.org.uk
X-Sa-Exim-Version	4.2.1 (built Mon, 26 Dec 2011 16:24:06 +0000)
X-Sa-Exim-Scanned	Yes (on mezzanine.sirena.org.uk)
Sender	robomod@news.nic.it
List-ID	<linux-kernel.vger.kernel.org>
X-Mailing-List	linux-kernel@vger.kernel.org
Approved	robomod@news.nic.it
Lines	46
Organization	linux.* mail to news gateway
X-Original-Cc	Adam Thomson <Adam.Thomson.Opensource@diasemi.com>, Liam Girdwood <lgirdwood@gmail.com>, Jaroslav Kysela <perex@perex.cz>, Takashi Iwai <tiwai@suse.com>, Pawel Moll <pawel.moll@arm.com>, Mark Rutland <mark.rutland@arm.com>, Ian Campbell <ijc+devicetree@hellion.org.uk>, Kumar Gala <galak@codeaurora.org>, alsa-devel@alsa-project.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Support Opensource <support.opensource@diasemi.com>
X-Original-Date	Sat, 21 Nov 2015 12:52:52 +0000
X-Original-Message-ID	<20151121125252.GD26072@sirena.org.uk>
X-Original-References	<cover.1447941395.git.Adam.Thomson.Opensource@diasemi.com> <add67c0ebac7a516d3cf515fea10a08b7cabadbc.1447941395.git.Adam.Thomson.Opensource@diasemi.com> <20151120155351.GA13917@rob-hp-laptop>
X-Original-Sender	linux-kernel-owner@vger.kernel.org
Xref	csiph.com linux.kernel:1274643

Show key headers only | 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.

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