Re: [PATCH v6 0/10] rcar-vin: Add support for parallel input on Gen3

[jacopo mondi <jacopo@jmondi.org>]

[-- Attachment #1: Type: text/plain, Size: 2921 bytes --]

HI Hans,

On Tue, Jul 17, 2018 at 03:00:20PM +0200, Hans Verkuil wrote:
> On 03/07/18 20:02, jacopo mondi wrote:
> > Hi Hans,
> >    As the VIN driver went through your tree, could you please pick
> > this series up now that all patches have been acked-by/reviewed-by and
> > tested on several platforms?
> >
> > Thank you...
>
> This got merged about 2 weeks ago. You were CC-ed on my pull request...

In facts, my original email was from 3rd of July :D

>
> Regards,
>
> 	Hans
>
> >
> > On Mon, Jul 02, 2018 at 08:11:04PM +0200, Niklas Söderlund wrote:
> >> Hi Jacopo,
> >>
> >> Nice work, I'm happy with the work you have done thanks!
> >>
> >> I have tested this series on M3-N to make sure it don't break anything
> >> on existing Gen3. Tested on V3M to make sure you can switch between
> >> CSI-2 and parallel input at runtime. And last I tested it on Koelsch to
> >> make sure Gen2 still works. All tests looks good.
> >>
> >> Thanks for your effort!
> >
> > Thanks for doing this Niklas...
> >
> > With your recent testing this series has been tested on:
> >
> > D3: parallel input
> > M3W: parallel and CSI-2
> > M3N: CSI-2
> > V3M: parallel and CSI-2
> > Gen2 Koelsch: parallel
> >
> > I guess is then good to go.
> >
> > Thanks
> >    j
> >
> >
> >>
> >> On 2018-06-12 11:43:22 +0200, Jacopo Mondi wrote:
> >>> Hello,
> >>>    this series adds support for parallel video input to the Gen3 version of
> >>> rcar-vin driver.
> >>>
> >>> Few changes compared to v5, closing a few comments from Kieran and Niklas,
> >>> and fixed the label names I forgot to change in previous version.
> >>>
> >>> Changlog in the individual patches when relevant.
> >>>
> >>> A few patches have not yet been acked-by, but things look smooth and we
> >>> should be close to have this finalized.
> >>>
> >>> Thanks
> >>>    j
> >>>
> >>> Jacopo Mondi (10):
> >>>   media: rcar-vin: Rename 'digital' to 'parallel'
> >>>   media: rcar-vin: Remove two empty lines
> >>>   media: rcar-vin: Create a group notifier
> >>>   media: rcar-vin: Cleanup notifier in error path
> >>>   media: rcar-vin: Cache the mbus configuration flags
> >>>   media: rcar-vin: Parse parallel input on Gen3
> >>>   media: rcar-vin: Link parallel input media entities
> >>>   media: rcar-vin: Handle parallel subdev in link_notify
> >>>   media: rcar-vin: Rename _rcar_info to rcar_info
> >>>   media: rcar-vin: Add support for R-Car R8A77995 SoC
> >>>
> >>>  drivers/media/platform/rcar-vin/rcar-core.c | 265 ++++++++++++++++++----------
> >>>  drivers/media/platform/rcar-vin/rcar-dma.c  |  36 ++--
> >>>  drivers/media/platform/rcar-vin/rcar-v4l2.c |  12 +-
> >>>  drivers/media/platform/rcar-vin/rcar-vin.h  |  29 +--
> >>>  4 files changed, 223 insertions(+), 119 deletions(-)
> >>>
> >>> --
> >>> 2.7.4
> >>>
> >>
> >> --
> >> Regards,
> >> Niklas Söderlund
>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]