From 6605610587d459c5836dcb1cbb325f3212cf8f17 Mon Sep 17 00:00:00 2001 From: kettenis Date: Thu, 30 Aug 2018 10:11:34 +0000 Subject: [PATCH] Move softbutton check before the check that skips attaching certain acpi devices. Fixes regression caused by matching on _CID in addition to matching on _HID. ok matthieu@, mlarkin@ --- sys/dev/acpi/acpi.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/sys/dev/acpi/acpi.c b/sys/dev/acpi/acpi.c index 7f44f8b0778..8cb5bb39d60 100644 --- a/sys/dev/acpi/acpi.c +++ b/sys/dev/acpi/acpi.c @@ -1,4 +1,4 @@ -/* $OpenBSD: acpi.c,v 1.358 2018/08/25 09:39:20 kettenis Exp $ */ +/* $OpenBSD: acpi.c,v 1.359 2018/08/30 10:11:34 kettenis Exp $ */ /* * Copyright (c) 2005 Thorsten Lockert * Copyright (c) 2005 Jordan Hargrave @@ -3065,10 +3065,6 @@ acpi_foundhid(struct aml_node *node, void *arg) aaa.aaa_dev = dev; aaa.aaa_cdev = cdev; - if (acpi_matchhids(&aaa, acpi_skip_hids, "none") || - acpi_matchhids(&aaa, acpi_isa_hids, "none")) - return (0); - #ifndef SMALL_KERNEL if (!strcmp(cdev, ACPI_DEV_MOUSE)) { for (i = 0; i < nitems(sbtn_pnp); i++) { @@ -3080,6 +3076,10 @@ acpi_foundhid(struct aml_node *node, void *arg) } #endif + if (acpi_matchhids(&aaa, acpi_skip_hids, "none") || + acpi_matchhids(&aaa, acpi_isa_hids, "none")) + return (0); + if (!node->parent->attached) { node->parent->attached = 1; config_found(self, &aaa, acpi_print); -- 2.20.1