diff options
| author | Mark Kettenis <kettenis@cvs.openbsd.org> | 2016-01-13 23:11:23 +0000 |
|---|---|---|
| committer | Mark Kettenis <kettenis@cvs.openbsd.org> | 2016-01-13 23:11:23 +0000 |
| commit | 0266bc4fff3b9f50badd8cf517f162f80cd73e3c (patch) | |
| tree | 0956523787cfae3fea3afd36af277fd1312e688e /sys/dev/acpi/dsdt.h | |
| parent | 034ac86a8271f72b97dd4bcd569127b7d3989645 (diff) | |
Change aml_find_node() such that it only walks down the tree and doesn't
traverse sideways. This seems to be what all callersexpect it to do, and
fixes a bug in dwiic(4) where it would try to access i2c devices on busses
they're not attached to.
If there is any fallout from this change, the right thing to do is probably
to make sure callers pass the right node.
While there, change the return type to void, as the return value was useless
and none of the callers looked at it.
ok mlarkin@
Diffstat (limited to 'sys/dev/acpi/dsdt.h')
| -rw-r--r-- | sys/dev/acpi/dsdt.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/dev/acpi/dsdt.h b/sys/dev/acpi/dsdt.h index 8f04ef2ad23..bcecbae824e 100644 --- a/sys/dev/acpi/dsdt.h +++ b/sys/dev/acpi/dsdt.h @@ -1,4 +1,4 @@ -/* $OpenBSD: dsdt.h,v 1.66 2016/01/13 10:11:43 kettenis Exp $ */ +/* $OpenBSD: dsdt.h,v 1.67 2016/01/13 23:11:22 kettenis Exp $ */ /* * Copyright (c) 2005 Marco Peereboom <marco@openbsd.org> * @@ -53,7 +53,7 @@ void aml_showvalue(struct aml_value *, int); void aml_walkroot(void); void aml_walktree(struct aml_node *); -int aml_find_node(struct aml_node *, const char *, +void aml_find_node(struct aml_node *, const char *, int (*)(struct aml_node *, void *), void *); int acpi_parse_aml(struct acpi_softc *, u_int8_t *, u_int32_t); |