[PATCH] dt-bindings: usb: ti,da830-ohci: Convert to DT schema

[PATCH] dt-bindings: usb: ti,da830-ohci: Convert to DT schema

[Bhargav Joshi]

Convert Texas Instruments da830-ohci bindings from legacy text to DT
schema. Name of binding is changed to match primary compatible property.

Signed-off-by: Bhargav Joshi <j.bhargav.u@gmail.com>
---
 .../devicetree/bindings/usb/ohci-da8xx.txt         | 23 ----------
 .../devicetree/bindings/usb/ti,da830-ohci.yaml     | 50 ++++++++++++++++++++++
 2 files changed, 50 insertions(+), 23 deletions(-)

diff --git a/Documentation/devicetree/bindings/usb/ohci-da8xx.txt b/Documentation/devicetree/bindings/usb/ohci-da8xx.txt
deleted file mode 100644
index 24a826d5015e..000000000000
--- a/Documentation/devicetree/bindings/usb/ohci-da8xx.txt
+++ /dev/null
@@ -1,23 +0,0 @@
-DA8XX USB OHCI controller
-
-Required properties:
-
- - compatible: Should be "ti,da830-ohci"
- - reg:        Should contain one register range i.e. start and length
- - interrupts: Description of the interrupt line
- - phys:       Phandle for the PHY device
- - phy-names:  Should be "usb-phy"
-
-Optional properties:
- - vbus-supply: phandle of regulator that controls vbus power / over-current
-
-Example:
-
-ohci: usb@225000 {
-        compatible = "ti,da830-ohci";
-        reg = <0x225000 0x1000>;
-        interrupts = <59>;
-        phys = <&usb_phy 1>;
-        phy-names = "usb-phy";
-        vbus-supply = <&reg_usb_ohci>;
-};
diff --git a/Documentation/devicetree/bindings/usb/ti,da830-ohci.yaml b/Documentation/devicetree/bindings/usb/ti,da830-ohci.yaml
new file mode 100644
index 000000000000..ad23da646648
--- /dev/null
+++ b/Documentation/devicetree/bindings/usb/ti,da830-ohci.yaml
@@ -0,0 +1,50 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/usb/ti,da830-ohci.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: DA8XX USB OHCI controller
+
+maintainers:
+  - Axel Haslam <ahaslam@baylibre.com>
+
+properties:
+  compatible:
+    const: ti,da830-ohci
+
+  reg:
+    maxItems: 1
+
+  interrupts:
+    maxItems: 1
+
+  phys:
+    maxItems: 1
+
+  phy-names:
+    const: usb-phy
+
+  vbus-supply:
+    description:
+      phandle of regulator that controls vbus power / over-current
+
+required:
+  - compatible
+  - reg
+  - interrupts
+  - phys
+  - phy-names
+
+additionalProperties: false
+
+examples:
+  - |
+    usb@225000 {
+        compatible = "ti,da830-ohci";
+        reg = <0x225000 0x1000>;
+        interrupts = <59>;
+        phys = <&usb_phy 1>;
+        phy-names = "usb-phy";
+        vbus-supply = <&reg_usb_ohci>;
+    };

---
base-commit: ef0c9f75a19532d7675384708fc8621e10850104
change-id: 20260622-ti-da830-ohci-2a2299d045e2

Best regards,
--  
Bhargav

Re: [PATCH] dt-bindings: usb: ti,da830-ohci: Convert to DT schema

[sashiko-bot]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Medium] The converted YAML schema omits the `clocks` and `oc-gpios` properties. Since `additionalProperties: false` is set, `make dtbs_check` will incorrectly fail for existing valid device trees that use these properties. Furthermore, `clocks` is a mandatory hardware requirement, as the driver's probe function will fail without it.
--

--- Patch [1]: [PATCH] dt-bindings: usb: ti,da830-ohci: Convert to DT schema ---
commit 7116233f47fab25581fee6b5a4f1df2b63c1d5a9
Author: Bhargav Joshi <j.bhargav.u@gmail.com>

dt-bindings: usb: ti,da830-ohci: Convert to DT schema

Convert Texas Instruments da830-ohci bindings from legacy text to DT
schema. Name of binding is changed to match primary compatible property.

> diff --git a/Documentation/devicetree/bindings/usb/ti,da830-ohci.yaml b/Documentation/devicetree/bindings/usb/ti,da830-ohci.yaml
> new file mode 100644
> index 0000000000000..ad23da6466487
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/usb/ti,da830-ohci.yaml
> @@ -0,0 +1,50 @@

