Received-SPF: pass (google.com: domain of linux-kernel+bounces-125648-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Precedence: bulk
Mime-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset=UTF-8
Date: Sat, 30 Mar 2024 10:33:53 +0100
Message-Id: <D06ZCKKYTQM5.3OJ6HCLHW3DZ9@matfyz.cz>
Cc: "Markuss Broks" <markuss.broks@gmail.com>,
        "Dmitry Torokhov"
 <dmitry.torokhov@gmail.com>,
        <linux-input@vger.kernel.org>, <duje.mihanovic@skole.hr>,
        <linux-kernel@vger.kernel.org>, <stable@vger.kernel.org>
Subject: Re: [PATCH AUTOSEL 6.8 78/98] input/touchscreen: imagis: add
 support for IST3032C
To: "Sasha Levin" <sashal@kernel.org>
From: "Karel Balej" <balejk@matfyz.cz>
References: <20240329123919.3087149-1-sashal@kernel.org>
 <20240329123919.3087149-78-sashal@kernel.org>
In-Reply-To: <20240329123919.3087149-78-sashal@kernel.org>

Sasha,

Sasha Levin, 2024-03-29T08:37:49-04:00:
> From: Karel Balej <balejk@matfyz.cz>
>
> [ Upstream commit 90cb57a6c5717b83a110c0da720a03ee32ed255e ]
>
> IST3032C is a touchscreen chip used for instance in the
> samsung,coreprimevelte smartphone, with which this was tested. Add the
> chip specific information to the driver.
>
> Reviewed-by: Markuss Broks <markuss.broks@gmail.com>
> Signed-off-by: Karel Balej <balejk@matfyz.cz>
> Link: https://lore.kernel.org/r/20240301164659.13240-6-karelb@gimli.ms.mf=
f.cuni.cz
> Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Signed-off-by: Sasha Levin <sashal@kernel.org>
> ---
>  drivers/input/touchscreen/imagis.c | 10 ++++++++++
>  1 file changed, 10 insertions(+)
>
> diff --git a/drivers/input/touchscreen/imagis.c b/drivers/input/touchscre=
en/imagis.c
> index 9af8a6332ae67..e1fafa561ee38 100644
> --- a/drivers/input/touchscreen/imagis.c
> +++ b/drivers/input/touchscreen/imagis.c
> @@ -11,6 +11,8 @@
>  #include <linux/property.h>
>  #include <linux/regulator/consumer.h>
> =20
> +#define IST3032C_WHOAMI			0x32c
> +
>  #define IST3038B_REG_STATUS		0x20
>  #define IST3038B_REG_CHIPID		0x30
>  #define IST3038B_WHOAMI			0x30380b
> @@ -363,6 +365,13 @@ static int imagis_resume(struct device *dev)
> =20
>  static DEFINE_SIMPLE_DEV_PM_OPS(imagis_pm_ops, imagis_suspend, imagis_re=
sume);
> =20
> +static const struct imagis_properties imagis_3032c_data =3D {
> +	.interrupt_msg_cmd =3D IST3038C_REG_INTR_MESSAGE,
> +	.touch_coord_cmd =3D IST3038C_REG_TOUCH_COORD,
> +	.whoami_cmd =3D IST3038C_REG_CHIPID,
> +	.whoami_val =3D IST3032C_WHOAMI,
> +};
> +
>  static const struct imagis_properties imagis_3038b_data =3D {
>  	.interrupt_msg_cmd =3D IST3038B_REG_STATUS,
>  	.touch_coord_cmd =3D IST3038B_REG_STATUS,
> @@ -380,6 +389,7 @@ static const struct imagis_properties imagis_3038c_da=
ta =3D {
> =20
>  #ifdef CONFIG_OF
>  static const struct of_device_id imagis_of_match[] =3D {
> +	{ .compatible =3D "imagis,ist3032c", .data =3D &imagis_3032c_data },
>  	{ .compatible =3D "imagis,ist3038b", .data =3D &imagis_3038b_data },
>  	{ .compatible =3D "imagis,ist3038c", .data =3D &imagis_3038c_data },
>  	{ },
> --=20
> 2.43.0

sorry if I'm missing something, but I don't see why this should be
backported: it doesn't fix anything, it's just adding support for new
hardware.

I can see that adding a device ID is permitted for -stable [1], but I
thought it still has to bear some signs of a fix, such as maybe here
[2].

Furthermore, you are also proposing to backport Duje's touch keys
support [3] which I think is unarguably a new feature and not a fix at
all. Of all the Imagis patches, only the touch area correction [4] seems
to make sense for backporting.

Could you please explain?

[1] https://docs.kernel.org/next/process/stable-kernel-rules.html
[2] https://git.kernel.org/pub/scm/linux/kernel/git/dtor/input.git/commit/?=
id=3Dd40e9edcf3eb925c259df9f9dd7319a4fcbc675b
[3] https://lore.kernel.org/stable/20240329123919.3087149-88-sashal@kernel.=
org/
[4] https://lore.kernel.org/stable/20240329123919.3087149-76-sashal@kernel.=
org/

Thank you, best regards,
K. B.