Hi, > Hans de Goede <hdegoede@xxxxxxxxxx> hat am 6. August 2016 um 12:00 > geschrieben: > > > Hi, > > On 06-08-16 04:22, Dmitry Torokhov wrote: > > On Fri, Aug 05, 2016 at 05:07:20PM +0200, Stefan Wahren wrote: > >> Hi Hans, > >> > >> Am 05.08.2016 um 09:24 schrieb Hans de Goede: > >>> The new(ish) ft6236 simply re-implements the M09 protocol of the > >>> (much) older edt-ft5x06.c driver. > >>> > >>> This commit removes this duplicate driver and adds the i2c ids and > >>> dt compatible string to the edt-ft5x06.c driver to keep compatibility. > >> > >> i think only remove of the driver is incorrect here. > >> > >> Since there are two similar but still different DT bindings > >> > >> Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.txt > >> > >> Documentation/devicetree/bindings/input/touchscreen/focaltech-ft6236.txt > >> > >> the DT handling of edt-ft5x06.c needs to be extended and then > >> finally driver and binding could be removed. > > > > I believe Hans made sure edt-ft5x06.c can handle focaltech-ft6236 > > bindings. Do you see a discrepancy? > > Correct I've double checked that both the i2c protocol and the > reset gpio binding are compatible. Devices using the focaltech-ft6236.txt > binding should work fine with the edt-ft5x06.c driver. i missed the point about touchscreen_parse_properties() and it's properties. Sorry for the noise. Everything okay. > > Now if someone can test and confirm that on actual hardware that would be > great... > > And we probably want to remove > Documentation/devicetree/bindings/input/touchscreen/focaltech-ft6236.txt > and instead add the compatible string that defines to > Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.txt > That would be great. Stefan > I can whip up a patch for that, but as said some testing on actual > hw would be good, preferably before I write said patch. > > Regards, > > Hans > > > > > > p.s. > > (the v3 in the subject is bogus my bad) -- 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