Re: [PATCH] arm64: dts: qcom: msm8916-samsung-grandmax: Add touchscreen

[Konrad Dybcio]

On 13.07.2024 7:27 PM, Lin, Meng-Bo wrote:
> Grand Max uses an Imagis IST3038 touchscreen that is connected to
> blsp_i2c5. Add it to the device tree.
> 
> Signed-off-by: "Lin, Meng-Bo" <linmengbo06...@proton.me>
> ---
>  .../dts/qcom/msm8916-samsung-grandmax.dts     | 24 ++++++++++++++++++-
>  1 file changed, 23 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/msm8916-samsung-grandmax.dts 
> b/arch/arm64/boot/dts/qcom/msm8916-samsung-grandmax.dts
> index 135df1739dbd..5806a28b7bec 100644
> --- a/arch/arm64/boot/dts/qcom/msm8916-samsung-grandmax.dts
> +++ b/arch/arm64/boot/dts/qcom/msm8916-samsung-grandmax.dts
> @@ -47,12 +47,34 @@ &battery {
>       constant-charge-voltage-max-microvolt = <4400000>;
>  };
>  
> +&blsp_i2c5 {
> +     status = "okay";
> +
> +     touchscreen@50 {
> +             compatible = "imagis,ist3038";
> +             reg = <0x50>;
> +
> +             interrupts-extended = <&tlmm 13 IRQ_TYPE_EDGE_FALLING>;
> +
> +             touchscreen-size-x = <720>;
> +             touchscreen-size-y = <1280>;
> +
> +             vdd-supply = <&reg_vdd_tsp_a>;
> +             vddio-supply = <&pm8916_l6>;
> +
> +             pinctrl-0 = <&ts_int_default>;
> +             pinctrl-names = "default";
> +
> +             linux,keycodes = <KEY_APPSELECT KEY_BACK>;
> +     };
> +};
> +
>  &reg_motor_vdd {
>       gpio = <&tlmm 72 GPIO_ACTIVE_HIGH>;
>  };
>  
>  &reg_touch_key {
> -     status = "disabled";
> +     status = "disabled"; /* Using Imagis touch key*/

key */

Reviewed-by: Konrad Dybcio <konrad.dyb...@linaro.org>

Konrad