Re: [PATCH 1/5] media: dt-bindings: media: rcar-csi2: Add r8a774a1 support

From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: "Fabrizio Castro" <fabrizio.castro@bp.renesas.com>,
	"Mauro Carvalho Chehab" <mchehab@kernel.org>,
	"Biju Das" <biju.das@bp.renesas.com>,
	"Niklas Söderlund" <niklas.soderlund@ragnatech.se>,
	"linux-media@vger.kernel.org" <linux-media@vger.kernel.org>,
	"linux-renesas-soc@vger.kernel.org"
	<linux-renesas-soc@vger.kernel.org>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"Simon Horman" <horms@verge.net.au>,
	"Geert Uytterhoeven" <geert+renesas@glider.be>,
	"Chris Paterson" <Chris.Paterson2@renesas.com>,
	"Rob Herring" <robh+dt@kernel.org>,
	"Mark Rutland" <mark.rutland@arm.com>
Subject: Re: [PATCH 1/5] media: dt-bindings: media: rcar-csi2: Add r8a774a1 support
Date: Fri, 1 Mar 2019 15:06:13 +0200	[thread overview]
Message-ID: <20190301130613.GB32244@pendragon.ideasonboard.com> (raw)
In-Reply-To: <8a5429a0-b4c5-a208-3e56-406bd031b01b@xs4all.nl>

Hi Hans,

On Fri, Mar 01, 2019 at 01:55:53PM +0100, Hans Verkuil wrote:
> Hi Fabrizio,
> 
> It looks like this series fell through the cracks.
> 
> I looked at it and the main problem is that it is missing a Reviewed-by
> from Rob Herring (devicetree maintainer). It's a bit surprising since he
> is usually fairly prompt.
> 
> I recommend that you rebase and repost it and I'll delegate the v2 series
> to me so we can hopefully get it in for 5.2 (5.1 is likely too late) once
> Rob reviews it.
> 
> BTW, I'm the one who usually processes rcar patches. But it was delegated in
> patchwork to Laurent, so I never saw it.

I handle the VSP and FDP patches. I propose delegating the CSI-2 and VIN
to Niklas.

> On 3/1/19 1:09 PM, Fabrizio Castro wrote:
> > Hello Mauro,
> > 
> > This patch has been around for some time now, do you think you can take it?
> > 
> > Cheers,
> > Fab
> > 
> >> From: Biju Das <biju.das@bp.renesas.com>
> >> Sent: 10 September 2018 15:31
> >> Subject: [PATCH 1/5] media: dt-bindings: media: rcar-csi2: Add r8a774a1 support
> >>
> >> Document RZ/G2M (R8A774A1) SoC bindings.
> >>
> >> The RZ/G2M SoC is similar to R-Car M3-W (R8A7796).
> >>
> >> Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> >> Reviewed-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
> >> ---
> >>  Documentation/devicetree/bindings/media/renesas,rcar-csi2.txt | 5 +++--
> >>  1 file changed, 3 insertions(+), 2 deletions(-)
> >>
> >> diff --git a/Documentation/devicetree/bindings/media/renesas,rcar-csi2.txt
> >> b/Documentation/devicetree/bindings/media/renesas,rcar-csi2.txt
> >> index 2d385b6..12fe685 100644
> >> --- a/Documentation/devicetree/bindings/media/renesas,rcar-csi2.txt
> >> +++ b/Documentation/devicetree/bindings/media/renesas,rcar-csi2.txt
> >> @@ -2,12 +2,13 @@ Renesas R-Car MIPI CSI-2
> >>  ------------------------
> >>
> >>  The R-Car CSI-2 receiver device provides MIPI CSI-2 capabilities for the
> >> -Renesas R-Car family of devices. It is used in conjunction with the
> >> -R-Car VIN module, which provides the video capture capabilities.
> >> +Renesas R-Car Gen3 and RZ/G2 family of devices. It is used in conjunction
> >> +with the R-Car VIN module, which provides the video capture capabilities.
> >>
> >>  Mandatory properties
> >>  --------------------
> >>   - compatible: Must be one or more of the following
> >> +   - "renesas,r8a774a1-csi2" for the R8A774A1 device.
> >>     - "renesas,r8a7795-csi2" for the R8A7795 device.
> >>     - "renesas,r8a7796-csi2" for the R8A7796 device.
> >>     - "renesas,r8a77965-csi2" for the R8A77965 device.

-- 
Regards,

Laurent Pinchart