Re: [v2 1/2] dt-bindings: HID: i2c-hid: elan: Introduce bindings for Ilitek ili2901

[Krzysztof Kozlowski]

On 26/12/2023 03:37, xiazhengqiao wrote:
> Because ilitek, ili2901 needs to use reset to pull down the time for 10ms,
> so we need to control the reset, use this drive control.

I don't see relation between commit msg and the patch itself. Perhaps
you wanted to say you document new device which is different than elan one?


Please use standard email subjects, so with the PATCH keyword in the
title. `git format-patch` helps here to create proper versioned patches.
Another useful tool is b4. Skipping the PATCH keyword makes filtering of
emails more difficult thus making the review process less convenient.

A nit, subject: drop second/last, redundant "bindings for". The
"dt-bindings" prefix is already stating that these are bindings.

> 
> Signed-off-by: xiazhengqiao <xiazhengqiao@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx>

Your name still does not look like in other reply.

> ---
>  Documentation/devicetree/bindings/input/elan,ekth6915.yaml | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 


What supplies does the device have? Not the driver, the device as
written in datasheet?

Best regards,
Krzysztof