linux-kernel - Re: [PATCH] driver: input: touchscreen: modify Raydium i2c touchscreen driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <YHiRP+UvUlXfr0J6@google.com>
Date:   Thu, 15 Apr 2021 12:17:19 -0700
From:   Dmitry Torokhov <dmitry.torokhov@...il.com>
To:     "simba.hsu" <simba.hsu@...dium.corp-partner.google.com>
Cc:     furquan@...gle.com, seanpaul@...omium.org, rrangel@...omium.org,
        simba.hsu@...-ic.com, jeffrey.lin@...-ic.com, KP.li@...-ic.com,
        linux-input@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] driver: input: touchscreen: modify Raydium i2c
 touchscreen driver

HI,

On Thu, Apr 15, 2021 at 04:58:29PM +0800, simba.hsu wrote:
> This path makes auto-update available when IC's status is
> Recovery mode.

Could you please explain in more detail what the issue is. Also please
improve the patch subject, as "modify Raydium i2c touchscreen driver"
does not really convey the substance of the patch.

Also, the patch does not apply to my tree, was it based on some other
tree?

> 
> Signed-off-by: simba.hsu@...dium.corp-partner.google.com

Please use your Raydium email for signoff.

> Change-Id: I5ae54896a201b949eba7514500a7e75574f5726b

No need to send change-id tags, they are not used in mainline kernel.

> ---
>  drivers/input/touchscreen/raydium_i2c_ts.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/input/touchscreen/raydium_i2c_ts.c b/drivers/input/touchscreen/raydium_i2c_ts.c
> index 444479ef699e..a97403c55f75 100644
> --- a/drivers/input/touchscreen/raydium_i2c_ts.c
> +++ b/drivers/input/touchscreen/raydium_i2c_ts.c
> @@ -298,6 +298,7 @@ static int raydium_i2c_query_ts_BL_info(struct raydium_data *ts)
>  					 0x10, 0xc0, 0x01, 0x00, 0x04, 0x00};
>  	int error;
>  	u8 rbuf[5] = {0, 0, 0, 0, 0};
> +	u32 tmpdata = 0;
>  
>  	error = raydium_i2c_send(client,
>  				 RM_CMD_BOOT_WRT, get_hwid, sizeof(get_hwid));
> @@ -315,7 +316,8 @@ static int raydium_i2c_query_ts_BL_info(struct raydium_data *ts)
>  	error = raydium_i2c_read(client,
>  				 RM_CMD_BOOT_CHK, rbuf, sizeof(rbuf));
>  	if (!error) {
> -		ts->info.hw_ver = cpu_to_le32(rbuf[1]<<24|rbuf[2]<<16|rbuf[3]<<8|rbuf[4]);
> +		tmpdata = (rbuf[1]<<24|rbuf[2]<<16|rbuf[3]<<8|rbuf[4]);
> +		ts->info.hw_ver = cpu_to_le32(tmpdata);

On the face of it I can't see why the code would behave differently,
nut then there is no raydium_i2c_query_ts_BL_info() in the copy of the
driver I have here.

>  		dev_err(&client->dev, "HWID %08X\n", ts->info.hw_ver);
>  	} else {
>  		ts->info.hw_ver = cpu_to_le32(0xffffffffUL);
> -- 
> 2.25.1
> 

Thanks.

-- 
Dmitry