All of lore.kernel.org
 help / color / mirror / Atom feed
From: Biju Das <biju.das@bp.renesas.com>
To: Hans Verkuil <hverkuil@xs4all.nl>,
	Mauro Carvalho Chehab <mchehab@kernel.org>,
	Rob Herring <robh+dt@kernel.org>,
	Mark Rutland <mark.rutland@arm.com>
Cc: "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>,
	"Fabrizio Castro" <fabrizio.castro@bp.renesas.com>
Subject: RE: [PATCH RESEND v2 1/2] media: dt-bindings: media: rcar-csi2: Add r8a774a1 support
Date: Fri, 1 Mar 2019 15:18:09 +0000	[thread overview]
Message-ID: <OSBPR01MB2103FD41B7BA18CDF16AAE36B8760@OSBPR01MB2103.jpnprd01.prod.outlook.com> (raw)
In-Reply-To: <OSBPR01MB2103FC038B64247EE41AE256B8760@OSBPR01MB2103.jpnprd01.prod.outlook.com>

Hi Hans,

> Subject: RE: [PATCH RESEND v2 1/2] media: dt-bindings: media: rcar-csi2: Add
> r8a774a1 support
>
> Hi Hans,
>
> Thanks for the feedback.
>
> > Subject: Re: [PATCH RESEND v2 1/2] media: dt-bindings: media:
> > rcar-csi2: Add
> > r8a774a1 support
> >
> > On 3/1/19 3:24 PM, Biju Das wrote:
> > > Document RZ/G2M (R8A774A1) SoC bindings.
> >
> > Please resend the whole series, not just the dt-bindings patches.
> >
> > Also note that the original v1 series said that there were 5 patches
> > in the series, but only the first 4 were received on linux-media. So I
> > have no idea what the 5th patch was (dts change perhaps?).
>
> Yes,  It is dts patch.

This 5th patch is already in media tree . commit 0c85e78fb1d3742c ("arm64: dts: renesas: r8a774a1: Add VIN and CSI-2 nodes")
So I will resend the patch series with first 4 patches.

Regards,
Biju

>
> > >
> > > 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>
> > > Acked-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
> > > Reviewed-by: Simon Horman <horms+renesas@verge.net.au>
> > > Reviewed-by: Rob Herring <robh@kernel.org>
> > > ---
> > > V1->V2
> > >    * No change
> > > ---
> > >  Documentation/devicetree/bindings/media/renesas,rcar-csi2.txt | 1 +
> > >  1 file changed, 1 insertion(+)
> > >
> > > diff --git
> > > a/Documentation/devicetree/bindings/media/renesas,rcar-csi2.txt
> > > b/Documentation/devicetree/bindings/media/renesas,rcar-csi2.txt
> > > index d63275e..9932458 100644
> > > --- a/Documentation/devicetree/bindings/media/renesas,rcar-csi2.txt
> > > +++ b/Documentation/devicetree/bindings/media/renesas,rcar-csi2.txt
> > > @@ -8,6 +8,7 @@ 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,r8a774c0-csi2" for the R8A774C0 device.
> > >     - "renesas,r8a7795-csi2" for the R8A7795 device.
> > >     - "renesas,r8a7796-csi2" for the R8A7796 device.
> > >



Renesas Electronics Europe GmbH,Geschaeftsfuehrer/President : Michael Hannawald, Sitz der Gesellschaft/Registered office: Duesseldorf, Arcadiastrasse 10, 40472 Duesseldorf, Germany,Handelsregister/Commercial Register: Duesseldorf, HRB 3708 USt-IDNr./Tax identification no.: DE 119353406 WEEE-Reg.-Nr./WEEE reg. no.: DE 14978647

      reply	other threads:[~2019-03-01 15:18 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-03-01 14:24 [PATCH RESEND v2 1/2] media: dt-bindings: media: rcar-csi2: Add r8a774a1 support Biju Das
2019-03-01 14:24 ` [PATCH RESEND v2 2/2] media: dt-bindings: media: rcar_vin: " Biju Das
2019-03-01 14:35 ` [PATCH RESEND v2 1/2] media: dt-bindings: media: rcar-csi2: " Hans Verkuil
2019-03-01 14:52   ` Biju Das
2019-03-01 15:18     ` Biju Das [this message]

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=OSBPR01MB2103FD41B7BA18CDF16AAE36B8760@OSBPR01MB2103.jpnprd01.prod.outlook.com \
    --to=biju.das@bp.renesas.com \
    --cc=Chris.Paterson2@renesas.com \
    --cc=devicetree@vger.kernel.org \
    --cc=fabrizio.castro@bp.renesas.com \
    --cc=geert+renesas@glider.be \
    --cc=horms@verge.net.au \
    --cc=hverkuil@xs4all.nl \
    --cc=linux-media@vger.kernel.org \
    --cc=linux-renesas-soc@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=mchehab@kernel.org \
    --cc=niklas.soderlund@ragnatech.se \
    --cc=robh+dt@kernel.org \
    /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.