* [PATCH v2 0/2] arm64: dts: renesas: r8a7799[05]: Add cpg reset for LVDS Interface
@ 2019-06-17 8:37 Simon Horman
2019-06-17 8:37 ` [PATCH v2 1/2] arm64: dts: renesas: r8a77990: " Simon Horman
2019-06-17 8:37 ` [PATCH v2 2/2] arm64: dts: renesas: r8a77995: " Simon Horman
0 siblings, 2 replies; 7+ messages in thread
From: Simon Horman @ 2019-06-17 8:37 UTC (permalink / raw)
To: linux-renesas-soc
Cc: Yoshihiro Kaneko, Magnus Damm, Geert Uytterhoeven,
Laurent Pinchart, Simon Horman, linux-arm-kernel
Hi,
this short series adds CPG reset for LVS interfaces of the
R-Car E3 (r8a77995) and D3 (r8a77990) SoCs.
Changes since v1
- only add one reset entry per group
Takeshi Kihara (1):
arm64: dts: renesas: r8a77990: Add cpg reset for LVDS Interface
Yoshihiro Kaneko (1):
arm64: dts: renesas: r8a77995: Add cpg reset for LVDS Interface
arch/arm64/boot/dts/renesas/r8a77990.dtsi | 2 ++
arch/arm64/boot/dts/renesas/r8a77995.dtsi | 2 ++
2 files changed, 4 insertions(+)
--
2.11.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v2 1/2] arm64: dts: renesas: r8a77990: Add cpg reset for LVDS Interface
2019-06-17 8:37 [PATCH v2 0/2] arm64: dts: renesas: r8a7799[05]: Add cpg reset for LVDS Interface Simon Horman
@ 2019-06-17 8:37 ` Simon Horman
2019-06-17 8:43 ` Geert Uytterhoeven
2019-06-17 8:37 ` [PATCH v2 2/2] arm64: dts: renesas: r8a77995: " Simon Horman
1 sibling, 1 reply; 7+ messages in thread
From: Simon Horman @ 2019-06-17 8:37 UTC (permalink / raw)
To: linux-renesas-soc
Cc: Takeshi Kihara, Yoshihiro Kaneko, Magnus Damm,
Geert Uytterhoeven, Laurent Pinchart, Simon Horman,
linux-arm-kernel
From: Takeshi Kihara <takeshi.kihara.df@renesas.com>
It is necessary to reset the LVDS Interface according to display on/off.
Therefore, this patch adds CPG reset properties in DU device node
for the R8A77990 SoC.
According to Laurent Pinchart, R-Car Gen3 reset is handled at the group
level so specifying one reset entry per group is sufficient. For this
reason <&cpg 724> is not listed as a reset for "du.1" as was the case in an
earlier revision of this patch.
Signed-off-by: Takeshi Kihara <takeshi.kihara.df@renesas.com>
Signed-off-by: Yoshihiro Kaneko <ykaneko0929@gmail.com>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
v2 [Simon Horman]
- only add one reset entry per group
v1 [Yoshihiro Kaneko]
v0 [Takeshi Kihara]
---
arch/arm64/boot/dts/renesas/r8a77990.dtsi | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm64/boot/dts/renesas/r8a77990.dtsi b/arch/arm64/boot/dts/renesas/r8a77990.dtsi
index b4318661f35e..84d1f58e73e7 100644
--- a/arch/arm64/boot/dts/renesas/r8a77990.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a77990.dtsi
@@ -1766,6 +1766,8 @@
clocks = <&cpg CPG_MOD 724>,
<&cpg CPG_MOD 723>;
clock-names = "du.0", "du.1";
+ resets = <&cpg 724>;
+ reset-names = "du.0";
vsps = <&vspd0 0 &vspd1 0>;
status = "disabled";
--
2.11.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH v2 2/2] arm64: dts: renesas: r8a77995: Add cpg reset for LVDS Interface
2019-06-17 8:37 [PATCH v2 0/2] arm64: dts: renesas: r8a7799[05]: Add cpg reset for LVDS Interface Simon Horman
2019-06-17 8:37 ` [PATCH v2 1/2] arm64: dts: renesas: r8a77990: " Simon Horman
@ 2019-06-17 8:37 ` Simon Horman
2019-06-17 8:43 ` Geert Uytterhoeven
1 sibling, 1 reply; 7+ messages in thread
From: Simon Horman @ 2019-06-17 8:37 UTC (permalink / raw)
To: linux-renesas-soc
Cc: Yoshihiro Kaneko, Magnus Damm, Geert Uytterhoeven,
Laurent Pinchart, Simon Horman, linux-arm-kernel
From: Yoshihiro Kaneko <ykaneko0929@gmail.com>
It is necessary to reset the LVDS Interface according to display on/off.
Therefore, this patch adds CPG reset properties in DU device node
for the R8A77995 SoC.
This patch was inspired by a patch in the BSP by Takeshi Kihara
<takeshi.kihara.df@renesas.com>.
According to Laurent Pinchart, R-Car Gen3 reset is handled at the group
level so specifying one reset entry per group is sufficient. For
this reason <&cpg 724> is not listed as a reset for "du.1" as
was the case in an earlier revision of this patch.
Signed-off-by: Yoshihiro Kaneko <ykaneko0929@gmail.com>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
v2 [Simon Horman]
- only add one reset entry per group
v1 [Yoshihiro Kaneko]
---
arch/arm64/boot/dts/renesas/r8a77995.dtsi | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm64/boot/dts/renesas/r8a77995.dtsi b/arch/arm64/boot/dts/renesas/r8a77995.dtsi
index 0a344eb55094..ca6aeabd6d04 100644
--- a/arch/arm64/boot/dts/renesas/r8a77995.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a77995.dtsi
@@ -1001,6 +1001,8 @@
clocks = <&cpg CPG_MOD 724>,
<&cpg CPG_MOD 723>;
clock-names = "du.0", "du.1";
+ resets = <&cpg 724>;
+ reset-names = "du.0";
vsps = <&vspd0 0 &vspd1 0>;
status = "disabled";
--
2.11.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v2 1/2] arm64: dts: renesas: r8a77990: Add cpg reset for LVDS Interface
2019-06-17 8:37 ` [PATCH v2 1/2] arm64: dts: renesas: r8a77990: " Simon Horman
@ 2019-06-17 8:43 ` Geert Uytterhoeven
2019-06-18 10:44 ` Simon Horman
0 siblings, 1 reply; 7+ messages in thread
From: Geert Uytterhoeven @ 2019-06-17 8:43 UTC (permalink / raw)
To: Simon Horman
Cc: Takeshi Kihara, Yoshihiro Kaneko, Magnus Damm, Linux-Renesas,
Laurent Pinchart, Linux ARM
Hi Simon,
On Mon, Jun 17, 2019 at 10:37 AM Simon Horman
<horms+renesas@verge.net.au> wrote:
> From: Takeshi Kihara <takeshi.kihara.df@renesas.com>
>
> It is necessary to reset the LVDS Interface according to display on/off.
This is not the LVDS interface.
The LVDS interface has its own device node.
> Therefore, this patch adds CPG reset properties in DU device node
> for the R8A77990 SoC.
>
> According to Laurent Pinchart, R-Car Gen3 reset is handled at the group
> level so specifying one reset entry per group is sufficient. For this
> reason <&cpg 724> is not listed as a reset for "du.1" as was the case in an
> earlier revision of this patch.
>
> Signed-off-by: Takeshi Kihara <takeshi.kihara.df@renesas.com>
> Signed-off-by: Yoshihiro Kaneko <ykaneko0929@gmail.com>
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> ---
> v2 [Simon Horman]
> - only add one reset entry per group
>
> v1 [Yoshihiro Kaneko]
>
> v0 [Takeshi Kihara]
> ---
> arch/arm64/boot/dts/renesas/r8a77990.dtsi | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/renesas/r8a77990.dtsi b/arch/arm64/boot/dts/renesas/r8a77990.dtsi
> index b4318661f35e..84d1f58e73e7 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77990.dtsi
> +++ b/arch/arm64/boot/dts/renesas/r8a77990.dtsi
> @@ -1766,6 +1766,8 @@
> clocks = <&cpg CPG_MOD 724>,
> <&cpg CPG_MOD 723>;
> clock-names = "du.0", "du.1";
> + resets = <&cpg 724>;
> + reset-names = "du.0";
> vsps = <&vspd0 0 &vspd1 0>;
> status = "disabled";
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
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 2/2] arm64: dts: renesas: r8a77995: Add cpg reset for LVDS Interface
2019-06-17 8:37 ` [PATCH v2 2/2] arm64: dts: renesas: r8a77995: " Simon Horman
@ 2019-06-17 8:43 ` Geert Uytterhoeven
0 siblings, 0 replies; 7+ messages in thread
From: Geert Uytterhoeven @ 2019-06-17 8:43 UTC (permalink / raw)
To: Simon Horman
Cc: Linux-Renesas, Yoshihiro Kaneko, Magnus Damm, Laurent Pinchart,
Linux ARM
Hi Simon,
On Mon, Jun 17, 2019 at 10:37 AM Simon Horman
<horms+renesas@verge.net.au> wrote:
> From: Yoshihiro Kaneko <ykaneko0929@gmail.com>
>
> It is necessary to reset the LVDS Interface according to display on/off.
This is not the LVDS interface.
The LVDS interface has its own device node.
> Therefore, this patch adds CPG reset properties in DU device node
> for the R8A77995 SoC.
>
> This patch was inspired by a patch in the BSP by Takeshi Kihara
> <takeshi.kihara.df@renesas.com>.
>
> According to Laurent Pinchart, R-Car Gen3 reset is handled at the group
> level so specifying one reset entry per group is sufficient. For
> this reason <&cpg 724> is not listed as a reset for "du.1" as
> was the case in an earlier revision of this patch.
>
> Signed-off-by: Yoshihiro Kaneko <ykaneko0929@gmail.com>
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> ---
> v2 [Simon Horman]
> - only add one reset entry per group
>
> v1 [Yoshihiro Kaneko]
> --- a/arch/arm64/boot/dts/renesas/r8a77995.dtsi
> +++ b/arch/arm64/boot/dts/renesas/r8a77995.dtsi
> @@ -1001,6 +1001,8 @@
> clocks = <&cpg CPG_MOD 724>,
> <&cpg CPG_MOD 723>;
> clock-names = "du.0", "du.1";
> + resets = <&cpg 724>;
> + reset-names = "du.0";
> vsps = <&vspd0 0 &vspd1 0>;
> status = "disabled";
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
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 1/2] arm64: dts: renesas: r8a77990: Add cpg reset for LVDS Interface
2019-06-17 8:43 ` Geert Uytterhoeven
@ 2019-06-18 10:44 ` Simon Horman
2019-06-18 11:19 ` Geert Uytterhoeven
0 siblings, 1 reply; 7+ messages in thread
From: Simon Horman @ 2019-06-18 10:44 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Takeshi Kihara, Yoshihiro Kaneko, Magnus Damm, Linux-Renesas,
Laurent Pinchart, Linux ARM
On Mon, Jun 17, 2019 at 10:43:09AM +0200, Geert Uytterhoeven wrote:
> Hi Simon,
>
> On Mon, Jun 17, 2019 at 10:37 AM Simon Horman
> <horms+renesas@verge.net.au> wrote:
> > From: Takeshi Kihara <takeshi.kihara.df@renesas.com>
> >
> > It is necessary to reset the LVDS Interface according to display on/off.
>
> This is not the LVDS interface.
> The LVDS interface has its own device node.
Thanks, how about a changelog more like this?
arm64: dts: renesas: r8a77990: Add cpg reset for DU
Add CPG reset properties to DU node of E3 (r8a77990) SoC.
According to Laurent Pinchart, R-Car Gen3 reset is handled at the group
level so specifying one reset entry per group is sufficient. For this
reason <&cpg 724> is not listed as a reset for "du.1" as was the case in an
earlier revision of this patch.
Signed-off-by: Takeshi Kihara <takeshi.kihara.df@renesas.com>
Signed-off-by: Yoshihiro Kaneko <ykaneko0929@gmail.com>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> > Therefore, this patch adds CPG reset properties in DU device node
> > for the R8A77990 SoC.
> >
> > According to Laurent Pinchart, R-Car Gen3 reset is handled at the group
> > level so specifying one reset entry per group is sufficient. For this
> > reason <&cpg 724> is not listed as a reset for "du.1" as was the case in an
> > earlier revision of this patch.
> >
> > Signed-off-by: Takeshi Kihara <takeshi.kihara.df@renesas.com>
> > Signed-off-by: Yoshihiro Kaneko <ykaneko0929@gmail.com>
> > Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> > ---
> > v2 [Simon Horman]
> > - only add one reset entry per group
> >
> > v1 [Yoshihiro Kaneko]
> >
> > v0 [Takeshi Kihara]
> > ---
> > arch/arm64/boot/dts/renesas/r8a77990.dtsi | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/renesas/r8a77990.dtsi b/arch/arm64/boot/dts/renesas/r8a77990.dtsi
> > index b4318661f35e..84d1f58e73e7 100644
> > --- a/arch/arm64/boot/dts/renesas/r8a77990.dtsi
> > +++ b/arch/arm64/boot/dts/renesas/r8a77990.dtsi
> > @@ -1766,6 +1766,8 @@
> > clocks = <&cpg CPG_MOD 724>,
> > <&cpg CPG_MOD 723>;
> > clock-names = "du.0", "du.1";
> > + resets = <&cpg 724>;
> > + reset-names = "du.0";
> > vsps = <&vspd0 0 &vspd1 0>;
> > status = "disabled";
>
> 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
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 1/2] arm64: dts: renesas: r8a77990: Add cpg reset for LVDS Interface
2019-06-18 10:44 ` Simon Horman
@ 2019-06-18 11:19 ` Geert Uytterhoeven
0 siblings, 0 replies; 7+ messages in thread
From: Geert Uytterhoeven @ 2019-06-18 11:19 UTC (permalink / raw)
To: Simon Horman
Cc: Takeshi Kihara, Yoshihiro Kaneko, Magnus Damm, Linux-Renesas,
Laurent Pinchart, Linux ARM
Hi Simon,
On Tue, Jun 18, 2019 at 12:45 PM Simon Horman <horms@verge.net.au> wrote:
> On Mon, Jun 17, 2019 at 10:43:09AM +0200, Geert Uytterhoeven wrote:
> > On Mon, Jun 17, 2019 at 10:37 AM Simon Horman
> > <horms+renesas@verge.net.au> wrote:
> > > From: Takeshi Kihara <takeshi.kihara.df@renesas.com>
> > >
> > > It is necessary to reset the LVDS Interface according to display on/off.
> >
> > This is not the LVDS interface.
> > The LVDS interface has its own device node.
>
> Thanks, how about a changelog more like this?
>
> arm64: dts: renesas: r8a77990: Add cpg reset for DU
>
> Add CPG reset properties to DU node of E3 (r8a77990) SoC.
Thanks, much better.
> According to Laurent Pinchart, R-Car Gen3 reset is handled at the group
> level so specifying one reset entry per group is sufficient. For this
> reason <&cpg 724> is not listed as a reset for "du.1" as was the case in an
> earlier revision of this patch.
Do we need this last sentence?
Note that "dt-bindings: display: renesas: du: Document optional reset
properties"
hasn't been accepted in -next yet.
> Signed-off-by: Takeshi Kihara <takeshi.kihara.df@renesas.com>
> Signed-off-by: Yoshihiro Kaneko <ykaneko0929@gmail.com>
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
>
> > > Therefore, this patch adds CPG reset properties in DU device node
> > > for the R8A77990 SoC.
> > >
> > > According to Laurent Pinchart, R-Car Gen3 reset is handled at the group
> > > level so specifying one reset entry per group is sufficient. For this
> > > reason <&cpg 724> is not listed as a reset for "du.1" as was the case in an
> > > earlier revision of this patch.
> > >
> > > Signed-off-by: Takeshi Kihara <takeshi.kihara.df@renesas.com>
> > > Signed-off-by: Yoshihiro Kaneko <ykaneko0929@gmail.com>
> > > Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> > > ---
> > > v2 [Simon Horman]
> > > - only add one reset entry per group
> > > --- a/arch/arm64/boot/dts/renesas/r8a77990.dtsi
> > > +++ b/arch/arm64/boot/dts/renesas/r8a77990.dtsi
> > > @@ -1766,6 +1766,8 @@
> > > clocks = <&cpg CPG_MOD 724>,
> > > <&cpg CPG_MOD 723>;
> > > clock-names = "du.0", "du.1";
> > > + resets = <&cpg 724>;
> > > + reset-names = "du.0";
> > > vsps = <&vspd0 0 &vspd1 0>;
> > > status = "disabled";
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
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
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2019-06-18 11:19 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-06-17 8:37 [PATCH v2 0/2] arm64: dts: renesas: r8a7799[05]: Add cpg reset for LVDS Interface Simon Horman
2019-06-17 8:37 ` [PATCH v2 1/2] arm64: dts: renesas: r8a77990: " Simon Horman
2019-06-17 8:43 ` Geert Uytterhoeven
2019-06-18 10:44 ` Simon Horman
2019-06-18 11:19 ` Geert Uytterhoeven
2019-06-17 8:37 ` [PATCH v2 2/2] arm64: dts: renesas: r8a77995: " Simon Horman
2019-06-17 8:43 ` Geert Uytterhoeven
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).