[coreboot-gerrit] Change in coreboot[master]: device/xhci: Add an xhci driver to enumerate capabilities

[Raul Rangel (Code Review)]

Raul Rangel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41899 )

Change subject: device/xhci: Add an xhci driver to enumerate capabilities
......................................................................

Uploaded patch set 2.

(3 comments)

https://review.coreboot.org/c/coreboot/+/41899/1/src/device/xhci.c 
File src/device/xhci.c:

https://review.coreboot.org/c/coreboot/+/41899/1/src/device/xhci.c@42 
PS1, Line 42: 	ext_caps_offset = read16(bar + XHCI_HCCPARAMS1_XECP);
> I guess I can make it print an error and return CB_ERR.

Done

https://review.coreboot.org/c/coreboot/+/41899/1/src/device/xhci.c@66 
PS1, Line 66: 			ext_caps_offset = 0;
> So this basically terminates the while loop? Why not use `break`?

Done

https://review.coreboot.org/c/coreboot/+/41899/1/src/device/xhci.c@68 
PS1, Line 68:
> I can add an xhci_print_supported_protocol that the callbacks can call if they want to debug.

Done