On Fri 15 Jan 07:27 CST 2021, Felipe Balbi wrote: > Hi, > > Shawn Guo <shawn.guo@xxxxxxxxxx> writes: > > For sdm845 ACPI boot, the URS (USB Role Switch) node in ACPI DSDT table > > holds the memory resource, while interrupt resources reside in the child > > nodes USB0 and UFN0. It adds USB0 host support by probing URS node, > > creating platform device for USB0 node, and then retrieve interrupt > > resources from USB0 platform device. > > just so I understand this: the interrupt was listed under a separate > device altogether? > Correct, this seems to be the standard structure for a role-switching usb controller, as shown under "ACPI System configuration" at https://docs.microsoft.com/en-us/windows-hardware/drivers/usbcon/usb-dual-role-driver-stack-architecture Regards, Bjorn > Nothing wrong with the patch itself, but just curious. > > -- > balbi
