From: Jyri Sarha <jsarha@ti.com>
To: Mark Brown <broonie@kernel.org>, Xiubo Li <Li.Xiubo@freescale.com>
Cc: mark.rutland@arm.com, moinejf@free.fr,
alsa-devel@alsa-project.org, pawel.moll@arm.com,
kuninori.morimoto.gx@renesas.com, andrew@lunn.ch, tiwai@suse.de,
linux-kernel@vger.kernel.org, ijc+devicetree@hellion.org.uk,
lgirdwood@gmail.com, devicetree@vger.kernel.org,
robh+dt@kernel.org, galak@codeaurora.org
Subject: Re: [PATCHv3 3/3] ASoC: simple-card: binding: update binding to support the new style.
Date: Tue, 9 Sep 2014 16:25:29 +0300 [thread overview]
Message-ID: <540EFFC9.4050207@ti.com> (raw)
In-Reply-To: <20140909120948.GZ2601@sirena.org.uk>
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.
next prev parent reply other threads:[~2014-09-09 13:25 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-03 2:23 [PATCHv3 0/3] ASoC: simple-card: simplify the code Xiubo Li
2014-09-03 2:23 ` [PATCHv3 1/3] ASoC: simple-card: Merge single and muti DAI link(s) code Xiubo Li
2014-09-09 12:02 ` Mark Brown
2014-09-03 2:23 ` [PATCHv3 2/3] ASoC: simple-card: Adjust the comments of simple card Xiubo Li
2014-09-09 12:02 ` Mark Brown
2014-09-10 1:43 ` Li.Xiubo
2014-09-03 2:23 ` [PATCHv3 3/3] ASoC: simple-card: binding: update binding to support the new style Xiubo Li
2014-09-09 12:09 ` Mark Brown
2014-09-09 13:25 ` Jyri Sarha [this message]
[not found] ` <540EFFC9.4050207-l0cyMroinI0@public.gmane.org>
2014-09-10 2:51 ` Li.Xiubo-KZfg59tc24xl57MIdRCFDg
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=540EFFC9.4050207@ti.com \
--to=jsarha@ti.com \
--cc=Li.Xiubo@freescale.com \
--cc=alsa-devel@alsa-project.org \
--cc=andrew@lunn.ch \
--cc=broonie@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=galak@codeaurora.org \
--cc=ijc+devicetree@hellion.org.uk \
--cc=kuninori.morimoto.gx@renesas.com \
--cc=lgirdwood@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=moinejf@free.fr \
--cc=pawel.moll@arm.com \
--cc=robh+dt@kernel.org \
--cc=tiwai@suse.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).