Re: [PATCH v2 0/6] Add Wacom I2C support to rM2

From: Marco Felsch
Date: Fri Jan 22 2021 - 12:11:20 EST


Hi Alistair,

thanks for the patches. Before getting into the series please check all
your commit subjects. Also please check that you do only one logical
change per patch: e.g. adding DT-Support means: Add the dt-table and not
more.

I looking forward to your v2 :)

Regards,
Marco

On 21-01-20 22:56, Alistair Francis wrote:
> Add support to the reMarkable2 (rM2) for the Wacom I2C device.
>
> This is based on the reMarkable Linux fork and with this series I am
> able to probe the Wacom digitiser.
>
> Alistair Francis (6):
> devicetree/bindings: Initial commit of wacom,wacom-i2c
> input/touchscreen: Add device tree support to wacom_i2c
> touchscreen/wacom_i2c: Add support for distance and tilt x/y
> touchscreen/wacom_i2c: Clean up the query device fields
> touchscreen/wacom_i2c: Add support for vdd regulator
> arch/arm: reMarkable2: Enable wacom_i2c
>
> .../input/touchscreen/wacom,wacom-i2c.yaml | 48 +++++++
> .../devicetree/bindings/vendor-prefixes.yaml | 2 +
> arch/arm/boot/dts/imx7d-remarkable2.dts | 41 ++++++
> arch/arm/configs/imx_v6_v7_defconfig | 1 +
> drivers/input/touchscreen/wacom_i2c.c | 136 +++++++++++++++---
> 5 files changed, 205 insertions(+), 23 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/input/touchscreen/wacom,wacom-i2c.yaml
>
> --
> 2.29.2
>
>
>

--
Pengutronix e.K. | |
Steuerwalder Str. 21 | http://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |