Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1161043AbcKPA15 (ORCPT ); Tue, 15 Nov 2016 19:27:57 -0500 Received: from hera.aquilenet.fr ([141.255.128.1]:38928 "EHLO hera.aquilenet.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753053AbcKPA1x (ORCPT ); Tue, 15 Nov 2016 19:27:53 -0500 X-Greylist: delayed 474 seconds by postgrey-1.27 at vger.kernel.org; Tue, 15 Nov 2016 19:27:53 EST Date: Wed, 16 Nov 2016 01:19:53 +0100 From: Samuel Thibault To: "Maciej S. Szmigiero" Cc: Greg Kroah-Hartman , Jiri Slaby , Dmitry Torokhov , linux-kernel Subject: Re: [PATCH] vt: fix Scroll Lock LED trigger name Message-ID: <20161116001953.GQ2448@var.home> Mail-Followup-To: Samuel Thibault , "Maciej S. Szmigiero" , Greg Kroah-Hartman , Jiri Slaby , Dmitry Torokhov , linux-kernel References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21+34 (58baf7c9f32f) (2010-12-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1668 Lines: 41 Maciej S. Szmigiero, on Wed 16 Nov 2016 00:55:57 +0100, wrote: > There is a disagreement between drivers/tty/vt/keyboard.c and > drivers/input/input-leds.c with regard to what is a Scroll Lock LED > trigger name: input calls it "kbd-scrolllock", but vt calls it > "kbd-scrollock" (two l's). > This prevents Scroll Lock LED trigger from binding to this LED by default. > > Since it is a scroLL Lock LED, this interface was introduced only about a > year ago and in an Internet search people seem to reference this trigger > only to set it to this LED let's simply rename it to "kbd-scrolllock". > > Also, it looks like this was supposed to be changed before this code was > merged: https://lkml.org/lkml/2015/6/9/697 but it was done only on > the input side. > > Signed-off-by: Maciej S. Szmigiero Acked-by: Samuel Thibault > --- > drivers/tty/vt/keyboard.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/tty/vt/keyboard.c b/drivers/tty/vt/keyboard.c > index d5d81d4d3c04..3dd6a491cdba 100644 > --- a/drivers/tty/vt/keyboard.c > +++ b/drivers/tty/vt/keyboard.c > @@ -982,7 +982,7 @@ static void kbd_led_trigger_activate(struct led_classdev *cdev) > KBD_LED_TRIGGER((_led_bit) + 8, _name) > > static struct kbd_led_trigger kbd_led_triggers[] = { > - KBD_LED_TRIGGER(VC_SCROLLOCK, "kbd-scrollock"), > + KBD_LED_TRIGGER(VC_SCROLLOCK, "kbd-scrolllock"), > KBD_LED_TRIGGER(VC_NUMLOCK, "kbd-numlock"), > KBD_LED_TRIGGER(VC_CAPSLOCK, "kbd-capslock"), > KBD_LED_TRIGGER(VC_KANALOCK, "kbd-kanalock"), > -- Samuel "2 + 2 = 5 pour d'assez grandes valeurs de 2"