Julius Werner has posted comments on this change. ( https://review.coreboot.org/27477 )
Change subject: libpayload/generic_hub: Detect port disconnect after reset ......................................................................
Patch Set 1:
I don't understand what this patch is needed for. The general concept for libpayload's USB stack is that the payload keeps calling usb_poll() in regular intervals. So if a device transforms upon reset like this, the new SuperSpeed port should just get enumerated as part of the next usb_poll(). Why do we need to add a bunch of custom code to enumerate it right now? Calling code should generally not be written in a way that relies on all USB devices being enumerated immediately with the first usb_poll().