* [PATCH] arm: dts: kirkwood: fix error in #sound-dai-cells size
@ 2018-03-01 20:25 Rob Herring
2018-03-02 13:50 ` Andrew Lunn
2018-03-05 13:48 ` Gregory CLEMENT
0 siblings, 2 replies; 3+ messages in thread
From: Rob Herring @ 2018-03-01 20:25 UTC (permalink / raw)
To: linux-arm-kernel, devicetree
Cc: Gregory Clement, Andrew Lunn, arm, Jason Cooper, Sebastian Hesselbarth
dtc now gives the following warning:
arch/arm/boot/dts/kirkwood-t5325.dtb: Warning (sound_dai_property): /sound/simple-audio-card,cpu:sound-dai: property size (4) too small for cell size 1
The binding documentation gives no indication what the cell size should
be (which should be fixed too), so just assume 0 is fine as that is what
the consumer has.
Cc: Jason Cooper <jason@lakedaemon.net>
Cc: Andrew Lunn <andrew@lunn.ch>
Cc: Gregory Clement <gregory.clement@free-electrons.com>
Cc: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
Signed-off-by: Rob Herring <robh@kernel.org>
---
arch/arm/boot/dts/kirkwood.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/kirkwood.dtsi b/arch/arm/boot/dts/kirkwood.dtsi
index eb2bf7409655..81c7eda2c442 100644
--- a/arch/arm/boot/dts/kirkwood.dtsi
+++ b/arch/arm/boot/dts/kirkwood.dtsi
@@ -382,7 +382,7 @@
audio0: audio-controller@a0000 {
compatible = "marvell,kirkwood-audio";
- #sound-dai-cells = <1>;
+ #sound-dai-cells = <0>;
reg = <0xa0000 0x2210>;
interrupts = <24>;
clocks = <&gate_clk 9>;
--
2.14.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] arm: dts: kirkwood: fix error in #sound-dai-cells size
2018-03-01 20:25 [PATCH] arm: dts: kirkwood: fix error in #sound-dai-cells size Rob Herring
@ 2018-03-02 13:50 ` Andrew Lunn
2018-03-05 13:48 ` Gregory CLEMENT
1 sibling, 0 replies; 3+ messages in thread
From: Andrew Lunn @ 2018-03-02 13:50 UTC (permalink / raw)
To: Rob Herring
Cc: devicetree, Jason Cooper, arm, Gregory Clement, linux-arm-kernel,
Sebastian Hesselbarth
On Thu, Mar 01, 2018 at 02:25:37PM -0600, Rob Herring wrote:
> dtc now gives the following warning:
>
> arch/arm/boot/dts/kirkwood-t5325.dtb: Warning (sound_dai_property): /sound/simple-audio-card,cpu:sound-dai: property size (4) too small for cell size 1
>
> The binding documentation gives no indication what the cell size should
> be (which should be fixed too), so just assume 0 is fine as that is what
> the consumer has.
>
> Cc: Jason Cooper <jason@lakedaemon.net>
> Cc: Andrew Lunn <andrew@lunn.ch>
> Cc: Gregory Clement <gregory.clement@free-electrons.com>
> Cc: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
> Signed-off-by: Rob Herring <robh@kernel.org>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Andrew
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] arm: dts: kirkwood: fix error in #sound-dai-cells size
2018-03-01 20:25 [PATCH] arm: dts: kirkwood: fix error in #sound-dai-cells size Rob Herring
2018-03-02 13:50 ` Andrew Lunn
@ 2018-03-05 13:48 ` Gregory CLEMENT
1 sibling, 0 replies; 3+ messages in thread
From: Gregory CLEMENT @ 2018-03-05 13:48 UTC (permalink / raw)
To: Rob Herring
Cc: devicetree, Jason Cooper, Andrew Lunn, arm, Gregory Clement,
linux-arm-kernel, Sebastian Hesselbarth
Hi Rob,
On jeu., mars 01 2018, Rob Herring <robh@kernel.org> wrote:
> dtc now gives the following warning:
>
> arch/arm/boot/dts/kirkwood-t5325.dtb: Warning (sound_dai_property): /sound/simple-audio-card,cpu:sound-dai: property size (4) too small for cell size 1
>
> The binding documentation gives no indication what the cell size should
> be (which should be fixed too), so just assume 0 is fine as that is what
> the consumer has.
>
> Cc: Jason Cooper <jason@lakedaemon.net>
> Cc: Andrew Lunn <andrew@lunn.ch>
> Cc: Gregory Clement <gregory.clement@free-electrons.com>
> Cc: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
> Signed-off-by: Rob Herring <robh@kernel.org>
Applied on mvebu/dt with the reviewed-by flag from Andrew.
Thanks,
Gregory
> ---
> arch/arm/boot/dts/kirkwood.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/kirkwood.dtsi b/arch/arm/boot/dts/kirkwood.dtsi
> index eb2bf7409655..81c7eda2c442 100644
> --- a/arch/arm/boot/dts/kirkwood.dtsi
> +++ b/arch/arm/boot/dts/kirkwood.dtsi
> @@ -382,7 +382,7 @@
>
> audio0: audio-controller@a0000 {
> compatible = "marvell,kirkwood-audio";
> - #sound-dai-cells = <1>;
> + #sound-dai-cells = <0>;
> reg = <0xa0000 0x2210>;
> interrupts = <24>;
> clocks = <&gate_clk 9>;
> --
> 2.14.1
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
--
Gregory Clement, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
http://bootlin.com
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-03-05 13:48 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-03-01 20:25 [PATCH] arm: dts: kirkwood: fix error in #sound-dai-cells size Rob Herring
2018-03-02 13:50 ` Andrew Lunn
2018-03-05 13:48 ` Gregory CLEMENT
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).