[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <YrveNToV1Sqs1X4X@google.com>
Date: Tue, 28 Jun 2022 22:08:05 -0700
From: Dmitry Torokhov <dmitry.torokhov@...il.com>
To: Dario Binacchi <dario.binacchi@...rulasolutions.com>
Cc: linux-kernel@...r.kernel.org,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Marco Felsch <m.felsch@...gutronix.de>,
Michael Trimarchi <michael@...rulasolutions.com>,
Oliver Graute <oliver.graute@...oconnector.com>,
Rob Herring <robh+dt@...nel.org>, devicetree@...r.kernel.org,
linux-input@...r.kernel.org
Subject: Re: [RESEND PATCH v4 0/6] Input: edt-ft5x06 - Improve configuration
On Tue, Jun 21, 2022 at 02:39:31PM +0200, Dario Binacchi wrote:
> The series was born from the analysis and mitigation of a crc problem
> raised by an M06 type device. The added sysfs attributes were helpful
> in debugging the problem. Patches that change the report rate on driver
> probing, mitigated crc errors on kernel bootup. The patch to get/set
> report rate by sysfs for an M12 device, has been tested.
>
> Changes in v4:
> - Add Rob Herring 'Acked-by' tag.
>
> Changes in v3:
> - Add hz unit suffix.
> - Add '|' to description.
> - Check the lower and upper limits of the report-rate-hz value
> - Convert the M06 report-rate-hz value
>
> Changes in v2:
> - Add Oliver Graute's 'Acked-by' tag to:
> * Input: edt-ft5x06 - show model name by sysfs
> * Input: edt-ft5x06 - show firmware version by sysfs
> - Fix yaml file. Tested with `make DT_CHECKER_FLAGS=-m dt_binding_check'.
>
> Dario Binacchi (6):
> dt-bindings: input: touchscreen: edt-ft5x06: add report-rate-hz
> Input: edt-ft5x06 - get/set M12 report rate by sysfs
> Input: edt-ft5x06 - set report rate by dts property
> Input: edt-ft5x06 - show model name by sysfs
> Input: edt-ft5x06 - show firmware version by sysfs
> Input: edt-ft5x06 - show crc and header errors by sysfs
>
> .../input/touchscreen/edt-ft5x06.yaml | 8 ++
> drivers/input/touchscreen/edt-ft5x06.c | 103 ++++++++++++++++--
> 2 files changed, 103 insertions(+), 8 deletions(-)
Applied the lot with a few small adjustments, thank you.
--
Dmitry
Powered by blists - more mailing lists