Re: [PATCH v1 1/3] dt-bindings: usb: dwc3: Add JHB100 device controller PHY adapter

[Conor Dooley <conor@kernel.org>]

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

On Fri, Sep 18, 2026 at 03:11:01PM +0800, Minda Chen wrote:
> Add dt-binding of JHB100 server host field USB device controller PHY
> adapter register IO region and its regs-name.

I don't understand the rationale here or the other binding patch.
Why is it not always 2 regs? Either the device has 2 or it has 1, there
shouldn't be variability unless there's more than one instance and the
dfferent instances have different numbers?

Cheers,
Conor.

> 
> Signed-off-by: Minda Chen <minda.chen@starfivetech.com>
> ---
>  .../devicetree/bindings/usb/starfive,jhb100-dwc3.yaml | 11 ++++++++++-
>  1 file changed, 10 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/usb/starfive,jhb100-dwc3.yaml b/Documentation/devicetree/bindings/usb/starfive,jhb100-dwc3.yaml
> index fbabe99e9d5c..b2fd98c06ba0 100644
> --- a/Documentation/devicetree/bindings/usb/starfive,jhb100-dwc3.yaml
> +++ b/Documentation/devicetree/bindings/usb/starfive,jhb100-dwc3.yaml
> @@ -20,7 +20,16 @@ properties:
>      const: starfive,jhb100-dwc3
>  
>    reg:
> -    maxItems: 1
> +    minItems: 1
> +    items:
> +      - description: Core DWC3 region
> +      - description: DWC3 device controller adapter region
> +
> +  reg-names:
> +    minItems: 1
> +    items:
> +      - const: core
> +      - const: adapter
>  
>    interrupts:
>      maxItems: 1
> -- 
> 2.17.1
> 

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