* [PATCH] dt-bindings: phy: Convert ti,keystone-usbphy to DT schema
@ 2025-06-07 21:26 ` Rob Herring (Arm)
0 siblings, 0 replies; 3+ messages in thread
From: Rob Herring (Arm) @ 2025-06-07 21:26 UTC (permalink / raw)
To: Vinod Koul, Kishon Vijay Abraham I, Krzysztof Kozlowski,
Conor Dooley, Nishanth Menon, Santosh Shilimkar
Cc: linux-phy, devicetree, linux-kernel
Convert the TI Keystone USB PHY binding to DT schema format. Drop the
"#address-cells" and "#size-cells" properties which don't make sense
without any child nodes.
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
.../bindings/phy/keystone-usb-phy.txt | 19 ----------
.../bindings/phy/ti,keystone-usbphy.yaml | 37 +++++++++++++++++++
2 files changed, 37 insertions(+), 19 deletions(-)
delete mode 100644 Documentation/devicetree/bindings/phy/keystone-usb-phy.txt
create mode 100644 Documentation/devicetree/bindings/phy/ti,keystone-usbphy.yaml
diff --git a/Documentation/devicetree/bindings/phy/keystone-usb-phy.txt b/Documentation/devicetree/bindings/phy/keystone-usb-phy.txt
deleted file mode 100644
index 300830dda0bf..000000000000
--- a/Documentation/devicetree/bindings/phy/keystone-usb-phy.txt
+++ /dev/null
@@ -1,19 +0,0 @@
-TI Keystone USB PHY
-
-Required properties:
- - compatible: should be "ti,keystone-usbphy".
- - #address-cells, #size-cells : should be '1' if the device has sub-nodes
- with 'reg' property.
- - reg : Address and length of the usb phy control register set.
-
-The main purpose of this PHY driver is to enable the USB PHY reference clock
-gate on the Keystone SOC for both the USB2 and USB3 PHY. Otherwise it is just
-an NOP PHY driver. Hence this node is referenced as both the usb2 and usb3
-phy node in the USB Glue layer driver node.
-
-usb_phy: usb_phy@2620738 {
- compatible = "ti,keystone-usbphy";
- #address-cells = <1>;
- #size-cells = <1>;
- reg = <0x2620738 32>;
-};
diff --git a/Documentation/devicetree/bindings/phy/ti,keystone-usbphy.yaml b/Documentation/devicetree/bindings/phy/ti,keystone-usbphy.yaml
new file mode 100644
index 000000000000..08dc18e7feea
--- /dev/null
+++ b/Documentation/devicetree/bindings/phy/ti,keystone-usbphy.yaml
@@ -0,0 +1,37 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/phy/ti,keystone-usbphy.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: TI Keystone USB PHY
+
+maintainers:
+ - Nishanth Menon <nm@ti.com>
+ - Santosh Shilimkar <ssantosh@kernel.org>
+
+description:
+ The main purpose of this PHY driver is to enable the USB PHY reference clock
+ gate on the Keystone SOC for both the USB2 and USB3 PHY. Otherwise it is just
+ an NOP PHY driver. Hence this node is referenced as both the usb2 and usb3
+ phy node in the USB Glue layer driver node.
+
+properties:
+ compatible:
+ const: ti,keystone-usbphy
+
+ reg:
+ maxItems: 1
+
+required:
+ - compatible
+ - reg
+
+additionalProperties: false
+
+examples:
+ - |
+ usb-phy@2620738 {
+ compatible = "ti,keystone-usbphy";
+ reg = <0x2620738 32>;
+ };
--
2.47.2
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH] dt-bindings: phy: Convert ti,keystone-usbphy to DT schema
@ 2025-06-07 21:26 ` Rob Herring (Arm)
0 siblings, 0 replies; 3+ messages in thread
From: Rob Herring (Arm) @ 2025-06-07 21:26 UTC (permalink / raw)
To: Vinod Koul, Kishon Vijay Abraham I, Krzysztof Kozlowski,
Conor Dooley, Nishanth Menon, Santosh Shilimkar
Cc: linux-phy, devicetree, linux-kernel
Convert the TI Keystone USB PHY binding to DT schema format. Drop the
"#address-cells" and "#size-cells" properties which don't make sense
without any child nodes.
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
.../bindings/phy/keystone-usb-phy.txt | 19 ----------
.../bindings/phy/ti,keystone-usbphy.yaml | 37 +++++++++++++++++++
2 files changed, 37 insertions(+), 19 deletions(-)
delete mode 100644 Documentation/devicetree/bindings/phy/keystone-usb-phy.txt
create mode 100644 Documentation/devicetree/bindings/phy/ti,keystone-usbphy.yaml
diff --git a/Documentation/devicetree/bindings/phy/keystone-usb-phy.txt b/Documentation/devicetree/bindings/phy/keystone-usb-phy.txt
deleted file mode 100644
index 300830dda0bf..000000000000
--- a/Documentation/devicetree/bindings/phy/keystone-usb-phy.txt
+++ /dev/null
@@ -1,19 +0,0 @@
-TI Keystone USB PHY
-
-Required properties:
- - compatible: should be "ti,keystone-usbphy".
- - #address-cells, #size-cells : should be '1' if the device has sub-nodes
- with 'reg' property.
- - reg : Address and length of the usb phy control register set.
-
-The main purpose of this PHY driver is to enable the USB PHY reference clock
-gate on the Keystone SOC for both the USB2 and USB3 PHY. Otherwise it is just
-an NOP PHY driver. Hence this node is referenced as both the usb2 and usb3
-phy node in the USB Glue layer driver node.
-
-usb_phy: usb_phy@2620738 {
- compatible = "ti,keystone-usbphy";
- #address-cells = <1>;
- #size-cells = <1>;
- reg = <0x2620738 32>;
-};
diff --git a/Documentation/devicetree/bindings/phy/ti,keystone-usbphy.yaml b/Documentation/devicetree/bindings/phy/ti,keystone-usbphy.yaml
new file mode 100644
index 000000000000..08dc18e7feea
--- /dev/null
+++ b/Documentation/devicetree/bindings/phy/ti,keystone-usbphy.yaml
@@ -0,0 +1,37 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/phy/ti,keystone-usbphy.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: TI Keystone USB PHY
+
+maintainers:
+ - Nishanth Menon <nm@ti.com>
+ - Santosh Shilimkar <ssantosh@kernel.org>
+
+description:
+ The main purpose of this PHY driver is to enable the USB PHY reference clock
+ gate on the Keystone SOC for both the USB2 and USB3 PHY. Otherwise it is just
+ an NOP PHY driver. Hence this node is referenced as both the usb2 and usb3
+ phy node in the USB Glue layer driver node.
+
+properties:
+ compatible:
+ const: ti,keystone-usbphy
+
+ reg:
+ maxItems: 1
+
+required:
+ - compatible
+ - reg
+
+additionalProperties: false
+
+examples:
+ - |
+ usb-phy@2620738 {
+ compatible = "ti,keystone-usbphy";
+ reg = <0x2620738 32>;
+ };
--
2.47.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] dt-bindings: phy: Convert ti,keystone-usbphy to DT schema
@ 2025-06-08 17:06 kernel test robot
0 siblings, 0 replies; 3+ messages in thread
From: kernel test robot @ 2025-06-08 17:06 UTC (permalink / raw)
To: oe-kbuild; +Cc: lkp
::::::
:::::: Manual check reason: "dtcheck: binding changes may go via different trees"
::::::
BCC: lkp@intel.com
CC: oe-kbuild-all@lists.linux.dev
In-Reply-To: <20250607212641.744683-1-robh@kernel.org>
References: <20250607212641.744683-1-robh@kernel.org>
TO: "Rob Herring (Arm)" <robh@kernel.org>
Hi Rob,
kernel test robot noticed the following build warnings:
[auto build test WARNING on robh/for-next]
[also build test WARNING on linus/master v6.15 next-20250606]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Rob-Herring-Arm/dt-bindings-phy-Convert-ti-keystone-usbphy-to-DT-schema/20250608-053412
base: https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
patch link: https://lore.kernel.org/r/20250607212641.744683-1-robh%40kernel.org
patch subject: [PATCH] dt-bindings: phy: Convert ti,keystone-usbphy to DT schema
:::::: branch date: 19 hours ago
:::::: commit date: 19 hours ago
config: arm-randconfig-053-20250608 (https://download.01.org/0day-ci/archive/20250609/202506090046.r16mryyj-lkp@intel.com/config)
compiler: arm-linux-gnueabi-gcc (GCC) 10.5.0
dtschema version: 2025.3.dev27+g32749b3
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250609/202506090046.r16mryyj-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/r/202506090046.r16mryyj-lkp@intel.com/
dtcheck warnings: (new ones prefixed by >>)
arch/arm/boot/dts/ti/keystone/keystone-k2hk-evm.dtb: /soc@0/clocks/clkxge@23500c8: failed to match any schema with compatible: ['ti,keystone,psc-clock']
arch/arm/boot/dts/ti/keystone/keystone-k2hk-evm.dtb: /soc@0/i2c@2530000: failed to match any schema with compatible: ['ti,davinci-i2c']
arch/arm/boot/dts/ti/keystone/keystone-k2hk-evm.dtb: /soc@0/i2c@2530400: failed to match any schema with compatible: ['ti,davinci-i2c']
arch/arm/boot/dts/ti/keystone/keystone-k2hk-evm.dtb: /soc@0/i2c@2530800: failed to match any schema with compatible: ['ti,davinci-i2c']
arch/arm/boot/dts/ti/keystone/keystone-k2hk-evm.dtb: /soc@0/spi@21000400: failed to match any schema with compatible: ['ti,keystone-spi', 'ti,dm6441-spi']
arch/arm/boot/dts/ti/keystone/keystone-k2hk-evm.dtb: /soc@0/spi@21000400: failed to match any schema with compatible: ['ti,keystone-spi', 'ti,dm6441-spi']
arch/arm/boot/dts/ti/keystone/keystone-k2hk-evm.dtb: /soc@0/spi@21000600: failed to match any schema with compatible: ['ti,keystone-spi', 'ti,dm6441-spi']
arch/arm/boot/dts/ti/keystone/keystone-k2hk-evm.dtb: /soc@0/spi@21000600: failed to match any schema with compatible: ['ti,keystone-spi', 'ti,dm6441-spi']
arch/arm/boot/dts/ti/keystone/keystone-k2hk-evm.dtb: /soc@0/spi@21000800: failed to match any schema with compatible: ['ti,keystone-spi', 'ti,dm6441-spi']
arch/arm/boot/dts/ti/keystone/keystone-k2hk-evm.dtb: /soc@0/spi@21000800: failed to match any schema with compatible: ['ti,keystone-spi', 'ti,dm6441-spi']
>> arch/arm/boot/dts/ti/keystone/keystone-k2hk-evm.dtb: usb_phy@2620738 (ti,keystone-usbphy): '#address-cells', '#size-cells' do not match any of the regexes: '^pinctrl-[0-9]+$'
from schema $id: http://devicetree.org/schemas/phy/ti,keystone-usbphy.yaml#
arch/arm/boot/dts/ti/keystone/keystone-k2hk-evm.dtb: usb@2680000 (ti,keystone-dwc3): 'clock-names' does not match any of the regexes: '^pinctrl-[0-9]+$', 'usb@[a-f0-9]+$'
from schema $id: http://devicetree.org/schemas/usb/ti,keystone-dwc3.yaml#
arch/arm/boot/dts/ti/keystone/keystone-k2hk-evm.dtb: wdt@22f0080 (ti,keystone-wdt): $nodename:0: 'wdt@22f0080' does not match '^(timer|watchdog)(@.*|-([0-9]|[1-9][0-9]+))?$'
from schema $id: http://devicetree.org/schemas/watchdog/ti,davinci-wdt.yaml#
arch/arm/boot/dts/ti/keystone/keystone-k2hk-evm.dtb: /soc@0/timer@22f0000: failed to match any schema with compatible: ['ti,keystone-timer']
arch/arm/boot/dts/ti/keystone/keystone-k2hk-evm.dtb: /soc@0/aemif@21000a00: failed to match any schema with compatible: ['ti,keystone-aemif', 'ti,davinci-aemif']
arch/arm/boot/dts/ti/keystone/keystone-k2hk-evm.dtb: /soc@0/aemif@21000a00: failed to match any schema with compatible: ['ti,keystone-aemif', 'ti,davinci-aemif']
arch/arm/boot/dts/ti/keystone/keystone-k2hk-evm.dtb: cs0: 'clocks' is a dependency of 'clock-ranges'
from schema $id: http://devicetree.org/schemas/clock/clock.yaml#
--
arch/arm/boot/dts/ti/keystone/keystone-k2l-evm.dtb: /soc@0/clocks/clkuart3@2350000: failed to match any schema with compatible: ['ti,keystone,psc-clock']
arch/arm/boot/dts/ti/keystone/keystone-k2l-evm.dtb: /soc@0/i2c@2530000: failed to match any schema with compatible: ['ti,davinci-i2c']
arch/arm/boot/dts/ti/keystone/keystone-k2l-evm.dtb: /soc@0/i2c@2530400: failed to match any schema with compatible: ['ti,davinci-i2c']
arch/arm/boot/dts/ti/keystone/keystone-k2l-evm.dtb: /soc@0/i2c@2530800: failed to match any schema with compatible: ['ti,davinci-i2c']
arch/arm/boot/dts/ti/keystone/keystone-k2l-evm.dtb: /soc@0/spi@21000400: failed to match any schema with compatible: ['ti,keystone-spi', 'ti,dm6441-spi']
arch/arm/boot/dts/ti/keystone/keystone-k2l-evm.dtb: /soc@0/spi@21000400: failed to match any schema with compatible: ['ti,keystone-spi', 'ti,dm6441-spi']
arch/arm/boot/dts/ti/keystone/keystone-k2l-evm.dtb: /soc@0/spi@21000600: failed to match any schema with compatible: ['ti,keystone-spi', 'ti,dm6441-spi']
arch/arm/boot/dts/ti/keystone/keystone-k2l-evm.dtb: /soc@0/spi@21000600: failed to match any schema with compatible: ['ti,keystone-spi', 'ti,dm6441-spi']
arch/arm/boot/dts/ti/keystone/keystone-k2l-evm.dtb: /soc@0/spi@21000800: failed to match any schema with compatible: ['ti,keystone-spi', 'ti,dm6441-spi']
arch/arm/boot/dts/ti/keystone/keystone-k2l-evm.dtb: /soc@0/spi@21000800: failed to match any schema with compatible: ['ti,keystone-spi', 'ti,dm6441-spi']
>> arch/arm/boot/dts/ti/keystone/keystone-k2l-evm.dtb: usb_phy@2620738 (ti,keystone-usbphy): '#address-cells', '#size-cells' do not match any of the regexes: '^pinctrl-[0-9]+$'
from schema $id: http://devicetree.org/schemas/phy/ti,keystone-usbphy.yaml#
arch/arm/boot/dts/ti/keystone/keystone-k2l-evm.dtb: usb@2680000 (ti,keystone-dwc3): 'clock-names' does not match any of the regexes: '^pinctrl-[0-9]+$', 'usb@[a-f0-9]+$'
from schema $id: http://devicetree.org/schemas/usb/ti,keystone-dwc3.yaml#
arch/arm/boot/dts/ti/keystone/keystone-k2l-evm.dtb: wdt@22f0080 (ti,keystone-wdt): $nodename:0: 'wdt@22f0080' does not match '^(timer|watchdog)(@.*|-([0-9]|[1-9][0-9]+))?$'
from schema $id: http://devicetree.org/schemas/watchdog/ti,davinci-wdt.yaml#
arch/arm/boot/dts/ti/keystone/keystone-k2l-evm.dtb: /soc@0/timer@22f0000: failed to match any schema with compatible: ['ti,keystone-timer']
arch/arm/boot/dts/ti/keystone/keystone-k2l-evm.dtb: /soc@0/aemif@21000a00: failed to match any schema with compatible: ['ti,keystone-aemif', 'ti,davinci-aemif']
arch/arm/boot/dts/ti/keystone/keystone-k2l-evm.dtb: /soc@0/aemif@21000a00: failed to match any schema with compatible: ['ti,keystone-aemif', 'ti,davinci-aemif']
arch/arm/boot/dts/ti/keystone/keystone-k2l-evm.dtb: cs0: 'clocks' is a dependency of 'clock-ranges'
from schema $id: http://devicetree.org/schemas/clock/clock.yaml#
--
arch/arm/boot/dts/ti/keystone/keystone-k2e-evm.dtb: /soc@0/clocks/clkxge@23500c8: failed to match any schema with compatible: ['ti,keystone,psc-clock']
arch/arm/boot/dts/ti/keystone/keystone-k2e-evm.dtb: /soc@0/i2c@2530000: failed to match any schema with compatible: ['ti,davinci-i2c']
arch/arm/boot/dts/ti/keystone/keystone-k2e-evm.dtb: /soc@0/i2c@2530400: failed to match any schema with compatible: ['ti,davinci-i2c']
arch/arm/boot/dts/ti/keystone/keystone-k2e-evm.dtb: /soc@0/i2c@2530800: failed to match any schema with compatible: ['ti,davinci-i2c']
arch/arm/boot/dts/ti/keystone/keystone-k2e-evm.dtb: /soc@0/spi@21000400: failed to match any schema with compatible: ['ti,keystone-spi', 'ti,dm6441-spi']
arch/arm/boot/dts/ti/keystone/keystone-k2e-evm.dtb: /soc@0/spi@21000400: failed to match any schema with compatible: ['ti,keystone-spi', 'ti,dm6441-spi']
arch/arm/boot/dts/ti/keystone/keystone-k2e-evm.dtb: /soc@0/spi@21000600: failed to match any schema with compatible: ['ti,keystone-spi', 'ti,dm6441-spi']
arch/arm/boot/dts/ti/keystone/keystone-k2e-evm.dtb: /soc@0/spi@21000600: failed to match any schema with compatible: ['ti,keystone-spi', 'ti,dm6441-spi']
arch/arm/boot/dts/ti/keystone/keystone-k2e-evm.dtb: /soc@0/spi@21000800: failed to match any schema with compatible: ['ti,keystone-spi', 'ti,dm6441-spi']
arch/arm/boot/dts/ti/keystone/keystone-k2e-evm.dtb: /soc@0/spi@21000800: failed to match any schema with compatible: ['ti,keystone-spi', 'ti,dm6441-spi']
>> arch/arm/boot/dts/ti/keystone/keystone-k2e-evm.dtb: usb_phy@2620738 (ti,keystone-usbphy): '#address-cells', '#size-cells' do not match any of the regexes: '^pinctrl-[0-9]+$'
from schema $id: http://devicetree.org/schemas/phy/ti,keystone-usbphy.yaml#
arch/arm/boot/dts/ti/keystone/keystone-k2e-evm.dtb: usb@2680000 (ti,keystone-dwc3): 'clock-names' does not match any of the regexes: '^pinctrl-[0-9]+$', 'usb@[a-f0-9]+$'
from schema $id: http://devicetree.org/schemas/usb/ti,keystone-dwc3.yaml#
arch/arm/boot/dts/ti/keystone/keystone-k2e-evm.dtb: wdt@22f0080 (ti,keystone-wdt): $nodename:0: 'wdt@22f0080' does not match '^(timer|watchdog)(@.*|-([0-9]|[1-9][0-9]+))?$'
from schema $id: http://devicetree.org/schemas/watchdog/ti,davinci-wdt.yaml#
arch/arm/boot/dts/ti/keystone/keystone-k2e-evm.dtb: /soc@0/timer@22f0000: failed to match any schema with compatible: ['ti,keystone-timer']
arch/arm/boot/dts/ti/keystone/keystone-k2e-evm.dtb: /soc@0/aemif@21000a00: failed to match any schema with compatible: ['ti,keystone-aemif', 'ti,davinci-aemif']
arch/arm/boot/dts/ti/keystone/keystone-k2e-evm.dtb: /soc@0/aemif@21000a00: failed to match any schema with compatible: ['ti,keystone-aemif', 'ti,davinci-aemif']
arch/arm/boot/dts/ti/keystone/keystone-k2e-evm.dtb: cs0: 'clocks' is a dependency of 'clock-ranges'
from schema $id: http://devicetree.org/schemas/clock/clock.yaml#
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-06-08 17:07 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-06-07 21:26 [PATCH] dt-bindings: phy: Convert ti,keystone-usbphy to DT schema Rob Herring (Arm)
2025-06-07 21:26 ` Rob Herring (Arm)
-- strict thread matches above, loose matches on Subject: below --
2025-06-08 17:06 kernel test robot
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.