acpi: Add support for writing a Power Resource

These are used in ACPI to disable power to various pats of the system when
in sleep. Add a way to create a power resource, with the caller finishing
off the details.

Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Wolfgang Wallner <wolfgang.wallner@br-automation.com>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>

diff --git a/include/acpi/acpigen.h b/include/acpi/acpigen.h
index 2dd806aa273bbf93de618356863bd9169c393794..2c07a5620824a329ed513a81b4fba50658bd7a4d 100644 (file)
--- a/include/acpi/acpigen.h
+++ b/include/acpi/acpigen.h
@@ -51,6 +51,7 @@ enum {
        AND_OP                  = 0x7b,
        OR_OP                   = 0x7d,
        NOT_OP                  = 0x80,
+       POWER_RES_OP            = 0x84,
        RETURN_OP               = 0xa4,
 };
 
@@ -321,4 +322,25 @@ void acpigen_write_and(struct acpi_ctx *ctx, u8 arg1, u8 arg2, u8 res);
  */
 void acpigen_write_not(struct acpi_ctx *ctx, u8 arg, u8 res);
 
+/**
+ * acpigen_write_power_res() - Write a power resource
+ *
+ * Name (_PRx, Package(One) { name })
+ * ...
+ * PowerResource (name, level, order)
+ *
+ * The caller should fill in the rest of the power resource and then call
+ * acpigen_pop_len() to close it off
+ *
+ * @ctx: ACPI context pointer
+ * @name: Name of power resource (e.g. "PRIC")
+ * @level: Deepest sleep level that this resource must be kept on (0=S0, 3=S3)
+ * @order: Order that this must be enabled/disabled (e.g. 0)
+ * @dev_stats: List of states to define, e.g. {"_PR0", "_PR3"}
+ * @dev_states_count: Number of dev states
+ */
+void acpigen_write_power_res(struct acpi_ctx *ctx, const char *name, uint level,
+                            uint order, const char *const dev_states[],
+                            size_t dev_states_count);
+
 #endif

diff --git a/lib/acpi/acpigen.c b/lib/acpi/acpigen.c
index 4fd29ccb81e9ee6091027e1601d36e36cd6a29fc..70d734245bb9d03ccb796d9f3d33ce58d5da425e 100644 (file)
--- a/lib/acpi/acpigen.c
+++ b/lib/acpi/acpigen.c
@@ -328,6 +328,28 @@ int acpigen_write_uuid(struct acpi_ctx *ctx, const char *uuid)
        return 0;
 }
 
+void acpigen_write_power_res(struct acpi_ctx *ctx, const char *name, uint level,
+                            uint order, const char *const dev_states[],
+                            size_t dev_states_count)
+{
+       size_t i;
+
+       for (i = 0; i < dev_states_count; i++) {
+               acpigen_write_name(ctx, dev_states[i]);
+               acpigen_write_package(ctx, 1);
+               acpigen_emit_simple_namestring(ctx, name);
+               acpigen_pop_len(ctx);           /* Package */
+       }
+
+       acpigen_emit_ext_op(ctx, POWER_RES_OP);
+
+       acpigen_write_len_f(ctx);
+
+       acpigen_emit_simple_namestring(ctx, name);
+       acpigen_emit_byte(ctx, level);
+       acpigen_emit_word(ctx, order);
+}
+
 /* Sleep (ms) */
 void acpigen_write_sleep(struct acpi_ctx *ctx, u64 sleep_ms)
 {

diff --git a/test/dm/acpigen.c b/test/dm/acpigen.c
index 63744454539c8b4aefc43a76872ceddb29979f32..df062a210c73dcdb1b04a9b8fe0345f00b70e9fd 100644 (file)
--- a/test/dm/acpigen.c
+++ b/test/dm/acpigen.c
@@ -701,3 +701,44 @@ static int dm_test_acpi_misc(struct unit_test_state *uts)
        return 0;
 }
 DM_TEST(dm_test_acpi_misc, 0);
+
+/* Test writing an ACPI power resource */
+static int dm_test_acpi_power_res(struct unit_test_state *uts)
+{
+       const char *const states[] = { "_PR0", "_PR3" };
+       const char *name = "PRIC";
+       const int level = 3;
+       const int order = 2;
+       struct acpi_ctx *ctx;
+       u8 *ptr;
+
+       ut_assertok(alloc_context(&ctx));
+
+       ptr = acpigen_get_current(ctx);
+
+       /* PowerResource (PRIC, 0, 0) */
+       acpigen_write_power_res(ctx, name, level, order, states,
+                               ARRAY_SIZE(states));
+       ut_asserteq(0x28, acpigen_get_current(ctx) - ptr);
+       ut_asserteq(NAME_OP, ptr[0]);
+       ut_asserteq_strn(states[0], (char *)ptr + 1);
+       ut_asserteq(8, acpi_test_get_length(ptr + 6));
+       ut_asserteq_strn(name, (char *)ptr + 0xa);
+
+       ut_asserteq_strn(states[1], (char *)ptr + 0xf);
+       ut_asserteq(8, acpi_test_get_length(ptr + 0x14));
+       ut_asserteq_strn(name, (char *)ptr + 0x18);
+
+       ut_asserteq(POWER_RES_OP, ptr[0x1d]);
+       ut_asserteq_strn(name, (char *)ptr + 0x21);
+       ut_asserteq(level, ptr[0x25]);
+       ut_asserteq(order, get_unaligned((u16 *)(ptr + 0x26)));
+
+       /* The length is not set - caller must use acpigen_pop_len() */
+       ut_asserteq(1, ctx->ltop);
+
+       free_context(&ctx);
+
+       return 0;
+}
+DM_TEST(dm_test_acpi_power_res, 0);