Message ID | 20201013150150.14801-5-fabrizio.castro.jz@renesas.com (mailing list archive) |
---|---|
State | Superseded, archived |
Headers |
Received: from vger.kernel.org ([23.128.96.18]) by www.linuxtv.org with esmtp (Exim 4.92) (envelope-from <linux-media-owner@vger.kernel.org>) id 1kSLiS-00GReF-RR; Tue, 13 Oct 2020 14:55:53 +0000 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2389126AbgJMPCf (ORCPT <rfc822;mkrufky@linuxtv.org> + 1 other); Tue, 13 Oct 2020 11:02:35 -0400 Received: from relmlor1.renesas.com ([210.160.252.171]:42312 "EHLO relmlie5.idc.renesas.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S2387958AbgJMPCS (ORCPT <rfc822;linux-media@vger.kernel.org>); Tue, 13 Oct 2020 11:02:18 -0400 X-IronPort-AV: E=Sophos;i="5.77,371,1596466800"; d="scan'208";a="59674410" Received: from unknown (HELO relmlir5.idc.renesas.com) ([10.200.68.151]) by relmlie5.idc.renesas.com with ESMTP; 14 Oct 2020 00:02:17 +0900 Received: from devel.example.org?044ree.adwin.renesas.com (unknown [10.226.36.120]) by relmlir5.idc.renesas.com (Postfix) with ESMTP id 933C94009410; Wed, 14 Oct 2020 00:02:14 +0900 (JST) From: Fabrizio Castro <fabrizio.castro.jz@renesas.com> To: Mauro Carvalho Chehab <mchehab@kernel.org>, Rob Herring <robh+dt@kernel.org>, Geert Uytterhoeven <geert+renesas@glider.be> Cc: Fabrizio Castro <fabrizio.castro.jz@renesas.com>, Ramesh Shanmugasundaram <rashanmu@gmail.com>, linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-renesas-soc@vger.kernel.org, Chris Paterson <Chris.Paterson2@renesas.com>, Biju Das <biju.das.jz@bp.renesas.com>, Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@bp.renesas.com>, Laurent Pinchart <laurent.pinchart@ideasonboard.com> Subject: [PATCH v3 4/5] media: dt-bindings: media: renesas,drif: Add r8a77965 support Date: Tue, 13 Oct 2020 16:01:49 +0100 Message-Id: <20201013150150.14801-5-fabrizio.castro.jz@renesas.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20201013150150.14801-1-fabrizio.castro.jz@renesas.com> References: <20201013150150.14801-1-fabrizio.castro.jz@renesas.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: <linux-media.vger.kernel.org> X-Mailing-List: linux-media@vger.kernel.org X-LSpam-Score: -2.4 (--) X-LSpam-Report: No, score=-2.4 required=5.0 tests=BAYES_00=-1.9,HEADER_FROM_DIFFERENT_DOMAINS=0.5,MAILING_LIST_MULTI=-1 autolearn=ham autolearn_force=no |
Series |
Add r8a77965 DRIF support
|
|
Commit Message
Fabrizio Castro
Oct. 13, 2020, 3:01 p.m. UTC
The r8a77965 (a.k.a. R-Car M3-N) device tree schema is
compatible with the already documented R-Car Gen3 devices.
Document r8a77965 support within renesas,drif.yaml.
Signed-off-by: Fabrizio Castro <fabrizio.castro.jz@renesas.com>
---
v2->v3:
* New patch
Documentation/devicetree/bindings/media/renesas,drif.yaml | 1 +
1 file changed, 1 insertion(+)
Comments
Hi Fabrizio, Thank you for the patch. On Tue, Oct 13, 2020 at 04:01:49PM +0100, Fabrizio Castro wrote: > The r8a77965 (a.k.a. R-Car M3-N) device tree schema is > compatible with the already documented R-Car Gen3 devices. > > Document r8a77965 support within renesas,drif.yaml. > > Signed-off-by: Fabrizio Castro <fabrizio.castro.jz@renesas.com> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > --- > v2->v3: > * New patch > > Documentation/devicetree/bindings/media/renesas,drif.yaml | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/media/renesas,drif.yaml b/Documentation/devicetree/bindings/media/renesas,drif.yaml > index ae50b1448320..89445ddd598e 100644 > --- a/Documentation/devicetree/bindings/media/renesas,drif.yaml > +++ b/Documentation/devicetree/bindings/media/renesas,drif.yaml > @@ -53,6 +53,7 @@ properties: > - enum: > - renesas,r8a7795-drif # R-Car H3 > - renesas,r8a7796-drif # R-Car M3-W > + - renesas,r8a77965-drif # R-Car M3-N > - renesas,r8a77990-drif # R-Car E3 > - const: renesas,rcar-gen3-drif # Generic R-Car Gen3 compatible device >
Hi Fabrizio, Thank you for the patch. On Tue, Oct 13, 2020 at 6:25 PM Fabrizio Castro <fabrizio.castro.jz@renesas.com> wrote: > > The r8a77965 (a.k.a. R-Car M3-N) device tree schema is > compatible with the already documented R-Car Gen3 devices. > > Document r8a77965 support within renesas,drif.yaml. > > Signed-off-by: Fabrizio Castro <fabrizio.castro.jz@renesas.com> > --- > v2->v3: > * New patch > > Documentation/devicetree/bindings/media/renesas,drif.yaml | 1 + > 1 file changed, 1 insertion(+) > Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> Cheers, Prabhakar > diff --git a/Documentation/devicetree/bindings/media/renesas,drif.yaml b/Documentation/devicetree/bindings/media/renesas,drif.yaml > index ae50b1448320..89445ddd598e 100644 > --- a/Documentation/devicetree/bindings/media/renesas,drif.yaml > +++ b/Documentation/devicetree/bindings/media/renesas,drif.yaml > @@ -53,6 +53,7 @@ properties: > - enum: > - renesas,r8a7795-drif # R-Car H3 > - renesas,r8a7796-drif # R-Car M3-W > + - renesas,r8a77965-drif # R-Car M3-N > - renesas,r8a77990-drif # R-Car E3 > - const: renesas,rcar-gen3-drif # Generic R-Car Gen3 compatible device > > -- > 2.25.1 >
Hi Fabrizio, On Tue, Oct 13, 2020 at 5:02 PM Fabrizio Castro <fabrizio.castro.jz@renesas.com> wrote: > The r8a77965 (a.k.a. R-Car M3-N) device tree schema is > compatible with the already documented R-Car Gen3 devices. > > Document r8a77965 support within renesas,drif.yaml. > > Signed-off-by: Fabrizio Castro <fabrizio.castro.jz@renesas.com> Thanks for your patch! Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> > --- a/Documentation/devicetree/bindings/media/renesas,drif.yaml > +++ b/Documentation/devicetree/bindings/media/renesas,drif.yaml > @@ -53,6 +53,7 @@ properties: > - enum: > - renesas,r8a7795-drif # R-Car H3 > - renesas,r8a7796-drif # R-Car M3-W > + - renesas,r8a77965-drif # R-Car M3-N > - renesas,r8a77990-drif # R-Car E3 > - const: renesas,rcar-gen3-drif # Generic R-Car Gen3 compatible device I guess you're aware M3-N (and E3) have an extra register? Probably the driver just relies on its initial value, but it never hurts to be explicit and initialize it properly. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
Hi Geert, Thank you for your feedback! > From: Geert Uytterhoeven <geert@linux-m68k.org> > Sent: 14 October 2020 14:38 > Subject: Re: [PATCH v3 4/5] media: dt-bindings: media: renesas,drif: Add > r8a77965 support > > Hi Fabrizio, > > On Tue, Oct 13, 2020 at 5:02 PM Fabrizio Castro > <fabrizio.castro.jz@renesas.com> wrote: > > The r8a77965 (a.k.a. R-Car M3-N) device tree schema is > > compatible with the already documented R-Car Gen3 devices. > > > > Document r8a77965 support within renesas,drif.yaml. > > > > Signed-off-by: Fabrizio Castro <fabrizio.castro.jz@renesas.com> > > Thanks for your patch! > > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> > > > --- a/Documentation/devicetree/bindings/media/renesas,drif.yaml > > +++ b/Documentation/devicetree/bindings/media/renesas,drif.yaml > > @@ -53,6 +53,7 @@ properties: > > - enum: > > - renesas,r8a7795-drif # R-Car H3 > > - renesas,r8a7796-drif # R-Car M3-W > > + - renesas,r8a77965-drif # R-Car M3-N > > - renesas,r8a77990-drif # R-Car E3 > > - const: renesas,rcar-gen3-drif # Generic R-Car Gen3 compatible device > > I guess you're aware M3-N (and E3) have an extra register? > Probably the driver just relies on its initial value, but it never hurts to be > explicit and initialize it properly. Yes, I am aware of the extra register, and that's reflected in the DRIF nodes definition within the SoC specific device trees. I'll tackle initialization and configuration of the extra register at some point, do you think we could use the default value for now? > > Gr{oetje,eeting}s, > > Geert > > > -- > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux- > m68k.org > > In personal conversations with technical people, I call myself a hacker. But > when I'm talking to journalists I just say "programmer" or something like that. > -- Linus Torvalds Renesas Electronics Europe GmbH, Geschaeftsfuehrer/President: Carsten Jauch, 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
Hi Fabrizio, On Wed, Oct 14, 2020 at 4:35 PM Fabrizio Castro <fabrizio.castro.jz@renesas.com> wrote: > > From: Geert Uytterhoeven <geert@linux-m68k.org> > > On Tue, Oct 13, 2020 at 5:02 PM Fabrizio Castro > > <fabrizio.castro.jz@renesas.com> wrote: > > > The r8a77965 (a.k.a. R-Car M3-N) device tree schema is > > > compatible with the already documented R-Car Gen3 devices. > > > > > > Document r8a77965 support within renesas,drif.yaml. > > > > > > Signed-off-by: Fabrizio Castro <fabrizio.castro.jz@renesas.com> > > > > Thanks for your patch! > > > > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> > > > > > --- a/Documentation/devicetree/bindings/media/renesas,drif.yaml > > > +++ b/Documentation/devicetree/bindings/media/renesas,drif.yaml > > > @@ -53,6 +53,7 @@ properties: > > > - enum: > > > - renesas,r8a7795-drif # R-Car H3 > > > - renesas,r8a7796-drif # R-Car M3-W > > > + - renesas,r8a77965-drif # R-Car M3-N > > > - renesas,r8a77990-drif # R-Car E3 > > > - const: renesas,rcar-gen3-drif # Generic R-Car Gen3 compatible device > > > > I guess you're aware M3-N (and E3) have an extra register? > > Probably the driver just relies on its initial value, but it never hurts to be > > explicit and initialize it properly. > > Yes, I am aware of the extra register, and that's reflected in the DRIF nodes > definition within the SoC specific device trees. > I'll tackle initialization and configuration of the extra register at some point, > do you think we could use the default value for now? Yes, that's fine for me, if it works with the current driver. Gr{oetje,eeting}s, Geert
Hi Geert, > From: Geert Uytterhoeven <geert@linux-m68k.org> > Sent: 14 October 2020 15:52 > Subject: Re: [PATCH v3 4/5] media: dt-bindings: media: renesas,drif: Add > r8a77965 support > > Hi Fabrizio, > > On Wed, Oct 14, 2020 at 4:35 PM Fabrizio Castro > <fabrizio.castro.jz@renesas.com> wrote: > > > From: Geert Uytterhoeven <geert@linux-m68k.org> > > > On Tue, Oct 13, 2020 at 5:02 PM Fabrizio Castro > > > <fabrizio.castro.jz@renesas.com> wrote: > > > > The r8a77965 (a.k.a. R-Car M3-N) device tree schema is > > > > compatible with the already documented R-Car Gen3 devices. > > > > > > > > Document r8a77965 support within renesas,drif.yaml. > > > > > > > > Signed-off-by: Fabrizio Castro <fabrizio.castro.jz@renesas.com> > > > > > > Thanks for your patch! > > > > > > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> > > > > > > > --- a/Documentation/devicetree/bindings/media/renesas,drif.yaml > > > > +++ b/Documentation/devicetree/bindings/media/renesas,drif.yaml > > > > @@ -53,6 +53,7 @@ properties: > > > > - enum: > > > > - renesas,r8a7795-drif # R-Car H3 > > > > - renesas,r8a7796-drif # R-Car M3-W > > > > + - renesas,r8a77965-drif # R-Car M3-N > > > > - renesas,r8a77990-drif # R-Car E3 > > > > - const: renesas,rcar-gen3-drif # Generic R-Car Gen3 compatible > device > > > > > > I guess you're aware M3-N (and E3) have an extra register? > > > Probably the driver just relies on its initial value, but it never hurts to be > > > explicit and initialize it properly. > > > > Yes, I am aware of the extra register, and that's reflected in the DRIF nodes > > definition within the SoC specific device trees. > > I'll tackle initialization and configuration of the extra register at some point, > > do you think we could use the default value for now? > > Yes, that's fine for me, if it works with the current driver. Yes, and thank you. Cheers, Fab > > Gr{oetje,eeting}s, > > Geert > > -- > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux- > m68k.org > > In personal conversations with technical people, I call myself a hacker. But > when I'm talking to journalists I just say "programmer" or something like that. > -- Linus Torvalds Renesas Electronics Europe GmbH, Geschaeftsfuehrer/President: Carsten Jauch, 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
diff --git a/Documentation/devicetree/bindings/media/renesas,drif.yaml b/Documentation/devicetree/bindings/media/renesas,drif.yaml index ae50b1448320..89445ddd598e 100644 --- a/Documentation/devicetree/bindings/media/renesas,drif.yaml +++ b/Documentation/devicetree/bindings/media/renesas,drif.yaml @@ -53,6 +53,7 @@ properties: - enum: - renesas,r8a7795-drif # R-Car H3 - renesas,r8a7796-drif # R-Car M3-W + - renesas,r8a77965-drif # R-Car M3-N - renesas,r8a77990-drif # R-Car E3 - const: renesas,rcar-gen3-drif # Generic R-Car Gen3 compatible device