Hi Benjamin, On Mon, Jan 30, 2017 at 02:05:39PM +0100, Benjamin Tissoires wrote: > Hi Dmitri, > > On Tue, Jan 10, 2017 at 5:11 PM, Benjamin Tissoires > <benjamin.tissoires@xxxxxxxxxx> wrote: > > Hi, > > > > Well, this is the last series which enables RMI4 over SMBus for the Thinkpad > > t*40, t*50, t*60 series. Few comments on the patches: > > > > patches 1 to 3 allows the re-routing of the trackstick buttons from the touchpad > > to the trackstick > > patch 4 to 7 allow rmi_smbus to actually control the PS/2 node to not resume and > > mess up the configuration after a resume > > patch 8 was originally sent as a RMI4 specific driver, but I came to realise > > that Elan could also benefit from the same driver > > Any comments on the first 8 patches? I think that by trying to "unwind" unsuccessful SMbus initialization you make the code much more complicated and fragile. I think we should select a path (PS/2 or SMbus) and commit to it. If we commit to SMBus then we need to communicate that fact to psmouse core so that it does not create input devices or psmouse attributes, and "short-circuit" the reconnect() routines to simply ignore requests and always report success. Thanks. -- Dmitry -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
