Hi Michał, On 10/8/22 20:49, Michał Szczepaniak wrote: > Thinklight has only two values, on/off so it's reasonable for max_brightness to be 0 and 1 as if you write anything between 0 and 255 it will be 255 anyway so there's no point for it to be 255. > > Also sorry for repeated messages, I got no reply so I'm trying to figure out what I did wrong. Your patch is missing a Signed-off-by line in the commit-message. I can only accept patches with a Signed-off-by line in the commit-message like this: Signed-off-by: Your Real Name <email@your.domain> See: https://www.kernel.org/doc/html/latest/process/submitting-patches.html#sign-your-work-the-developer-s-certificate-of-origin Also your patch is malformed (it has been reflowed by your email client) and it does not apply properly. Since this is only a trivial single line change I've just manually recreated the patch myself, with me as author to avoid the S-o-b problem. I have given you credit by adding: Reported-by: Michał Szczepaniak <m.szczepaniak.000@xxxxxxxxx> to the commit message of the patch. Regards, Hans > > > --- > drivers/platform/x86/thinkpad_acpi.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/platform/x86/thinkpad_acpi.c b/drivers/platform/x86/thinkpad_acpi.c > index 22d4e8633e30e9..775c2f327dc3a8 100644 > --- a/drivers/platform/x86/thinkpad_acpi.c > +++ b/drivers/platform/x86/thinkpad_acpi.c > @@ -5562,6 +5562,7 @@ static enum led_brightness light_sysfs_get(struct led_classdev *led_cdev) > static struct tpacpi_led_classdev tpacpi_led_thinklight = { > .led_classdev = { > .name = "tpacpi::thinklight", > + .max_brightness = 1, > .brightness_set_blocking = &light_sysfs_set, > .brightness_get = &light_sysfs_get, > } > > > >
