* [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
* 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 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
* [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 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
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).