Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37473 )
Change subject: driver/intel/usb4: Add new driver for USB4 Root Port and Resource Allocation ......................................................................
Patch Set 1:
Patch Set 1:
Patch Set 1:
Patch Set 1:
Patch Set 1:
Patch Set 1:
There was also this patch: https://review.coreboot.org/c/coreboot/+/35946
Brandon, you were in the following thread as well. I was trying to find another thread where we discussed some more on this topic that I think it is important for context. Let me continue digging there.
https://mail.coreboot.org/hyperkitty/list/coreboot@coreboot.org/thread/QW3NV...
Found it. It was on a review thread. I think that holds somewhat of context related to this one as well. The other CL I commented with is definitely related and we should align direction there.
Apparently I can't paste correctly. Here's the link: https://review.coreboot.org/c/coreboot/+/36221
I had not seen this patch I will take a look and better understand it
The patch itself isn't entirely related, but the conversation is certainly related. For Chromebooks, we don't have to enumerate nor even assign resources to the pcie root ports that go to usb4 capable ports because we could just let the OS deal with that. So, in this patch, having fixed resource sizes (address space and buses) is not entirely necessary if the follow-on payload has no intention of enabling usb4, e.g. Something to keep in mind.