2022-05-27 12:18:53

by Neil Armstrong

[permalink] [raw]
Subject: Re: [PATCH 2/2] ARM: dts: meson: adjust whitespace around '='

On 26/05/2022 22:45, Krzysztof Kozlowski wrote:
> Fix whitespace coding style: use single space instead of tabs or
> multiple spaces around '=' sign in property assignment. No functional
> changes (same DTB).
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>
> ---
>
> Output compared with dtx_diff and fdtdump.
> ---
> arch/arm/boot/dts/meson.dtsi | 4 ++--
> arch/arm/boot/dts/meson8.dtsi | 2 +-
> arch/arm/boot/dts/meson8b.dtsi | 2 +-
> 3 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm/boot/dts/meson.dtsi b/arch/arm/boot/dts/meson.dtsi
> index 26eaba3fa96f..8e3860d5d916 100644
> --- a/arch/arm/boot/dts/meson.dtsi
> +++ b/arch/arm/boot/dts/meson.dtsi
> @@ -214,14 +214,14 @@ aobus: aobus@c8100000 {
> ranges = <0x0 0xc8100000 0x100000>;
>
> ao_arc_rproc: remoteproc@1c {
> - compatible= "amlogic,meson-mx-ao-arc";
> + compatible = "amlogic,meson-mx-ao-arc";
> reg = <0x1c 0x8>, <0x38 0x8>;
> reg-names = "remap", "cpu";
> status = "disabled";
> };
>
> ir_receiver: ir-receiver@480 {
> - compatible= "amlogic,meson6-ir";
> + compatible = "amlogic,meson6-ir";
> reg = <0x480 0x20>;
> interrupts = <GIC_SPI 15 IRQ_TYPE_EDGE_RISING>;
> status = "disabled";
> diff --git a/arch/arm/boot/dts/meson8.dtsi b/arch/arm/boot/dts/meson8.dtsi
> index 9997a5d0333a..0f8bac8bac8b 100644
> --- a/arch/arm/boot/dts/meson8.dtsi
> +++ b/arch/arm/boot/dts/meson8.dtsi
> @@ -430,7 +430,7 @@ mux {
> };
>
> &ao_arc_rproc {
> - compatible= "amlogic,meson8-ao-arc", "amlogic,meson-mx-ao-arc";
> + compatible = "amlogic,meson8-ao-arc", "amlogic,meson-mx-ao-arc";
> amlogic,secbus2 = <&secbus2>;
> sram = <&ao_arc_sram>;
> resets = <&reset RESET_MEDIA_CPU>;
> diff --git a/arch/arm/boot/dts/meson8b.dtsi b/arch/arm/boot/dts/meson8b.dtsi
> index 94f1c03decce..cf9c04a61ba3 100644
> --- a/arch/arm/boot/dts/meson8b.dtsi
> +++ b/arch/arm/boot/dts/meson8b.dtsi
> @@ -384,7 +384,7 @@ mux {
> };
>
> &ao_arc_rproc {
> - compatible= "amlogic,meson8b-ao-arc", "amlogic,meson-mx-ao-arc";
> + compatible = "amlogic,meson8b-ao-arc", "amlogic,meson-mx-ao-arc";
> amlogic,secbus2 = <&secbus2>;
> sram = <&ao_arc_sram>;
> resets = <&reset RESET_MEDIA_CPU>;

Reviewed-by: Neil Armstrong <[email protected]>