summaryrefslogtreecommitdiff
path: root/payloads/coreinfo/pci_module.c
diff options
context:
space:
mode:
authorReka Norman <rekanorman@google.com>2022-11-30 16:33:16 +1100
committerEric Lai <eric_lai@quanta.corp-partner.google.com>2022-12-12 01:29:27 +0000
commit1d49d3e40b1bc1dcd25e26781277f3fa6409551b (patch)
tree98c4854612f674f4cff667d14b7fda050ee6a5d7 /payloads/coreinfo/pci_module.c
parent122e1dfe5d992788b636e803dd9aa76ba5497220 (diff)
mb/google/brya: Don't add MPTS to both DSDT and SSDT
commit 52ccd293d7 ("mb/google/brya: Implement shutdown function for dGPU") started unconditionally adding MPTS to the SSDT. On variants with HAVE_WWAN_POWER_SEQUENCE selected, MPTS is already added to the DSDT via wwan_power.asl. The duplicate definition results in a kernel error: ERR kernel: [ 0.109237] ACPI BIOS Error (bug): Failure creating named object [\_SB.MPTS], AE_ALREADY_EXISTS (20210730/dswload2-327) ERR kernel: [ 0.109242] ACPI Error: AE_ALREADY_EXISTS, During name lookup/catalog (20210730/psobject-220) Don't add MPTS to the SSDT if HAVE_WWAN_POWER_SEQUENCE is selected. There are no variants which use both, so this should only result in empty MPTS methods being removed. BUG=b:260380268 TEST=On pujjo, the SSDT no longer contains an empty MPTS method, there's no kernel error, and the WWAN power-off sequence is met. Change-Id: I9f411aae81ea87aa9c8fc7754c3709e398771a32 Signed-off-by: Reka Norman <rekanorman@chromium.org> Reviewed-on: https://review.coreboot.org/c/coreboot/+/70146 Reviewed-by: Kangheui Won <khwon@chromium.org> Reviewed-by: Subrata Banik <subratabanik@google.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'payloads/coreinfo/pci_module.c')
0 files changed, 0 insertions, 0 deletions