Attention is currently required from: Lance Zhao, Tim Wawrzynczak, Tim Van Patten. Raul Rangel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63558 )
Change subject: drivers/usb/acpi: acpi_power_res_params: Add use_gpio_for_status ......................................................................
Patch Set 2:
(3 comments)
File src/acpi/device.c:
PS2: Split this into its own commit.
https://review.coreboot.org/c/coreboot/+/63558/comment/6d5433a8_6eed6168 PS2, Line 698: ; How about: ``` if (params->use_gpio_for_status) { // ACPI IF (_STA() == ACPI_POWER_RESOURCE_STATUS_ON_OP) RETURN } ```
The _STA method in this case verifies if all the GPIOs are in the On state. If not do the full flow.
https://review.coreboot.org/c/coreboot/+/63558/comment/12b8db33_60047528 PS2, Line 705: acpigen_write_gpio_status_check Hrmm I don't think we can just bail since the other GPIOs might be in a different state.