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? Thanks. -- Dmitry -- 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