| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <8890e541-54cb-448a-8883-0465e32383c9@microchip.com>
Date: Thu, 9 Jan 2025 03:01:54 +0000
From: <Manikandan.M@...rochip.com>
To: <krzysztof.kozlowski@...aro.org>, <linus.walleij@...aro.org>,
<robh@...nel.org>, <krzk+dt@...nel.org>, <conor+dt@...nel.org>,
<Nicolas.Ferre@...rochip.com>, <alexandre.belloni@...tlin.com>,
<claudiu.beznea@...on.dev>, <michal.simek@....com>, <tychang@...ltek.com>,
<sai.krishna.potthuri@....com>, <linux-gpio@...r.kernel.org>,
<devicetree@...r.kernel.org>, <linux-arm-kernel@...ts.infradead.org>,
<linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] dt-bindings: pinctrl: Correct indentation and style in
DTS example
On 07/01/25 6:28 pm, Krzysztof Kozlowski wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>
> DTS example in the bindings should be indented with 2- or 4-spaces and
> aligned with opening '- |', so correct any differences like 3-spaces or
> mixtures 2- and 4-spaces in one binding.
>
> No functional changes here, but saves some comments during reviews of
> new patches built on existing code.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
Reviewed-by: Manikandan Muralidharan <manikandan.m@...rochip.com>
> ---
> .../pinctrl/atmel,at91rm9200-pinctrl.yaml | 64 +++++++++----------
> .../pinctrl/realtek,rtd1315e-pinctrl.yaml | 50 +++++++--------
> .../pinctrl/realtek,rtd1319d-pinctrl.yaml | 50 +++++++--------
> .../pinctrl/realtek,rtd1619b-pinctrl.yaml | 50 +++++++--------
> .../bindings/pinctrl/xlnx,pinctrl-zynq.yaml | 53 +++++++--------
> 5 files changed, 130 insertions(+), 137 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/pinctrl/atmel,at91rm9200-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/atmel,at91rm9200-pinctrl.yaml
> index 1bb386b42039..a7ede29c1444 100644
> --- a/Documentation/devicetree/bindings/pinctrl/atmel,at91rm9200-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/atmel,at91rm9200-pinctrl.yaml
> @@ -145,40 +145,40 @@ additionalProperties:
>
> examples:
> - |
> - #include <dt-bindings/clock/at91.h>
> - #include <dt-bindings/interrupt-controller/irq.h>
> - #include <dt-bindings/pinctrl/at91.h>
> + #include <dt-bindings/clock/at91.h>
> + #include <dt-bindings/interrupt-controller/irq.h>
> + #include <dt-bindings/pinctrl/at91.h>
>
> - pinctrl@...ff400 {
> - #address-cells = <1>;
> - #size-cells = <1>;
> - compatible = "atmel,at91rm9200-pinctrl", "simple-mfd";
> - ranges = <0xfffff400 0xfffff400 0x600>;
> + pinctrl@...ff400 {
> + #address-cells = <1>;
> + #size-cells = <1>;
> + compatible = "atmel,at91rm9200-pinctrl", "simple-mfd";
> + ranges = <0xfffff400 0xfffff400 0x600>;
>
> - atmel,mux-mask = <
> - /* A B */
> - 0xffffffff 0xffc00c3b /* pioA */
> - 0xffffffff 0x7fff3ccf /* pioB */
> - 0xffffffff 0x007fffff /* pioC */
> - >;
> + atmel,mux-mask = <
> + /* A B */
> + 0xffffffff 0xffc00c3b /* pioA */
> + 0xffffffff 0x7fff3ccf /* pioB */
> + 0xffffffff 0x007fffff /* pioC */
> + >;
>
> - dbgu {
> - pinctrl_dbgu: dbgu-0 {
> - atmel,pins =
> - <AT91_PIOB 14 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
> - AT91_PIOB 15 AT91_PERIPH_A AT91_PINCTRL_NONE>;
> - };
> - };
> + dbgu {
> + dbgu-0 {
> + atmel,pins =
> + <AT91_PIOB 14 AT91_PERIPH_A AT91_PINCTRL_PULL_UP
> + AT91_PIOB 15 AT91_PERIPH_A AT91_PINCTRL_NONE>;
> + };
> + };
>
> - pioA: gpio@...ff400 {
> - compatible = "atmel,at91rm9200-gpio";
> - reg = <0xfffff400 0x200>;
> - interrupts = <2 IRQ_TYPE_LEVEL_HIGH 1>;
> - #gpio-cells = <2>;
> - gpio-controller;
> - interrupt-controller;
> - #interrupt-cells = <2>;
> - clocks = <&pmc PMC_TYPE_PERIPHERAL 2>;
> - };
> - };
> + gpio@...ff400 {
> + compatible = "atmel,at91rm9200-gpio";
> + reg = <0xfffff400 0x200>;
> + interrupts = <2 IRQ_TYPE_LEVEL_HIGH 1>;
> + #gpio-cells = <2>;
> + gpio-controller;
> + interrupt-controller;
> + #interrupt-cells = <2>;
> + clocks = <&pmc PMC_TYPE_PERIPHERAL 2>;
> + };
> + };
> ...
> diff --git a/Documentation/devicetree/bindings/pinctrl/realtek,rtd1315e-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/realtek,rtd1315e-pinctrl.yaml
> index fc6c65fea73b..90bd49d87d2e 100644
> --- a/Documentation/devicetree/bindings/pinctrl/realtek,rtd1315e-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/realtek,rtd1315e-pinctrl.yaml
> @@ -159,30 +159,30 @@ additionalProperties: false
>
> examples:
> - |
> - pinctrl@...00 {
> - compatible = "realtek,rtd1315e-pinctrl";
> - reg = <0x4e000 0x130>;
> + pinctrl@...00 {
> + compatible = "realtek,rtd1315e-pinctrl";
> + reg = <0x4e000 0x130>;
>
> - emmc-hs200-pins {
> - pins = "emmc_clk",
> - "emmc_cmd",
> - "emmc_data_0",
> - "emmc_data_1",
> - "emmc_data_2",
> - "emmc_data_3",
> - "emmc_data_4",
> - "emmc_data_5",
> - "emmc_data_6",
> - "emmc_data_7";
> - function = "emmc";
> - realtek,drive-strength-p = <0x2>;
> - realtek,drive-strength-n = <0x2>;
> - };
> + emmc-hs200-pins {
> + pins = "emmc_clk",
> + "emmc_cmd",
> + "emmc_data_0",
> + "emmc_data_1",
> + "emmc_data_2",
> + "emmc_data_3",
> + "emmc_data_4",
> + "emmc_data_5",
> + "emmc_data_6",
> + "emmc_data_7";
> + function = "emmc";
> + realtek,drive-strength-p = <0x2>;
> + realtek,drive-strength-n = <0x2>;
> + };
>
> - i2c-0-pins {
> - pins = "gpio_12",
> - "gpio_13";
> - function = "i2c0";
> - drive-strength = <4>;
> - };
> - };
> + i2c-0-pins {
> + pins = "gpio_12",
> + "gpio_13";
> + function = "i2c0";
> + drive-strength = <4>;
> + };
> + };
> diff --git a/Documentation/devicetree/bindings/pinctrl/realtek,rtd1319d-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/realtek,rtd1319d-pinctrl.yaml
> index f07361d60acd..b6211c8544ca 100644
> --- a/Documentation/devicetree/bindings/pinctrl/realtek,rtd1319d-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/realtek,rtd1319d-pinctrl.yaml
> @@ -158,30 +158,30 @@ additionalProperties: false
>
> examples:
> - |
> - pinctrl@...00 {
> - compatible = "realtek,rtd1319d-pinctrl";
> - reg = <0x4e000 0x130>;
> + pinctrl@...00 {
> + compatible = "realtek,rtd1319d-pinctrl";
> + reg = <0x4e000 0x130>;
>
> - emmc-hs200-pins {
> - pins = "emmc_clk",
> - "emmc_cmd",
> - "emmc_data_0",
> - "emmc_data_1",
> - "emmc_data_2",
> - "emmc_data_3",
> - "emmc_data_4",
> - "emmc_data_5",
> - "emmc_data_6",
> - "emmc_data_7";
> - function = "emmc";
> - realtek,drive-strength-p = <0x2>;
> - realtek,drive-strength-n = <0x2>;
> - };
> + emmc-hs200-pins {
> + pins = "emmc_clk",
> + "emmc_cmd",
> + "emmc_data_0",
> + "emmc_data_1",
> + "emmc_data_2",
> + "emmc_data_3",
> + "emmc_data_4",
> + "emmc_data_5",
> + "emmc_data_6",
> + "emmc_data_7";
> + function = "emmc";
> + realtek,drive-strength-p = <0x2>;
> + realtek,drive-strength-n = <0x2>;
> + };
>
> - i2c-0-pins {
> - pins = "gpio_12",
> - "gpio_13";
> - function = "i2c0";
> - drive-strength = <4>;
> - };
> - };
> + i2c-0-pins {
> + pins = "gpio_12",
> + "gpio_13";
> + function = "i2c0";
> + drive-strength = <4>;
> + };
> + };
> diff --git a/Documentation/devicetree/bindings/pinctrl/realtek,rtd1619b-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/realtek,rtd1619b-pinctrl.yaml
> index 671e4ec84624..e88bc649cc73 100644
> --- a/Documentation/devicetree/bindings/pinctrl/realtek,rtd1619b-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/realtek,rtd1619b-pinctrl.yaml
> @@ -157,30 +157,30 @@ additionalProperties: false
>
> examples:
> - |
> - pinctrl@...00 {
> - compatible = "realtek,rtd1619b-pinctrl";
> - reg = <0x4e000 0x130>;
> + pinctrl@...00 {
> + compatible = "realtek,rtd1619b-pinctrl";
> + reg = <0x4e000 0x130>;
>
> - emmc-hs200-pins {
> - pins = "emmc_clk",
> - "emmc_cmd",
> - "emmc_data_0",
> - "emmc_data_1",
> - "emmc_data_2",
> - "emmc_data_3",
> - "emmc_data_4",
> - "emmc_data_5",
> - "emmc_data_6",
> - "emmc_data_7";
> - function = "emmc";
> - realtek,drive-strength-p = <0x2>;
> - realtek,drive-strength-n = <0x2>;
> - };
> + emmc-hs200-pins {
> + pins = "emmc_clk",
> + "emmc_cmd",
> + "emmc_data_0",
> + "emmc_data_1",
> + "emmc_data_2",
> + "emmc_data_3",
> + "emmc_data_4",
> + "emmc_data_5",
> + "emmc_data_6",
> + "emmc_data_7";
> + function = "emmc";
> + realtek,drive-strength-p = <0x2>;
> + realtek,drive-strength-n = <0x2>;
> + };
>
> - i2c-0-pins {
> - pins = "gpio_12",
> - "gpio_13";
> - function = "i2c0";
> - drive-strength = <4>;
> - };
> - };
> + i2c-0-pins {
> + pins = "gpio_12",
> + "gpio_13";
> + function = "i2c0";
> + drive-strength = <4>;
> + };
> + };
> diff --git a/Documentation/devicetree/bindings/pinctrl/xlnx,pinctrl-zynq.yaml b/Documentation/devicetree/bindings/pinctrl/xlnx,pinctrl-zynq.yaml
> index de6c10ba36c4..70548cb37ada 100644
> --- a/Documentation/devicetree/bindings/pinctrl/xlnx,pinctrl-zynq.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/xlnx,pinctrl-zynq.yaml
> @@ -180,38 +180,31 @@ additionalProperties: false
> examples:
> - |
> #include <dt-bindings/pinctrl/pinctrl-zynq.h>
> - pinctrl0: pinctrl@700 {
> - compatible = "xlnx,pinctrl-zynq";
> - reg = <0x700 0x200>;
> - syscon = <&slcr>;
> + pinctrl@700 {
> + compatible = "xlnx,pinctrl-zynq";
> + reg = <0x700 0x200>;
> + syscon = <&slcr>;
>
> - pinctrl_uart1_default: uart1-default {
> - mux {
> - groups = "uart1_10_grp";
> - function = "uart1";
> - };
> + uart1-default {
> + mux {
> + groups = "uart1_10_grp";
> + function = "uart1";
> + };
>
> - conf {
> - groups = "uart1_10_grp";
> - slew-rate = <0>;
> - power-source = <IO_STANDARD_LVCMOS18>;
> - };
> + conf {
> + groups = "uart1_10_grp";
> + slew-rate = <0>;
> + power-source = <IO_STANDARD_LVCMOS18>;
> + };
>
> - conf-rx {
> - pins = "MIO49";
> - bias-high-impedance;
> - };
> + conf-rx {
> + pins = "MIO49";
> + bias-high-impedance;
> + };
>
> - conf-tx {
> - pins = "MIO48";
> - bias-disable;
> - };
> - };
> + conf-tx {
> + pins = "MIO48";
> + bias-disable;
> + };
> + };
> };
> -
> - uart1 {
> - pinctrl-names = "default";
> - pinctrl-0 = <&pinctrl_uart1_default>;
> - };
> -
> -...
> --
> 2.43.0
>
--
Thanks and Regards,
Manikandan M.
Powered by blists - more mailing lists