Re: [PATCH v4 5/7] ARM: dts: meson6: switch to new bindings for UART nodes

From: Jerome Brunet
Date: Mon Jun 12 2017 - 05:43:06 EST


On Fri, 2017-06-09 at 11:49 +0200, Neil Armstrong wrote:
> Switch to the stable UART bindings by adding a XTAL node and using the
> proper compatible strings.
>
> Signed-off-by: Neil Armstrong <narmstrong@xxxxxxxxxxxx>

Shouldn't this patch be squashed with patch 4 ?


> ---
> Âarch/arm/boot/dts/meson6.dtsi | 28 ++++++++++++++++++++++++++++
> Â1 file changed, 28 insertions(+)
>
> diff --git a/arch/arm/boot/dts/meson6.dtsi b/arch/arm/boot/dts/meson6.dtsi
> index b0fc91f..a334fbe 100644
> --- a/arch/arm/boot/dts/meson6.dtsi
> +++ b/arch/arm/boot/dts/meson6.dtsi
> @@ -70,9 +70,37 @@
> Â };
> Â };
> Â
> + xtal: xtal-clk {
> + compatible = "fixed-clock";
> + clock-frequency = <24000000>;
> + clock-output-names = "xtal";
> + #clock-cells = <0>;
> + };
> +
> Â clk81: clk@0 {
> Â #clock-cells = <0>;
> Â compatible = "fixed-clock";
> Â clock-frequency = <200000000>;
> Â };
> Â}; /* end of / */
> +
> +
> +&uart_AO {
> + clocks = <&xtal>, <&clk81>, <&clk81>;
> + clock-names = "xtal", "pclk", "baud";
> +};
> +
> +&uart_A {
> + clocks = <&xtal>, <&clk81>, <&clk81>;
> + clock-names = "xtal", "pclk", "baud";
> +};
> +
> +&uart_B {
> + clocks = <&xtal>, <&clk81>, <&clk81>;
> + clock-names = "xtal", "pclk", "baud";
> +};
> +
> +&uart_C {
> + clocks = <&xtal>, <&clk81>, <&clk81>;
> + clock-names = "xtal", "pclk", "baud";
> +};