diff options
| author | Harsha Priya <harshapriya.n@intel.com> | 2016-07-15 17:31:43 -0700 |
|---|---|---|
| committer | Duncan Laurie <dlaurie@chromium.org> | 2016-07-17 02:26:13 +0200 |
| commit | 3a96ac44e275eca84baea513bc0802f62fe83fd3 (patch) | |
| tree | 40d3624043fa572d34d52c73cd0b46ecec6c556c | |
| parent | 6101dfbaf34224db9743f807f97bc4f61106a390 (diff) | |
acpi: Change API called to write the name for ACPI_DP_TYPE_CHILD
The API called to write the name of the child table in the
dp entry (type ACPI_DP_TYPE_CHILD) was not including the
quotes, e.g., it was DAAD and not "DAAD". Thus, the kernel driver
did not get the right information from SSDT.
Change the API to acpigen_write_string() to fix the issue.
Signed-off-by: Harsha Priya <harshapriya.n@intel.com>
Change-Id: Id33ad29e637bf1fe6b02e8a4b0fd9e220e8984e7
Reviewed-on: https://review.coreboot.org/15724
Tested-by: build bot (Jenkins)
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
| -rw-r--r-- | src/arch/x86/acpi_device.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/arch/x86/acpi_device.c b/src/arch/x86/acpi_device.c index ce31dc8e21..2982999fe5 100644 --- a/src/arch/x86/acpi_device.c +++ b/src/arch/x86/acpi_device.c @@ -498,10 +498,10 @@ static void acpi_dp_write_value(const struct acpi_dp *prop) acpigen_write_integer(prop->integer); break; case ACPI_DP_TYPE_STRING: + case ACPI_DP_TYPE_CHILD: acpigen_write_string(prop->string); break; case ACPI_DP_TYPE_REFERENCE: - case ACPI_DP_TYPE_CHILD: acpigen_emit_namestring(prop->string); break; case ACPI_DP_TYPE_ARRAY: |