Re: [PATCH v5 1/2] dt-bindings: usb: dwc3: Clean up hs_phy_irq in binding

[Krishna Kurapati PSSNV <quic_kriskura@xxxxxxxxxxx>]

> > > >            interrupt-names:
> > > > -          minItems: 3
> > > >              items:
> > > > +            - const: pwr_event
> > > >                - const: hs_phy_irq
> > > > -            - const: dp_hs_phy_irq
> > > > -            - const: dm_hs_phy_irq
> > > > +            - const: qusb2_phy
> > >
> > > Why qusb2_phy is after hs_phy_irq? In the earlier if:then: it is the
> > > second one.
> >
> > In v3 as well, the hs_phy_irq is before qusb2_phy interrupt:
> > https://lore.kernel.org/all/20231211121124.4194-2-quic_kriskura@xxxxxxxxxxx/
>
> ? How v3 matters?

I was thinking whether I modified the order between v3 and v5 and didn't 
mention in change log and hence I compared with v3. Thanks for the catch.

I made qusb2_phy the second one in the list and pushed v6: 
https://lore.kernel.org/all/20231227091951.685-1-quic_kriskura@xxxxxxxxxxx/

Can you help provide you review on v6 as well.

Regards,
Krishna,