diff mbox

ACPI: unsigned cannot be less than 0.

Message ID 49AB0AA1.1090207@gmail.com (mailing list archive)
State Accepted
Headers show

Commit Message

Roel Kluin March 1, 2009, 10:22 p.m. UTC
include/linux/pci-acpi.h:74:
typedef u32                 acpi_status;
------------------------------>8-------------8<---------------------------------
result is unsigned, so an error returned by acpi_bus_register_driver() will not
be noticed.

Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
---
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Len Brown March 16, 2009, 2:42 a.m. UTC | #1
applied
thanks,
Len Brown, Intel Open Source Technology Center

On Sun, 1 Mar 2009, Roel Kluin wrote:

> include/linux/pci-acpi.h:74:
> typedef u32                 acpi_status;
> ------------------------------>8-------------8<---------------------------------
> result is unsigned, so an error returned by acpi_bus_register_driver() will not
> be noticed.
> 
> Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
> ---
> diff --git a/drivers/platform/x86/wmi.c b/drivers/platform/x86/wmi.c
> index 8a8b377..2f269e1 100644
> --- a/drivers/platform/x86/wmi.c
> +++ b/drivers/platform/x86/wmi.c
> @@ -708,7 +708,7 @@ static int __init acpi_wmi_add(struct acpi_device *device)
>  
>  static int __init acpi_wmi_init(void)
>  {
> -	acpi_status result;
> +	int result;
>  
>  	INIT_LIST_HEAD(&wmi_blocks.list);
>  
> --
> To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/platform/x86/wmi.c b/drivers/platform/x86/wmi.c
index 8a8b377..2f269e1 100644
--- a/drivers/platform/x86/wmi.c
+++ b/drivers/platform/x86/wmi.c
@@ -708,7 +708,7 @@  static int __init acpi_wmi_add(struct acpi_device *device)
 
 static int __init acpi_wmi_init(void)
 {
-	acpi_status result;
+	int result;
 
 	INIT_LIST_HEAD(&wmi_blocks.list);