Re: [alsa-devel] [PATCH 01/14] Documentation: Add SoundWire summary

From: Mark Brown
Date: Mon Oct 23 2017 - 05:26:19 EST


On Sun, Oct 22, 2017 at 03:36:32PM +0530, Pierre-Louis Bossart wrote:
> On 10/21/17 4:58 PM, Vinod Koul wrote:
> > On Sat, Oct 21, 2017 at 09:57:44AM +0100, Mark Brown wrote:

> > > There's lots of perfectly normal nouns in this document like Slave here
> > > which are randomly capitalized. Is there some great reason for this?
> > > It makes the document pretty distracting to read.

> > Slave, SoundWire etc are MIPI definitions hence capitalized.

> I insisted to follow the conventions in the specification, it's not random
> at all.

It's making it pretty painful to read as kernel documentation.

> We can put together a DT binding document that follows the Disco spec but
> it'd be a bad idea to change the definitions or come up with new ones...

Given that there's a goal to do this via the device property stuff with
the apparent goal that we also use this stuff for DT it should at least
be run past some DT people. It may be that everything is totally fine as
is but we should at least check.

Attachment: signature.asc
Description: PGP signature