android/android_kernel_samsung_sm8650
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

platform/x86: sony-laptop: Don't turn off 0x153 keyboard backlight during probe

Browse Source 
commit ad75bd85b1db69c97eefea07b375567821f6ef58 upstream.

The 0x153 version of the kbd backlight control SNC handle has no separate
address to probe if the backlight is there.

This turns the probe call into a set keyboard backlight call with a value
of 0 turning off the keyboard backlight.

Skip probing when there is no separate probe address to avoid this.

Link: https://bugzilla.redhat.com/show_bug.cgi?id=1583752
Fixes: 800f20170d ("Keyboard backlight control for some Vaio Fit models")
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Reviewed-by: Mattia Dongili <malattia@linux.it>
Link: https://lore.kernel.org/r/20221213122943.11123-1-hdegoede@redhat.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Hans de Goede 2022-12-13 13:29:43 +01:00 committed by Greg Kroah-Hartman
parent 0f21e225ff
commit fb8534b796
1 changed files with 14 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
drivers/platform/x86/sony-laptop.c
View File

@ -1888,6 +1888,12 @@ static int sony_nc_kbd_backlight_setup(struct platform_device *pd,
break; break;
} }
/*
* Only probe if there is a separate probe_base, otherwise the probe call
* is equivalent to __sony_nc_kbd_backlight_mode_set(0), resulting in
* the keyboard backlight being turned off.
*/
if (probe_base) {
ret = sony_call_snc_handle(handle, probe_base, &result); ret = sony_call_snc_handle(handle, probe_base, &result);
if (ret) if (ret)
return ret; return ret;
@ -1897,6 +1903,7 @@ static int sony_nc_kbd_backlight_setup(struct platform_device *pd,
dprintk("no backlight keyboard found\n"); dprintk("no backlight keyboard found\n");
return 0; return 0;
} }
}
kbdbl_ctl = kzalloc(sizeof(*kbdbl_ctl), GFP_KERNEL); kbdbl_ctl = kzalloc(sizeof(*kbdbl_ctl), GFP_KERNEL);
if (!kbdbl_ctl) if (!kbdbl_ctl)