Hello Nick, On 12/16/2014 05:39 PM, Nick Dyer wrote: > On 16/12/14 16:34, Javier Martinez Canillas wrote: >> On 12/16/2014 05:16 PM, Nick Dyer wrote: >>>> Nick Dyer (1): >>>> Input: atmel_mxt_ts - Implement support for T100 touch object >>> I posted an updated version of this patch on 9th December and would prefer >>> you apply that rather than this version, since it contains some useful >>> additional work. >>> >>> A link to it: >>> http://www.spinics.net/lists/kernel/msg1885519.html >> >> Great, I didn't know about your patch since I was not in the cc list. >> >> Thanks for posting and feel free to discard these patches. > > No worries! Do you want me to put you on CC in future? > Yes, please cc me and also to Sjoerd since we are working on machines that have Atmel maxtouch chips. > I will add a todo to pick up your second change "Split out touchpad > initialisation logic" into patches I have waiting for upstream, it is a > sensible refactoring I think. > Awesome, what do you think about the change to have a common input device initialization function that I squashed in your original patch? mxt_initialize_t100_input_device() and mxt_initialize_t9_input_device() are very similar so I think that is a sensible refactoring as well. If you agree with the change I can post it on top of your patch once it lands in mainline. Best regards, Javier -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html