summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDuncan Laurie <dlaurie@chromium.org>2017-02-17 17:05:03 -0800
committerMartin Roth <martinroth@google.com>2017-02-19 21:34:12 +0100
commitbd73dbbc38649bd487a44b0bb49fdaa19a248634 (patch)
tree8706041da979ef86fb71ccd85fa04270fab0677b
parentf9d5308690e96002dbeab77b78e499dcd2d3a42f (diff)
downloadcoreboot-bd73dbbc38649bd487a44b0bb49fdaa19a248634.tar.xz
acpi_device: Move power resource function to generic code
Move the function that adds a power resource block from i2c/generic to the acpi device code at src/arch/x86/acpi_device.c so it can be used by more drivers. BUG=chrome-os-partner:61233 TEST=verify SSDT table generation is unchanged Change-Id: I0ffb61a4f46028cbe912e85c0124d9f5200b9c76 Signed-off-by: Duncan Laurie <dlaurie@chromium.org> Reviewed-on: https://review.coreboot.org/18391 Tested-by: build bot (Jenkins) Reviewed-by: Furquan Shaikh <furquan@google.com>
-rw-r--r--src/arch/x86/acpi_device.c46
-rw-r--r--src/arch/x86/include/arch/acpi_device.h9
-rw-r--r--src/drivers/i2c/generic/generic.c49
3 files changed, 59 insertions, 45 deletions
diff --git a/src/arch/x86/acpi_device.c b/src/arch/x86/acpi_device.c
index 7cf292fbee..4fe16a138f 100644
--- a/src/arch/x86/acpi_device.c
+++ b/src/arch/x86/acpi_device.c
@@ -490,6 +490,52 @@ void acpi_device_write_spi(const struct acpi_spi *spi)
acpi_device_fill_len(desc_length);
}
+/* PowerResource() with Enable and/or Reset control */
+void acpi_device_add_power_res(
+ struct acpi_gpio *reset, unsigned reset_delay_ms,
+ struct acpi_gpio *enable, unsigned enable_delay_ms)
+{
+ const char *power_res_dev_states[] = { "_PR0", "_PR3" };
+ unsigned reset_gpio = reset->pins[0];
+ unsigned enable_gpio = enable->pins[0];
+
+ if (!reset_gpio && !enable_gpio)
+ return;
+
+ /* PowerResource (PRIC, 0, 0) */
+ acpigen_write_power_res("PRIC", 0, 0, power_res_dev_states,
+ ARRAY_SIZE(power_res_dev_states));
+
+ /* Method (_STA, 0, NotSerialized) { Return (0x1) } */
+ acpigen_write_STA(0x1);
+
+ /* Method (_ON, 0, Serialized) */
+ acpigen_write_method_serialized("_ON", 0);
+ if (reset_gpio)
+ acpigen_soc_set_tx_gpio(reset_gpio);
+ if (enable_gpio) {
+ acpigen_soc_set_tx_gpio(enable_gpio);
+ if (enable_delay_ms)
+ acpigen_write_sleep(enable_delay_ms);
+ }
+ if (reset_gpio) {
+ acpigen_soc_clear_tx_gpio(reset_gpio);
+ if (reset_delay_ms)
+ acpigen_write_sleep(reset_delay_ms);
+ }
+ acpigen_pop_len(); /* _ON method */
+
+ /* Method (_OFF, 0, Serialized) */
+ acpigen_write_method_serialized("_OFF", 0);
+ if (reset_gpio)
+ acpigen_soc_set_tx_gpio(reset_gpio);
+ if (enable_gpio)
+ acpigen_soc_clear_tx_gpio(enable_gpio);
+ acpigen_pop_len(); /* _OFF method */
+
+ acpigen_pop_len(); /* PowerResource PRIC */
+}
+
static void acpi_dp_write_array(const struct acpi_dp *array);
static void acpi_dp_write_value(const struct acpi_dp *prop)
{
diff --git a/src/arch/x86/include/arch/acpi_device.h b/src/arch/x86/include/arch/acpi_device.h
index aba5c2d1a6..7d4cb8fd89 100644
--- a/src/arch/x86/include/arch/acpi_device.h
+++ b/src/arch/x86/include/arch/acpi_device.h
@@ -240,6 +240,15 @@ struct acpi_spi {
void acpi_device_write_spi(const struct acpi_spi *spi);
/*
+ * Add a basic PowerResource block for a device that includes
+ * GPIOs for enable and/or reset control of the device. Each
+ * GPIO is optional, but at least one must be provided.
+ */
+void acpi_device_add_power_res(
+ struct acpi_gpio *reset, unsigned reset_delay_ms,
+ struct acpi_gpio *enable, unsigned enable_delay_ms);
+
+/*
* Writing Device Properties objects via _DSD
*
* http://uefi.org/sites/default/files/resources/_DSD-device-properties-UUID.pdf
diff --git a/src/drivers/i2c/generic/generic.c b/src/drivers/i2c/generic/generic.c
index a90947f1b4..dea78d5e89 100644
--- a/src/drivers/i2c/generic/generic.c
+++ b/src/drivers/i2c/generic/generic.c
@@ -26,50 +26,6 @@
#if IS_ENABLED(CONFIG_HAVE_ACPI_TABLES)
-static void i2c_generic_add_power_res(struct drivers_i2c_generic_config *config)
-{
- const char *power_res_dev_states[] = { "_PR0", "_PR3" };
- unsigned reset_gpio = config->reset_gpio.pins[0];
- unsigned enable_gpio = config->enable_gpio.pins[0];
-
- if (!config->has_power_resource)
- return;
-
- if (!reset_gpio && !enable_gpio)
- return;
-
- /* PowerResource (PRIC, 0, 0) */
- acpigen_write_power_res("PRIC", 0, 0, power_res_dev_states,
- ARRAY_SIZE(power_res_dev_states));
-
- /* Method (_STA, 0, NotSerialized) { Return (0x1) } */
- acpigen_write_STA(0x1);
-
- /* Method (_ON, 0, Serialized) */
- acpigen_write_method_serialized("_ON", 0);
- if (reset_gpio)
- acpigen_soc_set_tx_gpio(reset_gpio);
- if (enable_gpio) {
- acpigen_soc_set_tx_gpio(enable_gpio);
- acpigen_write_sleep(config->enable_delay_ms);
- }
- if (reset_gpio) {
- acpigen_soc_clear_tx_gpio(reset_gpio);
- acpigen_write_sleep(config->reset_delay_ms);
- }
- acpigen_pop_len(); /* _ON method */
-
- /* Method (_OFF, 0, Serialized) */
- acpigen_write_method_serialized("_OFF", 0);
- if (reset_gpio)
- acpigen_soc_set_tx_gpio(reset_gpio);
- if (enable_gpio)
- acpigen_soc_clear_tx_gpio(enable_gpio);
- acpigen_pop_len(); /* _OFF method */
-
- acpigen_pop_len(); /* PowerResource PRIC */
-}
-
static bool i2c_generic_add_gpios_to_crs(struct drivers_i2c_generic_config *cfg)
{
/*
@@ -170,7 +126,10 @@ void i2c_generic_fill_ssdt(struct device *dev,
}
/* Power Resource */
- i2c_generic_add_power_res(config);
+ if (config->has_power_resource)
+ acpi_device_add_power_res(
+ &config->reset_gpio, config->reset_delay_ms,
+ &config->enable_gpio, config->enable_delay_ms);
/* Callback if any. */
if (callback)