Re: [PATCH v2 06/18] media: i2c: max9271: Check max9271_write() return

2021-03-15 Thread Laurent Pinchart
Hi Jacopo, Thank you for the patch. On Mon, Mar 15, 2021 at 02:15:00PM +0100, Jacopo Mondi wrote: > Check the return value of the max9271_write() function in the > max9271 library driver. > > While at it, modify an existing condition to be made identical > to other checks. > > Signed-off-by: Ja

Re: [PATCH v2 06/18] media: i2c: max9271: Check max9271_write() return

2021-03-15 Thread Kieran Bingham
Hi Jacopo, On 15/03/2021 13:15, Jacopo Mondi wrote: > Check the return value of the max9271_write() function in the > max9271 library driver. > > While at it, modify an existing condition to be made identical > to other checks. Excellent, much better to catch write errors early. Reviewed-by: Ki

[PATCH v2 06/18] media: i2c: max9271: Check max9271_write() return

2021-03-15 Thread Jacopo Mondi
Check the return value of the max9271_write() function in the max9271 library driver. While at it, modify an existing condition to be made identical to other checks. Signed-off-by: Jacopo Mondi --- drivers/media/i2c/max9271.c | 30 +++--- 1 file changed, 23 insertions(+)