diff options
author | Bob Moore <robert.moore@intel.com> | 2009-09-02 21:55:40 -0400 |
---|---|---|
committer | Len Brown <len.brown@intel.com> | 2009-09-08 22:28:33 -0400 |
commit | 307a042416dfc2216251a85b79e8578b65fdc0e7 (patch) | |
tree | 917923b931b3fc2fa6da212169d1758e2d39d3bf /drivers/acpi/acpica | |
parent | 718fb0de8ff88f71b3b91a8ee8e42e60c88e5128 (diff) |
ACPICA: Fix extraneous warning if _DSM returns a package
_DSM can return any type of object, so validation on the return
type cannot be performed. ACPICA BZ 802.
http://www.acpica.org/bugzilla/show_bug.cgi?id=802
Signed-off-by: Bob Moore <robert.moore@intel.com>
Signed-off-by: Lin Ming <ming.m.lin@intel.com>
Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'drivers/acpi/acpica')
-rw-r--r-- | drivers/acpi/acpica/nspredef.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/drivers/acpi/acpica/nspredef.c b/drivers/acpi/acpica/nspredef.c index 8314e6a9e72..f8427afeebd 100644 --- a/drivers/acpi/acpica/nspredef.c +++ b/drivers/acpi/acpica/nspredef.c | |||
@@ -193,11 +193,15 @@ acpi_ns_check_predefined_names(struct acpi_namespace_node *node, | |||
193 | } | 193 | } |
194 | 194 | ||
195 | /* | 195 | /* |
196 | * We have a return value, but if one wasn't expected, just exit, this is | 196 | * 1) We have a return value, but if one wasn't expected, just exit, this is |
197 | * not a problem. For example, if the "Implicit Return" feature is | 197 | * not a problem. For example, if the "Implicit Return" feature is |
198 | * enabled, methods will always return a value. | 198 | * enabled, methods will always return a value. |
199 | * | ||
200 | * 2) If the return value can be of any type, then we cannot perform any | ||
201 | * validation, exit. | ||
199 | */ | 202 | */ |
200 | if (!predefined->info.expected_btypes) { | 203 | if ((!predefined->info.expected_btypes) || |
204 | (predefined->info.expected_btypes == ACPI_RTYPE_ALL)) { | ||
201 | goto cleanup; | 205 | goto cleanup; |
202 | } | 206 | } |
203 | 207 | ||