Re: [PATCH 1/2] media: ov5645: Fix write_reg return code

2018-03-06 Thread Sakari Ailus
HI Mauro, On Tue, Mar 06, 2018 at 10:40:10AM -0300, Mauro Carvalho Chehab wrote: > Em Thu, 8 Feb 2018 11:41:59 +0200 > Todor Tomov escreveu: > > > I2C transfer functions return number of successful operations (on success). > > > > Do not return the received positive

Re: [PATCH 1/2] media: ov5645: Fix write_reg return code

2018-03-06 Thread Sakari Ailus
HI Mauro, On Tue, Mar 06, 2018 at 10:40:10AM -0300, Mauro Carvalho Chehab wrote: > Em Thu, 8 Feb 2018 11:41:59 +0200 > Todor Tomov escreveu: > > > I2C transfer functions return number of successful operations (on success). > > > > Do not return the received positive return code but instead

Re: [PATCH 1/2] media: ov5645: Fix write_reg return code

2018-03-06 Thread Mauro Carvalho Chehab
Em Thu, 8 Feb 2018 11:41:59 +0200 Todor Tomov escreveu: > I2C transfer functions return number of successful operations (on success). > > Do not return the received positive return code but instead return 0 on > success. The users of write_reg function already use this

Re: [PATCH 1/2] media: ov5645: Fix write_reg return code

2018-03-06 Thread Mauro Carvalho Chehab
Em Thu, 8 Feb 2018 11:41:59 +0200 Todor Tomov escreveu: > I2C transfer functions return number of successful operations (on success). > > Do not return the received positive return code but instead return 0 on > success. The users of write_reg function already use this logic. > >

[PATCH 1/2] media: ov5645: Fix write_reg return code

2018-02-08 Thread Todor Tomov
I2C transfer functions return number of successful operations (on success). Do not return the received positive return code but instead return 0 on success. The users of write_reg function already use this logic. Signed-off-by: Todor Tomov --- drivers/media/i2c/ov5645.c

[PATCH 1/2] media: ov5645: Fix write_reg return code

2018-02-08 Thread Todor Tomov
I2C transfer functions return number of successful operations (on success). Do not return the received positive return code but instead return 0 on success. The users of write_reg function already use this logic. Signed-off-by: Todor Tomov --- drivers/media/i2c/ov5645.c | 6 -- 1 file