Alex Levin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48518 )
Change subject: mb/google/volteer: Assert BT_DISABLE_L (GPP_A13) in early_gpio_table ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/c/coreboot/+/48518/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/48518/1//COMMIT_MSG@13 PS1, Line 13: scope shows assertion of the signal
Are there any timing requirements for how long to hold the pin in reset?
no. see the bug for a detailed discussion
https://review.coreboot.org/c/coreboot/+/48518/1/src/mainboard/google/voltee... File src/mainboard/google/volteer/variants/baseboard/gpio.c:
https://review.coreboot.org/c/coreboot/+/48518/1/src/mainboard/google/voltee... PS1, Line 405: GPP_A13
This will drive it low. […]
Nick, please read the bug for the background. The intent is to drive 0 and later to drive 1.
I have made sure that all the variant indeed drive 1 in ram stage.