Re: [PATCH v4 2/8] ARM: dts: Specify clocks for UARTs on bcm11351
On Thu, Dec 5, 2013 at 11:20 AM, Tim Kryger wrote: > The frequency property in "snps,dw-apb-uart" entries are no longer > required if the rate of the external clock can be determined using the > clk api (see e302cd9 serial: 8250_dw: add support for clk api). > > This patch replaces the frequency property in the UART nodes of > bcm11351.dtsi with references to the relevant clocks following the > common clock binding. > > Signed-off-by: Tim Kryger > Reviewed-by: Markus Mayer > Reviewed-by: Matt Porter > --- > arch/arm/boot/dts/bcm11351.dtsi | 8 > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/arch/arm/boot/dts/bcm11351.dtsi b/arch/arm/boot/dts/bcm11351.dtsi > index eca6fbc..9b99c52 100644 > --- a/arch/arm/boot/dts/bcm11351.dtsi > +++ b/arch/arm/boot/dts/bcm11351.dtsi > @@ -43,7 +43,7 @@ > compatible = "brcm,bcm11351-dw-apb-uart", "snps,dw-apb-uart"; > status = "disabled"; > reg = <0x3e00 0x1000>; > - clock-frequency = <1300>; > + clocks = <_clk>; > interrupts = ; > reg-shift = <2>; > reg-io-width = <4>; > @@ -53,7 +53,7 @@ > compatible = "brcm,bcm11351-dw-apb-uart", "snps,dw-apb-uart"; > status = "disabled"; > reg = <0x3e001000 0x1000>; > - clock-frequency = <1300>; > + clocks = <_clk>; > interrupts = ; > reg-shift = <2>; > reg-io-width = <4>; > @@ -63,7 +63,7 @@ > compatible = "brcm,bcm11351-dw-apb-uart", "snps,dw-apb-uart"; > status = "disabled"; > reg = <0x3e002000 0x1000>; > - clock-frequency = <1300>; > + clocks = <_clk>; > interrupts = ; > reg-shift = <2>; > reg-io-width = <4>; > @@ -73,7 +73,7 @@ > compatible = "brcm,bcm11351-dw-apb-uart", "snps,dw-apb-uart"; > status = "disabled"; > reg = <0x3e003000 0x1000>; > - clock-frequency = <1300>; > + clocks = <_clk>; > interrupts = ; > reg-shift = <2>; > reg-io-width = <4>; > -- > 1.8.0.1 > Reviewed-by: Christian Daudt -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH v4 2/8] ARM: dts: Specify clocks for UARTs on bcm11351
On Thu, Dec 5, 2013 at 11:20 AM, Tim Kryger tim.kry...@linaro.org wrote: The frequency property in snps,dw-apb-uart entries are no longer required if the rate of the external clock can be determined using the clk api (see e302cd9 serial: 8250_dw: add support for clk api). This patch replaces the frequency property in the UART nodes of bcm11351.dtsi with references to the relevant clocks following the common clock binding. Signed-off-by: Tim Kryger tim.kry...@linaro.org Reviewed-by: Markus Mayer markus.ma...@linaro.org Reviewed-by: Matt Porter matt.por...@linaro.org --- arch/arm/boot/dts/bcm11351.dtsi | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/arch/arm/boot/dts/bcm11351.dtsi b/arch/arm/boot/dts/bcm11351.dtsi index eca6fbc..9b99c52 100644 --- a/arch/arm/boot/dts/bcm11351.dtsi +++ b/arch/arm/boot/dts/bcm11351.dtsi @@ -43,7 +43,7 @@ compatible = brcm,bcm11351-dw-apb-uart, snps,dw-apb-uart; status = disabled; reg = 0x3e00 0x1000; - clock-frequency = 1300; + clocks = uartb_clk; interrupts = GIC_SPI 67 IRQ_TYPE_LEVEL_HIGH; reg-shift = 2; reg-io-width = 4; @@ -53,7 +53,7 @@ compatible = brcm,bcm11351-dw-apb-uart, snps,dw-apb-uart; status = disabled; reg = 0x3e001000 0x1000; - clock-frequency = 1300; + clocks = uartb2_clk; interrupts = GIC_SPI 66 IRQ_TYPE_LEVEL_HIGH; reg-shift = 2; reg-io-width = 4; @@ -63,7 +63,7 @@ compatible = brcm,bcm11351-dw-apb-uart, snps,dw-apb-uart; status = disabled; reg = 0x3e002000 0x1000; - clock-frequency = 1300; + clocks = uartb3_clk; interrupts = GIC_SPI 65 IRQ_TYPE_LEVEL_HIGH; reg-shift = 2; reg-io-width = 4; @@ -73,7 +73,7 @@ compatible = brcm,bcm11351-dw-apb-uart, snps,dw-apb-uart; status = disabled; reg = 0x3e003000 0x1000; - clock-frequency = 1300; + clocks = uartb4_clk; interrupts = GIC_SPI 64 IRQ_TYPE_LEVEL_HIGH; reg-shift = 2; reg-io-width = 4; -- 1.8.0.1 Reviewed-by: Christian Daudt b...@fixthebug.org -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/