From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>
Cc: Jacopo Mondi <jacopo+renesas@jmondi.org>,
geert@linux-m68k.org, horms@verge.net.au, uli+renesas@fpond.eu,
VenkataRajesh.Kalakodima@in.bosch.com, airlied@linux.ie,
daniel@ffwll.ch, koji.matsuoka.xm@renesas.com, muroya@ksk.co.jp,
Harsha.ManjulaMallikarjun@in.bosch.com, ezequiel@collabora.com,
seanpaul@chromium.org, linux-renesas-soc@vger.kernel.org,
dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org,
devicetree@vger.kernel.org, robh+dt@kernel.org,
mark.rutland@arm.com
Subject: Re: [PATCH v5 7/8] arm64: dts: renesas: Add CMM units to Gen3 SoCs
Date: Tue, 15 Oct 2019 21:06:05 +0300 [thread overview]
Message-ID: <20191015180605.GO4875@pendragon.ideasonboard.com> (raw)
In-Reply-To: <84f7b344-6a3a-edcc-3f3d-588825516bc2@ideasonboard.com>
Hi Jacopo and Kieran,
On Tue, Oct 15, 2019 at 01:52:29PM +0100, Kieran Bingham wrote:
> On 15/10/2019 11:46, Jacopo Mondi wrote:
> > Add CMM units to Renesas R-Car Gen3 SoC that support it, and reference them
> > from the Display Unit they are connected to.
> >
> > Sort the 'vsps', 'renesas,cmm' and 'status' properties in the DU unit
> > consistently in all the involved DTS.
>
> Going through this, I think I'm happy, except for a 'future' gotcha
> detailed below.
>
> The H3-N is possibly going to cause some issues (not
> supporting/connecting/using the CMM2) ... but as we don't really have
> that yet ... I'm going to say "la la la " ... and put this here:
>
> Reviewed-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>
>
> > Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>
> > ---
> > arch/arm64/boot/dts/renesas/r8a7795.dtsi | 39 +++++++++++++++++++++++
> > arch/arm64/boot/dts/renesas/r8a7796.dtsi | 31 +++++++++++++++++-
> > arch/arm64/boot/dts/renesas/r8a77965.dtsi | 31 +++++++++++++++++-
> > arch/arm64/boot/dts/renesas/r8a77990.dtsi | 21 ++++++++++++
> > arch/arm64/boot/dts/renesas/r8a77995.dtsi | 21 ++++++++++++
> > 5 files changed, 141 insertions(+), 2 deletions(-)
> >
> > diff --git a/arch/arm64/boot/dts/renesas/r8a7795.dtsi b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
> > index 6675462f7585..e16757af8c27 100644
> > --- a/arch/arm64/boot/dts/renesas/r8a7795.dtsi
> > +++ b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
> > @@ -2939,6 +2939,42 @@
> > iommus = <&ipmmu_vi1 10>;
> > };
> >
> > + cmm0: cmm@fea40000 {
> > + compatible = "renesas,r8a7795-cmm",
> > + "renesas,rcar-gen3-cmm";
> > + reg = <0 0xfea40000 0 0x1000>;
> > + power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
> > + clocks = <&cpg CPG_MOD 711>;
> > + resets = <&cpg 711>;
> > + };
> > +
> > + cmm1: cmm@fea50000 {
> > + compatible = "renesas,r8a7795-cmm",
> > + "renesas,rcar-gen3-cmm";
> > + reg = <0 0xfea50000 0 0x1000>;
> > + power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
> > + clocks = <&cpg CPG_MOD 710>;
> > + resets = <&cpg 710>;
> > + };
> > +
> > + cmm2: cmm@fea60000 {
> > + compatible = "renesas,r8a7795-cmm",
> > + "renesas,rcar-gen3-cmm";
> > + reg = <0 0xfea60000 0 0x1000>;
> > + power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
> > + clocks = <&cpg CPG_MOD 709>;
> > + resets = <&cpg 709>;
> > + };
>
> Yeouch. CMM2 is not available on the H3-N - but as far as I can tell the
> H3-N is an R8A7795 ...
>
> Geert, How will we differentiate this, or perhaps it just won't matter.
>
> The key part here will be handling it in the DU perhaps anyway.
I think we'll figure it out when we'll have more information about the
H3-N, and in particular if DU2 will be present (but not usable) or
completely absent. In the latter case we'll need a separate .dtsi.
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> > +
> > + cmm3: cmm@fea70000 {
> > + compatible = "renesas,r8a7795-cmm",
> > + "renesas,rcar-gen3-cmm";
> > + reg = <0 0xfea70000 0 0x1000>;
> > + power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
> > + clocks = <&cpg CPG_MOD 708>;
> > + resets = <&cpg 708>;
> > + };
> > +
> > csi20: csi2@fea80000 {
> > compatible = "renesas,r8a7795-csi2";
> > reg = <0 0xfea80000 0 0x10000>;
> > @@ -3142,7 +3178,10 @@
> > <&cpg CPG_MOD 722>,
> > <&cpg CPG_MOD 721>;
> > clock-names = "du.0", "du.1", "du.2", "du.3";
> > +
> > + renesas,cmms = <&cmm0>, <&cmm1>, <&cmm2>, <&cmm3>;
> > vsps = <&vspd0 0>, <&vspd1 0>, <&vspd2 0>, <&vspd0 1>;
> > +
> > status = "disabled";
> >
> > ports {
> > diff --git a/arch/arm64/boot/dts/renesas/r8a7796.dtsi b/arch/arm64/boot/dts/renesas/r8a7796.dtsi
> > index 822c96601d3c..597c47f3f994 100644
> > --- a/arch/arm64/boot/dts/renesas/r8a7796.dtsi
> > +++ b/arch/arm64/boot/dts/renesas/r8a7796.dtsi
> > @@ -2641,6 +2641,33 @@
> > renesas,fcp = <&fcpvi0>;
> > };
> >
> > + cmm0: cmm@fea40000 {
> > + compatible = "renesas,r8a7796-cmm",
> > + "renesas,rcar-gen3-cmm";
> > + reg = <0 0xfea40000 0 0x1000>;
> > + power-domains = <&sysc R8A7796_PD_ALWAYS_ON>;
> > + clocks = <&cpg CPG_MOD 711>;
> > + resets = <&cpg 711>;
> > + };
> > +
> > + cmm1: cmm@fea50000 {
> > + compatible = "renesas,r8a7796-cmm",
> > + "renesas,rcar-gen3-cmm";
> > + reg = <0 0xfea50000 0 0x1000>;
> > + power-domains = <&sysc R8A7796_PD_ALWAYS_ON>;
> > + clocks = <&cpg CPG_MOD 710>;
> > + resets = <&cpg 710>;
> > + };
> > +
> > + cmm2: cmm@fea60000 {
> > + compatible = "renesas,r8a7796-cmm",
> > + "renesas,rcar-gen3-cmm";
> > + reg = <0 0xfea60000 0 0x1000>;
> > + power-domains = <&sysc R8A7796_PD_ALWAYS_ON>;
> > + clocks = <&cpg CPG_MOD 709>;
> > + resets = <&cpg 709>;
> > + };
> > +
> > csi20: csi2@fea80000 {
> > compatible = "renesas,r8a7796-csi2";
> > reg = <0 0xfea80000 0 0x10000>;
> > @@ -2791,10 +2818,12 @@
> > <&cpg CPG_MOD 723>,
> > <&cpg CPG_MOD 722>;
> > clock-names = "du.0", "du.1", "du.2";
> > - status = "disabled";
> >
> > + renesas,cmms = <&cmm0>, <&cmm1>, <&cmm2>;
> > vsps = <&vspd0 0>, <&vspd1 0>, <&vspd2 0>;
> >
> > + status = "disabled";
> > +
> > ports {
> > #address-cells = <1>;
> > #size-cells = <0>;
> > diff --git a/arch/arm64/boot/dts/renesas/r8a77965.dtsi b/arch/arm64/boot/dts/renesas/r8a77965.dtsi
> > index 4ae163220f60..c3da8d26ccba 100644
> > --- a/arch/arm64/boot/dts/renesas/r8a77965.dtsi
> > +++ b/arch/arm64/boot/dts/renesas/r8a77965.dtsi
> > @@ -2320,6 +2320,33 @@
> > resets = <&cpg 611>;
> > };
> >
> > + cmm0: cmm@fea40000 {
> > + compatible = "renesas,r8a77965-cmm",
> > + "renesas,rcar-gen3-cmm";
> > + reg = <0 0xfea40000 0 0x1000>;
> > + power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
> > + clocks = <&cpg CPG_MOD 711>;
> > + resets = <&cpg 711>;
> > + };
> > +
> > + cmm1: cmm@fea50000 {
> > + compatible = "renesas,r8a77965-cmm",
> > + "renesas,rcar-gen3-cmm";
> > + reg = <0 0xfea50000 0 0x1000>;
> > + power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
> > + clocks = <&cpg CPG_MOD 710>;
> > + resets = <&cpg 710>;
> > + };
> > +
> > + cmm3: cmm@fea70000 {
> > + compatible = "renesas,r8a77965-cmm",
> > + "renesas,rcar-gen3-cmm";
> > + reg = <0 0xfea70000 0 0x1000>;
> > + power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
> > + clocks = <&cpg CPG_MOD 708>;
> > + resets = <&cpg 708>;
> > + };
> > +
> > csi20: csi2@fea80000 {
> > compatible = "renesas,r8a77965-csi2";
> > reg = <0 0xfea80000 0 0x10000>;
> > @@ -2467,10 +2494,12 @@
> > <&cpg CPG_MOD 723>,
> > <&cpg CPG_MOD 721>;
> > clock-names = "du.0", "du.1", "du.3";
> > - status = "disabled";
> >
> > + renesas,cmms = <&cmm0>, <&cmm1>, <&cmm3>;
> > vsps = <&vspd0 0>, <&vspd1 0>, <&vspd0 1>;
> >
> > + status = "disabled";
> > +
> > ports {
> > #address-cells = <1>;
> > #size-cells = <0>;
> > diff --git a/arch/arm64/boot/dts/renesas/r8a77990.dtsi b/arch/arm64/boot/dts/renesas/r8a77990.dtsi
> > index 455954c3d98e..bab9b7f96c72 100644
> > --- a/arch/arm64/boot/dts/renesas/r8a77990.dtsi
> > +++ b/arch/arm64/boot/dts/renesas/r8a77990.dtsi
> > @@ -1727,6 +1727,24 @@
> > iommus = <&ipmmu_vi0 9>;
> > };
> >
> > + cmm0: cmm@fea40000 {
> > + compatible = "renesas,r8a77990-cmm",
> > + "renesas,rcar-gen3-cmm";
> > + reg = <0 0xfea40000 0 0x1000>;
> > + power-domains = <&sysc R8A77990_PD_ALWAYS_ON>;
> > + clocks = <&cpg CPG_MOD 711>;
> > + resets = <&cpg 711>;
> > + };
> > +
> > + cmm1: cmm@fea50000 {
> > + compatible = "renesas,r8a77990-cmm",
> > + "renesas,rcar-gen3-cmm";
> > + reg = <0 0xfea50000 0 0x1000>;
> > + power-domains = <&sysc R8A77990_PD_ALWAYS_ON>;
> > + clocks = <&cpg CPG_MOD 710>;
> > + resets = <&cpg 710>;
> > + };
> > +
> > csi40: csi2@feaa0000 {
> > compatible = "renesas,r8a77990-csi2";
> > reg = <0 0xfeaa0000 0 0x10000>;
> > @@ -1768,7 +1786,10 @@
> > clock-names = "du.0", "du.1";
> > resets = <&cpg 724>;
> > reset-names = "du.0";
> > +
> > + renesas,cmms = <&cmm0>, <&cmm1>;
> > vsps = <&vspd0 0>, <&vspd1 0>;
> > +
> > status = "disabled";
> >
> > ports {
> > diff --git a/arch/arm64/boot/dts/renesas/r8a77995.dtsi b/arch/arm64/boot/dts/renesas/r8a77995.dtsi
> > index 183fef86cf7c..871c70cc2d2e 100644
> > --- a/arch/arm64/boot/dts/renesas/r8a77995.dtsi
> > +++ b/arch/arm64/boot/dts/renesas/r8a77995.dtsi
> > @@ -993,6 +993,24 @@
> > iommus = <&ipmmu_vi0 9>;
> > };
> >
> > + cmm0: cmm@fea40000 {
> > + compatible = "renesas,r8a77995-cmm",
> > + "renesas,rcar-gen3-cmm";
> > + reg = <0 0xfea40000 0 0x1000>;
> > + power-domains = <&sysc R8A77995_PD_ALWAYS_ON>;
> > + clocks = <&cpg CPG_MOD 711>;
> > + resets = <&cpg 711>;
> > + };
> > +
> > + cmm1: cmm@fea50000 {
> > + compatible = "renesas,r8a77995-cmm",
> > + "renesas,rcar-gen3-cmm";
> > + reg = <0 0xfea50000 0 0x1000>;
> > + power-domains = <&sysc R8A77995_PD_ALWAYS_ON>;
> > + clocks = <&cpg CPG_MOD 710>;
> > + resets = <&cpg 710>;
> > + };
> > +
> > du: display@feb00000 {
> > compatible = "renesas,du-r8a77995";
> > reg = <0 0xfeb00000 0 0x40000>;
> > @@ -1003,7 +1021,10 @@
> > clock-names = "du.0", "du.1";
> > resets = <&cpg 724>;
> > reset-names = "du.0";
> > +
> > + renesas,cmms = <&cmm0>, <&cmm1>;
> > vsps = <&vspd0 0>, <&vspd1 0>;
> > +
> > status = "disabled";
> >
> > ports {
--
Regards,
Laurent Pinchart
next prev parent reply other threads:[~2019-10-15 18:06 UTC|newest]
Thread overview: 76+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-10-15 10:46 [PATCH v5 0/8] drm: rcar-du: Add Color Management Module (CMM) Jacopo Mondi
2019-10-15 10:46 ` [PATCH v5 1/8] dt-bindings: display: renesas,cmm: Add R-Car CMM documentation Jacopo Mondi
2019-10-15 11:38 ` Kieran Bingham
2019-10-15 14:03 ` Geert Uytterhoeven
2019-10-15 14:03 ` Geert Uytterhoeven
2019-10-15 10:46 ` [PATCH v5 2/8] dt-bindings: display, renesas,du: Document cmms property Jacopo Mondi
2019-10-15 10:46 ` [PATCH v5 3/8] drm: rcar-du: Add support for CMM Jacopo Mondi
2019-10-15 11:53 ` Kieran Bingham
2019-10-15 11:53 ` Kieran Bingham
2019-10-15 13:17 ` Kieran Bingham
2019-10-15 13:33 ` Jacopo Mondi
2019-10-15 13:33 ` Jacopo Mondi
2019-10-15 14:26 ` Kieran Bingham
2019-10-15 14:26 ` Kieran Bingham
2019-10-15 17:25 ` Laurent Pinchart
2019-10-15 10:46 ` [PATCH v5 4/8] drm: rcar-du: kms: Initialize CMM instances Jacopo Mondi
2019-10-15 10:46 ` [PATCH v5 5/8] drm: rcar-du: crtc: Control CMM operations Jacopo Mondi
2019-10-15 13:15 ` Kieran Bingham
2019-10-15 13:15 ` Kieran Bingham
2019-10-15 13:37 ` Jacopo Mondi
2019-10-15 17:54 ` Laurent Pinchart
2019-10-15 17:54 ` Laurent Pinchart
2019-10-15 19:17 ` Jacopo Mondi
2019-10-15 19:17 ` Jacopo Mondi
2019-10-15 19:53 ` Laurent Pinchart
2019-10-15 10:46 ` [PATCH v5 6/8] drm: rcar-du: crtc: Register GAMMA_LUT properties Jacopo Mondi
2019-10-15 10:46 ` [PATCH v5 7/8] arm64: dts: renesas: Add CMM units to Gen3 SoCs Jacopo Mondi
2019-10-15 12:52 ` Kieran Bingham
2019-10-15 18:06 ` Laurent Pinchart [this message]
2019-10-15 10:46 ` [PATCH v5 8/8] drm: rcar-du: kms: Expand comment in vsps parsing routine Jacopo Mondi
2019-10-15 13:04 ` Kieran Bingham
2019-10-15 16:54 ` [PATCH v5 0/8] drm: rcar-du: Add Color Management Module (CMM) Laurent Pinchart
2019-11-11 11:21 ` Kalakodima Venkata Rajesh (RBEI/ECF3)
2019-11-11 11:21 ` Kalakodima Venkata Rajesh (RBEI/ECF3)
2019-11-11 11:21 ` Kalakodima Venkata Rajesh (RBEI/ECF3)
2019-11-11 13:06 ` Jacopo Mondi
2019-11-11 13:06 ` Jacopo Mondi
2019-11-11 13:06 ` Jacopo Mondi
2020-05-27 7:15 ` Eugeniu Rosca
2020-05-27 7:15 ` Eugeniu Rosca
2020-05-27 7:34 ` Geert Uytterhoeven
2020-05-27 7:34 ` Geert Uytterhoeven
2020-05-27 7:40 ` Gotthard Voellmeke
2020-05-27 7:40 ` Gotthard Voellmeke
2020-05-27 7:44 ` Eugeniu Rosca
2020-05-27 7:44 ` Eugeniu Rosca
2020-06-05 13:29 ` Jacopo Mondi
2020-06-05 13:29 ` Jacopo Mondi
2020-06-05 13:41 ` Eugeniu Rosca
2020-06-05 13:41 ` Eugeniu Rosca
2020-06-05 13:53 ` Jacopo Mondi
2020-06-05 13:53 ` Jacopo Mondi
2020-06-07 2:41 ` Laurent Pinchart
2020-06-07 2:41 ` Laurent Pinchart
2020-06-08 9:44 ` Eugeniu Rosca
2020-06-08 9:44 ` Eugeniu Rosca
2020-06-12 15:12 ` Jacopo Mondi
2020-06-12 15:12 ` Jacopo Mondi
2020-06-15 14:17 ` Eugeniu Rosca
2020-06-15 14:17 ` Eugeniu Rosca
2020-07-17 15:06 ` Jacopo Mondi
2020-07-17 15:06 ` Jacopo Mondi
2020-06-09 14:29 ` Eugeniu Rosca
2020-06-09 14:29 ` Eugeniu Rosca
2020-06-12 15:00 ` Jacopo Mondi
2020-06-12 15:00 ` Jacopo Mondi
2020-06-12 15:10 ` Laurent Pinchart
2020-06-12 15:10 ` Laurent Pinchart
2020-06-12 15:36 ` Eugeniu Rosca
2020-06-12 15:36 ` Eugeniu Rosca
2020-06-12 15:50 ` Laurent Pinchart
2020-06-12 15:50 ` Laurent Pinchart
2020-08-18 9:50 ` Geert Uytterhoeven
2020-08-18 9:50 ` Geert Uytterhoeven
2020-06-05 19:05 ` Laurent Pinchart
2020-06-05 19:05 ` Laurent Pinchart
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=20191015180605.GO4875@pendragon.ideasonboard.com \
--to=laurent.pinchart@ideasonboard.com \
--cc=Harsha.ManjulaMallikarjun@in.bosch.com \
--cc=VenkataRajesh.Kalakodima@in.bosch.com \
--cc=airlied@linux.ie \
--cc=daniel@ffwll.ch \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=ezequiel@collabora.com \
--cc=geert@linux-m68k.org \
--cc=horms@verge.net.au \
--cc=jacopo+renesas@jmondi.org \
--cc=kieran.bingham+renesas@ideasonboard.com \
--cc=koji.matsuoka.xm@renesas.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=muroya@ksk.co.jp \
--cc=robh+dt@kernel.org \
--cc=seanpaul@chromium.org \
--cc=uli+renesas@fpond.eu \
/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.