linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Hans Verkuil <hverkuil@xs4all.nl>
To: Fabrizio Castro <fabrizio.castro@bp.renesas.com>,
	Mauro Carvalho Chehab <mchehab@kernel.org>
Cc: "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>,
	"Laurent Pinchart" <laurent.pinchart@ideasonboard.com>
Subject: Re: [PATCH 1/5] media: dt-bindings: media: rcar-csi2: Add r8a774a1 support
Date: Fri, 1 Mar 2019 13:55:53 +0100	[thread overview]
Message-ID: <8a5429a0-b4c5-a208-3e56-406bd031b01b@xs4all.nl> (raw)
In-Reply-To: <TYXPR01MB1775F18270FB477D010C180EC0760@TYXPR01MB1775.jpnprd01.prod.outlook.com>

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.

Regards,

	Hans

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.
>> --
>> 2.7.4
> 


  reply	other threads:[~2019-03-01 12:56 UTC|newest]

Thread overview: 46+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-09-10 14:31 [PATCH 0/5] Add VIN support to RZ/G2M Biju Das
2018-09-10 14:31 ` [PATCH 1/5] media: dt-bindings: media: rcar-csi2: Add r8a774a1 support Biju Das
2018-09-17 11:04   ` Niklas Söderlund
2018-11-08 12:50   ` Fabrizio Castro
2018-11-08 14:08     ` Simon Horman
2018-11-08 14:19       ` Fabrizio Castro
2018-11-08 14:09   ` Simon Horman
2018-11-08 14:18   ` Fabrizio Castro
2019-01-21 17:17     ` Fabrizio Castro
2019-03-01 12:09   ` Fabrizio Castro
2019-03-01 12:55     ` Hans Verkuil [this message]
2019-03-01 12:58       ` Geert Uytterhoeven
2019-03-01 13:08         ` Hans Verkuil
2019-03-01 13:12           ` Biju Das
2019-03-01 13:19             ` Hans Verkuil
2019-03-01 13:21               ` Biju Das
2019-03-01 13:13           ` Geert Uytterhoeven
2019-03-01 13:06       ` Laurent Pinchart
2019-03-07  0:44         ` Niklas Söderlund
     [not found]   ` <5b9f3f73.1c69fb81.a4966.287a@mx.google.com>
2019-03-01 13:28     ` FW: " Biju Das
2018-09-10 14:31 ` [PATCH 2/5] media: rcar-csi2: Enable support for r8a774a1 Biju Das
2018-09-17 11:05   ` Niklas Söderlund
2018-11-08 12:51   ` Fabrizio Castro
2018-11-08 14:10     ` Simon Horman
2018-11-08 14:24       ` Fabrizio Castro
2018-11-08 14:23   ` Fabrizio Castro
2019-01-21 17:18     ` Fabrizio Castro
2018-11-08 14:25   ` Simon Horman
2019-03-01 12:11   ` Fabrizio Castro
2018-09-10 14:31 ` [PATCH 3/5] media: dt-bindings: media: rcar_vin: Add r8a774a1 support Biju Das
2018-09-17 11:07   ` Niklas Söderlund
2018-11-08 12:52   ` Fabrizio Castro
2018-11-08 14:20     ` Simon Horman
2018-11-08 14:25       ` Fabrizio Castro
2018-11-08 14:25   ` Fabrizio Castro
2019-01-21 17:19     ` Fabrizio Castro
2018-11-08 14:25   ` Simon Horman
2019-03-01 12:12   ` Fabrizio Castro
     [not found]   ` <5b9f3f74.1c69fb81.f4e64.f599@mx.google.com>
2019-03-01 13:29     ` FW: " Biju Das
2018-09-10 14:31 ` [PATCH 4/5] media: rcar-vin: Enable support for r8a774a1 Biju Das
2018-09-17 11:08   ` Niklas Söderlund
2018-11-08 12:53   ` Fabrizio Castro
2018-11-08 14:26   ` Simon Horman
2018-11-29 15:12   ` Biju Das
2019-01-21 17:19     ` Fabrizio Castro
2019-03-01 12:14   ` Fabrizio Castro

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=8a5429a0-b4c5-a208-3e56-406bd031b01b@xs4all.nl \
    --to=hverkuil@xs4all.nl \
    --cc=Chris.Paterson2@renesas.com \
    --cc=biju.das@bp.renesas.com \
    --cc=devicetree@vger.kernel.org \
    --cc=fabrizio.castro@bp.renesas.com \
    --cc=geert+renesas@glider.be \
    --cc=horms@verge.net.au \
    --cc=laurent.pinchart@ideasonboard.com \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).