From: Dan Carpenter Date: Tue, 15 Mar 2011 07:06:23 +0000 (+0300) Subject: asus-wmi: signedness bug in read_brightness() X-Git-Tag: v3.0~1130^2^2~7 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=0986f25fbba8827762b35222bb45d2fbb2d72fa4;p=platform%2Fkernel%2Flinux-amlogic.git asus-wmi: signedness bug in read_brightness() "err" needs to be signed for the error handling to work. Signed-off-by: Dan Carpenter Signed-off-by: Matthew Garrett --- 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);