On Tue, Mar 15, 2011 at 8:06 AM, Dan Carpenter <[email protected]> wrote:
> "err" needs to be signed for the error handling to work.
>
> Signed-off-by: Dan Carpenter <[email protected]>
>
> diff --git a/drivers/platform/x86/asus-wmi.c b/drivers/platform/x86/asus-wmi.c
> index 5b779a9..a038595 100644
> --- a/drivers/platform/x86/asus-wmi.c
> +++ b/drivers/platform/x86/asus-wmi.c
> @@ -946,7 +946,8 @@ static int read_brightness_max(struct asus_wmi *asus)
>  static int read_brightness(struct backlight_device *bd)
>  {
>        struct asus_wmi *asus = bl_get_data(bd);
> -       u32 retval, err;
> +       u32 retval;
> +       int err;
>
>        err = asus_wmi_get_devstate(asus, ASUS_WMI_DEVID_BRIGHTNESS, &retval);

Hm, Thanks,

Acked-by: Corentin Chary <[email protected]>

-- 
Corentin Chary
http://xf.iksaif.net
--
To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" 
in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to