diff options
| author | Furquan Shaikh <furquan@chromium.org> | 2017-08-28 17:20:49 -0700 |
|---|---|---|
| committer | Furquan Shaikh <furquan@google.com> | 2017-08-30 16:40:14 +0000 |
| commit | edf459fe65f9166a06899b2ed86a517ffe8bfdd9 (patch) | |
| tree | 641b67d1e8af49e8bf38f5263199b0f150285c4e /src/drivers/spi/acpi | |
| parent | a62520bc79388f8ed2a33473d76959a18255c76e (diff) | |
acpigen: Add stop gpio control to power resource
There is at least one I2C device (being used by Soraka) that has 3
controls -- enable, reset and stop. If the stop gpio is not put into
the right state when turning off the device in suspend mode, then it
causes leakage. Thus, we need control in power resource to be able to
stop the device when entering suspend state.
BUG=b:64987428
TEST=Verified on soraka that touchscreen stop is correctly configured
on suspend.
Change-Id: Iae5ec7eb3972c5c7f80956d60d0d3c321bbefb0f
Signed-off-by: Furquan Shaikh <furquan@chromium.org>
Reviewed-on: https://review.coreboot.org/21249
Reviewed-by: Duncan Laurie <dlaurie@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/drivers/spi/acpi')
| -rw-r--r-- | src/drivers/spi/acpi/acpi.c | 3 | ||||
| -rw-r--r-- | src/drivers/spi/acpi/chip.h | 5 |
2 files changed, 7 insertions, 1 deletions
diff --git a/src/drivers/spi/acpi/acpi.c b/src/drivers/spi/acpi/acpi.c index 102780f65d..2a39814c3d 100644 --- a/src/drivers/spi/acpi/acpi.c +++ b/src/drivers/spi/acpi/acpi.c @@ -168,7 +168,8 @@ static void spi_acpi_fill_ssdt_generator(struct device *dev) if (config->has_power_resource) acpi_device_add_power_res( &config->reset_gpio, config->reset_delay_ms, - &config->enable_gpio, config->enable_delay_ms); + &config->enable_gpio, config->enable_delay_ms, + &config->stop_gpio, config->stop_delay_ms); acpigen_pop_len(); /* Device */ acpigen_pop_len(); /* Scope */ diff --git a/src/drivers/spi/acpi/chip.h b/src/drivers/spi/acpi/chip.h index 6b54100536..35341d33be 100644 --- a/src/drivers/spi/acpi/chip.h +++ b/src/drivers/spi/acpi/chip.h @@ -47,6 +47,11 @@ struct drivers_spi_acpi_config { struct acpi_gpio enable_gpio; /* Delay to be inserted after device is enabled. */ unsigned enable_delay_ms; + + /* GPIO used to stop operation of device. */ + struct acpi_gpio stop_gpio; + /* Delay to be inserted after disabling stop. */ + unsigned stop_delay_ms; }; #endif /* __SPI_ACPI_CHIP_H__ */ |