diff options
| author | Furquan Shaikh <furquan@google.com> | 2021-02-19 10:23:17 -0800 |
|---|---|---|
| committer | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2021-02-20 21:32:17 +0000 |
| commit | ac204ba6b59bfc7d0f8c063722050b2fdb08fbb9 (patch) | |
| tree | 570b6ad7fada9e9d5c33b7ef61cf48d59edf5675 | |
| parent | 9de060fd34e53a797528e61b8d2f4e1afe99be00 (diff) | |
soc/amd/stoneyridge: Fix _INI method in SSDT for HDA
CB:40785 ("soc/amd/hda: Move HDA PCI device from DSDT to SSDT") moved
the HDA device in ACPI from DSDT to SSDT. During this, _INI method
generated in SSDT incorrectly inverted the values for NSEN, NSDO and
NSDI. This change fixes the mistake so that the _INI in SSDT matches
the original _INI in DSDT for HDA device.
Change-Id: I294b561a479b77ab8afb5f3e0de367ad24f3a764
Reported-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Signed-off-by: Furquan Shaikh <furquan@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/50927
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
| -rw-r--r-- | src/soc/amd/stoneyridge/northbridge.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/soc/amd/stoneyridge/northbridge.c b/src/soc/amd/stoneyridge/northbridge.c index c5ac90afa9..2c90d04ff9 100644 --- a/src/soc/amd/stoneyridge/northbridge.c +++ b/src/soc/amd/stoneyridge/northbridge.c @@ -530,9 +530,9 @@ void hda_soc_ssdt_quirks(const struct device *dev) acpigen_write_if_lequal_namestr_int("OSVR", 0x03); - acpigen_write_store_op_to_namestr(ONE_OP, "NSEN"); - acpigen_write_store_op_to_namestr(ZERO_OP, "NSDO"); - acpigen_write_store_op_to_namestr(ZERO_OP, "NSDI"); + acpigen_write_store_op_to_namestr(ZERO_OP, "NSEN"); + acpigen_write_store_op_to_namestr(ONE_OP, "NSDO"); + acpigen_write_store_op_to_namestr(ONE_OP, "NSDI"); acpigen_pop_len(); /* If */ |