On Tue, Sep 24, 2019 at 01:52:43PM +0200, Marek Szyprowski wrote: > From: Maciej Falkowski <m.falkowski@xxxxxxxxxxx> > > '#clock-cells' is a required property for clocks provider, missing > may cause compilation errors. This needs to be explained better. What compilation errors did you see (or expect)? As of now, there is no clock consumer for i2s clocks. After adding clock-cells, the driver will register/provide its clocks, so you add quite visible change without mentioning the effects in change log. Best regards, Krzysztof > > Signed-off-by: Maciej Falkowski <m.falkowski@xxxxxxxxxxx> > Signed-off-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx> > --- > arch/arm/boot/dts/exynos3250.dtsi | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm/boot/dts/exynos3250.dtsi b/arch/arm/boot/dts/exynos3250.dtsi > index 190d9160a5d1..bf32e00334ce 100644 > --- a/arch/arm/boot/dts/exynos3250.dtsi > +++ b/arch/arm/boot/dts/exynos3250.dtsi > @@ -687,6 +687,7 @@ > interrupts = <GIC_SPI 126 IRQ_TYPE_LEVEL_HIGH>; > clocks = <&cmu CLK_I2S>, <&cmu CLK_SCLK_I2S>; > clock-names = "iis", "i2s_opclk0"; > + #clock-cells = <1>; > dmas = <&pdma0 14>, <&pdma0 13>; > dma-names = "tx", "rx"; > pinctrl-0 = <&i2s2_bus>; > -- > 2.17.1 > > >
