Hi Jelle, On Wed, Jan 18, 2017 at 02:49:46PM -0800, Dmitry Torokhov wrote: > On Wed, Jan 18, 2017 at 09:57:14PM +0100, Jelle van der Waa wrote: > > On 01/14/17 at 11:14am, Dmitry Torokhov wrote: > > > > + > > > > +static const struct of_device_id zet6223_of_match[] = { > > > > + { .compatible = "zeitec", "zet6223" }, > > > > > > The compatible should be "zeitec,zet6223", what you have here is > > > equivalent of: > > > > > > { .compatible = "zeitec", .data = "zet6223" }, > > > > Ah that was silly, thanks :) > > > > > I also have been looking at you previosu submission and had some draft > > > changes. I reconciled them in the patch below, if it still works for you > > > then I'll fold everything together and apply. Please let me know. > > > > I'll find some time to test the patch below, I've one small suggestion > > to the patch below. It would be very helpful if you let me know if the patch actually works on the hardware: the merge window for 4.11 will be opening fairly soon. 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