Re: [PATCH v5 0/3] usb: cdns3: Add CIX Sky1 USBSSP controller support

["Hongliang Yang (杨宏亮)" <hongliang.yang@cixtech.com>]

Sorry for not responding to the earlier rounds of feedback - that was my
mistake.

The v5 USB cover was missing the changelog entirely; v6 has a complete
"Changes in v6" section covering each patch.

On the DT model: you were right that a shared register region is not a
simple-bus and that the representation was fake. v6 drops the container:
each controller node now sits at the address of its own glue registers
(usb@91c0314 and usb@91c0324), so the unit address matches real hardware,
and the Cadence core is a child at its real address (usb@91d0000 and
usb@91e0000) with an empty ranges property.

On the ABI: the usbN aliases and the id property are gone. The mode strap
register offset and bit position are passed as arguments of the
cix,usb-syscon phandle (0x424, bit 8 for USB4 and bit 10 for USB5), and
#syscon-cells = <2> is documented in the Sky1 system controller binding.
Nothing undocumented is parsed anymore.

Thanks,
Hongliang