mirror of
https://mirrors.bfsu.edu.cn/git/linux.git
synced 2024-11-26 21:54:11 +08:00
Input: elants_i2c - check Remark ID when attempting firmware update
To avoid flashing incompatible firmware onto a device we should check whether "Remark ID" in firmware matches with the one in the controller. This function is supported by Elan's latest version of boot code, so the driver decides whether to perform the check based on the boot code version. Signed-off-by: Johnny Chuang <johnny.chuang@emc.com.tw> Link: https://lore.kernel.org/r/00a901d5af3c$193e9cd0$4bbbd670$@emc.com.tw Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
This commit is contained in:
parent
a284e11c37
commit
f0b57e1901
@ -59,8 +59,10 @@
|
||||
#define CMD_HEADER_WRITE 0x54
|
||||
#define CMD_HEADER_READ 0x53
|
||||
#define CMD_HEADER_6B_READ 0x5B
|
||||
#define CMD_HEADER_ROM_READ 0x96
|
||||
#define CMD_HEADER_RESP 0x52
|
||||
#define CMD_HEADER_6B_RESP 0x9B
|
||||
#define CMD_HEADER_ROM_RESP 0x95
|
||||
#define CMD_HEADER_HELLO 0x55
|
||||
#define CMD_HEADER_REK 0x66
|
||||
|
||||
@ -200,6 +202,10 @@ static int elants_i2c_execute_command(struct i2c_client *client,
|
||||
expected_response = CMD_HEADER_6B_RESP;
|
||||
break;
|
||||
|
||||
case CMD_HEADER_ROM_READ:
|
||||
expected_response = CMD_HEADER_ROM_RESP;
|
||||
break;
|
||||
|
||||
default:
|
||||
dev_err(&client->dev, "%s: invalid command %*ph\n",
|
||||
__func__, (int)cmd_size, cmd);
|
||||
@ -556,6 +562,8 @@ static int elants_i2c_initialize(struct elants_data *ts)
|
||||
|
||||
/* hw version is available even if device in recovery state */
|
||||
error2 = elants_i2c_query_hw_version(ts);
|
||||
if (!error2)
|
||||
error2 = elants_i2c_query_bc_version(ts);
|
||||
if (!error)
|
||||
error = error2;
|
||||
|
||||
@ -563,8 +571,6 @@ static int elants_i2c_initialize(struct elants_data *ts)
|
||||
error = elants_i2c_query_fw_version(ts);
|
||||
if (!error)
|
||||
error = elants_i2c_query_test_version(ts);
|
||||
if (!error)
|
||||
error = elants_i2c_query_bc_version(ts);
|
||||
if (!error)
|
||||
error = elants_i2c_query_ts_info(ts);
|
||||
|
||||
@ -613,39 +619,94 @@ static int elants_i2c_fw_write_page(struct i2c_client *client,
|
||||
return error;
|
||||
}
|
||||
|
||||
static int elants_i2c_validate_remark_id(struct elants_data *ts,
|
||||
const struct firmware *fw)
|
||||
{
|
||||
struct i2c_client *client = ts->client;
|
||||
int error;
|
||||
const u8 cmd[] = { CMD_HEADER_ROM_READ, 0x80, 0x1F, 0x00, 0x00, 0x21 };
|
||||
u8 resp[6] = { 0 };
|
||||
u16 ts_remark_id = 0;
|
||||
u16 fw_remark_id = 0;
|
||||
|
||||
/* Compare TS Remark ID and FW Remark ID */
|
||||
error = elants_i2c_execute_command(client, cmd, sizeof(cmd),
|
||||
resp, sizeof(resp));
|
||||
if (error) {
|
||||
dev_err(&client->dev, "failed to query Remark ID: %d\n", error);
|
||||
return error;
|
||||
}
|
||||
|
||||
ts_remark_id = get_unaligned_be16(&resp[3]);
|
||||
|
||||
fw_remark_id = get_unaligned_le16(&fw->data[fw->size - 4]);
|
||||
|
||||
if (fw_remark_id != ts_remark_id) {
|
||||
dev_err(&client->dev,
|
||||
"Remark ID Mismatched: ts_remark_id=0x%04x, fw_remark_id=0x%04x.\n",
|
||||
ts_remark_id, fw_remark_id);
|
||||
return -EINVAL;
|
||||
}
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
||||
static int elants_i2c_do_update_firmware(struct i2c_client *client,
|
||||
const struct firmware *fw,
|
||||
bool force)
|
||||
{
|
||||
struct elants_data *ts = i2c_get_clientdata(client);
|
||||
const u8 enter_iap[] = { 0x45, 0x49, 0x41, 0x50 };
|
||||
const u8 enter_iap2[] = { 0x54, 0x00, 0x12, 0x34 };
|
||||
const u8 iap_ack[] = { 0x55, 0xaa, 0x33, 0xcc };
|
||||
const u8 close_idle[] = {0x54, 0x2c, 0x01, 0x01};
|
||||
const u8 close_idle[] = { 0x54, 0x2c, 0x01, 0x01 };
|
||||
u8 buf[HEADER_SIZE];
|
||||
u16 send_id;
|
||||
int page, n_fw_pages;
|
||||
int error;
|
||||
bool check_remark_id = ts->iap_version >= 0x60;
|
||||
|
||||
/* Recovery mode detection! */
|
||||
if (force) {
|
||||
dev_dbg(&client->dev, "Recovery mode procedure\n");
|
||||
|
||||
if (check_remark_id) {
|
||||
error = elants_i2c_validate_remark_id(ts, fw);
|
||||
if (error)
|
||||
return error;
|
||||
}
|
||||
|
||||
error = elants_i2c_send(client, enter_iap2, sizeof(enter_iap2));
|
||||
if (error) {
|
||||
dev_err(&client->dev, "failed to enter IAP mode: %d\n",
|
||||
error);
|
||||
return error;
|
||||
}
|
||||
} else {
|
||||
/* Start IAP Procedure */
|
||||
dev_dbg(&client->dev, "Normal IAP procedure\n");
|
||||
|
||||
/* Close idle mode */
|
||||
error = elants_i2c_send(client, close_idle, sizeof(close_idle));
|
||||
if (error)
|
||||
dev_err(&client->dev, "Failed close idle: %d\n", error);
|
||||
msleep(60);
|
||||
|
||||
elants_i2c_sw_reset(client);
|
||||
msleep(20);
|
||||
error = elants_i2c_send(client, enter_iap, sizeof(enter_iap));
|
||||
}
|
||||
|
||||
if (error) {
|
||||
dev_err(&client->dev, "failed to enter IAP mode: %d\n", error);
|
||||
return error;
|
||||
if (check_remark_id) {
|
||||
error = elants_i2c_validate_remark_id(ts, fw);
|
||||
if (error)
|
||||
return error;
|
||||
}
|
||||
|
||||
error = elants_i2c_send(client, enter_iap, sizeof(enter_iap));
|
||||
if (error) {
|
||||
dev_err(&client->dev, "failed to enter IAP mode: %d\n",
|
||||
error);
|
||||
return error;
|
||||
}
|
||||
}
|
||||
|
||||
msleep(20);
|
||||
|
Loading…
Reference in New Issue
Block a user