]> git.ipfire.org Git - thirdparty/qemu.git/commitdiff
hw/i386: Fix _STA return value for ACPI0017
authorJonathan Cameron <Jonathan.Cameron@huawei.com>
Fri, 26 Jan 2024 12:01:31 +0000 (12:01 +0000)
committerMichael Tokarev <mjt@tls.msk.ru>
Thu, 15 Feb 2024 08:28:42 +0000 (11:28 +0300)
Found whilst testing a series for the linux kernel that actually
bothers to check if enabled is set. 0xB is the option used
for vast majority of DSDT entries in QEMU.
It is a little odd for a device that doesn't really exist and
is simply a hook to tell the OS there is a CEDT table but 0xB
seems a reasonable choice and avoids need to special case
this device in the OS.

Means:
* Device present.
* Device enabled and decoding it's resources.
* Not shown in UI
* Functioning properly
* No battery (on this device!)

Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Message-Id: <20240126120132.24248-12-Jonathan.Cameron@huawei.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
(cherry picked from commit d9ae5802f656f6fb53b788747ba557a826b6e740)
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
hw/i386/acpi-build.c

index d9eaa5fc4d83a412b2c594fa197d93bb18f4dbd7..f9cdacadb16e1ee14795ff1181761147e38fb5de 100644 (file)
@@ -1311,7 +1311,7 @@ static void build_acpi0017(Aml *table)
     aml_append(dev, aml_name_decl("_HID", aml_string("ACPI0017")));
 
     method = aml_method("_STA", 0, AML_NOTSERIALIZED);
-    aml_append(method, aml_return(aml_int(0x01)));
+    aml_append(method, aml_return(aml_int(0x0B)));
     aml_append(dev, method);
 
     aml_append(scope, dev);