* [PATCH 0/2] ARM: dts: keystone-k2g: Fix usb-nop-xceiv dtbs_check warnings
@ 2026-09-10 20:52 Bhargav Joshi
2026-09-10 20:52 ` [PATCH 1/2] ARM: dts: keystone-k2g: Add required #phy-cells to usb-nop-xceiv nodes Bhargav Joshi
2026-09-10 20:52 ` [PATCH 2/2] ARM: dts: keystone-k2g: Move usb-phy nodes out of simple-bus Bhargav Joshi
0 siblings, 2 replies; 3+ messages in thread
From: Bhargav Joshi @ 2026-09-10 20:52 UTC (permalink / raw)
To: Nishanth Menon, Santosh Shilimkar, Rob Herring,
Krzysztof Kozlowski, Conor Dooley
Cc: linux-arm-kernel, devicetree, linux-kernel, j.bhargav.u
usb0_phy and usb1_phy fail dtbs_check because:
- missing the required #phy-cells.
- simple-bus wrapper that forces unnecessary reg.
Patch 1 adds #phy-cells with value 0 as required by binding.
Patch 2 drops the wrapper and moves the PHYs to root.
Signed-off-by: Bhargav Joshi <j.bhargav.u@gmail.com>
---
Bhargav Joshi (2):
ARM: dts: keystone-k2g: Add required #phy-cells to usb-nop-xceiv nodes
ARM: dts: keystone-k2g: Move usb-phy nodes out of simple-bus
arch/arm/boot/dts/ti/keystone/keystone-k2g.dtsi | 24 +++++++++---------------
1 file changed, 9 insertions(+), 15 deletions(-)
---
base-commit: 50d05c7c76c96b90462f24debacca971d2e86713
change-id: 20260911-ti-keystone-usb-phy-fix-12deee938735
Best regards,
--
Bhargav
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH 1/2] ARM: dts: keystone-k2g: Add required #phy-cells to usb-nop-xceiv nodes
2026-09-10 20:52 [PATCH 0/2] ARM: dts: keystone-k2g: Fix usb-nop-xceiv dtbs_check warnings Bhargav Joshi
@ 2026-09-10 20:52 ` Bhargav Joshi
2026-09-10 20:52 ` [PATCH 2/2] ARM: dts: keystone-k2g: Move usb-phy nodes out of simple-bus Bhargav Joshi
1 sibling, 0 replies; 3+ messages in thread
From: Bhargav Joshi @ 2026-09-10 20:52 UTC (permalink / raw)
To: Nishanth Menon, Santosh Shilimkar, Rob Herring,
Krzysztof Kozlowski, Conor Dooley
Cc: linux-arm-kernel, devicetree, linux-kernel, j.bhargav.u
The usb-nop-xceiv binding requires "#phy-cells" with a value of 0.
usb0_phy and usb1_phy are missing it. Add the required property.
Signed-off-by: Bhargav Joshi <j.bhargav.u@gmail.com>
---
arch/arm/boot/dts/ti/keystone/keystone-k2g.dtsi | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm/boot/dts/ti/keystone/keystone-k2g.dtsi b/arch/arm/boot/dts/ti/keystone/keystone-k2g.dtsi
index dafe485dfe19..5a63874090eb 100644
--- a/arch/arm/boot/dts/ti/keystone/keystone-k2g.dtsi
+++ b/arch/arm/boot/dts/ti/keystone/keystone-k2g.dtsi
@@ -77,12 +77,14 @@ usbphy {
usb0_phy: usb-phy@0 {
compatible = "usb-nop-xceiv";
reg = <0>;
+ #phy-cells = <0>;
status = "disabled";
};
usb1_phy: usb-phy@1 {
compatible = "usb-nop-xceiv";
reg = <1>;
+ #phy-cells = <0>;
status = "disabled";
};
};
--
2.55.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH 2/2] ARM: dts: keystone-k2g: Move usb-phy nodes out of simple-bus
2026-09-10 20:52 [PATCH 0/2] ARM: dts: keystone-k2g: Fix usb-nop-xceiv dtbs_check warnings Bhargav Joshi
2026-09-10 20:52 ` [PATCH 1/2] ARM: dts: keystone-k2g: Add required #phy-cells to usb-nop-xceiv nodes Bhargav Joshi
@ 2026-09-10 20:52 ` Bhargav Joshi
1 sibling, 0 replies; 3+ messages in thread
From: Bhargav Joshi @ 2026-09-10 20:52 UTC (permalink / raw)
To: Nishanth Menon, Santosh Shilimkar, Rob Herring,
Krzysztof Kozlowski, Conor Dooley
Cc: linux-arm-kernel, devicetree, linux-kernel, j.bhargav.u
usb0_phy and usb1_phy are usb-nop-xceiv NOP PHYs with no registers. They
sit under a usbphy simple-bus wrapper with unit-address names, which
forces a meaningless reg.
Drop the wrapper, reg and unit-addresses, move the PHYs to root with
plain unique names to fix dtbs_check errors.
Signed-off-by: Bhargav Joshi <j.bhargav.u@gmail.com>
---
arch/arm/boot/dts/ti/keystone/keystone-k2g.dtsi | 26 +++++++++----------------
1 file changed, 9 insertions(+), 17 deletions(-)
diff --git a/arch/arm/boot/dts/ti/keystone/keystone-k2g.dtsi b/arch/arm/boot/dts/ti/keystone/keystone-k2g.dtsi
index 5a63874090eb..c18bec46d9bf 100644
--- a/arch/arm/boot/dts/ti/keystone/keystone-k2g.dtsi
+++ b/arch/arm/boot/dts/ti/keystone/keystone-k2g.dtsi
@@ -69,24 +69,16 @@ pmu {
interrupts = <GIC_SPI 4 IRQ_TYPE_EDGE_RISING>;
};
- usbphy {
- #address-cells = <1>;
- #size-cells = <0>;
- compatible = "simple-bus";
-
- usb0_phy: usb-phy@0 {
- compatible = "usb-nop-xceiv";
- reg = <0>;
- #phy-cells = <0>;
- status = "disabled";
- };
+ usb0_phy: usb0-phy {
+ compatible = "usb-nop-xceiv";
+ #phy-cells = <0>;
+ status = "disabled";
+ };
- usb1_phy: usb-phy@1 {
- compatible = "usb-nop-xceiv";
- reg = <1>;
- #phy-cells = <0>;
- status = "disabled";
- };
+ usb1_phy: usb1-phy {
+ compatible = "usb-nop-xceiv";
+ #phy-cells = <0>;
+ status = "disabled";
};
soc0: soc@0 {
--
2.55.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2026-09-10 20:53 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-09-10 20:52 [PATCH 0/2] ARM: dts: keystone-k2g: Fix usb-nop-xceiv dtbs_check warnings Bhargav Joshi
2026-09-10 20:52 ` [PATCH 1/2] ARM: dts: keystone-k2g: Add required #phy-cells to usb-nop-xceiv nodes Bhargav Joshi
2026-09-10 20:52 ` [PATCH 2/2] ARM: dts: keystone-k2g: Move usb-phy nodes out of simple-bus Bhargav Joshi
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox