mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Li.Xiubo@freescale.com" <Li.Xiubo@freescale.com>
To: Jyri Sarha <jsarha@ti.com>, Mark Brown <broonie@kernel.org>
Cc: "perex@perex.cz" <perex@perex.cz>,
	"lgirdwood@gmail.com" <lgirdwood@gmail.com>,
	"tiwai@suse.de" <tiwai@suse.de>,
	"moinejf@free.fr" <moinejf@free.fr>,
	"andrew@lunn.ch" <andrew@lunn.ch>,
	"kuninori.morimoto.gx@renesas.com"
	<kuninori.morimoto.gx@renesas.com>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>,
	"robh+dt@kernel.org" <robh+dt@kernel.org>,
	"pawel.moll@arm.com" <pawel.moll@arm.com>,
	"mark.rutland@arm.com" <mark.rutland@arm.com>,
	"ijc+devicetree@hellion.org.uk" <ijc+devicetree@hellion.org.uk>,
	"galak@codeaurora.org" <galak@codeaurora.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCHv3 3/3] ASoC: simple-card: binding: update binding to support the new style.
Date: Wed, 10 Sep 2014 02:51:16 +0000	[thread overview]
Message-ID: <755d5dc6d85243c0bd209aea49eef6c0@BY2PR0301MB0613.namprd03.prod.outlook.com> (raw)
In-Reply-To: <540EFFC9.4050207@ti.com>

Hi,

> 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 will post one new version of this patch with some very small fix, and
I will also add this.

> 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.

Yes, I have one patch about the new style in my local machine, for some
reason I will post it later.

Thanks,

BRs
Xiubo




      reply	other threads:[~2014-09-10  2:51 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
2014-09-10  2:51       ` Li.Xiubo [this message]

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=755d5dc6d85243c0bd209aea49eef6c0@BY2PR0301MB0613.namprd03.prod.outlook.com \
    --to=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=jsarha@ti.com \
    --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=perex@perex.cz \
    --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

all inboxes | Powered by JetHome®