erebion/coreboot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

soc/intel/common/block/sata: Fix scope for SATA ACPI device

Browse source 
acpi_device_path() includes the device name, so we end up with:
Scope (\_SB.PCI0.SATA) {
    Device (SATA) {
...

Fix this by using acpi_device_scope() instead.

TEST=build/boot purism librem_cml (Mini v2), dump ACPI and verify
SATA device scope correct.

Change-Id: Ibbc8890d93b22f0ecba4b3a9b0531994574b3d55
Signed-off-by: Matt DeVillier <matt.devillier@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/80554
Reviewed-by: Subrata Banik <subratabanik@google.com>
Reviewed-by: Sean Rhodes <sean@starlabs.systems>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Reviewed-by: Felix Singer <service+coreboot-gerrit@felixsinger.de>
This commit is contained in:
Matt DeVillier 2024-02-15 14:58:14 -06:00 • committed by Felix Singer
commit 1f250767ed
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
src/soc/intel/common/block/sata/sata.c
View file

@ -9,7 +9,7 @@
static void sata_acpi_fill_ssdt(const struct device *dev)
{
const char *scope = acpi_device_path(dev);
const char *scope = acpi_device_scope(dev);
if (!scope)
return;