diff options
| author | Mike Larkin <mlarkin@cvs.openbsd.org> | 2014-05-18 20:16:30 +0000 |
|---|---|---|
| committer | Mike Larkin <mlarkin@cvs.openbsd.org> | 2014-05-18 20:16:30 +0000 |
| commit | e53f000172179cfd54e5d0674f07e4cd518c7e95 (patch) | |
| tree | 9094bc211fb088b509b78edc7410b9f4dc9ded9a /sys/dev | |
| parent | ac317b60340d7940e9d4175349465d873fd48944 (diff) | |
_PIC is an optional method, so we shouldn't be aborting the rest of the
MADT processing if it is missing. This, combined with an earlier diff
from kettenis@ to fix MADT interrupt polarity processing, should resolve
the issues reported with QEMU/KVM ACPI power button presses when using
OpenBSD guests.
ok kettenis@, pirofti@
tested by myself and kettenis for the past few weeks
Diffstat (limited to 'sys/dev')
| -rw-r--r-- | sys/dev/acpi/acpimadt.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/sys/dev/acpi/acpimadt.c b/sys/dev/acpi/acpimadt.c index f012cf37365..1fc2e64aebf 100644 --- a/sys/dev/acpi/acpimadt.c +++ b/sys/dev/acpi/acpimadt.c @@ -1,4 +1,4 @@ -/* $OpenBSD: acpimadt.c,v 1.26 2012/01/07 20:13:16 kettenis Exp $ */ +/* $OpenBSD: acpimadt.c,v 1.27 2014/05/18 20:16:29 mlarkin Exp $ */ /* * Copyright (c) 2006 Mark Kettenis <kettenis@openbsd.org> * @@ -216,8 +216,7 @@ acpimadt_attach(struct device *parent, struct device *self, void *aux) arg.type = AML_OBJTYPE_INTEGER; arg.v_integer = 1; - if (aml_evalname(acpi_sc, NULL, "\\_PIC", 1, &arg, NULL) != 0) - return; + aml_evalname(acpi_sc, NULL, "\\_PIC", 1, &arg, NULL); mp_busses = acpimadt_busses; mp_nbusses = nitems(acpimadt_busses); |