[PATCH v3 1/5] media: hi846: fix hi846_write_reg_16 handling

Markus Elfring Markus.Elfring at web.de
Mon May 11 02:36:20 PDT 2026


…
> Fix this by resetting *err to 0 only when it is positive.

How do you think about to handle return values by an adjusted interface?
https://elixir.bootlin.com/linux/v7.1-rc2/source/drivers/media/i2c/hi846.c#L1265-L1282

Would a function signature (like the following) be nicer?

static int hi846_write_reg_16(struct hi846 *hi846, u16 reg, u16 val);

Regards,
Markus



More information about the linux-arm-kernel mailing list