On Wed Sep 30 14:41:27 2026 +0800, Hermes Wu wrote:
> it6625_hpd_delayed_work(): wrap the container_of() assignment after
> '=' instead of splitting the argument list mid-parenthesis.
>
> it6625_cec_adap_enable(): assign the cmds[] buffer at declaration
> instead of two separate statements, matching the pattern already used
> elsewhere in this file.
>
> it6625_cec_adap_log_addr(): add the missing spaces inside the cmds[]
> initializer's braces.
>
> Signed-off-by: Hermes Wu <[email protected]>
> Signed-off-by: Hans Verkuil <[email protected]>
Patch committed.
Thanks,
Hans Verkuil
drivers/media/i2c/it6625.c | 10 ++++------
1 file changed, 4 insertions(+), 6 deletions(-)
---
diff --git a/drivers/media/i2c/it6625.c b/drivers/media/i2c/it6625.c
index eaf8f939a0a4..61e021afa41b 100644
--- a/drivers/media/i2c/it6625.c
+++ b/drivers/media/i2c/it6625.c
@@ -760,8 +760,8 @@ static void it6625_enable_hpd(struct it6625 *it6625)
static void it6625_hpd_delayed_work(struct work_struct *work)
{
- struct it6625 *it6625 = container_of(work,
- struct it6625, hpd_delayed_work.work);
+ struct it6625 *it6625 =
+ container_of(work, struct it6625, hpd_delayed_work.work);
int val = 0;
guard(mutex)(&it6625->it6625_lock);
@@ -999,10 +999,8 @@ static int it6625_initial_setup(struct it6625 *it6625)
static int it6625_cec_adap_enable(struct cec_adapter *adap, bool enable)
{
struct it6625 *it6625 = adap->priv;
- u8 cmds[2];
+ u8 cmds[2] = { CMD_SET_CEC_ENABLE, enable ? 1 : 0 };
- cmds[0] = CMD_SET_CEC_ENABLE;
- cmds[1] = enable ? 1 : 0;
guard(mutex)(&it6625->it6625_lock);
it6625_write_command(it6625, cmds, sizeof(cmds));
@@ -1028,7 +1026,7 @@ static void it6625_cec_reset_la(struct it6625 *it6625,
bool keep_enabled)
static int it6625_cec_adap_log_addr(struct cec_adapter *adap, u8 log_addr)
{
struct it6625 *it6625 = adap->priv;
- u8 cmds[2] = {CMD_SET_CEC_LA, log_addr};
+ u8 cmds[2] = { CMD_SET_CEC_LA, log_addr };
dev_dbg(it6625->dev, "%s: la=%d", __func__, log_addr);
_______________________________________________
linuxtv-commits mailing list -- [email protected]
To unsubscribe send an email to [email protected]
