Re: [PATCH v10 10/30] rcar-vin: fix handling of single field frames (top, bottom and alternate fields)

2018-02-13 Thread Laurent Pinchart
Hi Niklas, On Wednesday, 14 February 2018 01:12:50 EET Niklas Söderlund wrote: > On 2018-02-14 00:31:21 +0200, Laurent Pinchart wrote: > > On Tuesday, 13 February 2018 18:47:04 EET Niklas Söderlund wrote: > >> On 2018-02-13 18:26:34 +0200, Laurent Pinchart wrote: > >>> On Monday, 29 January 2018 1

Re: [PATCH v10 10/30] rcar-vin: fix handling of single field frames (top, bottom and alternate fields)

2018-02-13 Thread Niklas Söderlund
Hi Laurent, On 2018-02-14 00:31:21 +0200, Laurent Pinchart wrote: > Hi Niklas, > > On Tuesday, 13 February 2018 18:47:04 EET Niklas Söderlund wrote: > > On 2018-02-13 18:26:34 +0200, Laurent Pinchart wrote: > > > On Monday, 29 January 2018 18:34:15 EET Niklas Söderlund wrote: > > >> There was nev

Re: [PATCH v10 10/30] rcar-vin: fix handling of single field frames (top, bottom and alternate fields)

2018-02-13 Thread Laurent Pinchart
Hi Niklas, On Tuesday, 13 February 2018 18:47:04 EET Niklas Söderlund wrote: > On 2018-02-13 18:26:34 +0200, Laurent Pinchart wrote: > > On Monday, 29 January 2018 18:34:15 EET Niklas Söderlund wrote: > >> There was never proper support in the VIN driver to deliver ALTERNATING > >> field format to

Re: [PATCH v10 10/30] rcar-vin: fix handling of single field frames (top, bottom and alternate fields)

2018-02-13 Thread Niklas Söderlund
Hi Laurent, On 2018-02-13 18:26:34 +0200, Laurent Pinchart wrote: > Hi Niklas, > > Thank you for the patch. Thanks for your comments. > > On Monday, 29 January 2018 18:34:15 EET Niklas Söderlund wrote: > > There was never proper support in the VIN driver to deliver ALTERNATING > > field format

Re: [PATCH v10 10/30] rcar-vin: fix handling of single field frames (top, bottom and alternate fields)

2018-02-13 Thread Laurent Pinchart
Hi Niklas, Thank you for the patch. On Monday, 29 January 2018 18:34:15 EET Niklas Söderlund wrote: > There was never proper support in the VIN driver to deliver ALTERNATING > field format to user-space, remove this field option. The problem is > that ALTERNATING filed order requires the sequence