[ ... ]

> +  vbus-supply:
> +    description:
> +      phandle of regulator that controls vbus power / over-current
> +
> +required:
> +  - compatible
> +  - reg
> +  - interrupts
> +  - phys
> +  - phy-names
> +
> +additionalProperties: false

[Severity: Medium]
Because this schema enforces additionalProperties: false, won't make
dtbs_check incorrectly report validation errors for existing device trees
that include clocks or oc-gpios?

Existing device trees like arch/arm/boot/dts/ti/davinci/da850.dtsi define
a clocks property for this device:

arch/arm/boot/dts/ti/davinci/da850.dtsi:usb1() {
        ...
        phy-names = "usb-phy";
        clocks = <&psc1 2>;
        ...
}

Additionally, the driver drivers/usb/host/ohci-da8xx.c checks for an optional
"oc" GPIO (which translates to oc-gpios in the device tree):

drivers/usb/host/ohci-da8xx.c:ohci_da8xx_probe() {
        ...
        da8xx_ohci->oc_gpio = devm_gpiod_get_optional(dev, "oc", GPIOD_IN);
        ...
}

Furthermore, should clocks be added to the required list? The driver's probe
function appears to fail if the clock is missing:

drivers/usb/host/ohci-da8xx.c:ohci_da8xx_probe() {
        ...
        da8xx_ohci->usb11_clk = devm_clk_get(dev, NULL);
        if (IS_ERR(da8xx_ohci->usb11_clk)) {
        ...
}

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260717-ti-da830-ohci-v1-1-3f2d111a2042@gmail.com?part=1

Re: [PATCH] dt-bindings: usb: ti,da830-ohci: Convert to DT schema

[kernel test robot]

:::::: 
:::::: 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: <20260717-ti-da830-ohci-v1-1-3f2d111a2042@gmail.com>
References: <20260717-ti-da830-ohci-v1-1-3f2d111a2042@gmail.com>
TO: Bhargav Joshi <j.bhargav.u@gmail.com>
TO: "Greg Kroah-Hartman" <gregkh@linuxfoundation.org>
TO: Rob Herring <robh@kernel.org>
TO: Krzysztof Kozlowski <krzk@kernel.org>
TO: Conor Dooley <conor+dt@kernel.org>
TO: Axel Haslam <ahaslam@baylibre.com>
CC: linux-usb@vger.kernel.org
CC: devicetree@vger.kernel.org
CC: linux-kernel@vger.kernel.org
CC: goledhruva@gmail.com
CC: m-chawdhry@ti.com
CC: daniel.baluta@gmail.com
CC: simona.toaca@nxp.com
CC: j.bhargav.u@gmail.com

Hi Bhargav,

kernel test robot noticed the following build warnings:

[auto build test WARNING on ef0c9f75a19532d7675384708fc8621e10850104]

url:    https://github.com/intel-lab-lkp/linux/commits/Bhargav-Joshi/dt-bindings-usb-ti-da830-ohci-Convert-to-DT-schema/20260717-054535
base:   ef0c9f75a19532d7675384708fc8621e10850104
patch link:    https://lore.kernel.org/r/20260717-ti-da830-ohci-v1-1-3f2d111a2042%40gmail.com
patch subject: [PATCH] dt-bindings: usb: ti,da830-ohci: Convert to DT schema
:::::: branch date: 2 days ago
:::::: commit date: 2 days ago
config: arm-randconfig-2052-20260717 (https://download.01.org/0day-ci/archive/20260718/202607181400.9LRShl0x-lkp@intel.com/config)
compiler: arm-linux-gnueabi-gcc (GCC) 10.5.0
dtschema: 2026.7.dev1+g2203c1720
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260718/202607181400.9LRShl0x-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/202607181400.9LRShl0x-lkp@intel.com/

dtcheck warnings: (new ones prefixed by >>)
   arch/arm/boot/dts/ti/davinci/da850-lcdk.dtb: /soc@1c00000/spi@41000: failed to match any schema with compatible: ['ti,da830-spi']
   arch/arm/boot/dts/ti/davinci/da850-lcdk.dtb: /soc@1c00000/spi@30e000: failed to match any schema with compatible: ['ti,da830-spi']
   arch/arm/boot/dts/ti/davinci/da850-lcdk.dtb: /soc@1c00000/usb@200000: failed to match any schema with compatible: ['ti,da830-musb']
   arch/arm/boot/dts/ti/davinci/da850-lcdk.dtb: /soc@1c00000/usb@200000/dma-controller@201000: failed to match any schema with compatible: ['ti,da830-cppi41']
   arch/arm/boot/dts/ti/davinci/da850-lcdk.dtb: sata@218000 (ti,da850-ahci): 'clock-names', 'clocks' do not match any of the regexes: '^pinctrl-[0-9]+$'
   	from schema $id: http://devicetree.org/schemas/ata/ti,da850-ahci.yaml
   arch/arm/boot/dts/ti/davinci/da850-lcdk.dtb: /soc@1c00000/clock-controller@21a000: failed to match any schema with compatible: ['ti,da850-pll1']
   arch/arm/boot/dts/ti/davinci/da850-lcdk.dtb: mdio@224000 (ti,davinci_mdio): Unevaluated properties are not allowed ('power-domains' was unexpected)
   	from schema $id: http://devicetree.org/schemas/net/ti,davinci-mdio.yaml
   arch/arm/boot/dts/ti/davinci/da850-lcdk.dtb: /soc@1c00000/ethernet@220000: failed to match any schema with compatible: ['ti,davinci-dm6467-emac']
>> arch/arm/boot/dts/ti/davinci/da850-lcdk.dtb: usb@225000 (ti,da830-ohci): 'clocks' does not match any of the regexes: '^pinctrl-[0-9]+$'
   	from schema $id: http://devicetree.org/schemas/usb/ti,da830-ohci.yaml
   arch/arm/boot/dts/ti/davinci/da850-lcdk.dtb: /soc@1c00000/clock-controller@227000: failed to match any schema with compatible: ['ti,da850-psc1']
   arch/arm/boot/dts/ti/davinci/da850-lcdk.dtb: /soc@1c00000/pin-controller@22c00c: failed to match any schema with compatible: ['ti,da850-pupd']
   arch/arm/boot/dts/ti/davinci/da850-lcdk.dtb: display@213000 (ti,da850-tilcdc): 'clock-names', 'clocks', 'power-domains' do not match any of the regexes: '^pinctrl-[0-9]+$'
   	from schema $id: http://devicetree.org/schemas/display/tilcdc/ti,am33xx-tilcdc.yaml
   arch/arm/boot/dts/ti/davinci/da850-lcdk.dtb: /aemif@68000000: failed to match any schema with compatible: ['ti,da850-aemif']
   arch/arm/boot/dts/ti/davinci/da850-lcdk.dtb: cs3: 'clocks' is a dependency of 'clock-ranges'
   	from schema $id: http://devicetree.org/schemas/clock/clock.yaml
   arch/arm/boot/dts/ti/davinci/da850-lcdk.dtb: nand@2000000,0 (ti,davinci-nand): #size-cells: 0 was expected
   	from schema $id: http://devicetree.org/schemas/mtd/ti,davinci-nand.yaml
--
   arch/arm/boot/dts/ti/davinci/da850-enbw-cmc.dtb: /soc@1c00000/usb@200000: failed to match any schema with compatible: ['ti,da830-musb']
   arch/arm/boot/dts/ti/davinci/da850-enbw-cmc.dtb: /soc@1c00000/usb@200000/dma-controller@201000: failed to match any schema with compatible: ['ti,da830-cppi41']
   arch/arm/boot/dts/ti/davinci/da850-enbw-cmc.dtb: sata@218000 (ti,da850-ahci): 'clock-names', 'clocks' do not match any of the regexes: '^pinctrl-[0-9]+$'
   	from schema $id: http://devicetree.org/schemas/ata/ti,da850-ahci.yaml
   arch/arm/boot/dts/ti/davinci/da850-enbw-cmc.dtb: /soc@1c00000/clock-controller@21a000: failed to match any schema with compatible: ['ti,da850-pll1']
   arch/arm/boot/dts/ti/davinci/da850-enbw-cmc.dtb: mdio@224000 (ti,davinci_mdio): 'bus_freq' is a required property
   	from schema $id: http://devicetree.org/schemas/net/ti,davinci-mdio.yaml
   arch/arm/boot/dts/ti/davinci/da850-enbw-cmc.dtb: mdio@224000 (ti,davinci_mdio): Unevaluated properties are not allowed ('power-domains' was unexpected)
   	from schema $id: http://devicetree.org/schemas/net/ti,davinci-mdio.yaml
   arch/arm/boot/dts/ti/davinci/da850-enbw-cmc.dtb: /soc@1c00000/ethernet@220000: failed to match any schema with compatible: ['ti,davinci-dm6467-emac']
>> arch/arm/boot/dts/ti/davinci/da850-enbw-cmc.dtb: usb@225000 (ti,da830-ohci): 'clocks' does not match any of the regexes: '^pinctrl-[0-9]+$'
   	from schema $id: http://devicetree.org/schemas/usb/ti,da830-ohci.yaml
   arch/arm/boot/dts/ti/davinci/da850-enbw-cmc.dtb: /soc@1c00000/clock-controller@227000: failed to match any schema with compatible: ['ti,da850-psc1']
   arch/arm/boot/dts/ti/davinci/da850-enbw-cmc.dtb: /soc@1c00000/pin-controller@22c00c: failed to match any schema with compatible: ['ti,da850-pupd']
   arch/arm/boot/dts/ti/davinci/da850-enbw-cmc.dtb: display@213000 (ti,da850-tilcdc): 'clock-names', 'clocks', 'power-domains' do not match any of the regexes: '^pinctrl-[0-9]+$'
   	from schema $id: http://devicetree.org/schemas/display/tilcdc/ti,am33xx-tilcdc.yaml
   arch/arm/boot/dts/ti/davinci/da850-enbw-cmc.dtb: /aemif@68000000: failed to match any schema with compatible: ['ti,da850-aemif']
--
   arch/arm/boot/dts/ti/davinci/da850-evm.dtb: /soc@1c00000/spi@41000: failed to match any schema with compatible: ['ti,da830-spi']
   arch/arm/boot/dts/ti/davinci/da850-evm.dtb: /soc@1c00000/spi@30e000: failed to match any schema with compatible: ['ti,da830-spi']
   arch/arm/boot/dts/ti/davinci/da850-evm.dtb: /soc@1c00000/usb@200000: failed to match any schema with compatible: ['ti,da830-musb']
   arch/arm/boot/dts/ti/davinci/da850-evm.dtb: /soc@1c00000/usb@200000/dma-controller@201000: failed to match any schema with compatible: ['ti,da830-cppi41']
   arch/arm/boot/dts/ti/davinci/da850-evm.dtb: sata@218000 (ti,da850-ahci): 'clock-names', 'clocks' do not match any of the regexes: '^pinctrl-[0-9]+$'
   	from schema $id: http://devicetree.org/schemas/ata/ti,da850-ahci.yaml
   arch/arm/boot/dts/ti/davinci/da850-evm.dtb: /soc@1c00000/clock-controller@21a000: failed to match any schema with compatible: ['ti,da850-pll1']
   arch/arm/boot/dts/ti/davinci/da850-evm.dtb: mdio@224000 (ti,davinci_mdio): Unevaluated properties are not allowed ('power-domains' was unexpected)
   	from schema $id: http://devicetree.org/schemas/net/ti,davinci-mdio.yaml
   arch/arm/boot/dts/ti/davinci/da850-evm.dtb: /soc@1c00000/ethernet@220000: failed to match any schema with compatible: ['ti,davinci-dm6467-emac']
>> arch/arm/boot/dts/ti/davinci/da850-evm.dtb: usb@225000 (ti,da830-ohci): 'clocks' does not match any of the regexes: '^pinctrl-[0-9]+$'
   	from schema $id: http://devicetree.org/schemas/usb/ti,da830-ohci.yaml
   arch/arm/boot/dts/ti/davinci/da850-evm.dtb: /soc@1c00000/clock-controller@227000: failed to match any schema with compatible: ['ti,da850-psc1']
   arch/arm/boot/dts/ti/davinci/da850-evm.dtb: /soc@1c00000/pin-controller@22c00c: failed to match any schema with compatible: ['ti,da850-pupd']
   arch/arm/boot/dts/ti/davinci/da850-evm.dtb: display@213000 (ti,da850-tilcdc): 'clock-names', 'clocks', 'power-domains' do not match any of the regexes: '^pinctrl-[0-9]+$'
   	from schema $id: http://devicetree.org/schemas/display/tilcdc/ti,am33xx-tilcdc.yaml
   arch/arm/boot/dts/ti/davinci/da850-evm.dtb: display@213000 (ti,da850-tilcdc): 'port' is a required property
   	from schema $id: http://devicetree.org/schemas/display/tilcdc/ti,am33xx-tilcdc.yaml
   arch/arm/boot/dts/ti/davinci/da850-evm.dtb: /aemif@68000000: failed to match any schema with compatible: ['ti,da850-aemif']
   arch/arm/boot/dts/ti/davinci/da850-evm.dtb: cs3: 'clocks' is a dependency of 'clock-ranges'
   	from schema $id: http://devicetree.org/schemas/clock/clock.yaml
--
   	from schema $id: http://devicetree.org/schemas/mtd/jedec,spi-nor.yaml
   arch/arm/boot/dts/ti/davinci/da850-lego-ev3.dtb: adc@3 (ti,ads7957): 'ti,spi-wdelay' does not match any of the regexes: '^pinctrl-[0-9]+$'
   	from schema $id: http://devicetree.org/schemas/iio/adc/ti,ads7950.yaml
   arch/arm/boot/dts/ti/davinci/da850-lego-ev3.dtb: /soc@1c00000/spi@30e000: failed to match any schema with compatible: ['ti,da830-spi']
   arch/arm/boot/dts/ti/davinci/da850-lego-ev3.dtb: /soc@1c00000/usb@200000: failed to match any schema with compatible: ['ti,da830-musb']
   arch/arm/boot/dts/ti/davinci/da850-lego-ev3.dtb: /soc@1c00000/usb@200000/dma-controller@201000: failed to match any schema with compatible: ['ti,da830-cppi41']
   arch/arm/boot/dts/ti/davinci/da850-lego-ev3.dtb: sata@218000 (ti,da850-ahci): 'clock-names', 'clocks' do not match any of the regexes: '^pinctrl-[0-9]+$'
   	from schema $id: http://devicetree.org/schemas/ata/ti,da850-ahci.yaml
   arch/arm/boot/dts/ti/davinci/da850-lego-ev3.dtb: /soc@1c00000/clock-controller@21a000: failed to match any schema with compatible: ['ti,da850-pll1']
   arch/arm/boot/dts/ti/davinci/da850-lego-ev3.dtb: /soc@1c00000/ethernet@220000: failed to match any schema with compatible: ['ti,davinci-dm6467-emac']
>> arch/arm/boot/dts/ti/davinci/da850-lego-ev3.dtb: usb@225000 (ti,da830-ohci): 'clocks' does not match any of the regexes: '^pinctrl-[0-9]+$'
   	from schema $id: http://devicetree.org/schemas/usb/ti,da830-ohci.yaml
   arch/arm/boot/dts/ti/davinci/da850-lego-ev3.dtb: /soc@1c00000/clock-controller@227000: failed to match any schema with compatible: ['ti,da850-psc1']
   arch/arm/boot/dts/ti/davinci/da850-lego-ev3.dtb: /soc@1c00000/pin-controller@22c00c: failed to match any schema with compatible: ['ti,da850-pupd']
   arch/arm/boot/dts/ti/davinci/da850-lego-ev3.dtb: display@213000 (ti,da850-tilcdc): 'clock-names', 'clocks', 'power-domains' do not match any of the regexes: '^pinctrl-[0-9]+$'
   	from schema $id: http://devicetree.org/schemas/display/tilcdc/ti,am33xx-tilcdc.yaml
   arch/arm/boot/dts/ti/davinci/da850-lego-ev3.dtb: /aemif@68000000: failed to match any schema with compatible: ['ti,da850-aemif']
   arch/arm/boot/dts/ti/davinci/da850-lego-ev3.dtb: gpio_keys (gpio-keys): 'back', 'center', 'down', 'left', 'right', 'up' do not match any of the regexes: '^(button|event|key|switch|(button|event|key|switch)-[a-z0-9-]+|[a-z0-9-]+-(button|event|key|switch))$', '^pinctrl-[0-9]+$'
   	from schema $id: http://devicetree.org/schemas/input/gpio-keys.yaml
   arch/arm/boot/dts/ti/davinci/da850-lego-ev3.dtb: leds (gpio-leds): 'left_green', 'left_red', 'right_green', 'right_red' do not match any of the regexes: '(^led-[0-9a-f]$|led)', '^pinctrl-[0-9]+$'
   	from schema $id: http://devicetree.org/schemas/leds/leds-gpio.yaml

--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki