[coreboot-gerrit] Change in coreboot[master]: mainboard/google/volteer: Enable long cr50 ready pulses

[Jes Klinke (Code Review)]

Jes Klinke has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44359 )

Change subject: mainboard/google/volteer: Enable long cr50 ready pulses
......................................................................

Patch Set 15:

(3 comments)

https://review.coreboot.org/c/coreboot/+/44359/12/src/mainboard/google/volte... 
File src/mainboard/google/volteer/mainboard.c:

https://review.coreboot.org/c/coreboot/+/44359/12/src/mainboard/google/volte... 
PS12, Line 35: mainboard_update_s0ix_disable_mask
> SGTM.

I have created the below CL to introduce a weak method for mainboard to override.
https://review.coreboot.org/c/coreboot/+/44914

https://review.coreboot.org/c/coreboot/+/44359/12/src/mainboard/google/volte... 
PS12, Line 46: gpio_override_pm
> You will have to drop this from devicetree.cb now: […]

Done

https://review.coreboot.org/c/coreboot/+/44359/12/src/mainboard/google/volte... 
PS12, Line 48: cfg->gpio_pm[i] = 0
> memset to 0?

Done