Hi Hans, Yes. Will do. Regards, Biju > -----Original Message----- > From: Hans Verkuil <hverkuil@xxxxxxxxx> > Sent: 01 March 2019 13:19 > To: Biju Das <biju.das@xxxxxxxxxxxxxx>; Geert Uytterhoeven > <geert@xxxxxxxxxxxxxx> > Cc: Fabrizio Castro <fabrizio.castro@xxxxxxxxxxxxxx>; Mauro Carvalho > Chehab <mchehab@xxxxxxxxxx>; Niklas Söderlund > <niklas.soderlund@xxxxxxxxxxxx>; linux-media@xxxxxxxxxxxxxxx; linux- > renesas-soc@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; Simon Horman > <horms@xxxxxxxxxxxx>; Geert Uytterhoeven <geert+renesas@xxxxxxxxx>; > Chris Paterson <Chris.Paterson2@xxxxxxxxxxx>; Rob Herring > <robh+dt@xxxxxxxxxx>; Mark Rutland <mark.rutland@xxxxxxx>; Laurent > Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > Subject: Re: [PATCH 1/5] media: dt-bindings: media: rcar-csi2: Add r8a774a1 > support > > Hi Biju, > > Can you do the following: > > 1) forward both of Rob's replies with his Reviewed-by tag to linux-media, > that way I have seen it. > 2) rebase the patch series and add all Reviewed-by etc. tags and post as > a v2. I'll pick it up and make sure it will get merged. Not sure if we > can manage 5.1, but it will certainly get in 5.2. > > Regards, > > Hans > > On 3/1/19 2:12 PM, Biju Das wrote: > > Hi Hans, > > > > Thanks for the feedback. > > > >> -----Original Message----- > >> From: Hans Verkuil <hverkuil@xxxxxxxxx> > >> Sent: 01 March 2019 13:09 > >> To: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> > >> Cc: Fabrizio Castro <fabrizio.castro@xxxxxxxxxxxxxx>; Mauro Carvalho > >> Chehab <mchehab@xxxxxxxxxx>; Biju Das <biju.das@xxxxxxxxxxxxxx>; > >> Niklas Söderlund <niklas.soderlund@xxxxxxxxxxxx>; linux- > >> media@xxxxxxxxxxxxxxx; linux-renesas-soc@xxxxxxxxxxxxxxx; > >> devicetree@xxxxxxxxxxxxxxx; Simon Horman <horms@xxxxxxxxxxxx>; > Geert > >> Uytterhoeven <geert+renesas@xxxxxxxxx>; Chris Paterson > >> <Chris.Paterson2@xxxxxxxxxxx>; Rob Herring <robh+dt@xxxxxxxxxx>; > Mark > >> Rutland <mark.rutland@xxxxxxx>; Laurent Pinchart > >> <laurent.pinchart@xxxxxxxxxxxxxxxx> > >> Subject: Re: [PATCH 1/5] media: dt-bindings: media: rcar-csi2: Add > >> r8a774a1 support > >> > >> On 3/1/19 1:58 PM, Geert Uytterhoeven wrote: > >>> Hi Hans, > >>> > >>> On Fri, Mar 1, 2019 at 1:55 PM Hans Verkuil <hverkuil@xxxxxxxxx> wrote: > >>>> 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. > >>> > >>> He actually did provide his Rb on Sep 17. > >> > >> Hmm, I don't see anything about that in my linux-media archive, and > >> patchwork didn't pick that up either. > >> > >> Was linux-media in the CC list of Rob's reply? > > > > Yes. Please see below. > > > >> -----Original Message----- > >> From: Rob Herring <robh@xxxxxxxxxx> > >> Sent: 17 September 2018 06:45 > >> To: Biju Das <biju.das@xxxxxxxxxxxxxx> > >> Cc: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>; Mark Rutland > >> <mark.rutland@xxxxxxx>; Biju Das <biju.das@xxxxxxxxxxxxxx>; Niklas > >> Söderlund <niklas.soderlund@xxxxxxxxxxxx>; > >> linux-media@xxxxxxxxxxxxxxx; linux-renesas-soc@xxxxxxxxxxxxxxx; > >> devicetree@xxxxxxxxxxxxxxx; Simon Horman <horms@xxxxxxxxxxxx>; > Geert > >> Uytterhoeven <geert+renesas@xxxxxxxxx>; Chris Paterson > >> <Chris.Paterson2@xxxxxxxxxxx>; Fabrizio Castro > >> <fabrizio.castro@xxxxxxxxxxxxxx> > >> Subject: Re: [PATCH 1/5] media: dt-bindings: media: rcar-csi2: Add > >> r8a774a1 support > >> > >> On Mon, 10 Sep 2018 15:31:14 +0100, Biju Das wrote: > >>> 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@xxxxxxxxxxxxxx> > >>> Reviewed-by: Fabrizio Castro <fabrizio.castro@xxxxxxxxxxxxxx> > >>> --- > >>> Documentation/devicetree/bindings/media/renesas,rcar-csi2.txt | 5 > >>> +++-- > >>> 1 file changed, 3 insertions(+), 2 deletions(-) > >>> > >> > >> Reviewed-by: Rob Herring <robh@xxxxxxxxxx> > > > > Regards, > > Biju > > > > > > 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 > > 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