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

[Minda Chen]

> 
> 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.
> 
Thanks for reviewing. The virtual device controller do not contain PHY
but need to add PHY adapter setting(self-develop by StarFive). So have 2 regs. 
Previous is BMC USB module do not contain it. But if PHY adapter be moved
to PHY related driver can remove this . But Thinh still no review the patch3.

> >
> > Signed-off-by: Minda Chen <minda.chen@xxxxxxxxxxxxxxxx>
> > ---
> >  .../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
> >