linux-kernel - Re: [PATCHv3 3/3] ASoC: simple-card: binding: update binding to support the new style.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 9 Sep 2014 16:25:29 +0300
From:	Jyri Sarha <jsarha@...com>
To:	Mark Brown <broonie@...nel.org>, Xiubo Li <Li.Xiubo@...escale.com>
CC:	<perex@...ex.cz>, <lgirdwood@...il.com>, <tiwai@...e.de>,
	<moinejf@...e.fr>, <andrew@...n.ch>,
	<kuninori.morimoto.gx@...esas.com>, <devicetree@...r.kernel.org>,
	<alsa-devel@...a-project.org>, <robh+dt@...nel.org>,
	<pawel.moll@....com>, <mark.rutland@....com>,
	<ijc+devicetree@...lion.org.uk>, <galak@...eaurora.org>,
	<linux-kernel@...r.kernel.org>
Subject: Re: [PATCHv3 3/3] ASoC: simple-card: binding: update binding to support
 the new style.

On 09/09/2014 03:09 PM, Mark Brown wrote:
> On Wed, Sep 03, 2014 at 10:23:41AM +0800, Xiubo Li wrote:
>> This update patch will split the DT node into old style and new style:
>> The new style will will be easier to add muti DAI links from old single
>> DAI link DTs.
>
>> This patch will maintian compatibility with the old DTs.
>
> Is everyone happy with this new style binding?
>

If I read the diff correctly the only change in the syntax is moving the 
bitclock-inversion and frame-inversion properties to DAI nodes also for 
the new style bindings.

I am Ok with that.

The document does not indicate that the same properties could be also 
placed to DAI link level like they used to, from where they would apply 
to both DAI nodes.

I am Ok with this too, but this breaks the backwards compatibility, 
which I guess is generally considered a problem. However, I doubt anyone 
but me has yet used this part of the new syntax and I am happy to fix my 
dts files.

Best regards,
Jyri

ps. I could not find the corresponding code changes in the previous 
patch, so I assume they are coming later.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/