Re: [PATCH] arm64: dts: renesas: adjust whitespace around '='

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Krzysztof,

On Thu, May 26, 2022 at 10:42 PM Krzysztof Kozlowski
<krzysztof.kozlowski@xxxxxxxxxx> 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 <krzysztof.kozlowski@xxxxxxxxxx>

Thanks for your patch!

> --- a/arch/arm64/boot/dts/renesas/draak.dtsi
> +++ b/arch/arm64/boot/dts/renesas/draak.dtsi
> @@ -630,7 +630,7 @@ rsnd_for_ak4613: endpoint {
>                                 bitclock-master = <&rsnd_for_ak4613>;
>                                 frame-master = <&rsnd_for_ak4613>;
>                                 playback = <&ssi3>, <&src5>, <&dvc0>;
> -                               capture  = <&ssi4>, <&src6>, <&dvc1>;
> +                               capture = <&ssi4>, <&src6>, <&dvc1>;

I think the idea was to align the playback and capture properties,
for easier comparison.
Hence I'm a bit undecided if this needs to be fixed or not...

>                         };
>                 };
>         };

> --- a/arch/arm64/boot/dts/renesas/ulcb-kf.dtsi
> +++ b/arch/arm64/boot/dts/renesas/ulcb-kf.dtsi
> @@ -206,12 +206,12 @@ pcm3168a: audio-codec@44 {
>                                 clocks = <&clksndsel>;
>                                 clock-names = "scki";
>
> -                               VDD1-supply     = <&snd_3p3v>;
> -                               VDD2-supply     = <&snd_3p3v>;
> -                               VCCAD1-supply   = <&snd_vcc5v>;
> -                               VCCAD2-supply   = <&snd_vcc5v>;
> -                               VCCDA1-supply   = <&snd_vcc5v>;
> -                               VCCDA2-supply   = <&snd_vcc5v>;
> +                               VDD1-supply = <&snd_3p3v>;
> +                               VDD2-supply = <&snd_3p3v>;
> +                               VCCAD1-supply = <&snd_vcc5v>;
> +                               VCCAD2-supply = <&snd_vcc5v>;
> +                               VCCDA1-supply = <&snd_vcc5v>;
> +                               VCCDA2-supply = <&snd_vcc5v>;

Same for the various supplies.

>
>                                 ports {
>                                         #address-cells = <1>;
> @@ -438,7 +438,7 @@ rsnd_for_pcm3168a_capture: endpoint {
>                                 bitclock-master;
>                                 frame-master;
>                                 dai-tdm-slot-num = <6>;
> -                               capture  = <&ssi4>;
> +                               capture = <&ssi4>;

Right, there's nothing to align here.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux