Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751952AbbKIOGa (ORCPT ); Mon, 9 Nov 2015 09:06:30 -0500 Received: from mail-wm0-f46.google.com ([74.125.82.46]:33231 "EHLO mail-wm0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751101AbbKIOG3 (ORCPT ); Mon, 9 Nov 2015 09:06:29 -0500 MIME-Version: 1.0 In-Reply-To: References: <1446766950-31213-1-git-send-email-aduggan@synaptics.com> Date: Mon, 9 Nov 2015 15:06:27 +0100 Message-ID: Subject: Re: [PATCH 22/26] Input: synaptics-rmi4 - Add F30 support From: Benjamin Tissoires To: Linus Walleij Cc: Andrew Duggan , Linux Input , "linux-kernel@vger.kernel.org" , Benjamin Tissoires , Dmitry Torokhov , Christopher Heiny , Stephen Chandler Paul , Allie Xiong Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2186 Lines: 55 Hey Linus, first thanks for the reviews. Much appreciated. On Mon, Nov 9, 2015 at 2:32 PM, Linus Walleij wrote: > On Fri, Nov 6, 2015 at 12:42 AM, Andrew Duggan wrote: > >> From: Benjamin Tissoires >> >> RMI4 Function 0x30 provides support for GPIOs, LEDs and mechanical >> buttons. In particular, the mechanical button support is used in >> an increasing number of touchpads. >> >> [BT] cured the code to rely only on the unified input node created >> by rmi_driver. >> >> Signed-off-by: Andrew Duggan >> Signed-off-by: Allie Xiong >> Signed-off-by: Benjamin Tissoires > > I see this function driver is not yet adding any gpio_chip or > LEDs class devices, which is fine, we can add that later when > we have something to test. Or is iit using that LED feature > in the input layer that corresponds to caps lock etc? Do not take my words as the official ones, but when we discussed with Synaptics about F30 (and unified input), they told us that they designed the driver based on the phone use case. In such use case, the power (and maybe LEDs) are handled through F30, and the touchscreen through F11/12. Problem is, I am not even sure there are phones around with such F30/F11 combination. So in the end, from what I can see, F30 is used for buttons on touchpads/clickpads, and LEDs when there are some on these touchpads. I don't know if the keyboards would use F30 for their LEDs though. That being said. Unless Synaptics tells us that there are uses of a non "unified" input device somewhere, I would also agree to only keep the "unified" input node, which would simplify both F11/12 and F30. > > I don't quite get it I guess :/ > > But I guess it should also be squashed into the original F30 driver. I think this is the original F30 driver :-) Cheers, Benjamin -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/