Re: [PATCH 2/2] media: i2c: adv748x: Remove PAGE_WAIT

2019-01-14 Thread Niklas Söderlund
Hi Kieran, Thanks for your patch. On 2019-01-11 17:41:41 +, Kieran Bingham wrote: > The ADV748X_PAGE_WAIT is a fake page to insert arbitrary delays in the > register tables. > > Its only usage was removed, so we can remove the handling and simplify > the code. > > Signed-off-by: Kieran Bing

Re: [PATCH 2/2] media: i2c: adv748x: Remove PAGE_WAIT

2019-01-12 Thread Kieran Bingham
Hi Laurent, On 11/01/2019 20:23, Laurent Pinchart wrote: > Hi Kieran, > > Thank you for the patch. > > On Friday, 11 January 2019 19:41:41 EET Kieran Bingham wrote: >> The ADV748X_PAGE_WAIT is a fake page to insert arbitrary delays in the >> register tables. >> >> Its only usage was removed, so

Re: [PATCH 2/2] media: i2c: adv748x: Remove PAGE_WAIT

2019-01-11 Thread Laurent Pinchart
Hi Kieran, Thank you for the patch. On Friday, 11 January 2019 19:41:41 EET Kieran Bingham wrote: > The ADV748X_PAGE_WAIT is a fake page to insert arbitrary delays in the > register tables. > > Its only usage was removed, so we can remove the handling and simplify > the code. > > Signed-off-by:

[PATCH 2/2] media: i2c: adv748x: Remove PAGE_WAIT

2019-01-11 Thread Kieran Bingham
The ADV748X_PAGE_WAIT is a fake page to insert arbitrary delays in the register tables. Its only usage was removed, so we can remove the handling and simplify the code. Signed-off-by: Kieran Bingham --- drivers/media/i2c/adv748x/adv748x-core.c | 17 ++--- drivers/media/i2c/adv748x/a