platform/x86: thinkpad_acpi: Stop setting led_classdev brightness directly
authorHans de Goede <hdegoede@redhat.com>
Thu, 9 Feb 2017 15:44:11 +0000 (16:44 +0100)
committerDarren Hart <dvhart@linux.intel.com>
Sat, 25 Feb 2017 07:48:51 +0000 (23:48 -0800)
There is no need to set the led_classdev's brightness value from
its set_brightness callback, this is taken care of by the led-core and
thinkpad_acpi really should not be mucking with it.

Note that kbdlight_set_level_and_update() is still used by the old
thinpad_acpi specific sysfs interface for the led, so we cannot
remove it.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Acked-by: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Acked-by: Pavel Machek <pavel@ucw.cz>
drivers/platform/x86/thinkpad_acpi.c

index cacb43fb1df7fcbd5f643788fa960a4cdb6e4e36..0680bb3952044a20606da23be1bb588762522ba5 100644 (file)
@@ -5095,8 +5095,6 @@ static int kbdlight_set_level(int level)
        return 0;
 }
 
-static int kbdlight_set_level_and_update(int level);
-
 static int kbdlight_get_level(void)
 {
        int status = 0;
@@ -5164,7 +5162,7 @@ static void kbdlight_set_worker(struct work_struct *work)
                        container_of(work, struct tpacpi_led_classdev, work);
 
        if (likely(tpacpi_lifecycle == TPACPI_LIFE_RUNNING))
-               kbdlight_set_level_and_update(data->new_state);
+               kbdlight_set_level(data->new_state);
 }
 
 static void kbdlight_sysfs_set(struct led_classdev *led_cdev,