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

From: Krishna Kurapati PSSNV
Date: Wed Jan 03 2024 - 09:52:46 EST



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,