Attention is currently required from: Chen Wisley, Tim Wawrzynczak. Wisley Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56428 )
Change subject: mb/google/brya/variants/redrix: Update gpio for redrix ......................................................................
Patch Set 5:
(3 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/56428/comment/f20256e9_8e2db131 PS3, Line 9: Update
`Add initial` […]
Done
File src/mainboard/google/brya/variants/redrix/gpio.c:
https://review.coreboot.org/c/coreboot/+/56428/comment/fb0a6daa_c9a9b255 PS3, Line 12: PAD_CFG_GPO(GPP_A17, 1, DEEP),
If this is a NC, why drive it high? Should this be […]
I correct it.
Thanks
https://review.coreboot.org/c/coreboot/+/56428/comment/0477f38d_4bcedfe3 PS3, Line 107: EN_PP3300_SD
Is this on a DB or on the MB?
It's on MB.