Shelley Chen has submitted this change. ( https://review.coreboot.org/c/coreboot/+/55378 ) Change subject: herobrine: Assert gpio for USB_HUB_LDO_EN ...................................................................... herobrine: Assert gpio for USB_HUB_LDO_EN Some herobrine variants have USB hub powered by discrete LDO that is controlled by USB_HUB_LDO_EN gpio. Assert the GPIO on boot. BUG=b:182963902 TEST=Validated on qualcomm sc7280 development board. Signed-off-by: Sandeep Maheswaram <sanm@codeaurora.org> Change-Id: Ia94e046f9eb0d3ce593f3445e0203a7391c14de2 Reviewed-on: https://review.coreboot.org/c/coreboot/+/55378 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Julius Werner <jwerner@chromium.org> --- M src/mainboard/google/herobrine/board.h M src/mainboard/google/herobrine/mainboard.c 2 files changed, 10 insertions(+), 0 deletions(-) Approvals: build bot (Jenkins): Verified Julius Werner: Looks good to me, approved diff --git a/src/mainboard/google/herobrine/board.h b/src/mainboard/google/herobrine/board.h index 0e4a760..4278c11 100644 --- a/src/mainboard/google/herobrine/board.h +++ b/src/mainboard/google/herobrine/board.h @@ -9,6 +9,13 @@ #define GPIO_SD_CD_L GPIO(91) +#if CONFIG(BOARD_GOOGLE_SENOR) || CONFIG(BOARD_GOOGLE_PIGLIN) || CONFIG(BOARD_GOOGLE_HOGLIN) +#define USB_HUB_LDO_EN GPIO(157) +#else +/* For Herobrine board and all variants */ +#define USB_HUB_LDO_EN GPIO(24) +#endif + #define QCOM_SC7280_SKU1 0x0 #define QCOM_SC7280_SKU2 0x1 #define QCOM_SC7280_SKU3 0x2 diff --git a/src/mainboard/google/herobrine/mainboard.c b/src/mainboard/google/herobrine/mainboard.c index e64fb15..a2e0786 100644 --- a/src/mainboard/google/herobrine/mainboard.c +++ b/src/mainboard/google/herobrine/mainboard.c @@ -24,6 +24,9 @@ static void setup_usb(void) { + /* Assert EN_PP3300_HUB for those board variants that use it. */ + gpio_output(USB_HUB_LDO_EN, 1); + setup_usb_host0(&usb0_board_data); } -- To view, visit https://review.coreboot.org/c/coreboot/+/55378 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ia94e046f9eb0d3ce593f3445e0203a7391c14de2 Gerrit-Change-Number: 55378 Gerrit-PatchSet: 56 Gerrit-Owner: Ravi kumar <rbokka@codeaurora.org> Gerrit-Reviewer: Julius Werner <jwerner@chromium.org> Gerrit-Reviewer: Shelley Chen <shchen@google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-CC: Paul Menzel <paulepanter@mailbox.org> Gerrit-CC: Ravi Kumar Bokka <c_rbokka@qualcomm.corp-partner.google.com> Gerrit-CC: Sandeep Maheswaram <sanm@codeaurora.org> Gerrit-MessageType: merged