[coreboot-gerrit] Change in ...coreboot[master]: drivers/generic/bayhub: Add reset capability

[Richard Spiegel (Code Review)]

Richard Spiegel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30319 )

Change subject: drivers/generic/bayhub: Add reset capability
......................................................................

Patch Set 3:

(1 comment)

https://review.coreboot.org/#/c/30319/2/src/drivers/generic/bayhub/bh720.c 
File src/drivers/generic/bayhub/bh720.c:

https://review.coreboot.org/#/c/30319/2/src/drivers/generic/bayhub/bh720.c@9... 
PS2, Line 91: static const unsigned short pci_device_ids[] = {
> Hmmm, that's really little. I'm confused. […]

Depthcharge is actually fast, it's just a secure loader. What takes time is the actual loading of the OS. But obviously, once the OS starts loading, eMMC is already working. Depthcharge is loaded from the flash, selects where to load from, checks some security (not sure which, but would assume TPM related). and loads the OS (normally chrome OS).