2024-05-01 20:48:49

by Andreas Kemnade

[permalink] [raw]
Subject: [PATCH v3 0/2] Input: add ft5426

Changes in v3:
- reorder compatible also in driver

Changes in v2:
- reorder compatible

Andreas Kemnade (2):
dt-bindings: input: touchscreen: edt-ft5x06: Add ft5426
Input: edt-ft5x06 - add ft5426

.../devicetree/bindings/input/touchscreen/edt-ft5x06.yaml | 1 +
drivers/input/touchscreen/edt-ft5x06.c | 1 +
2 files changed, 2 insertions(+)

--
2.39.2



2024-05-01 20:48:50

by Andreas Kemnade

[permalink] [raw]
Subject: [PATCH v3 1/2] dt-bindings: input: touchscreen: edt-ft5x06: Add ft5426

Add compatible for ft5426.
Searches for documentation reveal neither edt nor evervision
as some related company, only FocalTech.

Signed-off-by: Andreas Kemnade <[email protected]>
Acked-by: Krzysztof Kozlowski <[email protected]>
---
.../devicetree/bindings/input/touchscreen/edt-ft5x06.yaml | 1 +
1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml b/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml
index f2808cb4d99df..71fd3c66c966e 100644
--- a/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml
+++ b/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml
@@ -39,6 +39,7 @@ properties:
- edt,edt-ft5406
- edt,edt-ft5506
- evervision,ev-ft5726
+ - focaltech,ft5426
- focaltech,ft6236

reg:
--
2.39.2


2024-05-01 20:49:02

by Andreas Kemnade

[permalink] [raw]
Subject: [PATCH v3 2/2] Input: edt-ft5x06 - add ft5426

As ft5426 seems to be compatible with this driver, add it.
Debug output during identification: Model "generic ft5x06 (79)", Rev. "

Signed-off-by: Andreas Kemnade <[email protected]>
---
drivers/input/touchscreen/edt-ft5x06.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/drivers/input/touchscreen/edt-ft5x06.c b/drivers/input/touchscreen/edt-ft5x06.c
index 2a1db11344766..028ec66faa193 100644
--- a/drivers/input/touchscreen/edt-ft5x06.c
+++ b/drivers/input/touchscreen/edt-ft5x06.c
@@ -1486,6 +1486,7 @@ static const struct of_device_id edt_ft5x06_of_match[] = {
{ .compatible = "edt,edt-ft5406", .data = &edt_ft5x06_data },
{ .compatible = "edt,edt-ft5506", .data = &edt_ft5506_data },
{ .compatible = "evervision,ev-ft5726", .data = &edt_ft5506_data },
+ { .compatible = "focaltech,ft5426", .data = &edt_ft5506_data },
/* Note focaltech vendor prefix for compatibility with ft6236.c */
{ .compatible = "focaltech,ft6236", .data = &edt_ft6236_data },
{ /* sentinel */ }
--
2.39.2


2024-05-02 12:13:10

by Oliver Graute

[permalink] [raw]
Subject: Re: [PATCH v3 1/2] dt-bindings: input: touchscreen: edt-ft5x06: Add ft5426

On 01/05/24, Andreas Kemnade wrote:
> Add compatible for ft5426.
> Searches for documentation reveal neither edt nor evervision
> as some related company, only FocalTech.
>
> Signed-off-by: Andreas Kemnade <[email protected]>
> Acked-by: Krzysztof Kozlowski <[email protected]>
> ---
> .../devicetree/bindings/input/touchscreen/edt-ft5x06.yaml | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml b/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml
> index f2808cb4d99df..71fd3c66c966e 100644
> --- a/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml
> +++ b/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml
> @@ -39,6 +39,7 @@ properties:
> - edt,edt-ft5406
> - edt,edt-ft5506
> - evervision,ev-ft5726
> + - focaltech,ft5426
> - focaltech,ft6236
>
> reg:
> --
> 2.39.2
>
Reviewed-by: Oliver Graute <[email protected]>

2024-05-02 12:15:41

by Oliver Graute

[permalink] [raw]
Subject: Re: [PATCH v3 2/2] Input: edt-ft5x06 - add ft5426

On 01/05/24, Andreas Kemnade wrote:
> As ft5426 seems to be compatible with this driver, add it.
> Debug output during identification: Model "generic ft5x06 (79)", Rev. "
>
> Signed-off-by: Andreas Kemnade <[email protected]>
> ---
> drivers/input/touchscreen/edt-ft5x06.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/input/touchscreen/edt-ft5x06.c b/drivers/input/touchscreen/edt-ft5x06.c
> index 2a1db11344766..028ec66faa193 100644
> --- a/drivers/input/touchscreen/edt-ft5x06.c
> +++ b/drivers/input/touchscreen/edt-ft5x06.c
> @@ -1486,6 +1486,7 @@ static const struct of_device_id edt_ft5x06_of_match[] = {
> { .compatible = "edt,edt-ft5406", .data = &edt_ft5x06_data },
> { .compatible = "edt,edt-ft5506", .data = &edt_ft5506_data },
> { .compatible = "evervision,ev-ft5726", .data = &edt_ft5506_data },
> + { .compatible = "focaltech,ft5426", .data = &edt_ft5506_data },
> /* Note focaltech vendor prefix for compatibility with ft6236.c */
> { .compatible = "focaltech,ft6236", .data = &edt_ft6236_data },
> { /* sentinel */ }
> --
> 2.39.2

Reviewed-by: Oliver Graute <[email protected]>