[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAEnQRZAUOzmP2yPb4utyDTnYUDNyqesXJPb5-Ms0tfPy8TMBig@mail.gmail.com>
Date: Tue, 30 Jul 2019 15:02:30 +0300
From: Daniel Baluta <daniel.baluta@...il.com>
To: Nicolin Chen <nicoleotsuka@...il.com>
Cc: Daniel Baluta <daniel.baluta@....com>,
Devicetree List <devicetree@...r.kernel.org>,
Linux-ALSA <alsa-devel@...a-project.org>,
Pengutronix Kernel Team <kernel@...gutronix.de>,
Timur Tabi <timur@...nel.org>, Rob Herring <robh@...nel.org>,
"S.j. Wang" <shengjiu.wang@....com>,
"Angus Ainslie (Purism)" <angus@...ea.ca>,
Takashi Iwai <tiwai@...e.com>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Mark Brown <broonie@...nel.org>,
dl-linux-imx <linux-imx@....com>,
Viorel Suman <viorel.suman@....com>,
Fabio Estevam <festevam@...il.com>,
Mihai Serban <mihai.serban@...il.com>,
Lucas Stach <l.stach@...gutronix.de>
Subject: Re: [alsa-devel] [PATCH v2 7/7] ASoC: dt-bindings: Introduce
compatible strings for 7ULP and 8MQ
On Tue, Jul 30, 2019 at 11:02 AM Nicolin Chen <nicoleotsuka@...il.com> wrote:
>
> On Sun, Jul 28, 2019 at 10:24:29PM +0300, Daniel Baluta wrote:
> > For i.MX7ULP and i.MX8MQ register map is changed. Add two new compatbile
> > strings to differentiate this.
> >
> > Signed-off-by: Daniel Baluta <daniel.baluta@....com>
> > ---
> > Documentation/devicetree/bindings/sound/fsl-sai.txt | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/Documentation/devicetree/bindings/sound/fsl-sai.txt b/Documentation/devicetree/bindings/sound/fsl-sai.txt
> > index 2b38036a4883..b008e9cfedc1 100644
> > --- a/Documentation/devicetree/bindings/sound/fsl-sai.txt
> > +++ b/Documentation/devicetree/bindings/sound/fsl-sai.txt
> > @@ -8,7 +8,8 @@ codec/DSP interfaces.
> > Required properties:
> >
> > - compatible : Compatible list, contains "fsl,vf610-sai",
> > - "fsl,imx6sx-sai" or "fsl,imx6ul-sai"
> > + "fsl,imx6sx-sai", "fsl,imx6ul-sai",
> > + "fsl,imx7ulp-sai", "fsl,imx8mq-sai".
>
> A nit, could have that 'or' :)
I removed the 'or' on purpose because I don't want to move it
around each time we add a new compatible.
Anyhow, I can put it back if this is the convention.
Powered by blists - more mailing lists