* [PATCH] clk: rockchip: Add pclk_peri to critical clocks on RK3066/RK3188
@ 2015-08-21 18:10 ` Romain Perier
0 siblings, 0 replies; 9+ messages in thread
From: Romain Perier @ 2015-08-21 18:10 UTC (permalink / raw)
To: heiko; +Cc: linux-rockchip, linux-clk, linux-arm-kernel, mturquette
Clocks for bank GPIO3 and GPIO4 are supplied by pclk_peri. These clocks need to
stay on, otherwise pinctrl hangs the system when the driver is probed.
Signed-off-by: Romain Perier <romain.perier@gmail.com>
---
drivers/clk/rockchip/clk-rk3188.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/clk/rockchip/clk-rk3188.c b/drivers/clk/rockchip/clk-rk3188.c
index f63a642..596be81 100644
--- a/drivers/clk/rockchip/clk-rk3188.c
+++ b/drivers/clk/rockchip/clk-rk3188.c
@@ -717,6 +717,7 @@ static const char *const rk3188_critical_clocks[] __initconst = {
"aclk_peri",
"hclk_peri",
"pclk_cpu",
+ "pclk_peri",
};
static void __init rk3188_common_clk_init(struct device_node *np)
--
2.1.4
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH] clk: rockchip: Add pclk_peri to critical clocks on RK3066/RK3188
@ 2015-08-21 18:10 ` Romain Perier
0 siblings, 0 replies; 9+ messages in thread
From: Romain Perier @ 2015-08-21 18:10 UTC (permalink / raw)
To: heiko-4mtYJXux2i+zQB+pC5nmwQ
Cc: linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
mturquette-rdvid1DuHRBWk0Htik3J/w,
linux-clk-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
Clocks for bank GPIO3 and GPIO4 are supplied by pclk_peri. These clocks need to
stay on, otherwise pinctrl hangs the system when the driver is probed.
Signed-off-by: Romain Perier <romain.perier-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
---
drivers/clk/rockchip/clk-rk3188.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/clk/rockchip/clk-rk3188.c b/drivers/clk/rockchip/clk-rk3188.c
index f63a642..596be81 100644
--- a/drivers/clk/rockchip/clk-rk3188.c
+++ b/drivers/clk/rockchip/clk-rk3188.c
@@ -717,6 +717,7 @@ static const char *const rk3188_critical_clocks[] __initconst = {
"aclk_peri",
"hclk_peri",
"pclk_cpu",
+ "pclk_peri",
};
static void __init rk3188_common_clk_init(struct device_node *np)
--
2.1.4
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH] clk: rockchip: Add pclk_peri to critical clocks on RK3066/RK3188
@ 2015-08-21 18:10 ` Romain Perier
0 siblings, 0 replies; 9+ messages in thread
From: Romain Perier @ 2015-08-21 18:10 UTC (permalink / raw)
To: linux-arm-kernel
Clocks for bank GPIO3 and GPIO4 are supplied by pclk_peri. These clocks need to
stay on, otherwise pinctrl hangs the system when the driver is probed.
Signed-off-by: Romain Perier <romain.perier@gmail.com>
---
drivers/clk/rockchip/clk-rk3188.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/clk/rockchip/clk-rk3188.c b/drivers/clk/rockchip/clk-rk3188.c
index f63a642..596be81 100644
--- a/drivers/clk/rockchip/clk-rk3188.c
+++ b/drivers/clk/rockchip/clk-rk3188.c
@@ -717,6 +717,7 @@ static const char *const rk3188_critical_clocks[] __initconst = {
"aclk_peri",
"hclk_peri",
"pclk_cpu",
+ "pclk_peri",
};
static void __init rk3188_common_clk_init(struct device_node *np)
--
2.1.4
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] clk: rockchip: Add pclk_peri to critical clocks on RK3066/RK3188
@ 2015-08-21 18:24 ` Heiko Stuebner
0 siblings, 0 replies; 9+ messages in thread
From: Heiko Stuebner @ 2015-08-21 18:24 UTC (permalink / raw)
To: Romain Perier; +Cc: linux-rockchip, linux-clk, linux-arm-kernel, mturquette
Hi Romain,
Am Freitag, 21. August 2015, 18:10:27 schrieb Romain Perier:
> Clocks for bank GPIO3 and GPIO4 are supplied by pclk_peri. These clocks need
> to stay on, otherwise pinctrl hangs the system when the driver is probed.
I don't agree with the commit message.
The issue you see does not originate from the pinctrl driver itself. Instead
only the new flexible clock handling causes some other (still unhandled)
component reyling on pclk_peri to hang the system.
In the past the clocks for the gpio banks were simply enabled once during
probe, so pclk_peri as parent clock would also stay enabled for eternity. Now
we enable the gpio clocks only when necessary, which results in that pclk_peri
might get disabled too when no other component has claimed it - hence its
addition to critical clocks.
Also, please note that Stephen Boyd is now also clock maintainer.
>
> Signed-off-by: Romain Perier <romain.perier@gmail.com>
> ---
> drivers/clk/rockchip/clk-rk3188.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/clk/rockchip/clk-rk3188.c
> b/drivers/clk/rockchip/clk-rk3188.c index f63a642..596be81 100644
> --- a/drivers/clk/rockchip/clk-rk3188.c
> +++ b/drivers/clk/rockchip/clk-rk3188.c
> @@ -717,6 +717,7 @@ static const char *const rk3188_critical_clocks[]
> __initconst = { "aclk_peri",
> "hclk_peri",
> "pclk_cpu",
> + "pclk_peri",
> };
>
> static void __init rk3188_common_clk_init(struct device_node *np)
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] clk: rockchip: Add pclk_peri to critical clocks on RK3066/RK3188
@ 2015-08-21 18:24 ` Heiko Stuebner
0 siblings, 0 replies; 9+ messages in thread
From: Heiko Stuebner @ 2015-08-21 18:24 UTC (permalink / raw)
To: Romain Perier
Cc: linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
mturquette-rdvid1DuHRBWk0Htik3J/w,
linux-clk-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
Hi Romain,
Am Freitag, 21. August 2015, 18:10:27 schrieb Romain Perier:
> Clocks for bank GPIO3 and GPIO4 are supplied by pclk_peri. These clocks need
> to stay on, otherwise pinctrl hangs the system when the driver is probed.
I don't agree with the commit message.
The issue you see does not originate from the pinctrl driver itself. Instead
only the new flexible clock handling causes some other (still unhandled)
component reyling on pclk_peri to hang the system.
In the past the clocks for the gpio banks were simply enabled once during
probe, so pclk_peri as parent clock would also stay enabled for eternity. Now
we enable the gpio clocks only when necessary, which results in that pclk_peri
might get disabled too when no other component has claimed it - hence its
addition to critical clocks.
Also, please note that Stephen Boyd is now also clock maintainer.
>
> Signed-off-by: Romain Perier <romain.perier-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> ---
> drivers/clk/rockchip/clk-rk3188.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/clk/rockchip/clk-rk3188.c
> b/drivers/clk/rockchip/clk-rk3188.c index f63a642..596be81 100644
> --- a/drivers/clk/rockchip/clk-rk3188.c
> +++ b/drivers/clk/rockchip/clk-rk3188.c
> @@ -717,6 +717,7 @@ static const char *const rk3188_critical_clocks[]
> __initconst = { "aclk_peri",
> "hclk_peri",
> "pclk_cpu",
> + "pclk_peri",
> };
>
> static void __init rk3188_common_clk_init(struct device_node *np)
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH] clk: rockchip: Add pclk_peri to critical clocks on RK3066/RK3188
@ 2015-08-21 18:24 ` Heiko Stuebner
0 siblings, 0 replies; 9+ messages in thread
From: Heiko Stuebner @ 2015-08-21 18:24 UTC (permalink / raw)
To: linux-arm-kernel
Hi Romain,
Am Freitag, 21. August 2015, 18:10:27 schrieb Romain Perier:
> Clocks for bank GPIO3 and GPIO4 are supplied by pclk_peri. These clocks need
> to stay on, otherwise pinctrl hangs the system when the driver is probed.
I don't agree with the commit message.
The issue you see does not originate from the pinctrl driver itself. Instead
only the new flexible clock handling causes some other (still unhandled)
component reyling on pclk_peri to hang the system.
In the past the clocks for the gpio banks were simply enabled once during
probe, so pclk_peri as parent clock would also stay enabled for eternity. Now
we enable the gpio clocks only when necessary, which results in that pclk_peri
might get disabled too when no other component has claimed it - hence its
addition to critical clocks.
Also, please note that Stephen Boyd is now also clock maintainer.
>
> Signed-off-by: Romain Perier <romain.perier@gmail.com>
> ---
> drivers/clk/rockchip/clk-rk3188.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/clk/rockchip/clk-rk3188.c
> b/drivers/clk/rockchip/clk-rk3188.c index f63a642..596be81 100644
> --- a/drivers/clk/rockchip/clk-rk3188.c
> +++ b/drivers/clk/rockchip/clk-rk3188.c
> @@ -717,6 +717,7 @@ static const char *const rk3188_critical_clocks[]
> __initconst = { "aclk_peri",
> "hclk_peri",
> "pclk_cpu",
> + "pclk_peri",
> };
>
> static void __init rk3188_common_clk_init(struct device_node *np)
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] clk: rockchip: Add pclk_peri to critical clocks on RK3066/RK3188
@ 2015-08-22 15:05 ` Michael Niewoehner
0 siblings, 0 replies; 9+ messages in thread
From: Michael Niewoehner @ 2015-08-22 15:05 UTC (permalink / raw)
To: Romain Perier
Cc: Heiko Stuebner, linux-rockchip, mturquette, linux-clk, linux-arm-kernel
Am 21.08.2015 um 20:10 schrieb Romain Perier <romain.perier@gmail.com>:
> Clocks for bank GPIO3 and GPIO4 are supplied by pclk_peri. These =
clocks need to
> stay on, otherwise pinctrl hangs the system when the driver is probed.
>=20
> Signed-off-by: Romain Perier <romain.perier@gmail.com>
Tested-by: Michael Niewoehner <linux@mniewoehner.de>
> ---
> drivers/clk/rockchip/clk-rk3188.c | 1 +
> 1 file changed, 1 insertion(+)
>=20
> diff --git a/drivers/clk/rockchip/clk-rk3188.c =
b/drivers/clk/rockchip/clk-rk3188.c
> index f63a642..596be81 100644
> --- a/drivers/clk/rockchip/clk-rk3188.c
> +++ b/drivers/clk/rockchip/clk-rk3188.c
> @@ -717,6 +717,7 @@ static const char *const rk3188_critical_clocks[] =
__initconst =3D {
> "aclk_peri",
> "hclk_peri",
> "pclk_cpu",
> + "pclk_peri",
> };
>=20
> static void __init rk3188_common_clk_init(struct device_node *np)
> --=20
> 2.1.4
>=20
>=20
> _______________________________________________
> 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] 9+ messages in thread
* Re: [PATCH] clk: rockchip: Add pclk_peri to critical clocks on RK3066/RK3188
@ 2015-08-22 15:05 ` Michael Niewoehner
0 siblings, 0 replies; 9+ messages in thread
From: Michael Niewoehner @ 2015-08-22 15:05 UTC (permalink / raw)
To: Romain Perier
Cc: linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
mturquette-rdvid1DuHRBWk0Htik3J/w, Heiko Stuebner,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-clk-u79uwXL29TY76Z2rM5mHXA
Am 21.08.2015 um 20:10 schrieb Romain Perier <romain.perier-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>:
> Clocks for bank GPIO3 and GPIO4 are supplied by pclk_peri. These clocks need to
> stay on, otherwise pinctrl hangs the system when the driver is probed.
>
> Signed-off-by: Romain Perier <romain.perier-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Tested-by: Michael Niewoehner <linux-zzFNMPX9jIaDjmgdnaGrkw@public.gmane.org>
> ---
> drivers/clk/rockchip/clk-rk3188.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/clk/rockchip/clk-rk3188.c b/drivers/clk/rockchip/clk-rk3188.c
> index f63a642..596be81 100644
> --- a/drivers/clk/rockchip/clk-rk3188.c
> +++ b/drivers/clk/rockchip/clk-rk3188.c
> @@ -717,6 +717,7 @@ static const char *const rk3188_critical_clocks[] __initconst = {
> "aclk_peri",
> "hclk_peri",
> "pclk_cpu",
> + "pclk_peri",
> };
>
> static void __init rk3188_common_clk_init(struct device_node *np)
> --
> 2.1.4
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH] clk: rockchip: Add pclk_peri to critical clocks on RK3066/RK3188
@ 2015-08-22 15:05 ` Michael Niewoehner
0 siblings, 0 replies; 9+ messages in thread
From: Michael Niewoehner @ 2015-08-22 15:05 UTC (permalink / raw)
To: linux-arm-kernel
Am 21.08.2015 um 20:10 schrieb Romain Perier <romain.perier@gmail.com>:
> Clocks for bank GPIO3 and GPIO4 are supplied by pclk_peri. These clocks need to
> stay on, otherwise pinctrl hangs the system when the driver is probed.
>
> Signed-off-by: Romain Perier <romain.perier@gmail.com>
Tested-by: Michael Niewoehner <linux@mniewoehner.de>
> ---
> drivers/clk/rockchip/clk-rk3188.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/clk/rockchip/clk-rk3188.c b/drivers/clk/rockchip/clk-rk3188.c
> index f63a642..596be81 100644
> --- a/drivers/clk/rockchip/clk-rk3188.c
> +++ b/drivers/clk/rockchip/clk-rk3188.c
> @@ -717,6 +717,7 @@ static const char *const rk3188_critical_clocks[] __initconst = {
> "aclk_peri",
> "hclk_peri",
> "pclk_cpu",
> + "pclk_peri",
> };
>
> static void __init rk3188_common_clk_init(struct device_node *np)
> --
> 2.1.4
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2015-08-22 15:05 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-08-21 18:10 [PATCH] clk: rockchip: Add pclk_peri to critical clocks on RK3066/RK3188 Romain Perier
2015-08-21 18:10 ` Romain Perier
2015-08-21 18:10 ` Romain Perier
2015-08-21 18:24 ` Heiko Stuebner
2015-08-21 18:24 ` Heiko Stuebner
2015-08-21 18:24 ` Heiko Stuebner
2015-08-22 15:05 ` Michael Niewoehner
2015-08-22 15:05 ` Michael Niewoehner
2015-08-22 15:05 ` Michael Niewoehner
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.