Commit 9d29327d authored by Johan Fjeldtvedt's avatar Johan Fjeldtvedt Committed by Mauro Carvalho Chehab

[media] pulse8-cec: some small fixes

Fix some small things:
    - clean up setup function
    - use MSGEND instead of 0xfe
    - don't assign "return value" from cec_phys_addr to err,
      it has return type void.
Signed-off-by: default avatarJohan Fjeldtvedt <jaffe1@gmail.com>
Signed-off-by: default avatarHans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
parent 846c4a7b
...@@ -266,7 +266,7 @@ static int pulse8_send(struct serio *serio, const u8 *command, u8 cmd_len) ...@@ -266,7 +266,7 @@ static int pulse8_send(struct serio *serio, const u8 *command, u8 cmd_len)
} }
} }
if (!err) if (!err)
err = serio_write(serio, 0xfe); err = serio_write(serio, MSGEND);
return err; return err;
} }
...@@ -331,40 +331,29 @@ static int pulse8_setup(struct pulse8 *pulse8, struct serio *serio, ...@@ -331,40 +331,29 @@ static int pulse8_setup(struct pulse8 *pulse8, struct serio *serio,
u8 *data = pulse8->data + 1; u8 *data = pulse8->data + 1;
u8 cmd[2]; u8 cmd[2];
int err; int err;
struct tm tm;
time_t date;
pulse8->vers = 0; pulse8->vers = 0;
cmd[0] = MSGCODE_PING;
err = pulse8_send_and_wait(pulse8, cmd, 1,
MSGCODE_COMMAND_ACCEPTED, 0);
cmd[0] = MSGCODE_FIRMWARE_VERSION; cmd[0] = MSGCODE_FIRMWARE_VERSION;
if (!err) err = pulse8_send_and_wait(pulse8, cmd, 1, cmd[0], 2);
err = pulse8_send_and_wait(pulse8, cmd, 1, cmd[0], 2);
if (err) if (err)
return err; return err;
pulse8->vers = (data[0] << 8) | data[1]; pulse8->vers = (data[0] << 8) | data[1];
dev_info(pulse8->dev, "Firmware version %04x\n", pulse8->vers); dev_info(pulse8->dev, "Firmware version %04x\n", pulse8->vers);
if (pulse8->vers < 2) if (pulse8->vers < 2)
return 0; return 0;
cmd[0] = MSGCODE_GET_BUILDDATE; cmd[0] = MSGCODE_GET_BUILDDATE;
if (!err) err = pulse8_send_and_wait(pulse8, cmd, 1, cmd[0], 4);
err = pulse8_send_and_wait(pulse8, cmd, 1, cmd[0], 4);
if (!err) {
time_t date = (data[0] << 24) | (data[1] << 16) |
(data[2] << 8) | data[3];
struct tm tm;
time_to_tm(date, 0, &tm);
dev_info(pulse8->dev, "Firmware build date %04ld.%02d.%02d %02d:%02d:%02d\n",
tm.tm_year + 1900, tm.tm_mon + 1, tm.tm_mday,
tm.tm_hour, tm.tm_min, tm.tm_sec);
}
if (err) if (err)
return err; return err;
date = (data[0] << 24) | (data[1] << 16) | (data[2] << 8) | data[3];
time_to_tm(date, 0, &tm);
dev_info(pulse8->dev, "Firmware build date %04ld.%02d.%02d %02d:%02d:%02d\n",
tm.tm_year + 1900, tm.tm_mon + 1, tm.tm_mday,
tm.tm_hour, tm.tm_min, tm.tm_sec);
dev_dbg(pulse8->dev, "Persistent config:\n"); dev_dbg(pulse8->dev, "Persistent config:\n");
cmd[0] = MSGCODE_GET_AUTO_ENABLED; cmd[0] = MSGCODE_GET_AUTO_ENABLED;
...@@ -456,8 +445,6 @@ static int pulse8_apply_persistent_config(struct pulse8 *pulse8, ...@@ -456,8 +445,6 @@ static int pulse8_apply_persistent_config(struct pulse8 *pulse8,
return err; return err;
cec_s_phys_addr(pulse8->adap, pa, false); cec_s_phys_addr(pulse8->adap, pa, false);
if (err)
return err;
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment