[PATCH 0/2] usb: misc: onboard_usb_dev: Add Genesys Logic GL3525 hub support

[PATCH 0/2] usb: misc: onboard_usb_dev: Add Genesys Logic GL3525 hub support

[Fei Shao]

This series adds DT binding and onboard_usb_dev driver support for the
Genesys Logic GL3525 hub controller.


Fei Shao (2):
  dt-bindings: usb: Add Genesys Logic GL3525 hub controller
  usb: misc: onboard_usb_dev: Add Genesys Logic GL3525 hub support

 Documentation/devicetree/bindings/usb/genesys,gl850g.yaml | 4 ++++
 drivers/usb/misc/onboard_usb_dev.c                        | 2 ++
 drivers/usb/misc/onboard_usb_dev.h                        | 2 ++
 3 files changed, 8 insertions(+)

-- 
2.56.0.542.g3ef89b7020-goog

[PATCH 1/2] dt-bindings: usb: Add Genesys Logic GL3525 hub controller

[Fei Shao]

Add USB 3.2 (usb5e3,629) and USB 2.0 (usb5e3,630) compatibles for the
Genesys Logic GL3525 hub controller, which is powered by a 5V VDD supply
like GL3523.

Signed-off-by: Fei Shao <fshao@chromium.org>
---

 Documentation/devicetree/bindings/usb/genesys,gl850g.yaml | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/Documentation/devicetree/bindings/usb/genesys,gl850g.yaml b/Documentation/devicetree/bindings/usb/genesys,gl850g.yaml
index e8b8c03f87a0..5153a988a835 100644
--- a/Documentation/devicetree/bindings/usb/genesys,gl850g.yaml
+++ b/Documentation/devicetree/bindings/usb/genesys,gl850g.yaml
@@ -17,6 +17,8 @@ properties:
       - usb5e3,620
       - usb5e3,625
       - usb5e3,626
+      - usb5e3,629
+      - usb5e3,630
 
   reg: true
 
@@ -70,6 +72,8 @@ allOf:
             enum:
               - usb5e3,610
               - usb5e3,620
+              - usb5e3,629
+              - usb5e3,630
     then:
       properties:
         peer-hub: true
-- 
2.56.0.542.g3ef89b7020-goog

Re: [PATCH 1/2] dt-bindings: usb: Add Genesys Logic GL3525 hub controller

[sashiko-bot]

Thank you for your contribution! Sashiko AI review found 1 potential
issue to consider:
- [Low] dt-bindings: usb: unwieldy per-compatible conditionals in
  genesys,gl850g.yaml
--

commit 74488a0f4f0a5f370e0fe57ea84abc8dcd904262
Author: Fei Shao <fshao@chromium.org>

dt-bindings: usb: Add Genesys Logic GL3525 hub controller

This commit adds the USB 3.2 (usb5e3,629) and USB 2.0 (usb5e3,630)
compatible strings for the Genesys Logic GL3525 hub controller to
genesys,gl850g.yaml.

> diff --git a/Documentation/devicetree/bindings/usb/genesys,gl850g.yaml b/Documentation/devicetree/bindings/usb/genesys,gl850g.yaml
> index e8b8c03f87a0e..5153a988a835e 100644
> --- a/Documentation/devicetree/bindings/usb/genesys,gl850g.yaml
> +++ b/Documentation/devicetree/bindings/usb/genesys,gl850g.yaml
[ ... ]
> @@ -70,6 +72,8 @@ allOf:
>              enum:
>                - usb5e3,610
>                - usb5e3,620
> +              - usb5e3,629
> +              - usb5e3,630
>      then:
>        properties:
>          peer-hub: true

[Severity: Low]
This isn't a bug, but the per-compatible if-then conditionals in allOf are
becoming unwieldy and inconsistent as more hub variants with different
supply and peer-hub rules are added to genesys,gl850g.yaml.

In particular, setting peer-hub: true and vdd-supply: true in the then
blocks is redundant because those properties are already defined at the top
level, usb5e3,626 is omitted from the conditionals entirely (leaving
vdd12-supply allowed on usb5e3,626 while disallowing it here), and
usb5e3,610 is restricted to vdd12-supply: false even when paired as the USB
2.0 companion hub for GL3590 (usb5e3,625).  Could these variants be split
into separate schema documents, or the conditionals simplified to only
restrict the properties that do not apply to specific compatibles?

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261111103710.1393311-1-fshao@chromium.org?part=1

Re: [PATCH 1/2] dt-bindings: usb: Add Genesys Logic GL3525 hub controller

[Conor Dooley]

[-- Attachment #1: Type: text/plain, Size: 75 bytes --]

Acked-by: Conor Dooley <conor.dooley@microchip.com>
pw-bot: not-applicable

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]