From: jacopo mondi <jacopo@jmondi.org>
To: "Niklas Söderlund" <niklas.soderlund@ragnatech.se>,
"Hans Verkuil" <hverkuil@xs4all.nl>
Cc: Jacopo Mondi <jacopo+renesas@jmondi.org>,
laurent.pinchart@ideasonboard.com, mchehab@kernel.org,
linux-media@vger.kernel.org, linux-renesas-soc@vger.kernel.org
Subject: Re: [PATCH v6 0/10] rcar-vin: Add support for parallel input on Gen3
Date: Tue, 3 Jul 2018 20:02:10 +0200 [thread overview]
Message-ID: <20180703180210.GB5611@w540> (raw)
In-Reply-To: <20180702181104.GZ5237@bigcity.dyn.berto.se>
[-- Attachment #1: Type: text/plain, Size: 2425 bytes --]
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...
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 --]
next prev parent reply other threads:[~2018-07-03 18:02 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-06-12 9:43 [PATCH v6 0/10] rcar-vin: Add support for parallel input on Gen3 Jacopo Mondi
2018-06-12 9:43 ` [PATCH v6 01/10] media: rcar-vin: Rename 'digital' to 'parallel' Jacopo Mondi
2018-06-12 9:43 ` [PATCH v6 02/10] media: rcar-vin: Remove two empty lines Jacopo Mondi
2018-06-12 9:43 ` [PATCH v6 03/10] media: rcar-vin: Create a group notifier Jacopo Mondi
2018-07-02 18:07 ` Niklas Söderlund
2018-07-02 18:07 ` Niklas Söderlund
2018-06-12 9:43 ` [PATCH v6 04/10] media: rcar-vin: Cleanup notifier in error path Jacopo Mondi
2018-06-12 9:43 ` [PATCH v6 05/10] media: rcar-vin: Cache the mbus configuration flags Jacopo Mondi
2018-06-12 9:43 ` [PATCH v6 06/10] media: rcar-vin: Parse parallel input on Gen3 Jacopo Mondi
2018-07-02 18:08 ` Niklas Söderlund
2018-07-02 18:08 ` Niklas Söderlund
2018-06-12 9:43 ` [PATCH v6 07/10] media: rcar-vin: Link parallel input media entities Jacopo Mondi
2018-06-12 9:43 ` [PATCH v6 08/10] media: rcar-vin: Handle parallel subdev in link_notify Jacopo Mondi
2018-06-12 9:43 ` [PATCH v6 09/10] media: rcar-vin: Rename _rcar_info to rcar_info Jacopo Mondi
2018-06-12 9:43 ` [PATCH v6 10/10] media: rcar-vin: Add support for R-Car R8A77995 SoC Jacopo Mondi
2018-07-04 7:36 ` Hans Verkuil
2018-07-04 7:49 ` jacopo mondi
2018-07-04 8:00 ` Geert Uytterhoeven
2018-07-04 8:08 ` jacopo mondi
2018-07-04 8:15 ` Hans Verkuil
2018-07-04 8:35 ` jacopo mondi
2018-07-02 18:11 ` [PATCH v6 0/10] rcar-vin: Add support for parallel input on Gen3 Niklas Söderlund
2018-07-02 18:11 ` Niklas Söderlund
2018-07-03 18:02 ` jacopo mondi [this message]
2018-07-04 6:08 ` Niklas Söderlund
2018-07-04 6:08 ` Niklas Söderlund
2018-07-17 13:00 ` Hans Verkuil
2018-07-17 13:07 ` jacopo mondi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180703180210.GB5611@w540 \
--to=jacopo@jmondi.org \
--cc=hverkuil@xs4all.nl \
--cc=jacopo+renesas@jmondi.org \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-media@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=mchehab@kernel.org \
--cc=niklas.soderlund@ragnatech.se \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.