* [PATCH] dt-bindings: usb: Correct indentation and style in DTS example
@ 2025-01-07 13:10 Krzysztof Kozlowski
2025-01-07 17:26 ` Florian Fainelli
` (8 more replies)
0 siblings, 9 replies; 10+ messages in thread
From: Krzysztof Kozlowski @ 2025-01-07 13:10 UTC (permalink / raw)
To: Greg Kroah-Hartman, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Joel Stanley, Andrew Jeffery, Justin Chen,
Al Cooper, Broadcom internal kernel review list, Thierry Reding,
Jonathan Hunter, Geert Uytterhoeven, Magnus Damm,
Benjamin Herrenschmidt, Florian Fainelli, Benjamin Bara,
Bjorn Andersson, Ahmad Zainie, Del Regno, Tianping Fang,
Jassi Brar, Biju Das, Yoshihiro Shimoda, Francesco Dolcini,
Macpaul Lin, Alexander Stein, linux-usb, devicetree,
linux-arm-kernel, linux-aspeed, linux-kernel, linux-tegra,
linux-renesas-soc
Cc: Krzysztof Kozlowski
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@linaro.org>
---
.../bindings/usb/aspeed,usb-vhub.yaml | 40 +++++++++----------
.../devicetree/bindings/usb/brcm,bdc.yaml | 14 +++----
.../devicetree/bindings/usb/cypress,hx3.yaml | 24 +++++------
.../devicetree/bindings/usb/dwc2.yaml | 4 +-
.../devicetree/bindings/usb/fcs,fsa4480.yaml | 20 +++++-----
.../bindings/usb/intel,keembay-dwc3.yaml | 30 +++++++-------
.../devicetree/bindings/usb/ite,it5205.yaml | 18 ++++-----
.../bindings/usb/maxim,max3420-udc.yaml | 28 ++++++-------
.../bindings/usb/nvidia,tegra210-xusb.yaml | 4 +-
.../bindings/usb/renesas,rzv2m-usb3drd.yaml | 36 ++++++++---------
.../bindings/usb/renesas,usb3-peri.yaml | 24 +++++------
.../devicetree/bindings/usb/ti,hd3ss3220.yaml | 38 +++++++++---------
.../bindings/usb/ti,tusb73x0-pci.yaml | 6 +--
.../devicetree/bindings/usb/ti,usb8020b.yaml | 20 +++++-----
.../devicetree/bindings/usb/ti,usb8041.yaml | 16 ++++----
15 files changed, 161 insertions(+), 161 deletions(-)
diff --git a/Documentation/devicetree/bindings/usb/aspeed,usb-vhub.yaml b/Documentation/devicetree/bindings/usb/aspeed,usb-vhub.yaml
index a86bcd95100e..7f22f9c031b2 100644
--- a/Documentation/devicetree/bindings/usb/aspeed,usb-vhub.yaml
+++ b/Documentation/devicetree/bindings/usb/aspeed,usb-vhub.yaml
@@ -113,27 +113,27 @@ examples:
- |
#include <dt-bindings/clock/aspeed-clock.h>
vhub: usb-vhub@1e6a0000 {
- compatible = "aspeed,ast2500-usb-vhub";
- reg = <0x1e6a0000 0x300>;
- interrupts = <5>;
- clocks = <&syscon ASPEED_CLK_GATE_USBPORT1CLK>;
- aspeed,vhub-downstream-ports = <5>;
- aspeed,vhub-generic-endpoints = <15>;
- pinctrl-names = "default";
- pinctrl-0 = <&pinctrl_usb2ad_default>;
+ compatible = "aspeed,ast2500-usb-vhub";
+ reg = <0x1e6a0000 0x300>;
+ interrupts = <5>;
+ clocks = <&syscon ASPEED_CLK_GATE_USBPORT1CLK>;
+ aspeed,vhub-downstream-ports = <5>;
+ aspeed,vhub-generic-endpoints = <15>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinctrl_usb2ad_default>;
- vhub-vendor-id = <0x1d6b>;
- vhub-product-id = <0x0107>;
- vhub-device-revision = <0x0100>;
- vhub-strings {
- #address-cells = <1>;
- #size-cells = <0>;
+ vhub-vendor-id = <0x1d6b>;
+ vhub-product-id = <0x0107>;
+ vhub-device-revision = <0x0100>;
+ vhub-strings {
+ #address-cells = <1>;
+ #size-cells = <0>;
- string@409 {
- reg = <0x409>;
- manufacturer = "ASPEED";
- product = "USB Virtual Hub";
- serial-number = "0000";
- };
+ string@409 {
+ reg = <0x409>;
+ manufacturer = "ASPEED";
+ product = "USB Virtual Hub";
+ serial-number = "0000";
};
+ };
};
diff --git a/Documentation/devicetree/bindings/usb/brcm,bdc.yaml b/Documentation/devicetree/bindings/usb/brcm,bdc.yaml
index 9e561fee98f1..f9375c69e86b 100644
--- a/Documentation/devicetree/bindings/usb/brcm,bdc.yaml
+++ b/Documentation/devicetree/bindings/usb/brcm,bdc.yaml
@@ -41,10 +41,10 @@ additionalProperties: false
examples:
- |
- usb@f0b02000 {
- compatible = "brcm,bdc-udc-v2";
- reg = <0xf0b02000 0xfc4>;
- interrupts = <0x0 0x60 0x0>;
- phys = <&usbphy_0 0x0>;
- clocks = <&sw_usbd>;
- };
+ usb@f0b02000 {
+ compatible = "brcm,bdc-udc-v2";
+ reg = <0xf0b02000 0xfc4>;
+ interrupts = <0x0 0x60 0x0>;
+ phys = <&usbphy_0 0x0>;
+ clocks = <&sw_usbd>;
+ };
diff --git a/Documentation/devicetree/bindings/usb/cypress,hx3.yaml b/Documentation/devicetree/bindings/usb/cypress,hx3.yaml
index e44e88d993d0..1033b7a4b8f9 100644
--- a/Documentation/devicetree/bindings/usb/cypress,hx3.yaml
+++ b/Documentation/devicetree/bindings/usb/cypress,hx3.yaml
@@ -56,21 +56,21 @@ examples:
/* 2.0 hub on port 1 */
hub_2_0: hub@1 {
- compatible = "usb4b4,6504";
- reg = <1>;
- peer-hub = <&hub_3_0>;
- reset-gpios = <&gpio1 11 GPIO_ACTIVE_LOW>;
- vdd-supply = <®_1v2_usb>;
- vdd2-supply = <®_3v3_usb>;
+ compatible = "usb4b4,6504";
+ reg = <1>;
+ peer-hub = <&hub_3_0>;
+ reset-gpios = <&gpio1 11 GPIO_ACTIVE_LOW>;
+ vdd-supply = <®_1v2_usb>;
+ vdd2-supply = <®_3v3_usb>;
};
/* 3.0 hub on port 2 */
hub_3_0: hub@2 {
- compatible = "usb4b4,6506";
- reg = <2>;
- peer-hub = <&hub_2_0>;
- reset-gpios = <&gpio1 11 GPIO_ACTIVE_LOW>;
- vdd-supply = <®_1v2_usb>;
- vdd2-supply = <®_3v3_usb>;
+ compatible = "usb4b4,6506";
+ reg = <2>;
+ peer-hub = <&hub_2_0>;
+ reset-gpios = <&gpio1 11 GPIO_ACTIVE_LOW>;
+ vdd-supply = <®_1v2_usb>;
+ vdd2-supply = <®_3v3_usb>;
};
};
diff --git a/Documentation/devicetree/bindings/usb/dwc2.yaml b/Documentation/devicetree/bindings/usb/dwc2.yaml
index a5f2e3442a0e..e83d30a91b88 100644
--- a/Documentation/devicetree/bindings/usb/dwc2.yaml
+++ b/Documentation/devicetree/bindings/usb/dwc2.yaml
@@ -192,7 +192,7 @@ unevaluatedProperties: false
examples:
- |
- usb@101c0000 {
+ usb@101c0000 {
compatible = "rockchip,rk3066-usb", "snps,dwc2";
reg = <0x10180000 0x40000>;
interrupts = <18>;
@@ -200,6 +200,6 @@ examples:
clock-names = "otg";
phys = <&usbphy>;
phy-names = "usb2-phy";
- };
+ };
...
diff --git a/Documentation/devicetree/bindings/usb/fcs,fsa4480.yaml b/Documentation/devicetree/bindings/usb/fcs,fsa4480.yaml
index 8b25b9a01ced..e3a7df91f7f1 100644
--- a/Documentation/devicetree/bindings/usb/fcs,fsa4480.yaml
+++ b/Documentation/devicetree/bindings/usb/fcs,fsa4480.yaml
@@ -87,21 +87,21 @@ examples:
#size-cells = <0>;
typec-mux@42 {
- compatible = "fcs,fsa4480";
- reg = <0x42>;
+ compatible = "fcs,fsa4480";
+ reg = <0x42>;
- interrupts-extended = <&tlmm 2 IRQ_TYPE_LEVEL_LOW>;
+ interrupts-extended = <&tlmm 2 IRQ_TYPE_LEVEL_LOW>;
- vcc-supply = <&vreg_bob>;
+ vcc-supply = <&vreg_bob>;
- mode-switch;
- orientation-switch;
+ mode-switch;
+ orientation-switch;
- port {
- fsa4480_ept: endpoint {
- remote-endpoint = <&typec_controller>;
+ port {
+ fsa4480_ept: endpoint {
+ remote-endpoint = <&typec_controller>;
+ };
};
- };
};
};
...
diff --git a/Documentation/devicetree/bindings/usb/intel,keembay-dwc3.yaml b/Documentation/devicetree/bindings/usb/intel,keembay-dwc3.yaml
index d3511f48cd55..1a75544a8c31 100644
--- a/Documentation/devicetree/bindings/usb/intel,keembay-dwc3.yaml
+++ b/Documentation/devicetree/bindings/usb/intel,keembay-dwc3.yaml
@@ -58,20 +58,20 @@ examples:
#define KEEM_BAY_A53_AUX_USB_SUSPEND
usb {
- compatible = "intel,keembay-dwc3";
- clocks = <&scmi_clk KEEM_BAY_A53_AUX_USB>,
- <&scmi_clk KEEM_BAY_A53_AUX_USB_REF>,
- <&scmi_clk KEEM_BAY_A53_AUX_USB_ALT_REF>,
- <&scmi_clk KEEM_BAY_A53_AUX_USB_SUSPEND>;
- clock-names = "async_master", "ref", "alt_ref", "suspend";
- ranges;
- #address-cells = <1>;
- #size-cells = <1>;
+ compatible = "intel,keembay-dwc3";
+ clocks = <&scmi_clk KEEM_BAY_A53_AUX_USB>,
+ <&scmi_clk KEEM_BAY_A53_AUX_USB_REF>,
+ <&scmi_clk KEEM_BAY_A53_AUX_USB_ALT_REF>,
+ <&scmi_clk KEEM_BAY_A53_AUX_USB_SUSPEND>;
+ clock-names = "async_master", "ref", "alt_ref", "suspend";
+ ranges;
+ #address-cells = <1>;
+ #size-cells = <1>;
- usb@34000000 {
- compatible = "snps,dwc3";
- reg = <0x34000000 0x10000>;
- interrupts = <GIC_SPI 91 IRQ_TYPE_LEVEL_HIGH>;
- dr_mode = "peripheral";
- };
+ usb@34000000 {
+ compatible = "snps,dwc3";
+ reg = <0x34000000 0x10000>;
+ interrupts = <GIC_SPI 91 IRQ_TYPE_LEVEL_HIGH>;
+ dr_mode = "peripheral";
+ };
};
diff --git a/Documentation/devicetree/bindings/usb/ite,it5205.yaml b/Documentation/devicetree/bindings/usb/ite,it5205.yaml
index 36ec4251b5f2..889710733de5 100644
--- a/Documentation/devicetree/bindings/usb/ite,it5205.yaml
+++ b/Documentation/devicetree/bindings/usb/ite,it5205.yaml
@@ -54,19 +54,19 @@ examples:
#size-cells = <0>;
typec-mux@48 {
- compatible = "ite,it5205";
- reg = <0x48>;
+ compatible = "ite,it5205";
+ reg = <0x48>;
- mode-switch;
- orientation-switch;
+ mode-switch;
+ orientation-switch;
- vcc-supply = <&mt6359_vibr_ldo_reg>;
+ vcc-supply = <&mt6359_vibr_ldo_reg>;
- port {
- it5205_usbss_sbu: endpoint {
- remote-endpoint = <&typec_controller>;
+ port {
+ it5205_usbss_sbu: endpoint {
+ remote-endpoint = <&typec_controller>;
+ };
};
- };
};
};
...
diff --git a/Documentation/devicetree/bindings/usb/maxim,max3420-udc.yaml b/Documentation/devicetree/bindings/usb/maxim,max3420-udc.yaml
index 8e0f4ecc010d..6edb1fc5044e 100644
--- a/Documentation/devicetree/bindings/usb/maxim,max3420-udc.yaml
+++ b/Documentation/devicetree/bindings/usb/maxim,max3420-udc.yaml
@@ -50,18 +50,18 @@ additionalProperties: false
examples:
- |
- #include <dt-bindings/gpio/gpio.h>
- #include <dt-bindings/interrupt-controller/irq.h>
- spi {
- #address-cells = <1>;
- #size-cells = <0>;
+ #include <dt-bindings/gpio/gpio.h>
+ #include <dt-bindings/interrupt-controller/irq.h>
+ spi {
+ #address-cells = <1>;
+ #size-cells = <0>;
- udc@0 {
- compatible = "maxim,max3420-udc";
- reg = <0>;
- interrupt-parent = <&gpio>;
- interrupts = <0 IRQ_TYPE_EDGE_FALLING>, <10 IRQ_TYPE_EDGE_BOTH>;
- interrupt-names = "udc", "vbus";
- spi-max-frequency = <12500000>;
- };
- };
+ udc@0 {
+ compatible = "maxim,max3420-udc";
+ reg = <0>;
+ interrupt-parent = <&gpio>;
+ interrupts = <0 IRQ_TYPE_EDGE_FALLING>, <10 IRQ_TYPE_EDGE_BOTH>;
+ interrupt-names = "udc", "vbus";
+ spi-max-frequency = <12500000>;
+ };
+ };
diff --git a/Documentation/devicetree/bindings/usb/nvidia,tegra210-xusb.yaml b/Documentation/devicetree/bindings/usb/nvidia,tegra210-xusb.yaml
index 90296613b3a5..c0e313c70bba 100644
--- a/Documentation/devicetree/bindings/usb/nvidia,tegra210-xusb.yaml
+++ b/Documentation/devicetree/bindings/usb/nvidia,tegra210-xusb.yaml
@@ -189,7 +189,7 @@ examples:
#size-cells = <0>;
ethernet@1 {
- compatible = "usb955,9ff";
- reg = <1>;
+ compatible = "usb955,9ff";
+ reg = <1>;
};
};
diff --git a/Documentation/devicetree/bindings/usb/renesas,rzv2m-usb3drd.yaml b/Documentation/devicetree/bindings/usb/renesas,rzv2m-usb3drd.yaml
index ff625600d9af..b87e139c29e5 100644
--- a/Documentation/devicetree/bindings/usb/renesas,rzv2m-usb3drd.yaml
+++ b/Documentation/devicetree/bindings/usb/renesas,rzv2m-usb3drd.yaml
@@ -104,26 +104,26 @@ examples:
#size-cells = <1>;
usb3host: usb@85060000 {
- compatible = "renesas,r9a09g011-xhci",
- "renesas,rzv2m-xhci";
- reg = <0x85060000 0x2000>;
- interrupts = <GIC_SPI 245 IRQ_TYPE_LEVEL_HIGH>;
- clocks = <&cpg CPG_MOD R9A09G011_USB_ACLK_H>,
- <&cpg CPG_MOD R9A09G011_USB_PCLK>;
- clock-names = "axi", "reg";
- power-domains = <&cpg>;
- resets = <&cpg R9A09G011_USB_ARESETN_H>;
+ compatible = "renesas,r9a09g011-xhci",
+ "renesas,rzv2m-xhci";
+ reg = <0x85060000 0x2000>;
+ interrupts = <GIC_SPI 245 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&cpg CPG_MOD R9A09G011_USB_ACLK_H>,
+ <&cpg CPG_MOD R9A09G011_USB_PCLK>;
+ clock-names = "axi", "reg";
+ power-domains = <&cpg>;
+ resets = <&cpg R9A09G011_USB_ARESETN_H>;
};
usb3peri: usb3peri@85070000 {
- compatible = "renesas,r9a09g011-usb3-peri",
- "renesas,rzv2m-usb3-peri";
- reg = <0x85070000 0x400>;
- interrupts = <GIC_SPI 246 IRQ_TYPE_LEVEL_HIGH>;
- clocks = <&cpg CPG_MOD R9A09G011_USB_ACLK_P>,
- <&cpg CPG_MOD R9A09G011_USB_PCLK>;
- clock-names = "axi", "reg";
- power-domains = <&cpg>;
- resets = <&cpg R9A09G011_USB_ARESETN_P>;
+ compatible = "renesas,r9a09g011-usb3-peri",
+ "renesas,rzv2m-usb3-peri";
+ reg = <0x85070000 0x400>;
+ interrupts = <GIC_SPI 246 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&cpg CPG_MOD R9A09G011_USB_ACLK_P>,
+ <&cpg CPG_MOD R9A09G011_USB_PCLK>;
+ clock-names = "axi", "reg";
+ power-domains = <&cpg>;
+ resets = <&cpg R9A09G011_USB_ARESETN_P>;
};
};
diff --git a/Documentation/devicetree/bindings/usb/renesas,usb3-peri.yaml b/Documentation/devicetree/bindings/usb/renesas,usb3-peri.yaml
index b2b811a0ade8..4e56e4ffeaf2 100644
--- a/Documentation/devicetree/bindings/usb/renesas,usb3-peri.yaml
+++ b/Documentation/devicetree/bindings/usb/renesas,usb3-peri.yaml
@@ -132,19 +132,19 @@ examples:
usb-role-switch;
ports {
- #address-cells = <1>;
- #size-cells = <0>;
- port@0 {
- reg = <0>;
- usb3_hs_ep: endpoint {
- remote-endpoint = <&hs_ep>;
- };
+ #address-cells = <1>;
+ #size-cells = <0>;
+ port@0 {
+ reg = <0>;
+ usb3_hs_ep: endpoint {
+ remote-endpoint = <&hs_ep>;
};
- port@1 {
- reg = <1>;
- usb3_role_switch: endpoint {
- remote-endpoint = <&hd3ss3220_out_ep>;
- };
+ };
+ port@1 {
+ reg = <1>;
+ usb3_role_switch: endpoint {
+ remote-endpoint = <&hd3ss3220_out_ep>;
};
+ };
};
};
diff --git a/Documentation/devicetree/bindings/usb/ti,hd3ss3220.yaml b/Documentation/devicetree/bindings/usb/ti,hd3ss3220.yaml
index 54c6586cb56d..bec1c8047bc0 100644
--- a/Documentation/devicetree/bindings/usb/ti,hd3ss3220.yaml
+++ b/Documentation/devicetree/bindings/usb/ti,hd3ss3220.yaml
@@ -56,26 +56,26 @@ examples:
#size-cells = <0>;
hd3ss3220@47 {
- compatible = "ti,hd3ss3220";
- reg = <0x47>;
- interrupt-parent = <&gpio6>;
- interrupts = <3>;
+ compatible = "ti,hd3ss3220";
+ reg = <0x47>;
+ interrupt-parent = <&gpio6>;
+ interrupts = <3>;
- ports {
- #address-cells = <1>;
- #size-cells = <0>;
- port@0 {
- reg = <0>;
- hd3ss3220_in_ep: endpoint {
- remote-endpoint = <&ss_ep>;
- };
- };
- port@1 {
- reg = <1>;
- hd3ss3220_out_ep: endpoint {
- remote-endpoint = <&usb3_role_switch>;
- };
- };
+ ports {
+ #address-cells = <1>;
+ #size-cells = <0>;
+ port@0 {
+ reg = <0>;
+ hd3ss3220_in_ep: endpoint {
+ remote-endpoint = <&ss_ep>;
+ };
};
+ port@1 {
+ reg = <1>;
+ hd3ss3220_out_ep: endpoint {
+ remote-endpoint = <&usb3_role_switch>;
+ };
+ };
+ };
};
};
diff --git a/Documentation/devicetree/bindings/usb/ti,tusb73x0-pci.yaml b/Documentation/devicetree/bindings/usb/ti,tusb73x0-pci.yaml
index ddda734f36fb..c4a91b3d6612 100644
--- a/Documentation/devicetree/bindings/usb/ti,tusb73x0-pci.yaml
+++ b/Documentation/devicetree/bindings/usb/ti,tusb73x0-pci.yaml
@@ -48,8 +48,8 @@ examples:
device_type = "pci";
usb@0 {
- compatible = "pci104c,8241";
- reg = <0x0 0x0 0x0 0x0 0x0>;
- ti,pwron-active-high;
+ compatible = "pci104c,8241";
+ reg = <0x0 0x0 0x0 0x0 0x0>;
+ ti,pwron-active-high;
};
};
diff --git a/Documentation/devicetree/bindings/usb/ti,usb8020b.yaml b/Documentation/devicetree/bindings/usb/ti,usb8020b.yaml
index 8ef117793e11..61217da8b2f3 100644
--- a/Documentation/devicetree/bindings/usb/ti,usb8020b.yaml
+++ b/Documentation/devicetree/bindings/usb/ti,usb8020b.yaml
@@ -51,19 +51,19 @@ examples:
/* 2.0 hub on port 1 */
hub_2_0: hub@1 {
- compatible = "usb451,8027";
- reg = <1>;
- peer-hub = <&hub_3_0>;
- reset-gpios = <&pio 7 GPIO_ACTIVE_HIGH>;
- vdd-supply = <&usb_hub_fixed_3v3>;
+ compatible = "usb451,8027";
+ reg = <1>;
+ peer-hub = <&hub_3_0>;
+ reset-gpios = <&pio 7 GPIO_ACTIVE_HIGH>;
+ vdd-supply = <&usb_hub_fixed_3v3>;
};
/* 3.0 hub on port 2 */
hub_3_0: hub@2 {
- compatible = "usb451,8025";
- reg = <2>;
- peer-hub = <&hub_2_0>;
- reset-gpios = <&pio 7 GPIO_ACTIVE_HIGH>;
- vdd-supply = <&usb_hub_fixed_3v3>;
+ compatible = "usb451,8025";
+ reg = <2>;
+ peer-hub = <&hub_2_0>;
+ reset-gpios = <&pio 7 GPIO_ACTIVE_HIGH>;
+ vdd-supply = <&usb_hub_fixed_3v3>;
};
};
diff --git a/Documentation/devicetree/bindings/usb/ti,usb8041.yaml b/Documentation/devicetree/bindings/usb/ti,usb8041.yaml
index c2e29bd61e11..bce730a5e237 100644
--- a/Documentation/devicetree/bindings/usb/ti,usb8041.yaml
+++ b/Documentation/devicetree/bindings/usb/ti,usb8041.yaml
@@ -51,17 +51,17 @@ examples:
/* 2.0 hub on port 1 */
hub_2_0: hub@1 {
- compatible = "usb451,8142";
- reg = <1>;
- peer-hub = <&hub_3_0>;
- reset-gpios = <&gpio1 11 GPIO_ACTIVE_LOW>;
+ compatible = "usb451,8142";
+ reg = <1>;
+ peer-hub = <&hub_3_0>;
+ reset-gpios = <&gpio1 11 GPIO_ACTIVE_LOW>;
};
/* 3.0 hub on port 2 */
hub_3_0: hub@2 {
- compatible = "usb451,8140";
- reg = <2>;
- peer-hub = <&hub_2_0>;
- reset-gpios = <&gpio1 11 GPIO_ACTIVE_LOW>;
+ compatible = "usb451,8140";
+ reg = <2>;
+ peer-hub = <&hub_2_0>;
+ reset-gpios = <&gpio1 11 GPIO_ACTIVE_LOW>;
};
};
--
2.43.0
^ permalink raw reply related [flat|nested] 10+ messages in thread* Re: [PATCH] dt-bindings: usb: Correct indentation and style in DTS example
2025-01-07 13:10 [PATCH] dt-bindings: usb: Correct indentation and style in DTS example Krzysztof Kozlowski
@ 2025-01-07 17:26 ` Florian Fainelli
2025-01-07 17:33 ` Francesco Dolcini
` (7 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Florian Fainelli @ 2025-01-07 17:26 UTC (permalink / raw)
To: Krzysztof Kozlowski, Greg Kroah-Hartman, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Joel Stanley, Andrew Jeffery,
Justin Chen, Al Cooper, Broadcom internal kernel review list,
Thierry Reding, Jonathan Hunter, Geert Uytterhoeven, Magnus Damm,
Benjamin Herrenschmidt, Florian Fainelli, Benjamin Bara,
Bjorn Andersson, Ahmad Zainie, Del Regno, Tianping Fang,
Jassi Brar, Biju Das, Yoshihiro Shimoda, Francesco Dolcini,
Macpaul Lin, Alexander Stein, linux-usb, devicetree,
linux-arm-kernel, linux-aspeed, linux-kernel, linux-tegra,
linux-renesas-soc
On 1/7/25 05:10, Krzysztof Kozlowski wrote:
> 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@linaro.org>
> ---
> .../bindings/usb/aspeed,usb-vhub.yaml | 40 +++++++++----------
> .../devicetree/bindings/usb/brcm,bdc.yaml | 14 +++----
Acked-by: Florian Fainelli <florian.fainelli@broadcom.com> # brcm,bdc.yaml
--
Florian
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] dt-bindings: usb: Correct indentation and style in DTS example
2025-01-07 13:10 [PATCH] dt-bindings: usb: Correct indentation and style in DTS example Krzysztof Kozlowski
2025-01-07 17:26 ` Florian Fainelli
@ 2025-01-07 17:33 ` Francesco Dolcini
2025-01-08 0:35 ` Andrew Jeffery
` (6 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Francesco Dolcini @ 2025-01-07 17:33 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Greg Kroah-Hartman, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Joel Stanley, Andrew Jeffery, Justin Chen,
Al Cooper, Broadcom internal kernel review list, Thierry Reding,
Jonathan Hunter, Geert Uytterhoeven, Magnus Damm,
Benjamin Herrenschmidt, Florian Fainelli, Benjamin Bara,
Bjorn Andersson, Ahmad Zainie, Del Regno, Tianping Fang,
Jassi Brar, Biju Das, Yoshihiro Shimoda, Francesco Dolcini,
Macpaul Lin, Alexander Stein, linux-usb, devicetree,
linux-arm-kernel, linux-aspeed, linux-kernel, linux-tegra,
linux-renesas-soc
On Tue, Jan 07, 2025 at 02:10:13PM +0100, Krzysztof Kozlowski wrote:
> 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@linaro.org>
Acked-by: Francesco Dolcini <francesco.dolcini@toradex.com> # ti,tusb73x0-pci.yaml
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] dt-bindings: usb: Correct indentation and style in DTS example
2025-01-07 13:10 [PATCH] dt-bindings: usb: Correct indentation and style in DTS example Krzysztof Kozlowski
2025-01-07 17:26 ` Florian Fainelli
2025-01-07 17:33 ` Francesco Dolcini
@ 2025-01-08 0:35 ` Andrew Jeffery
2025-01-08 6:41 ` Benjamin Bara - SKIDATA
` (5 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Andrew Jeffery @ 2025-01-08 0:35 UTC (permalink / raw)
To: Krzysztof Kozlowski, Greg Kroah-Hartman, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Joel Stanley, Justin Chen,
Al Cooper, Broadcom internal kernel review list, Thierry Reding,
Jonathan Hunter, Geert Uytterhoeven, Magnus Damm,
Benjamin Herrenschmidt, Florian Fainelli, Benjamin Bara,
Bjorn Andersson, Ahmad Zainie, Del Regno, Tianping Fang,
Jassi Brar, Biju Das, Yoshihiro Shimoda, Francesco Dolcini,
Macpaul Lin, Alexander Stein, linux-usb, devicetree,
linux-arm-kernel, linux-aspeed, linux-kernel, linux-tegra,
linux-renesas-soc
On Tue, 2025-01-07 at 14:10 +0100, Krzysztof Kozlowski wrote:
> 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@linaro.org>
> ---
> .../bindings/usb/aspeed,usb-vhub.yaml | 40 +++++++++--------
> --
Acked-by: Andrew Jeffery <andrew@codeconstruct.com.au> # aspeed,usb-vhub.yaml
Cheers,
Andrew
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] dt-bindings: usb: Correct indentation and style in DTS example
2025-01-07 13:10 [PATCH] dt-bindings: usb: Correct indentation and style in DTS example Krzysztof Kozlowski
` (2 preceding siblings ...)
2025-01-08 0:35 ` Andrew Jeffery
@ 2025-01-08 6:41 ` Benjamin Bara - SKIDATA
2025-01-08 12:49 ` Geert Uytterhoeven
` (4 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Benjamin Bara - SKIDATA @ 2025-01-08 6:41 UTC (permalink / raw)
To: magnus.damm@gmail.com, krzysztof.kozlowski@linaro.org,
benh@kernel.crashing.org, conor+dt@kernel.org,
alcooperx@gmail.com, yoshihiro.shimoda.uh@renesas.com,
angelogioacchino.delregno@collabora.com,
andrew@codeconstruct.com.au, justin.chen@broadcom.com,
devicetree@vger.kernel.org, jonathanh@nvidia.com,
linux-kernel@vger.kernel.org, linux-renesas-soc@vger.kernel.org,
linux-usb@vger.kernel.org, macpaul.lin@mediatek.com,
alexander.stein@ew.tq-group.com, f.fainelli@gmail.com,
krzk+dt@kernel.org, andersson@kernel.org, geert+renesas@glider.be,
tianping.fang@mediatek.com, thierry.reding@gmail.com,
joel@jms.id.au, jaswinder.singh@linaro.org,
gregkh@linuxfoundation.org, linux-aspeed@lists.ozlabs.org,
robh@kernel.org, linux-arm-kernel@lists.infradead.org,
biju.das.jz@bp.renesas.com, linux-tegra@vger.kernel.org,
bcm-kernel-feedback-list@broadcom.com,
wan.ahmad.zainie.wan.mohamad@intel.com,
francesco.dolcini@toradex.com
On Tue, 2025-01-07 at 14:10 +0100, Krzysztof Kozlowski wrote:
> 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@linaro.org>
Acked-by: Benjamin Bara <benjamin.bara@skidata.com> # cypress,hx3.yaml
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] dt-bindings: usb: Correct indentation and style in DTS example
2025-01-07 13:10 [PATCH] dt-bindings: usb: Correct indentation and style in DTS example Krzysztof Kozlowski
` (3 preceding siblings ...)
2025-01-08 6:41 ` Benjamin Bara - SKIDATA
@ 2025-01-08 12:49 ` Geert Uytterhoeven
2025-01-08 16:38 ` Thierry Reding
` (3 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Geert Uytterhoeven @ 2025-01-08 12:49 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Greg Kroah-Hartman, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Joel Stanley, Andrew Jeffery, Justin Chen,
Al Cooper, Broadcom internal kernel review list, Thierry Reding,
Jonathan Hunter, Geert Uytterhoeven, Magnus Damm,
Benjamin Herrenschmidt, Florian Fainelli, Benjamin Bara,
Bjorn Andersson, Ahmad Zainie, Del Regno, Tianping Fang,
Jassi Brar, Biju Das, Yoshihiro Shimoda, Francesco Dolcini,
Macpaul Lin, Alexander Stein, linux-usb, devicetree,
linux-arm-kernel, linux-aspeed, linux-kernel, linux-tegra,
linux-renesas-soc
On Tue, Jan 7, 2025 at 2:10 PM Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:
> 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@linaro.org>
> .../bindings/usb/renesas,rzv2m-usb3drd.yaml | 36 ++++++++---------
> .../bindings/usb/renesas,usb3-peri.yaml | 24 +++++------
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> # renesas
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
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
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] dt-bindings: usb: Correct indentation and style in DTS example
2025-01-07 13:10 [PATCH] dt-bindings: usb: Correct indentation and style in DTS example Krzysztof Kozlowski
` (4 preceding siblings ...)
2025-01-08 12:49 ` Geert Uytterhoeven
@ 2025-01-08 16:38 ` Thierry Reding
2025-01-09 11:19 ` AngeloGioacchino Del Regno
` (2 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Thierry Reding @ 2025-01-08 16:38 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Greg Kroah-Hartman, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Joel Stanley, Andrew Jeffery, Justin Chen,
Al Cooper, Broadcom internal kernel review list, Jonathan Hunter,
Geert Uytterhoeven, Magnus Damm, Benjamin Herrenschmidt,
Florian Fainelli, Benjamin Bara, Bjorn Andersson, Ahmad Zainie,
Del Regno, Tianping Fang, Jassi Brar, Biju Das, Yoshihiro Shimoda,
Francesco Dolcini, Macpaul Lin, Alexander Stein, linux-usb,
devicetree, linux-arm-kernel, linux-aspeed, linux-kernel,
linux-tegra, linux-renesas-soc
[-- Attachment #1: Type: text/plain, Size: 1562 bytes --]
On Tue, Jan 07, 2025 at 02:10:13PM +0100, Krzysztof Kozlowski wrote:
> 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@linaro.org>
> ---
> .../bindings/usb/aspeed,usb-vhub.yaml | 40 +++++++++----------
> .../devicetree/bindings/usb/brcm,bdc.yaml | 14 +++----
> .../devicetree/bindings/usb/cypress,hx3.yaml | 24 +++++------
> .../devicetree/bindings/usb/dwc2.yaml | 4 +-
> .../devicetree/bindings/usb/fcs,fsa4480.yaml | 20 +++++-----
> .../bindings/usb/intel,keembay-dwc3.yaml | 30 +++++++-------
> .../devicetree/bindings/usb/ite,it5205.yaml | 18 ++++-----
> .../bindings/usb/maxim,max3420-udc.yaml | 28 ++++++-------
> .../bindings/usb/nvidia,tegra210-xusb.yaml | 4 +-
> .../bindings/usb/renesas,rzv2m-usb3drd.yaml | 36 ++++++++---------
> .../bindings/usb/renesas,usb3-peri.yaml | 24 +++++------
> .../devicetree/bindings/usb/ti,hd3ss3220.yaml | 38 +++++++++---------
> .../bindings/usb/ti,tusb73x0-pci.yaml | 6 +--
> .../devicetree/bindings/usb/ti,usb8020b.yaml | 20 +++++-----
> .../devicetree/bindings/usb/ti,usb8041.yaml | 16 ++++----
> 15 files changed, 161 insertions(+), 161 deletions(-)
Acked-by: Thierry Reding <treding@nvidia.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] dt-bindings: usb: Correct indentation and style in DTS example
2025-01-07 13:10 [PATCH] dt-bindings: usb: Correct indentation and style in DTS example Krzysztof Kozlowski
` (5 preceding siblings ...)
2025-01-08 16:38 ` Thierry Reding
@ 2025-01-09 11:19 ` AngeloGioacchino Del Regno
2025-01-10 15:45 ` Rob Herring (Arm)
2025-01-12 17:11 ` Jassi Brar
8 siblings, 0 replies; 10+ messages in thread
From: AngeloGioacchino Del Regno @ 2025-01-09 11:19 UTC (permalink / raw)
To: Krzysztof Kozlowski, Greg Kroah-Hartman, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Joel Stanley, Andrew Jeffery,
Justin Chen, Al Cooper, Broadcom internal kernel review list,
Thierry Reding, Jonathan Hunter, Geert Uytterhoeven, Magnus Damm,
Benjamin Herrenschmidt, Florian Fainelli, Benjamin Bara,
Bjorn Andersson, Ahmad Zainie, Tianping Fang, Jassi Brar,
Biju Das, Yoshihiro Shimoda, Francesco Dolcini, Macpaul Lin,
Alexander Stein, linux-usb, devicetree, linux-arm-kernel,
linux-aspeed, linux-kernel, linux-tegra, linux-renesas-soc
Il 07/01/25 14:10, Krzysztof Kozlowski ha scritto:
> 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@linaro.org>
For iTE IT5205:
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] dt-bindings: usb: Correct indentation and style in DTS example
2025-01-07 13:10 [PATCH] dt-bindings: usb: Correct indentation and style in DTS example Krzysztof Kozlowski
` (6 preceding siblings ...)
2025-01-09 11:19 ` AngeloGioacchino Del Regno
@ 2025-01-10 15:45 ` Rob Herring (Arm)
2025-01-12 17:11 ` Jassi Brar
8 siblings, 0 replies; 10+ messages in thread
From: Rob Herring (Arm) @ 2025-01-10 15:45 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Jassi Brar, Andrew Jeffery, Tianping Fang, Biju Das,
Benjamin Bara, linux-aspeed, Yoshihiro Shimoda, linux-kernel,
Al Cooper, Magnus Damm, Conor Dooley,
Broadcom internal kernel review list, Alexander Stein,
Justin Chen, linux-renesas-soc, Geert Uytterhoeven,
Jonathan Hunter, Thierry Reding, Francesco Dolcini,
Benjamin Herrenschmidt, linux-tegra, Bjorn Andersson, linux-usb,
linux-arm-kernel, devicetree, Ahmad Zainie, Del Regno,
Florian Fainelli, Greg Kroah-Hartman, Krzysztof Kozlowski,
Joel Stanley, Macpaul Lin
On Tue, 07 Jan 2025 14:10:13 +0100, Krzysztof Kozlowski wrote:
> 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@linaro.org>
> ---
> .../bindings/usb/aspeed,usb-vhub.yaml | 40 +++++++++----------
> .../devicetree/bindings/usb/brcm,bdc.yaml | 14 +++----
> .../devicetree/bindings/usb/cypress,hx3.yaml | 24 +++++------
> .../devicetree/bindings/usb/dwc2.yaml | 4 +-
> .../devicetree/bindings/usb/fcs,fsa4480.yaml | 20 +++++-----
> .../bindings/usb/intel,keembay-dwc3.yaml | 30 +++++++-------
> .../devicetree/bindings/usb/ite,it5205.yaml | 18 ++++-----
> .../bindings/usb/maxim,max3420-udc.yaml | 28 ++++++-------
> .../bindings/usb/nvidia,tegra210-xusb.yaml | 4 +-
> .../bindings/usb/renesas,rzv2m-usb3drd.yaml | 36 ++++++++---------
> .../bindings/usb/renesas,usb3-peri.yaml | 24 +++++------
> .../devicetree/bindings/usb/ti,hd3ss3220.yaml | 38 +++++++++---------
> .../bindings/usb/ti,tusb73x0-pci.yaml | 6 +--
> .../devicetree/bindings/usb/ti,usb8020b.yaml | 20 +++++-----
> .../devicetree/bindings/usb/ti,usb8041.yaml | 16 ++++----
> 15 files changed, 161 insertions(+), 161 deletions(-)
>
Acked-by: Rob Herring (Arm) <robh@kernel.org>
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] dt-bindings: usb: Correct indentation and style in DTS example
2025-01-07 13:10 [PATCH] dt-bindings: usb: Correct indentation and style in DTS example Krzysztof Kozlowski
` (7 preceding siblings ...)
2025-01-10 15:45 ` Rob Herring (Arm)
@ 2025-01-12 17:11 ` Jassi Brar
8 siblings, 0 replies; 10+ messages in thread
From: Jassi Brar @ 2025-01-12 17:11 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Greg Kroah-Hartman, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Joel Stanley, Andrew Jeffery, Justin Chen,
Al Cooper, Broadcom internal kernel review list, Thierry Reding,
Jonathan Hunter, Geert Uytterhoeven, Magnus Damm,
Benjamin Herrenschmidt, Florian Fainelli, Benjamin Bara,
Bjorn Andersson, Ahmad Zainie, Del Regno, Tianping Fang,
Jassi Brar, Biju Das, Yoshihiro Shimoda, Francesco Dolcini,
Macpaul Lin, Alexander Stein, linux-usb, devicetree,
linux-arm-kernel, linux-aspeed, linux-kernel, linux-tegra,
linux-renesas-soc
On Tue, Jan 7, 2025 at 7:10 AM Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:
>
> 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@linaro.org>
> ---
> .../bindings/usb/aspeed,usb-vhub.yaml | 40 +++++++++----------
> .../devicetree/bindings/usb/brcm,bdc.yaml | 14 +++----
> .../devicetree/bindings/usb/cypress,hx3.yaml | 24 +++++------
> .../devicetree/bindings/usb/dwc2.yaml | 4 +-
> .../devicetree/bindings/usb/fcs,fsa4480.yaml | 20 +++++-----
> .../bindings/usb/intel,keembay-dwc3.yaml | 30 +++++++-------
> .../devicetree/bindings/usb/ite,it5205.yaml | 18 ++++-----
> .../bindings/usb/maxim,max3420-udc.yaml | 28 ++++++-------
Acked-by: Jassi Brar <jassisinghbrar@gmail.com>
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2025-01-12 17:33 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-01-07 13:10 [PATCH] dt-bindings: usb: Correct indentation and style in DTS example Krzysztof Kozlowski
2025-01-07 17:26 ` Florian Fainelli
2025-01-07 17:33 ` Francesco Dolcini
2025-01-08 0:35 ` Andrew Jeffery
2025-01-08 6:41 ` Benjamin Bara - SKIDATA
2025-01-08 12:49 ` Geert Uytterhoeven
2025-01-08 16:38 ` Thierry Reding
2025-01-09 11:19 ` AngeloGioacchino Del Regno
2025-01-10 15:45 ` Rob Herring (Arm)
2025-01-12 17:11 ` Jassi Brar
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).