[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20200908065321.GB24227@pi3>
Date: Tue, 8 Sep 2020 08:53:21 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Jonathan Bakker <xc-racer2@...e.ca>
Cc: Kukjin Kim <kgene@...nel.org>, Rob Herring <robh+dt@...nel.org>,
Jonathan Cameron <jic23@...nel.org>,
Lars-Peter Clausen <lars@...afoo.de>,
Peter Meerwald-Stadler <pmeerw@...erw.net>,
Paweł Chmiel <pawel.mikolaj.chmiel@...il.com>,
linux-arm-kernel@...ts.infradead.org,
linux-samsung-soc@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-iio@...r.kernel.org
Subject: Re: [RFT 09/25] ARM: dts: s5pv210: fix number of I2S DAI cells
On Mon, Sep 07, 2020 at 04:55:26PM -0700, Jonathan Bakker wrote:
> Sadly, this is causing issues for me. The machine driver is no longer probing correctly
> on the Galaxy S.
>
> The failing call in sound/soc/samsung/aries_wm8994.c is
>
> /* Set CPU of_node for BT DAI */
> aries_dai[2].cpus->of_node = of_parse_phandle(cpu,
> "sound-dai", 1);
>
> where cpus->of_node is not set properly. Which is definitely weird because it doesn't
> look like this should affect that.
>
> Let me know if there's any specific test that you want me to do.
Thanks for the tests. I wonder now if this was working before because
really my change should not break it... I'll think more about it.
Best regards,
Krzysztof
>
> Thanks,
> Jonathan
>
>
> On 2020-09-07 9:11 a.m., Krzysztof Kozlowski wrote:
> > The bindings describe I2S DAI has 1 cells. This makes especially sense
> > for i2s0 which registers two DAIs. Adjust the cells to fix dtbs_check
> > warnings like:
> >
> > i2s@...00000: #sound-dai-cells:0:0: 1 was expected
> >
> > Signed-off-by: Krzysztof Kozlowski <krzk@...nel.org>
> > ---
> > arch/arm/boot/dts/s5pv210-fascinate4g.dts | 2 +-
> > arch/arm/boot/dts/s5pv210-galaxys.dts | 2 +-
> > arch/arm/boot/dts/s5pv210.dtsi | 6 +++---
> > 3 files changed, 5 insertions(+), 5 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/s5pv210-fascinate4g.dts b/arch/arm/boot/dts/s5pv210-fascinate4g.dts
> > index ca064359dd30..a6dc8a173af1 100644
> > --- a/arch/arm/boot/dts/s5pv210-fascinate4g.dts
> > +++ b/arch/arm/boot/dts/s5pv210-fascinate4g.dts
> > @@ -102,7 +102,7 @@
> > pinctrl-0 = <&headset_det &earpath_sel>;
> >
> > cpu {
> > - sound-dai = <&i2s0>, <&bt_codec>;
> > + sound-dai = <&i2s0 0>, <&bt_codec>;
> > };
> >
> > codec {
> > diff --git a/arch/arm/boot/dts/s5pv210-galaxys.dts b/arch/arm/boot/dts/s5pv210-galaxys.dts
> > index 560f830b6f6b..0eba06f56ac7 100644
> > --- a/arch/arm/boot/dts/s5pv210-galaxys.dts
> > +++ b/arch/arm/boot/dts/s5pv210-galaxys.dts
> > @@ -132,7 +132,7 @@
> > pinctrl-0 = <&headset_det &earpath_sel>;
> >
> > cpu {
> > - sound-dai = <&i2s0>, <&bt_codec>;
> > + sound-dai = <&i2s0 0>, <&bt_codec>;
> > };
> >
> > codec {
> > diff --git a/arch/arm/boot/dts/s5pv210.dtsi b/arch/arm/boot/dts/s5pv210.dtsi
> > index 2871351ab907..96e667ba1c3f 100644
> > --- a/arch/arm/boot/dts/s5pv210.dtsi
> > +++ b/arch/arm/boot/dts/s5pv210.dtsi
> > @@ -251,7 +251,7 @@
> > samsung,idma-addr = <0xc0010000>;
> > pinctrl-names = "default";
> > pinctrl-0 = <&i2s0_bus>;
> > - #sound-dai-cells = <0>;
> > + #sound-dai-cells = <1>;
> > status = "disabled";
> > };
> >
> > @@ -266,7 +266,7 @@
> > clocks = <&clocks CLK_I2S1>, <&clocks SCLK_AUDIO1>;
> > pinctrl-names = "default";
> > pinctrl-0 = <&i2s1_bus>;
> > - #sound-dai-cells = <0>;
> > + #sound-dai-cells = <1>;
> > status = "disabled";
> > };
> >
> > @@ -281,7 +281,7 @@
> > clocks = <&clocks CLK_I2S2>, <&clocks SCLK_AUDIO2>;
> > pinctrl-names = "default";
> > pinctrl-0 = <&i2s2_bus>;
> > - #sound-dai-cells = <0>;
> > + #sound-dai-cells = <1>;
> > status = "disabled";
> > };
> >
> >
Powered by blists - more mailing lists