Hi Simon,
sorry for the late answer,
On Tue, Jan 16, 2018 at 07:52:06PM +1100, Simon Shields wrote:
> Hi Andi,
>
> Thanks for the review!
>
> On Tue, Jan 16, 2018 at 04:56:11PM +0900, Andi Shyti wrote:
> > Hi Simon,
> >
> > On Sat, Jan 13, 2018 at 01:04:56PM +1100, Simon Shields wrote:
> > > The MMS114 platform data has no in-tree users, so drop it,
> > > and make the driver depend on CONFIG_OF.
> > >
> > > Switch to using the standard touchscreen properties via
> > > touchscreen_parse_properties(), and move the old DT parsing code
> > > to use device_property_*() APIs.
> > >
> > > Finally, use touchscreen_report_pos to report x/y coordinates
> > > and drop the custom x/y inversion code.
> > >
> > > Signed-off-by: Simon Shields <simon@xxxxxxxxxxxxx>
> > > ---
> > > .../bindings/input/touchscreen/mms114.txt | 29 ++--
> > > drivers/input/touchscreen/Kconfig | 1 +
> > > drivers/input/touchscreen/mms114.c | 152 +++++++++------------
> > > include/linux/platform_data/mms114.h | 24 ----
> > > 4 files changed, 83 insertions(+), 123 deletions(-)
> > > delete mode 100644 include/linux/platform_data/mms114.h
> > >
> >
> > The patch looks good, but you would also need to update the dtsi
> > files in this same patch:
> >
> > ./arch/arm/boot/dts/exynos4412-trats2.dts
> > ./arch/arm/boot/dts/exynos4210-trats.dts
> >
> > and Cc the Samsung-soc mailing list.
> >
> > For now it's a nack because the touchscreen would not work
> > anymore with the trats boards.
>
> This patch keeps support for the old bindings. I've verified that both
> the old and new bindings work on a GT-I9300 (trats2 with a different
> bootloader/partition layout).
Oh, yes, I read the patch partially, I didn't see that you
actually kept the back compatibility by doing this:
> + if (mms114_parse_dt(data) < 0) {
> + /* No valid legacy binding found, try the common one */
> + touchscreen_parse_properties(input_dev, true, &data->props);
[...]
Sorry for the confusion, please add:
Reviewed-by: Andi Shyti <andi.shyti@xxxxxxxxxxx>
Tested-by: Andi Shyti <andi.shyti@xxxxxxxxxxx>
And for reference to Dmitry, there is also Rob's ack.
Andi
--
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
