lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <9db96c99c805e615ba40ca7fd3632174d1e8d11f.camel@gmail.com> Date: Fri, 06 Sep 2024 10:38:35 +0200 From: Nuno Sá <noname.nuno@...il.com> To: Dmitry Torokhov <dmitry.torokhov@...il.com>, Arnd Bergmann <arnd@...db.de>, Michael Hennerich <michael.hennerich@...log.com> Cc: Linus Walleij <linus.walleij@...aro.org>, linux-input@...r.kernel.org, linux-kernel@...r.kernel.org, Lee Jones <lee@...nel.org>, linux-arm-kernel@...ts.infradead.org, Utsav Agarwal <utsav.agarwal@...log.com> Subject: Re: [PATCH] Input: keypad-nomadik-ske - remove the driver On Mon, 2024-08-19 at 07:54 -0700, Dmitry Torokhov wrote: > On Mon, Aug 19, 2024 at 11:29:32AM +0200, Arnd Bergmann wrote: > > On Fri, Aug 16, 2024, at 20:54, Dmitry Torokhov wrote: > > > The users of this driver were removed in 2013 in commit 28633c54bda6 > > > ("ARM: ux500: Rip out keypad initialisation which is no longer used"). > > > > > > Remove the driver as well. > > > > > > Signed-off-by: Dmitry Torokhov <dmitry.torokhov@...il.com> > > > --- > > > drivers/input/keyboard/Kconfig | 11 - > > > drivers/input/keyboard/Makefile | 1 - > > > drivers/input/keyboard/nomadik-ske-keypad.c | 378 ------------------ > > > .../linux/platform_data/keypad-nomadik-ske.h | 50 --- > > > 4 files changed, 440 deletions(-) > > > > > > > Acked-by: Arnd Bergmann <arnd@...db.de> > > > > I have a list of drivers that I determined to be likely > > unused as well and found a few more input drivers > > that were unused in 2022: > > > > CONFIG_KEYBOARD_ADP5520/CONFIG_PMIC_ADP5520 > > CONFIG_KEYBOARD_ADP5589 > > CONFIG_INPUT_AD714X > > CONFIG_TOUCHSCREEN_AD7877 > > > > As far as I can tell, these all lost their last device > > definition, or they never had one and are impossible to > > be used with device tree data. > > I asked Analog Devices folks (CCed) about 5589 and Nuno said that it is > still relevant and promised to do conversion to DT similar to adp5588. > > Nuno, Michale, what about the other drivers that Arnd listed? > Hi Dmitry, This is not forgotten and I plan to start working on this early next week. One thing I noticed and I might as well just ask before starting the work, is that the platform data allows, in theory, for you to have holes in your keymap [1]. Like enabling row 1 and 3 skipping 2. AFAICT, the matrix stuff does not allow this out of the box as we just define the number of rows/cols and then without any other property we assume (I think) that the map is contiguous. This is just early thinking but one way to support the current behavior would be 2 custom DT properties that would be 2 u32 arrays specifying the enabled columns and rows. Out of it, we could build row and col masks and get the total number of cols and rows that we could pass to matrix_keypad_build_keymap(). The question is... is it worth it? I'm aware that if we just assume a contiguous keymap we could break some old users. But I guess it would be only out of tree ones as we don't have any in kernel user of the platform data. On top of it, I guess it's sane to assume that one just wants a contiguous keymap... [1]: https://elixir.bootlin.com/linux/v6.10.8/source/drivers/input/keyboard/adp5589-keys.c#L630 Thanks! - Nuno Sá
Powered by blists - more mailing lists