On Tue, Dec 10, 2013 at 1:46 AM, Dmitry Torokhov <[email protected]> wrote: > On Mon, Dec 09, 2013 at 10:39:14PM -0800, Dmitry Torokhov wrote: >> On Mon, Dec 09, 2013 at 03:14:26PM -0500, Benjamin Tissoires wrote: >> > Hi Chris, >> > >> > On 05/12/13 19:29, Christopher Heiny wrote: >> > > This patch implements changes to the synaptics-rmi4 branch of >> > > Dmitry's input tree. The base for the patch is commit >> > > 8ca01dc61a42b6f7bcba052a8c084000f7057a34. >> > > >> > > This patch primarily reorders the various declarations in rmi_bus.c in >> > > order to >> > > group related elements together, along with some typo fixes. The code >> > > is still >> > > horribly broken, but this change should make the following fixes easier >> > > to >> > > review. >> > > >> > > Signed-off-by: Christopher Heiny <[email protected]> >> > > Cc: Dmitry Torokhov <[email protected]> >> > > Cc: Jean Delvare <[email protected]> >> > > Cc: Linus Walleij <[email protected]> >> > > Cc: Joerie de Gram <[email protected]> >> > > Cc: Benjamin Tissoires <[email protected]> >> > > >> > > --- >> > >> > FWIW, I made a review of the patch. >> > The patches does not only reorder the functions, but also fix some few >> > things I will detail later (plus fixes of whitespace/comments issues). >> > It also changes the exported functions as GPL. >> > >> > Dmitry, given the current state of the driver (which does not work at >> > all if I understood correctly), maybe you can pick this one in its >> > current state. >> >> Applied, thank you. > > Well, I had to pull up rmi_debugfs_root declaration to avoid: > > CC [M] drivers/input/rmi4/rmi_driver.o > drivers/input/rmi4/rmi_bus.c: In function ‘rmi_physical_setup_debugfs’: > drivers/input/rmi4/rmi_bus.c:51:10: error: ‘rmi_debugfs_root’ undeclared > (first use in this function) > rmi_debugfs_root); > > Guys, a bit better compile coverage would be appreciated.
oops, sorry for not having spot this one. My mistake. Thanks for fixing it. Cheers, Benjamin -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
