diff options
author | Jean Delvare <jdelvare@suse.de> | 2009-09-08 09:31:46 -0400 |
---|---|---|
committer | Len Brown <len.brown@intel.com> | 2009-09-27 03:12:13 -0400 |
commit | 14f03343ad1080c2fea29ab2c13f05b976c4584e (patch) | |
tree | ec3b36bc0a274ea382f5816a6919d82a59ac110f /drivers | |
parent | b684a3637e0887683a0a3d6fd471fc41d7c1606a (diff) |
ACPI: Clarify resource conflict message
The message "ACPI: Device needs an ACPI driver" is misleading. The
device _may_ need an ACPI driver, if the BIOS implemented a custom
API for the device in question (which, AFAIK, can't be checked.) If
not, then either a generic ACPI driver may be used (for example
"thermal"), or nothing can be done (other than a white list).
I propose to reword the message to:
ACPI: If an ACPI driver is available for this device, you should use
it instead of the native driver
which I think is more correct. Comments and suggestions welcome.
I also added a message warning about possible problems and system
instability when users pass acpi_enforce_resources=lax, as suggested
by Len.
Signed-off-by: Jean Delvare <jdelvare@suse.de>
Cc: Thomas Renninger <trenn@suse.de>
Cc: Alan Jenkins <sourcejedi.lkml@googlemail.com>
Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/acpi/osl.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/drivers/acpi/osl.c b/drivers/acpi/osl.c index 5633b86e3ed..7c1c59ea9ec 100644 --- a/drivers/acpi/osl.c +++ b/drivers/acpi/osl.c | |||
@@ -1161,7 +1161,13 @@ int acpi_check_resource_conflict(struct resource *res) | |||
1161 | res_list_elem->name, | 1161 | res_list_elem->name, |
1162 | (long long) res_list_elem->start, | 1162 | (long long) res_list_elem->start, |
1163 | (long long) res_list_elem->end); | 1163 | (long long) res_list_elem->end); |
1164 | printk(KERN_INFO "ACPI: Device needs an ACPI driver\n"); | 1164 | if (acpi_enforce_resources == ENFORCE_RESOURCES_LAX) |
1165 | printk(KERN_NOTICE "ACPI: This conflict may" | ||
1166 | " cause random problems and system" | ||
1167 | " instability\n"); | ||
1168 | printk(KERN_INFO "ACPI: If an ACPI driver is available" | ||
1169 | " for this device, you should use it instead of" | ||
1170 | " the native driver\n"); | ||
1165 | } | 1171 | } |
1166 | if (acpi_enforce_resources == ENFORCE_RESOURCES_STRICT) | 1172 | if (acpi_enforce_resources == ENFORCE_RESOURCES_STRICT) |
1167 | return -EBUSY; | 1173 | return -EBUSY; |