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

[Furquan Shaikh (Code Review)]

Furquan Shaikh 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 10:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/44359/10/src/mainboard/google/volte... 
PS10, Line 37: 		cfg->LpmStateDisableMask = LPM_S0i3_4;
I think you will also have to handle gpio_pm_config here:

https://source.chromium.org/chromiumos/chromiumos/codesearch/+/master:src/th...

i.e. by default those should be set to no override i.e. gpio_override_pm = "0" and here along with disabling of LPM_S0i3_4, you will also need to set gpio_override_pm = 1 and gpio_pm[0..4] to 0.