[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <a9059443-f125-f6dc-4686-8212165cd431@linaro.org>
Date: Thu, 19 Jan 2023 08:21:12 +0100
From: Neil Armstrong <neil.armstrong@...aro.org>
To: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
Mark Brown <broonie@...nel.org>,
Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Chen-Yu Tsai <wens@...e.org>,
Jernej Skrabec <jernej.skrabec@...il.com>,
Samuel Holland <samuel@...lland.org>,
Kevin Hilman <khilman@...libre.com>,
Jerome Brunet <jbrunet@...libre.com>,
Martin Blumenstingl <martin.blumenstingl@...glemail.com>,
Chin-Ting Kuo <chin-ting_kuo@...eedtech.com>,
Cédric Le Goater <clg@...d.org>,
Joel Stanley <joel@....id.au>,
Andrew Jeffery <andrew@...id.au>,
Kamal Dasu <kdasu.kdev@...il.com>,
Broadcom internal kernel review list
<bcm-kernel-feedback-list@...adcom.com>, Han Xu <han.xu@....com>,
Shawn Guo <shawnguo@...nel.org>,
Sascha Hauer <s.hauer@...gutronix.de>,
Pengutronix Kernel Team <kernel@...gutronix.de>,
Fabio Estevam <festevam@...il.com>,
NXP Linux Team <linux-imx@....com>,
Matthias Brugger <matthias.bgg@...il.com>,
Thierry Reding <thierry.reding@...il.com>,
Jonathan Hunter <jonathanh@...dia.com>,
Andy Gross <agross@...nel.org>,
Bjorn Andersson <andersson@...nel.org>,
Konrad Dybcio <konrad.dybcio@...aro.org>,
Serge Semin <fancer.lancer@...il.com>,
Haibo Chen <haibo.chen@....com>,
Yogesh Gaur <yogeshgaur.83@...il.com>,
Linus Walleij <linus.walleij@...aro.org>,
Heiko Stuebner <heiko@...ech.de>,
Palmer Dabbelt <palmer@...belt.com>,
Paul Walmsley <paul.walmsley@...ive.com>,
Li-hao Kuo <lhjeff911@...il.com>,
Michal Simek <michal.simek@...inx.com>,
Orson Zhai <orsonzhai@...il.com>,
Baolin Wang <baolin.wang@...ux.alibaba.com>,
Chunyan Zhang <zhang.lyra@...il.com>,
Maxime Coquelin <mcoquelin.stm32@...il.com>,
Alexandre Torgue <alexandre.torgue@...s.st.com>,
Maxime Ripard <mripard@...nel.org>,
�ecki <rafal@...ecki.pl>,
Vaishnav Achath <vaishnav.a@...com>,
Parshuram Thombare <pthombar@...ence.com>,
Leilk Liu <leilk.liu@...iatek.com>,
Gabor Juhos <juhosg@...nwrt.org>,
Bert Vermeulen <bert@...t.com>,
Miquel Raynal <miquel.raynal@...tlin.com>,
Marek Vasut <marex@...x.de>,
Birger Koblitz <mail@...ger-koblitz.de>,
Geert Uytterhoeven <geert+renesas@...der.be>,
Anson Huang <Anson.Huang@....com>,
Chris Packham <chris.packham@...iedtelesis.co.nz>,
Kuldeep Singh <singh.kuldeep87k@...il.com>,
Pragnesh Patel <pragnesh.patel@...ive.com>,
Christophe Kerello <christophe.kerello@...s.st.com>,
Patrice Chotard <patrice.chotard@...s.st.com>,
Erwan Leray <erwan.leray@...s.st.com>,
Fabrice Gasnier <fabrice.gasnier@...s.st.com>,
linux-spi@...r.kernel.org, devicetree@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-sunxi@...ts.linux.dev,
linux-kernel@...r.kernel.org, linux-amlogic@...ts.infradead.org,
linux-aspeed@...ts.ozlabs.org, openbmc@...ts.ozlabs.org,
linux-mediatek@...ts.infradead.org, linux-tegra@...r.kernel.org,
linux-arm-msm@...r.kernel.org, linux-rockchip@...ts.infradead.org,
linux-riscv@...ts.infradead.org,
linux-stm32@...md-mailman.stormreply.com
Subject: Re: [PATCH 2/2] spi: dt-bindings: cleanup examples - indentation,
lowercase hex
On 18/01/2023 18:39, Krzysztof Kozlowski wrote:
> Cleanup examples:
> - use 4-space indentation (for cases when it is neither 4 not 2 space),
> - drop redundant blank lines,
> - use lowercase hex.
>
> No functional impact except adjusting to preferred coding style.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
> ---
> .../bindings/spi/amlogic,meson-gx-spicc.yaml | 26 +--
> .../bindings/spi/amlogic,meson6-spifc.yaml | 22 +--
For meson changes:
Reviewed-by: Neil Armstrong <neil.armstrong@...aro.org>
> .../bindings/spi/aspeed,ast2600-fmc.yaml | 24 +--
> .../bindings/spi/brcm,spi-bcm-qspi.yaml | 156 +++++++++---------
> .../bindings/spi/cdns,qspi-nor.yaml | 4 +-
> .../bindings/spi/nvidia,tegra210-quad.yaml | 42 ++---
> .../bindings/spi/qcom,spi-qcom-qspi.yaml | 1 -
> .../devicetree/bindings/spi/renesas,rspi.yaml | 22 +--
> .../bindings/spi/spi-sunplus-sp7021.yaml | 4 +-
> .../devicetree/bindings/spi/st,stm32-spi.yaml | 1 -
> 10 files changed, 150 insertions(+), 152 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/spi/amlogic,meson-gx-spicc.yaml b/Documentation/devicetree/bindings/spi/amlogic,meson-gx-spicc.yaml
> index e5eca3a6f132..4e28e6e9d8e0 100644
> --- a/Documentation/devicetree/bindings/spi/amlogic,meson-gx-spicc.yaml
> +++ b/Documentation/devicetree/bindings/spi/amlogic,meson-gx-spicc.yaml
> @@ -100,17 +100,17 @@ unevaluatedProperties: false
> examples:
> - |
> spi@...08d80 {
> - compatible = "amlogic,meson-gx-spicc";
> - reg = <0xc1108d80 0x80>;
> - interrupts = <112>;
> - clocks = <&clk81>;
> - clock-names = "core";
> - #address-cells = <1>;
> - #size-cells = <0>;
> -
> - display@0 {
> - compatible = "lg,lg4573";
> - spi-max-frequency = <1000000>;
> - reg = <0>;
> - };
> + compatible = "amlogic,meson-gx-spicc";
> + reg = <0xc1108d80 0x80>;
> + interrupts = <112>;
> + clocks = <&clk81>;
> + clock-names = "core";
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + display@0 {
> + compatible = "lg,lg4573";
> + spi-max-frequency = <1000000>;
> + reg = <0>;
> + };
> };
> diff --git a/Documentation/devicetree/bindings/spi/amlogic,meson6-spifc.yaml b/Documentation/devicetree/bindings/spi/amlogic,meson6-spifc.yaml
> index 806043fed4d1..8e769ccda97f 100644
> --- a/Documentation/devicetree/bindings/spi/amlogic,meson6-spifc.yaml
> +++ b/Documentation/devicetree/bindings/spi/amlogic,meson6-spifc.yaml
> @@ -40,15 +40,15 @@ unevaluatedProperties: false
> examples:
> - |
> spi@...08c80 {
> - compatible = "amlogic,meson6-spifc";
> - reg = <0xc1108c80 0x80>;
> - clocks = <&clk81>;
> - #address-cells = <1>;
> - #size-cells = <0>;
> -
> - flash: flash@0 {
> - compatible = "spansion,m25p80", "jedec,spi-nor";
> - reg = <0>;
> - spi-max-frequency = <40000000>;
> - };
> + compatible = "amlogic,meson6-spifc";
> + reg = <0xc1108c80 0x80>;
> + clocks = <&clk81>;
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + flash: flash@0 {
> + compatible = "spansion,m25p80", "jedec,spi-nor";
> + reg = <0>;
> + spi-max-frequency = <40000000>;
> + };
> };
<snip>
Powered by blists - more mailing lists