On Thu, Mar 23, 2017 at 12:14 AM, Dmitry Torokhov <dmitry.torokhov@xxxxxxxxx> wrote: > On Thu, Mar 23, 2017 at 12:05:17AM +0530, Jagan Teki wrote: >> On Thu, Mar 23, 2017 at 12:00 AM, Dmitry Torokhov >> <dmitry.torokhov@xxxxxxxxx> wrote: >> > On Wed, Mar 22, 2017 at 04:36:46PM +0530, Jagan Teki wrote: >> >> From: Jagan Teki <jagan@xxxxxxxxxxxxxxxxxxxx> >> >> >> >> Added missing of_match_table for max11801_ts driver with >> >> compatible as "maxim,max11801_ts" >> >> >> > >> > Why not "maxim,max11801"? Also, I think we'd need a binding document. >> >> Compatibility purpose to make this is for touchscreen and few of other > > Compatibility with what? The i2c id is "max11801" (without the "-ts"). Do we need to sync with i2c id as well? egalax_ts.c follow "_ts" of-course the i2c there as follow the same. > >> *_ts.c files follow the same, and added binding documentation in 2/3 >> patch. > > Sorry, it seems I am only copied on 1/3 patch. Sorry, will add in next version patches. thanks! -- Jagan Teki Senior Linux Kernel Engineer | Amarula Solutions U-Boot, Linux | Upstream Maintainer Hyderabad, India. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html