* [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks
@ 2019-03-13 16:35 ` Gregory CLEMENT
0 siblings, 0 replies; 30+ messages in thread
From: Gregory CLEMENT @ 2019-03-13 16:35 UTC (permalink / raw)
To: Stephen Boyd, Mike Turquette, linux-clk, linux-kernel
Cc: Rafael J. Wysocki, Viresh Kumar, linux-pm, Christian Neubert,
Ilias Apalodimas, Vincent Guittot, Jason Cooper, Andrew Lunn,
Sebastian Hesselbarth, Gregory CLEMENT, Thomas Petazzoni,
linux-arm-kernel, Antoine Tenart, Miquèl Raynal,
Maxime Chevallier, stable
The clock parenting was not setup properly when DVFS was enabled. It was
expected that the same clock source was used with and without DVFS which
was not the case.
This patch fixes this issue, allowing to make the cpufreq support work
when the CPU clocks source are not the default ones.
Fixes: 92ce45fb875d ("cpufreq: Add DVFS support for Armada 37xx")
Cc: <stable@vger.kernel.org>
Reported-by: Christian Neubert <christian.neubert.86@gmail.com>
Reported-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>
Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com>
---
drivers/clk/mvebu/armada-37xx-periph.c | 11 +++++++++++
1 file changed, 11 insertions(+)
diff --git a/drivers/clk/mvebu/armada-37xx-periph.c b/drivers/clk/mvebu/armada-37xx-periph.c
index 1f1cff428d78..26ed3c18a239 100644
--- a/drivers/clk/mvebu/armada-37xx-periph.c
+++ b/drivers/clk/mvebu/armada-37xx-periph.c
@@ -671,6 +671,17 @@ static int armada_3700_add_composite_clk(const struct clk_periph_data *data,
map = syscon_regmap_lookup_by_compatible(
"marvell,armada-3700-nb-pm");
pmcpu_clk->nb_pm_base = map;
+
+ /*
+ * Use the same parent when DVFS is enabled that the
+ * default parent received at boot time. When this
+ * function is called, DVFS is not enabled yet, so we
+ * get the default parent and we can set the parent
+ * for DVFS.
+ */
+ if (clk_pm_cpu_set_parent(muxrate_hw,
+ clk_pm_cpu_get_parent(muxrate_hw)))
+ dev_warn(dev, "Failed to setup default parent clock for DVFS\n");
}
*hw = clk_hw_register_composite(dev, data->name, data->parent_names,
--
2.20.1
^ permalink raw reply related [flat|nested] 30+ messages in thread* [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks @ 2019-03-13 16:35 ` Gregory CLEMENT 0 siblings, 0 replies; 30+ messages in thread From: Gregory CLEMENT @ 2019-03-13 16:35 UTC (permalink / raw) To: Stephen Boyd, Mike Turquette, linux-clk, linux-kernel Cc: Andrew Lunn, Vincent Guittot, Jason Cooper, Christian Neubert, Antoine Tenart, Viresh Kumar, linux-pm, Ilias Apalodimas, Rafael J. Wysocki, stable, Maxime Chevallier, Thomas Petazzoni, Miquèl Raynal, Gregory CLEMENT, linux-arm-kernel, Sebastian Hesselbarth The clock parenting was not setup properly when DVFS was enabled. It was expected that the same clock source was used with and without DVFS which was not the case. This patch fixes this issue, allowing to make the cpufreq support work when the CPU clocks source are not the default ones. Fixes: 92ce45fb875d ("cpufreq: Add DVFS support for Armada 37xx") Cc: <stable@vger.kernel.org> Reported-by: Christian Neubert <christian.neubert.86@gmail.com> Reported-by: Ilias Apalodimas <ilias.apalodimas@linaro.org> Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com> --- drivers/clk/mvebu/armada-37xx-periph.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/drivers/clk/mvebu/armada-37xx-periph.c b/drivers/clk/mvebu/armada-37xx-periph.c index 1f1cff428d78..26ed3c18a239 100644 --- a/drivers/clk/mvebu/armada-37xx-periph.c +++ b/drivers/clk/mvebu/armada-37xx-periph.c @@ -671,6 +671,17 @@ static int armada_3700_add_composite_clk(const struct clk_periph_data *data, map = syscon_regmap_lookup_by_compatible( "marvell,armada-3700-nb-pm"); pmcpu_clk->nb_pm_base = map; + + /* + * Use the same parent when DVFS is enabled that the + * default parent received at boot time. When this + * function is called, DVFS is not enabled yet, so we + * get the default parent and we can set the parent + * for DVFS. + */ + if (clk_pm_cpu_set_parent(muxrate_hw, + clk_pm_cpu_get_parent(muxrate_hw))) + dev_warn(dev, "Failed to setup default parent clock for DVFS\n"); } *hw = clk_hw_register_composite(dev, data->name, data->parent_names, -- 2.20.1 _______________________________________________ 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] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks 2019-03-13 16:35 ` Gregory CLEMENT @ 2019-03-14 12:15 ` Ilias Apalodimas -1 siblings, 0 replies; 30+ messages in thread From: Ilias Apalodimas @ 2019-03-14 12:15 UTC (permalink / raw) To: Gregory CLEMENT Cc: Stephen Boyd, Mike Turquette, linux-clk, linux-kernel, Rafael J. Wysocki, Viresh Kumar, linux-pm, Christian Neubert, Vincent Guittot, Jason Cooper, Andrew Lunn, Sebastian Hesselbarth, Thomas Petazzoni, linux-arm-kernel, Antoine Tenart, Miquèl Raynal, Maxime Chevallier, stable Hi Gregory, > The clock parenting was not setup properly when DVFS was enabled. It was > expected that the same clock source was used with and without DVFS which > was not the case. > > This patch fixes this issue, allowing to make the cpufreq support work > when the CPU clocks source are not the default ones. > > Fixes: 92ce45fb875d ("cpufreq: Add DVFS support for Armada 37xx") > Cc: <stable@vger.kernel.org> > Reported-by: Christian Neubert <christian.neubert.86@gmail.com> > Reported-by: Ilias Apalodimas <ilias.apalodimas@linaro.org> > Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com> > --- > drivers/clk/mvebu/armada-37xx-periph.c | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/drivers/clk/mvebu/armada-37xx-periph.c b/drivers/clk/mvebu/armada-37xx-periph.c > index 1f1cff428d78..26ed3c18a239 100644 > --- a/drivers/clk/mvebu/armada-37xx-periph.c > +++ b/drivers/clk/mvebu/armada-37xx-periph.c > @@ -671,6 +671,17 @@ static int armada_3700_add_composite_clk(const struct clk_periph_data *data, > map = syscon_regmap_lookup_by_compatible( > "marvell,armada-3700-nb-pm"); > pmcpu_clk->nb_pm_base = map; > + > + /* > + * Use the same parent when DVFS is enabled that the > + * default parent received at boot time. When this > + * function is called, DVFS is not enabled yet, so we > + * get the default parent and we can set the parent > + * for DVFS. > + */ > + if (clk_pm_cpu_set_parent(muxrate_hw, > + clk_pm_cpu_get_parent(muxrate_hw))) > + dev_warn(dev, "Failed to setup default parent clock for DVFS\n"); > } > > *hw = clk_hw_register_composite(dev, data->name, data->parent_names, > -- > 2.20.1 > Applied this and selected only CONFIG_CPU_FREQ_DEFAULT_GOV_PERFORMANCE=y CONFIG_CPU_FREQ_GOV_PERFORMANCE=y CONFIG_CPU_FREQ_GOV_POWERSAVE=y After changing the governor from 'powersave' to 'performance' the board completely froze (i even lost access to the serial port) Cheers /Ilias ^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks @ 2019-03-14 12:15 ` Ilias Apalodimas 0 siblings, 0 replies; 30+ messages in thread From: Ilias Apalodimas @ 2019-03-14 12:15 UTC (permalink / raw) To: Gregory CLEMENT Cc: Andrew Lunn, Vincent Guittot, Jason Cooper, linux-pm, Antoine Tenart, Viresh Kumar, Mike Turquette, Christian Neubert, Rafael J. Wysocki, linux-kernel, stable, linux-clk, Thomas Petazzoni, Miquèl Raynal, Maxime Chevallier, Stephen Boyd, linux-arm-kernel, Sebastian Hesselbarth Hi Gregory, > The clock parenting was not setup properly when DVFS was enabled. It was > expected that the same clock source was used with and without DVFS which > was not the case. > > This patch fixes this issue, allowing to make the cpufreq support work > when the CPU clocks source are not the default ones. > > Fixes: 92ce45fb875d ("cpufreq: Add DVFS support for Armada 37xx") > Cc: <stable@vger.kernel.org> > Reported-by: Christian Neubert <christian.neubert.86@gmail.com> > Reported-by: Ilias Apalodimas <ilias.apalodimas@linaro.org> > Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com> > --- > drivers/clk/mvebu/armada-37xx-periph.c | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/drivers/clk/mvebu/armada-37xx-periph.c b/drivers/clk/mvebu/armada-37xx-periph.c > index 1f1cff428d78..26ed3c18a239 100644 > --- a/drivers/clk/mvebu/armada-37xx-periph.c > +++ b/drivers/clk/mvebu/armada-37xx-periph.c > @@ -671,6 +671,17 @@ static int armada_3700_add_composite_clk(const struct clk_periph_data *data, > map = syscon_regmap_lookup_by_compatible( > "marvell,armada-3700-nb-pm"); > pmcpu_clk->nb_pm_base = map; > + > + /* > + * Use the same parent when DVFS is enabled that the > + * default parent received at boot time. When this > + * function is called, DVFS is not enabled yet, so we > + * get the default parent and we can set the parent > + * for DVFS. > + */ > + if (clk_pm_cpu_set_parent(muxrate_hw, > + clk_pm_cpu_get_parent(muxrate_hw))) > + dev_warn(dev, "Failed to setup default parent clock for DVFS\n"); > } > > *hw = clk_hw_register_composite(dev, data->name, data->parent_names, > -- > 2.20.1 > Applied this and selected only CONFIG_CPU_FREQ_DEFAULT_GOV_PERFORMANCE=y CONFIG_CPU_FREQ_GOV_PERFORMANCE=y CONFIG_CPU_FREQ_GOV_POWERSAVE=y After changing the governor from 'powersave' to 'performance' the board completely froze (i even lost access to the serial port) Cheers /Ilias _______________________________________________ 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] 30+ messages in thread
[parent not found: <CAC5LXJcCs4nr-qFOWzUJpUBAJ9ngG-cgeTCVCFBKFc1SPzHMuQ@mail.gmail.com>]
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks [not found] ` <CAC5LXJcCs4nr-qFOWzUJpUBAJ9ngG-cgeTCVCFBKFc1SPzHMuQ@mail.gmail.com> @ 2019-03-14 13:44 ` Ilias Apalodimas 0 siblings, 0 replies; 30+ messages in thread From: Ilias Apalodimas @ 2019-03-14 13:44 UTC (permalink / raw) To: Christian Neubert Cc: Gregory CLEMENT, Stephen Boyd, Mike Turquette, linux-clk, linux-kernel, Rafael J. Wysocki, Viresh Kumar, linux-pm, Vincent Guittot, Jason Cooper, Andrew Lunn, Sebastian Hesselbarth, Thomas Petazzoni, linux-arm-kernel, Antoine Tenart, Miquèl Raynal, Maxime Chevallier, stable Hello Christian, > Hi, > > I assume you use the 1000 MHz firmware. This does also not work on my Rev 7 > board. But I'm pretty sure this is not a problem of the patches, because if > I take a newer kernel (4.19.20/27) without the patches it also does not > work. A kernel 4.19.17 does work for me. My opinion on that is that this is > another problem which does just occure now because now the cpu frequency > scaling is working with the right frequencies. I am not sure which firmware i am running, i did all my tests on 5.0.0 and changing between governors worked fine without the patches Regards /Ilias > > Ilias Apalodimas <ilias.apalodimas@linaro.org> schrieb am Do., 14. März > 2019, 13:15: > > > Hi Gregory, > > > The clock parenting was not setup properly when DVFS was enabled. It was > > > expected that the same clock source was used with and without DVFS which > > > was not the case. > > > > > > This patch fixes this issue, allowing to make the cpufreq support work > > > when the CPU clocks source are not the default ones. > > > > > > Fixes: 92ce45fb875d ("cpufreq: Add DVFS support for Armada 37xx") > > > Cc: <stable@vger.kernel.org> > > > Reported-by: Christian Neubert <christian.neubert.86@gmail.com> > > > Reported-by: Ilias Apalodimas <ilias.apalodimas@linaro.org> > > > Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com> > > > --- > > > drivers/clk/mvebu/armada-37xx-periph.c | 11 +++++++++++ > > > 1 file changed, 11 insertions(+) > > > > > > diff --git a/drivers/clk/mvebu/armada-37xx-periph.c > > b/drivers/clk/mvebu/armada-37xx-periph.c > > > index 1f1cff428d78..26ed3c18a239 100644 > > > --- a/drivers/clk/mvebu/armada-37xx-periph.c > > > +++ b/drivers/clk/mvebu/armada-37xx-periph.c > > > @@ -671,6 +671,17 @@ static int armada_3700_add_composite_clk(const > > struct clk_periph_data *data, > > > map = syscon_regmap_lookup_by_compatible( > > > "marvell,armada-3700-nb-pm"); > > > pmcpu_clk->nb_pm_base = map; > > > + > > > + /* > > > + * Use the same parent when DVFS is enabled that the > > > + * default parent received at boot time. When this > > > + * function is called, DVFS is not enabled yet, so we > > > + * get the default parent and we can set the parent > > > + * for DVFS. > > > + */ > > > + if (clk_pm_cpu_set_parent(muxrate_hw, > > > + > > clk_pm_cpu_get_parent(muxrate_hw))) > > > + dev_warn(dev, "Failed to setup default parent > > clock for DVFS\n"); > > > } > > > > > > *hw = clk_hw_register_composite(dev, data->name, > > data->parent_names, > > > -- > > > 2.20.1 > > > > > Applied this and selected only > > > > CONFIG_CPU_FREQ_DEFAULT_GOV_PERFORMANCE=y > > CONFIG_CPU_FREQ_GOV_PERFORMANCE=y > > CONFIG_CPU_FREQ_GOV_POWERSAVE=y > > > > After changing the governor from 'powersave' to 'performance' the board > > completely froze (i even lost access to the serial port) > > > > Cheers > > /Ilias > > ^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks @ 2019-03-14 13:44 ` Ilias Apalodimas 0 siblings, 0 replies; 30+ messages in thread From: Ilias Apalodimas @ 2019-03-14 13:44 UTC (permalink / raw) To: Christian Neubert Cc: Andrew Lunn, Vincent Guittot, Jason Cooper, linux-pm, Antoine Tenart, Stephen Boyd, Gregory CLEMENT, Miquèl Raynal, Rafael J. Wysocki, linux-kernel, stable, linux-clk, Thomas Petazzoni, Viresh Kumar, Maxime Chevallier, Mike Turquette, linux-arm-kernel, Sebastian Hesselbarth Hello Christian, > Hi, > > I assume you use the 1000 MHz firmware. This does also not work on my Rev 7 > board. But I'm pretty sure this is not a problem of the patches, because if > I take a newer kernel (4.19.20/27) without the patches it also does not > work. A kernel 4.19.17 does work for me. My opinion on that is that this is > another problem which does just occure now because now the cpu frequency > scaling is working with the right frequencies. I am not sure which firmware i am running, i did all my tests on 5.0.0 and changing between governors worked fine without the patches Regards /Ilias > > Ilias Apalodimas <ilias.apalodimas@linaro.org> schrieb am Do., 14. März > 2019, 13:15: > > > Hi Gregory, > > > The clock parenting was not setup properly when DVFS was enabled. It was > > > expected that the same clock source was used with and without DVFS which > > > was not the case. > > > > > > This patch fixes this issue, allowing to make the cpufreq support work > > > when the CPU clocks source are not the default ones. > > > > > > Fixes: 92ce45fb875d ("cpufreq: Add DVFS support for Armada 37xx") > > > Cc: <stable@vger.kernel.org> > > > Reported-by: Christian Neubert <christian.neubert.86@gmail.com> > > > Reported-by: Ilias Apalodimas <ilias.apalodimas@linaro.org> > > > Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com> > > > --- > > > drivers/clk/mvebu/armada-37xx-periph.c | 11 +++++++++++ > > > 1 file changed, 11 insertions(+) > > > > > > diff --git a/drivers/clk/mvebu/armada-37xx-periph.c > > b/drivers/clk/mvebu/armada-37xx-periph.c > > > index 1f1cff428d78..26ed3c18a239 100644 > > > --- a/drivers/clk/mvebu/armada-37xx-periph.c > > > +++ b/drivers/clk/mvebu/armada-37xx-periph.c > > > @@ -671,6 +671,17 @@ static int armada_3700_add_composite_clk(const > > struct clk_periph_data *data, > > > map = syscon_regmap_lookup_by_compatible( > > > "marvell,armada-3700-nb-pm"); > > > pmcpu_clk->nb_pm_base = map; > > > + > > > + /* > > > + * Use the same parent when DVFS is enabled that the > > > + * default parent received at boot time. When this > > > + * function is called, DVFS is not enabled yet, so we > > > + * get the default parent and we can set the parent > > > + * for DVFS. > > > + */ > > > + if (clk_pm_cpu_set_parent(muxrate_hw, > > > + > > clk_pm_cpu_get_parent(muxrate_hw))) > > > + dev_warn(dev, "Failed to setup default parent > > clock for DVFS\n"); > > > } > > > > > > *hw = clk_hw_register_composite(dev, data->name, > > data->parent_names, > > > -- > > > 2.20.1 > > > > > Applied this and selected only > > > > CONFIG_CPU_FREQ_DEFAULT_GOV_PERFORMANCE=y > > CONFIG_CPU_FREQ_GOV_PERFORMANCE=y > > CONFIG_CPU_FREQ_GOV_POWERSAVE=y > > > > After changing the governor from 'powersave' to 'performance' the board > > completely froze (i even lost access to the serial port) > > > > Cheers > > /Ilias > > _______________________________________________ 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] 30+ messages in thread
[parent not found: <CAC5LXJf=f2bZLroSXFkGiF2W=1XzAgb_wwtpvZY7O9xv28wbVA@mail.gmail.com>]
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks [not found] ` <CAC5LXJf=f2bZLroSXFkGiF2W=1XzAgb_wwtpvZY7O9xv28wbVA@mail.gmail.com> @ 2019-03-14 14:15 ` Ilias Apalodimas 0 siblings, 0 replies; 30+ messages in thread From: Ilias Apalodimas @ 2019-03-14 14:15 UTC (permalink / raw) To: Christian Neubert Cc: Gregory CLEMENT, Stephen Boyd, Mike Turquette, linux-clk, linux-kernel, Rafael J. Wysocki, Viresh Kumar, linux-pm, Vincent Guittot, Jason Cooper, Andrew Lunn, Sebastian Hesselbarth, Thomas Petazzoni, linux-arm-kernel, Antoine Tenart, Miquèl Raynal, Maxime Chevallier, stable Hi, > Hi, > > the cpu frequency scaling never worked right (only on the 4.4 kernel from > marvell). If you use the 1000 MHz firmware you are running with just 800 > MHz (this is the case on my board with a current firmware). > > Just have a look what the kernel thinks it is running at (frequency). Ok, probably my bad here. By 'worked fine' i mean that this didn't lead to any freezes or panics. I know the actual frequency wasn't set properly Regards /Ilias > > Regards, > Christian > > Ilias Apalodimas <ilias.apalodimas@linaro.org> schrieb am Do., 14. März > 2019, 14:44: > > > Hello Christian, > > > Hi, > > > > > > I assume you use the 1000 MHz firmware. This does also not work on my > > Rev 7 > > > board. But I'm pretty sure this is not a problem of the patches, because > > if > > > I take a newer kernel (4.19.20/27) without the patches it also does not > > > work. A kernel 4.19.17 does work for me. My opinion on that is that this > > is > > > another problem which does just occure now because now the cpu frequency > > > scaling is working with the right frequencies. > > I am not sure which firmware i am running, i did all my tests on 5.0.0 and > > changing between governors worked fine without the patches > > > > Regards > > /Ilias > > > > > > Ilias Apalodimas <ilias.apalodimas@linaro.org> schrieb am Do., 14. März > > > 2019, 13:15: > > > > > > > Hi Gregory, > > > > > The clock parenting was not setup properly when DVFS was enabled. It > > was > > > > > expected that the same clock source was used with and without DVFS > > which > > > > > was not the case. > > > > > > > > > > This patch fixes this issue, allowing to make the cpufreq support > > work > > > > > when the CPU clocks source are not the default ones. > > > > > > > > > > Fixes: 92ce45fb875d ("cpufreq: Add DVFS support for Armada 37xx") > > > > > Cc: <stable@vger.kernel.org> > > > > > Reported-by: Christian Neubert <christian.neubert.86@gmail.com> > > > > > Reported-by: Ilias Apalodimas <ilias.apalodimas@linaro.org> > > > > > Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com> > > > > > --- > > > > > drivers/clk/mvebu/armada-37xx-periph.c | 11 +++++++++++ > > > > > 1 file changed, 11 insertions(+) > > > > > > > > > > diff --git a/drivers/clk/mvebu/armada-37xx-periph.c > > > > b/drivers/clk/mvebu/armada-37xx-periph.c > > > > > index 1f1cff428d78..26ed3c18a239 100644 > > > > > --- a/drivers/clk/mvebu/armada-37xx-periph.c > > > > > +++ b/drivers/clk/mvebu/armada-37xx-periph.c > > > > > @@ -671,6 +671,17 @@ static int armada_3700_add_composite_clk(const > > > > struct clk_periph_data *data, > > > > > map = syscon_regmap_lookup_by_compatible( > > > > > "marvell,armada-3700-nb-pm"); > > > > > pmcpu_clk->nb_pm_base = map; > > > > > + > > > > > + /* > > > > > + * Use the same parent when DVFS is enabled that the > > > > > + * default parent received at boot time. When this > > > > > + * function is called, DVFS is not enabled yet, so we > > > > > + * get the default parent and we can set the parent > > > > > + * for DVFS. > > > > > + */ > > > > > + if (clk_pm_cpu_set_parent(muxrate_hw, > > > > > + > > > > clk_pm_cpu_get_parent(muxrate_hw))) > > > > > + dev_warn(dev, "Failed to setup default parent > > > > clock for DVFS\n"); > > > > > } > > > > > > > > > > *hw = clk_hw_register_composite(dev, data->name, > > > > data->parent_names, > > > > > -- > > > > > 2.20.1 > > > > > > > > > Applied this and selected only > > > > > > > > CONFIG_CPU_FREQ_DEFAULT_GOV_PERFORMANCE=y > > > > CONFIG_CPU_FREQ_GOV_PERFORMANCE=y > > > > CONFIG_CPU_FREQ_GOV_POWERSAVE=y > > > > > > > > After changing the governor from 'powersave' to 'performance' the board > > > > completely froze (i even lost access to the serial port) > > > > > > > > Cheers > > > > /Ilias > > > > > > ^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks @ 2019-03-14 14:15 ` Ilias Apalodimas 0 siblings, 0 replies; 30+ messages in thread From: Ilias Apalodimas @ 2019-03-14 14:15 UTC (permalink / raw) To: Christian Neubert Cc: Andrew Lunn, Vincent Guittot, Jason Cooper, linux-pm, Antoine Tenart, Stephen Boyd, Gregory CLEMENT, Miquèl Raynal, Rafael J. Wysocki, linux-kernel, stable, linux-clk, Thomas Petazzoni, Viresh Kumar, Maxime Chevallier, Mike Turquette, linux-arm-kernel, Sebastian Hesselbarth Hi, > Hi, > > the cpu frequency scaling never worked right (only on the 4.4 kernel from > marvell). If you use the 1000 MHz firmware you are running with just 800 > MHz (this is the case on my board with a current firmware). > > Just have a look what the kernel thinks it is running at (frequency). Ok, probably my bad here. By 'worked fine' i mean that this didn't lead to any freezes or panics. I know the actual frequency wasn't set properly Regards /Ilias > > Regards, > Christian > > Ilias Apalodimas <ilias.apalodimas@linaro.org> schrieb am Do., 14. März > 2019, 14:44: > > > Hello Christian, > > > Hi, > > > > > > I assume you use the 1000 MHz firmware. This does also not work on my > > Rev 7 > > > board. But I'm pretty sure this is not a problem of the patches, because > > if > > > I take a newer kernel (4.19.20/27) without the patches it also does not > > > work. A kernel 4.19.17 does work for me. My opinion on that is that this > > is > > > another problem which does just occure now because now the cpu frequency > > > scaling is working with the right frequencies. > > I am not sure which firmware i am running, i did all my tests on 5.0.0 and > > changing between governors worked fine without the patches > > > > Regards > > /Ilias > > > > > > Ilias Apalodimas <ilias.apalodimas@linaro.org> schrieb am Do., 14. März > > > 2019, 13:15: > > > > > > > Hi Gregory, > > > > > The clock parenting was not setup properly when DVFS was enabled. It > > was > > > > > expected that the same clock source was used with and without DVFS > > which > > > > > was not the case. > > > > > > > > > > This patch fixes this issue, allowing to make the cpufreq support > > work > > > > > when the CPU clocks source are not the default ones. > > > > > > > > > > Fixes: 92ce45fb875d ("cpufreq: Add DVFS support for Armada 37xx") > > > > > Cc: <stable@vger.kernel.org> > > > > > Reported-by: Christian Neubert <christian.neubert.86@gmail.com> > > > > > Reported-by: Ilias Apalodimas <ilias.apalodimas@linaro.org> > > > > > Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com> > > > > > --- > > > > > drivers/clk/mvebu/armada-37xx-periph.c | 11 +++++++++++ > > > > > 1 file changed, 11 insertions(+) > > > > > > > > > > diff --git a/drivers/clk/mvebu/armada-37xx-periph.c > > > > b/drivers/clk/mvebu/armada-37xx-periph.c > > > > > index 1f1cff428d78..26ed3c18a239 100644 > > > > > --- a/drivers/clk/mvebu/armada-37xx-periph.c > > > > > +++ b/drivers/clk/mvebu/armada-37xx-periph.c > > > > > @@ -671,6 +671,17 @@ static int armada_3700_add_composite_clk(const > > > > struct clk_periph_data *data, > > > > > map = syscon_regmap_lookup_by_compatible( > > > > > "marvell,armada-3700-nb-pm"); > > > > > pmcpu_clk->nb_pm_base = map; > > > > > + > > > > > + /* > > > > > + * Use the same parent when DVFS is enabled that the > > > > > + * default parent received at boot time. When this > > > > > + * function is called, DVFS is not enabled yet, so we > > > > > + * get the default parent and we can set the parent > > > > > + * for DVFS. > > > > > + */ > > > > > + if (clk_pm_cpu_set_parent(muxrate_hw, > > > > > + > > > > clk_pm_cpu_get_parent(muxrate_hw))) > > > > > + dev_warn(dev, "Failed to setup default parent > > > > clock for DVFS\n"); > > > > > } > > > > > > > > > > *hw = clk_hw_register_composite(dev, data->name, > > > > data->parent_names, > > > > > -- > > > > > 2.20.1 > > > > > > > > > Applied this and selected only > > > > > > > > CONFIG_CPU_FREQ_DEFAULT_GOV_PERFORMANCE=y > > > > CONFIG_CPU_FREQ_GOV_PERFORMANCE=y > > > > CONFIG_CPU_FREQ_GOV_POWERSAVE=y > > > > > > > > After changing the governor from 'powersave' to 'performance' the board > > > > completely froze (i even lost access to the serial port) > > > > > > > > Cheers > > > > /Ilias > > > > > > _______________________________________________ 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] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks 2019-03-14 13:44 ` Ilias Apalodimas @ 2019-03-14 14:20 ` Gregory CLEMENT -1 siblings, 0 replies; 30+ messages in thread From: Gregory CLEMENT @ 2019-03-14 14:20 UTC (permalink / raw) To: Ilias Apalodimas, Christian Neubert Cc: Stephen Boyd, Mike Turquette, linux-clk, linux-kernel, Rafael J. Wysocki, Viresh Kumar, linux-pm, Vincent Guittot, Jason Cooper, Andrew Lunn, Sebastian Hesselbarth, Thomas Petazzoni, linux-arm-kernel, Antoine Tenart, Miquèl Raynal, Maxime Chevallier, stable Hi Ilias, On jeu., mars 14 2019, Ilias Apalodimas <ilias.apalodimas@linaro.org> wrote: > Hello Christian, >> Hi, >> >> I assume you use the 1000 MHz firmware. This does also not work on my Rev 7 >> board. But I'm pretty sure this is not a problem of the patches, because if >> I take a newer kernel (4.19.20/27) without the patches it also does not >> work. A kernel 4.19.17 does work for me. My opinion on that is that this is >> another problem which does just occure now because now the cpu frequency >> scaling is working with the right frequencies. > I am not sure which firmware i am running, i did all my tests on 5.0.0 and > changing between governors worked fine without the patches Curently my espressobin is broken so I tested the patches on the Armada 3700 DB and I didn't observe the issue you had. The 3700 DB I used is configured to run at 800MHz. Could you apply the following patch and sent me the boot log? diff --git a/drivers/clk/mvebu/armada-37xx-periph.c b/drivers/clk/mvebu/armada-37xx-periph.c index 26ed3c18a239..f814ade5cd80 100644 --- a/drivers/clk/mvebu/armada-37xx-periph.c +++ b/drivers/clk/mvebu/armada-37xx-periph.c @@ -452,14 +452,17 @@ static int clk_pm_cpu_set_parent(struct clk_hw *hw, u8 index) /* Set the parent clock for all the load level */ for (load_level = 0; load_level < LOAD_LEVEL_NR; load_level++) { - unsigned int reg, mask, val, + unsigned int reg, mask, val, old, offset = ARMADA_37XX_NB_TBG_SEL_OFF; armada_3700_pm_dvfs_update_regs(load_level, ®, &offset); val = index << offset; mask = ARMADA_37XX_NB_TBG_SEL_MASK << offset; + regmap_read(base, reg, &old); regmap_update_bits(base, reg, mask, val); + pr_err("%s old=%X -> val=0x%X load_level=%d\n", + __func__, old, val, load_level); } return 0; } > > Regards > /Ilias >> >> Ilias Apalodimas <ilias.apalodimas@linaro.org> schrieb am Do., 14. März >> 2019, 13:15: >> >> > Hi Gregory, >> > > The clock parenting was not setup properly when DVFS was enabled. It was >> > > expected that the same clock source was used with and without DVFS which >> > > was not the case. >> > > >> > > This patch fixes this issue, allowing to make the cpufreq support work >> > > when the CPU clocks source are not the default ones. >> > > >> > > Fixes: 92ce45fb875d ("cpufreq: Add DVFS support for Armada 37xx") >> > > Cc: <stable@vger.kernel.org> >> > > Reported-by: Christian Neubert <christian.neubert.86@gmail.com> >> > > Reported-by: Ilias Apalodimas <ilias.apalodimas@linaro.org> >> > > Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com> >> > > --- >> > > drivers/clk/mvebu/armada-37xx-periph.c | 11 +++++++++++ >> > > 1 file changed, 11 insertions(+) >> > > >> > > diff --git a/drivers/clk/mvebu/armada-37xx-periph.c >> > b/drivers/clk/mvebu/armada-37xx-periph.c >> > > index 1f1cff428d78..26ed3c18a239 100644 >> > > --- a/drivers/clk/mvebu/armada-37xx-periph.c >> > > +++ b/drivers/clk/mvebu/armada-37xx-periph.c >> > > @@ -671,6 +671,17 @@ static int armada_3700_add_composite_clk(const >> > struct clk_periph_data *data, >> > > map = syscon_regmap_lookup_by_compatible( >> > > "marvell,armada-3700-nb-pm"); >> > > pmcpu_clk->nb_pm_base = map; >> > > + >> > > + /* >> > > + * Use the same parent when DVFS is enabled that the >> > > + * default parent received at boot time. When this >> > > + * function is called, DVFS is not enabled yet, so we >> > > + * get the default parent and we can set the parent >> > > + * for DVFS. >> > > + */ >> > > + if (clk_pm_cpu_set_parent(muxrate_hw, >> > > + >> > clk_pm_cpu_get_parent(muxrate_hw))) >> > > + dev_warn(dev, "Failed to setup default parent >> > clock for DVFS\n"); >> > > } >> > > >> > > *hw = clk_hw_register_composite(dev, data->name, >> > data->parent_names, >> > > -- >> > > 2.20.1 >> > > >> > Applied this and selected only >> > >> > CONFIG_CPU_FREQ_DEFAULT_GOV_PERFORMANCE=y >> > CONFIG_CPU_FREQ_GOV_PERFORMANCE=y >> > CONFIG_CPU_FREQ_GOV_POWERSAVE=y >> > >> > After changing the governor from 'powersave' to 'performance' the board >> > completely froze (i even lost access to the serial port) >> > >> > Cheers >> > /Ilias >> > -- Gregory Clement, Bootlin Embedded Linux and Kernel engineering http://bootlin.com ^ permalink raw reply related [flat|nested] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks @ 2019-03-14 14:20 ` Gregory CLEMENT 0 siblings, 0 replies; 30+ messages in thread From: Gregory CLEMENT @ 2019-03-14 14:20 UTC (permalink / raw) To: Ilias Apalodimas, Christian Neubert Cc: Andrew Lunn, Vincent Guittot, Jason Cooper, linux-pm, Antoine Tenart, Viresh Kumar, Mike Turquette, Rafael J. Wysocki, linux-kernel, stable, linux-clk, Thomas Petazzoni, Miquèl Raynal, Maxime Chevallier, Stephen Boyd, linux-arm-kernel, Sebastian Hesselbarth Hi Ilias, On jeu., mars 14 2019, Ilias Apalodimas <ilias.apalodimas@linaro.org> wrote: > Hello Christian, >> Hi, >> >> I assume you use the 1000 MHz firmware. This does also not work on my Rev 7 >> board. But I'm pretty sure this is not a problem of the patches, because if >> I take a newer kernel (4.19.20/27) without the patches it also does not >> work. A kernel 4.19.17 does work for me. My opinion on that is that this is >> another problem which does just occure now because now the cpu frequency >> scaling is working with the right frequencies. > I am not sure which firmware i am running, i did all my tests on 5.0.0 and > changing between governors worked fine without the patches Curently my espressobin is broken so I tested the patches on the Armada 3700 DB and I didn't observe the issue you had. The 3700 DB I used is configured to run at 800MHz. Could you apply the following patch and sent me the boot log? diff --git a/drivers/clk/mvebu/armada-37xx-periph.c b/drivers/clk/mvebu/armada-37xx-periph.c index 26ed3c18a239..f814ade5cd80 100644 --- a/drivers/clk/mvebu/armada-37xx-periph.c +++ b/drivers/clk/mvebu/armada-37xx-periph.c @@ -452,14 +452,17 @@ static int clk_pm_cpu_set_parent(struct clk_hw *hw, u8 index) /* Set the parent clock for all the load level */ for (load_level = 0; load_level < LOAD_LEVEL_NR; load_level++) { - unsigned int reg, mask, val, + unsigned int reg, mask, val, old, offset = ARMADA_37XX_NB_TBG_SEL_OFF; armada_3700_pm_dvfs_update_regs(load_level, ®, &offset); val = index << offset; mask = ARMADA_37XX_NB_TBG_SEL_MASK << offset; + regmap_read(base, reg, &old); regmap_update_bits(base, reg, mask, val); + pr_err("%s old=%X -> val=0x%X load_level=%d\n", + __func__, old, val, load_level); } return 0; } > > Regards > /Ilias >> >> Ilias Apalodimas <ilias.apalodimas@linaro.org> schrieb am Do., 14. März >> 2019, 13:15: >> >> > Hi Gregory, >> > > The clock parenting was not setup properly when DVFS was enabled. It was >> > > expected that the same clock source was used with and without DVFS which >> > > was not the case. >> > > >> > > This patch fixes this issue, allowing to make the cpufreq support work >> > > when the CPU clocks source are not the default ones. >> > > >> > > Fixes: 92ce45fb875d ("cpufreq: Add DVFS support for Armada 37xx") >> > > Cc: <stable@vger.kernel.org> >> > > Reported-by: Christian Neubert <christian.neubert.86@gmail.com> >> > > Reported-by: Ilias Apalodimas <ilias.apalodimas@linaro.org> >> > > Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com> >> > > --- >> > > drivers/clk/mvebu/armada-37xx-periph.c | 11 +++++++++++ >> > > 1 file changed, 11 insertions(+) >> > > >> > > diff --git a/drivers/clk/mvebu/armada-37xx-periph.c >> > b/drivers/clk/mvebu/armada-37xx-periph.c >> > > index 1f1cff428d78..26ed3c18a239 100644 >> > > --- a/drivers/clk/mvebu/armada-37xx-periph.c >> > > +++ b/drivers/clk/mvebu/armada-37xx-periph.c >> > > @@ -671,6 +671,17 @@ static int armada_3700_add_composite_clk(const >> > struct clk_periph_data *data, >> > > map = syscon_regmap_lookup_by_compatible( >> > > "marvell,armada-3700-nb-pm"); >> > > pmcpu_clk->nb_pm_base = map; >> > > + >> > > + /* >> > > + * Use the same parent when DVFS is enabled that the >> > > + * default parent received at boot time. When this >> > > + * function is called, DVFS is not enabled yet, so we >> > > + * get the default parent and we can set the parent >> > > + * for DVFS. >> > > + */ >> > > + if (clk_pm_cpu_set_parent(muxrate_hw, >> > > + >> > clk_pm_cpu_get_parent(muxrate_hw))) >> > > + dev_warn(dev, "Failed to setup default parent >> > clock for DVFS\n"); >> > > } >> > > >> > > *hw = clk_hw_register_composite(dev, data->name, >> > data->parent_names, >> > > -- >> > > 2.20.1 >> > > >> > Applied this and selected only >> > >> > CONFIG_CPU_FREQ_DEFAULT_GOV_PERFORMANCE=y >> > CONFIG_CPU_FREQ_GOV_PERFORMANCE=y >> > CONFIG_CPU_FREQ_GOV_POWERSAVE=y >> > >> > After changing the governor from 'powersave' to 'performance' the board >> > completely froze (i even lost access to the serial port) >> > >> > Cheers >> > /Ilias >> > -- Gregory Clement, Bootlin Embedded Linux and Kernel engineering http://bootlin.com _______________________________________________ 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] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks 2019-03-14 14:20 ` Gregory CLEMENT @ 2019-03-18 11:28 ` Ilias Apalodimas -1 siblings, 0 replies; 30+ messages in thread From: Ilias Apalodimas @ 2019-03-18 11:28 UTC (permalink / raw) To: Gregory CLEMENT Cc: Christian Neubert, Stephen Boyd, Mike Turquette, linux-clk, linux-kernel, Rafael J. Wysocki, Viresh Kumar, linux-pm, Vincent Guittot, Jason Cooper, Andrew Lunn, Sebastian Hesselbarth, Thomas Petazzoni, linux-arm-kernel, Antoine Tenart, Miquèl Raynal, Maxime Chevallier, stable Hi Gregory, > Hi Ilias, > > On jeu., mars 14 2019, Ilias Apalodimas <ilias.apalodimas@linaro.org> wrote: > > > Hello Christian, > >> Hi, > >> > >> I assume you use the 1000 MHz firmware. This does also not work on my Rev 7 > >> board. But I'm pretty sure this is not a problem of the patches, because if > >> I take a newer kernel (4.19.20/27) without the patches it also does not > >> work. A kernel 4.19.17 does work for me. My opinion on that is that this is > >> another problem which does just occure now because now the cpu frequency > >> scaling is working with the right frequencies. > > I am not sure which firmware i am running, i did all my tests on 5.0.0 and > > changing between governors worked fine without the patches > > Curently my espressobin is broken so I tested the patches on the Armada > 3700 DB and I didn't observe the issue you had. > > The 3700 DB I used is configured to run at 800MHz. > > Could you apply the following patch and sent me the boot log? > > diff --git a/drivers/clk/mvebu/armada-37xx-periph.c b/drivers/clk/mvebu/armada-37xx-periph.c > index 26ed3c18a239..f814ade5cd80 100644 > --- a/drivers/clk/mvebu/armada-37xx-periph.c > +++ b/drivers/clk/mvebu/armada-37xx-periph.c > @@ -452,14 +452,17 @@ static int clk_pm_cpu_set_parent(struct clk_hw *hw, u8 index) > > /* Set the parent clock for all the load level */ > for (load_level = 0; load_level < LOAD_LEVEL_NR; load_level++) { > - unsigned int reg, mask, val, > + unsigned int reg, mask, val, old, > offset = ARMADA_37XX_NB_TBG_SEL_OFF; > > armada_3700_pm_dvfs_update_regs(load_level, ®, &offset); > > val = index << offset; > mask = ARMADA_37XX_NB_TBG_SEL_MASK << offset; > + regmap_read(base, reg, &old); > regmap_update_bits(base, reg, mask, val); > + pr_err("%s old=%X -> val=0x%X load_level=%d\n", > + __func__, old, val, load_level); > } > return 0; > } > > > [ 14.909524] clk_pm_cpu_set_parent old=28004840 -> val=0x6000000 load_level=0 [ 14.916424] clk_pm_cpu_set_parent old=2E004840 -> val=0x600 load_level=1 [ 14.923315] clk_pm_cpu_set_parent old=8880A8C0 -> val=0x6000000 load_level=2 [ 14.930572] clk_pm_cpu_set_parent old=8E80A8C0 -> val=0x600 load_level=3 Let me know if you need anything else Regards /Ilias ^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks @ 2019-03-18 11:28 ` Ilias Apalodimas 0 siblings, 0 replies; 30+ messages in thread From: Ilias Apalodimas @ 2019-03-18 11:28 UTC (permalink / raw) To: Gregory CLEMENT Cc: Andrew Lunn, Vincent Guittot, Jason Cooper, Christian Neubert, Antoine Tenart, Stephen Boyd, Mike Turquette, linux-pm, Rafael J. Wysocki, linux-kernel, stable, Maxime Chevallier, Miquèl Raynal, Thomas Petazzoni, Viresh Kumar, linux-clk, linux-arm-kernel, Sebastian Hesselbarth Hi Gregory, > Hi Ilias, > > On jeu., mars 14 2019, Ilias Apalodimas <ilias.apalodimas@linaro.org> wrote: > > > Hello Christian, > >> Hi, > >> > >> I assume you use the 1000 MHz firmware. This does also not work on my Rev 7 > >> board. But I'm pretty sure this is not a problem of the patches, because if > >> I take a newer kernel (4.19.20/27) without the patches it also does not > >> work. A kernel 4.19.17 does work for me. My opinion on that is that this is > >> another problem which does just occure now because now the cpu frequency > >> scaling is working with the right frequencies. > > I am not sure which firmware i am running, i did all my tests on 5.0.0 and > > changing between governors worked fine without the patches > > Curently my espressobin is broken so I tested the patches on the Armada > 3700 DB and I didn't observe the issue you had. > > The 3700 DB I used is configured to run at 800MHz. > > Could you apply the following patch and sent me the boot log? > > diff --git a/drivers/clk/mvebu/armada-37xx-periph.c b/drivers/clk/mvebu/armada-37xx-periph.c > index 26ed3c18a239..f814ade5cd80 100644 > --- a/drivers/clk/mvebu/armada-37xx-periph.c > +++ b/drivers/clk/mvebu/armada-37xx-periph.c > @@ -452,14 +452,17 @@ static int clk_pm_cpu_set_parent(struct clk_hw *hw, u8 index) > > /* Set the parent clock for all the load level */ > for (load_level = 0; load_level < LOAD_LEVEL_NR; load_level++) { > - unsigned int reg, mask, val, > + unsigned int reg, mask, val, old, > offset = ARMADA_37XX_NB_TBG_SEL_OFF; > > armada_3700_pm_dvfs_update_regs(load_level, ®, &offset); > > val = index << offset; > mask = ARMADA_37XX_NB_TBG_SEL_MASK << offset; > + regmap_read(base, reg, &old); > regmap_update_bits(base, reg, mask, val); > + pr_err("%s old=%X -> val=0x%X load_level=%d\n", > + __func__, old, val, load_level); > } > return 0; > } > > > [ 14.909524] clk_pm_cpu_set_parent old=28004840 -> val=0x6000000 load_level=0 [ 14.916424] clk_pm_cpu_set_parent old=2E004840 -> val=0x600 load_level=1 [ 14.923315] clk_pm_cpu_set_parent old=8880A8C0 -> val=0x6000000 load_level=2 [ 14.930572] clk_pm_cpu_set_parent old=8E80A8C0 -> val=0x600 load_level=3 Let me know if you need anything else Regards /Ilias _______________________________________________ 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] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks 2019-03-18 11:28 ` Ilias Apalodimas @ 2019-03-18 11:40 ` Gregory CLEMENT -1 siblings, 0 replies; 30+ messages in thread From: Gregory CLEMENT @ 2019-03-18 11:40 UTC (permalink / raw) To: Ilias Apalodimas Cc: Christian Neubert, Stephen Boyd, Mike Turquette, linux-clk, linux-kernel, Rafael J. Wysocki, Viresh Kumar, linux-pm, Vincent Guittot, Jason Cooper, Andrew Lunn, Sebastian Hesselbarth, Thomas Petazzoni, linux-arm-kernel, Antoine Tenart, Miquèl Raynal, Maxime Chevallier, stable Hi Ilias, On lun., mars 18 2019, Ilias Apalodimas <ilias.apalodimas@linaro.org> wrote: > Hi Gregory, >> Hi Ilias, >> >> On jeu., mars 14 2019, Ilias Apalodimas <ilias.apalodimas@linaro.org> wrote: >> >> > Hello Christian, >> >> Hi, >> >> >> >> I assume you use the 1000 MHz firmware. This does also not work on my Rev 7 >> >> board. But I'm pretty sure this is not a problem of the patches, because if >> >> I take a newer kernel (4.19.20/27) without the patches it also does not >> >> work. A kernel 4.19.17 does work for me. My opinion on that is that this is >> >> another problem which does just occure now because now the cpu frequency >> >> scaling is working with the right frequencies. >> > I am not sure which firmware i am running, i did all my tests on 5.0.0 and >> > changing between governors worked fine without the patches >> >> Curently my espressobin is broken so I tested the patches on the Armada >> 3700 DB and I didn't observe the issue you had. >> >> The 3700 DB I used is configured to run at 800MHz. >> >> Could you apply the following patch and sent me the boot log? >> >> diff --git a/drivers/clk/mvebu/armada-37xx-periph.c b/drivers/clk/mvebu/armada-37xx-periph.c >> index 26ed3c18a239..f814ade5cd80 100644 >> --- a/drivers/clk/mvebu/armada-37xx-periph.c >> +++ b/drivers/clk/mvebu/armada-37xx-periph.c >> @@ -452,14 +452,17 @@ static int clk_pm_cpu_set_parent(struct clk_hw *hw, u8 index) >> >> /* Set the parent clock for all the load level */ >> for (load_level = 0; load_level < LOAD_LEVEL_NR; load_level++) { >> - unsigned int reg, mask, val, >> + unsigned int reg, mask, val, old, >> offset = ARMADA_37XX_NB_TBG_SEL_OFF; >> >> armada_3700_pm_dvfs_update_regs(load_level, ®, &offset); >> >> val = index << offset; >> mask = ARMADA_37XX_NB_TBG_SEL_MASK << offset; >> + regmap_read(base, reg, &old); >> regmap_update_bits(base, reg, mask, val); >> + pr_err("%s old=%X -> val=0x%X load_level=%d\n", >> + __func__, old, val, load_level); >> } >> return 0; >> } >> >> >> > [ 14.909524] clk_pm_cpu_set_parent old=28004840 -> val=0x6000000 load_level=0 > [ 14.916424] clk_pm_cpu_set_parent old=2E004840 -> val=0x600 load_level=1 > [ 14.923315] clk_pm_cpu_set_parent old=8880A8C0 -> val=0x6000000 load_level=2 > [ 14.930572] clk_pm_cpu_set_parent old=8E80A8C0 -> val=0x600 load_level=3 > > > Let me know if you need anything else Could you show me the output of "cat /sys/kernel/debug/clk/clk_summary" Also, during this week-end, Christian suggested that the issue might come from the AVS support. Could you disable it and check you still have the issue? For this, you just have to remove the avs node in arch/arm64/boot/dts/marvell/armada-37xx.dtsi and rebuild the dtb. Thanks, Gregory > > Regards > /Ilias -- Gregory Clement, Bootlin Embedded Linux and Kernel engineering http://bootlin.com ^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks @ 2019-03-18 11:40 ` Gregory CLEMENT 0 siblings, 0 replies; 30+ messages in thread From: Gregory CLEMENT @ 2019-03-18 11:40 UTC (permalink / raw) To: Ilias Apalodimas Cc: Andrew Lunn, Vincent Guittot, Jason Cooper, Christian Neubert, Antoine Tenart, Stephen Boyd, Mike Turquette, linux-pm, Rafael J. Wysocki, linux-kernel, stable, Maxime Chevallier, Miquèl Raynal, Thomas Petazzoni, Viresh Kumar, linux-clk, linux-arm-kernel, Sebastian Hesselbarth Hi Ilias, On lun., mars 18 2019, Ilias Apalodimas <ilias.apalodimas@linaro.org> wrote: > Hi Gregory, >> Hi Ilias, >> >> On jeu., mars 14 2019, Ilias Apalodimas <ilias.apalodimas@linaro.org> wrote: >> >> > Hello Christian, >> >> Hi, >> >> >> >> I assume you use the 1000 MHz firmware. This does also not work on my Rev 7 >> >> board. But I'm pretty sure this is not a problem of the patches, because if >> >> I take a newer kernel (4.19.20/27) without the patches it also does not >> >> work. A kernel 4.19.17 does work for me. My opinion on that is that this is >> >> another problem which does just occure now because now the cpu frequency >> >> scaling is working with the right frequencies. >> > I am not sure which firmware i am running, i did all my tests on 5.0.0 and >> > changing between governors worked fine without the patches >> >> Curently my espressobin is broken so I tested the patches on the Armada >> 3700 DB and I didn't observe the issue you had. >> >> The 3700 DB I used is configured to run at 800MHz. >> >> Could you apply the following patch and sent me the boot log? >> >> diff --git a/drivers/clk/mvebu/armada-37xx-periph.c b/drivers/clk/mvebu/armada-37xx-periph.c >> index 26ed3c18a239..f814ade5cd80 100644 >> --- a/drivers/clk/mvebu/armada-37xx-periph.c >> +++ b/drivers/clk/mvebu/armada-37xx-periph.c >> @@ -452,14 +452,17 @@ static int clk_pm_cpu_set_parent(struct clk_hw *hw, u8 index) >> >> /* Set the parent clock for all the load level */ >> for (load_level = 0; load_level < LOAD_LEVEL_NR; load_level++) { >> - unsigned int reg, mask, val, >> + unsigned int reg, mask, val, old, >> offset = ARMADA_37XX_NB_TBG_SEL_OFF; >> >> armada_3700_pm_dvfs_update_regs(load_level, ®, &offset); >> >> val = index << offset; >> mask = ARMADA_37XX_NB_TBG_SEL_MASK << offset; >> + regmap_read(base, reg, &old); >> regmap_update_bits(base, reg, mask, val); >> + pr_err("%s old=%X -> val=0x%X load_level=%d\n", >> + __func__, old, val, load_level); >> } >> return 0; >> } >> >> >> > [ 14.909524] clk_pm_cpu_set_parent old=28004840 -> val=0x6000000 load_level=0 > [ 14.916424] clk_pm_cpu_set_parent old=2E004840 -> val=0x600 load_level=1 > [ 14.923315] clk_pm_cpu_set_parent old=8880A8C0 -> val=0x6000000 load_level=2 > [ 14.930572] clk_pm_cpu_set_parent old=8E80A8C0 -> val=0x600 load_level=3 > > > Let me know if you need anything else Could you show me the output of "cat /sys/kernel/debug/clk/clk_summary" Also, during this week-end, Christian suggested that the issue might come from the AVS support. Could you disable it and check you still have the issue? For this, you just have to remove the avs node in arch/arm64/boot/dts/marvell/armada-37xx.dtsi and rebuild the dtb. Thanks, Gregory > > Regards > /Ilias -- Gregory Clement, Bootlin Embedded Linux and Kernel engineering http://bootlin.com _______________________________________________ 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] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks 2019-03-18 11:40 ` Gregory CLEMENT @ 2019-03-18 12:21 ` Ilias Apalodimas -1 siblings, 0 replies; 30+ messages in thread From: Ilias Apalodimas @ 2019-03-18 12:21 UTC (permalink / raw) To: Gregory CLEMENT Cc: Christian Neubert, Stephen Boyd, Mike Turquette, linux-clk, linux-kernel, Rafael J. Wysocki, Viresh Kumar, linux-pm, Vincent Guittot, Jason Cooper, Andrew Lunn, Sebastian Hesselbarth, Thomas Petazzoni, linux-arm-kernel, Antoine Tenart, Miquèl Raynal, Maxime Chevallier, stable Hi Gregory, > >> > > [ 14.909524] clk_pm_cpu_set_parent old=28004840 -> val=0x6000000 load_level=0 > > [ 14.916424] clk_pm_cpu_set_parent old=2E004840 -> val=0x600 load_level=1 > > [ 14.923315] clk_pm_cpu_set_parent old=8880A8C0 -> val=0x6000000 load_level=2 > > [ 14.930572] clk_pm_cpu_set_parent old=8E80A8C0 -> val=0x600 load_level=3 > > > > > > Let me know if you need anything else > > Could you show me the output of "cat /sys/kernel/debug/clk/clk_summary" > root@localhost ~ # cat /sys/kernel/debug/clk/clk_summary enable prepare protect duty clock count count count rate accuracy phase cycle --------------------------------------------------------------------------------------------- xtal 2 3 0 25000000 0 0 50000 i2c_1 0 0 0 25000000 0 0 50000 i2c_2 0 0 0 25000000 0 0 50000 avs 0 0 0 25000000 0 0 50000 TBG-B-S 2 2 0 1000000000 0 0 50000 cpu 0 0 0 200000000 0 0 50000 usb32_ss_sys 1 1 0 125000000 0 0 50000 usb32_usb2_sys 0 0 0 100000000 0 0 50000 gbe_125 0 0 0 125000000 0 0 50000 gbe0_125 0 0 0 125000000 0 0 50000 gbe1_125 0 0 0 125000000 0 0 50000 gbe_core 1 1 0 250000000 0 0 50000 gbe_bm 0 0 0 250000000 0 0 50000 gbe0_core 1 1 0 250000000 0 0 50000 gbe1_core 0 0 0 250000000 0 0 50000 eip97 0 0 0 500000000 0 0 50000 trace 0 0 0 1000000000 0 0 50000 ddr_fclk 0 0 0 125000000 0 0 50000 pwm 0 0 0 50000000 0 0 50000 tscem_tmx 0 0 0 1000000000 0 0 50000 sec_dap 0 0 0 100000000 0 0 50000 sec_at 0 0 0 200000000 0 0 50000 TBG-A-S 0 0 0 1600000000 0 0 50000 sdio 0 0 0 400000000 0 0 50000 ddr_phy 0 0 0 400000000 0 0 50000 mmc 0 0 0 400000000 0 0 50000 TBG-B-P 0 0 0 500000000 0 0 50000 gbe_50 0 0 0 50000000 0 0 50000 gbe0_50 0 0 0 50000000 0 0 50000 gbe1_50 0 0 0 50000000 0 0 50000 TBG-A-P 0 1 0 800000000 0 0 50000 counter 0 0 0 160000000 0 0 50000 sqf 0 1 0 200000000 0 0 50000 tscem 0 0 0 200000000 0 0 50000 sata_host 0 0 0 200000000 0 0 50000 root@localhost ~ # > Also, during this week-end, Christian suggested that the issue might > come from the AVS support. > > Could you disable it and check you still have the issue? > > For this, you just have to remove the avs node in > arch/arm64/boot/dts/marvell/armada-37xx.dtsi and rebuild the dtb. Sure. You'll have to wait for a week though. Currently on a trip. I'll run that once i return Cheers /Ilias ^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks @ 2019-03-18 12:21 ` Ilias Apalodimas 0 siblings, 0 replies; 30+ messages in thread From: Ilias Apalodimas @ 2019-03-18 12:21 UTC (permalink / raw) To: Gregory CLEMENT Cc: Andrew Lunn, Vincent Guittot, Jason Cooper, Christian Neubert, Antoine Tenart, Stephen Boyd, Mike Turquette, linux-pm, Rafael J. Wysocki, linux-kernel, stable, Maxime Chevallier, Miquèl Raynal, Thomas Petazzoni, Viresh Kumar, linux-clk, linux-arm-kernel, Sebastian Hesselbarth Hi Gregory, > >> > > [ 14.909524] clk_pm_cpu_set_parent old=28004840 -> val=0x6000000 load_level=0 > > [ 14.916424] clk_pm_cpu_set_parent old=2E004840 -> val=0x600 load_level=1 > > [ 14.923315] clk_pm_cpu_set_parent old=8880A8C0 -> val=0x6000000 load_level=2 > > [ 14.930572] clk_pm_cpu_set_parent old=8E80A8C0 -> val=0x600 load_level=3 > > > > > > Let me know if you need anything else > > Could you show me the output of "cat /sys/kernel/debug/clk/clk_summary" > root@localhost ~ # cat /sys/kernel/debug/clk/clk_summary enable prepare protect duty clock count count count rate accuracy phase cycle --------------------------------------------------------------------------------------------- xtal 2 3 0 25000000 0 0 50000 i2c_1 0 0 0 25000000 0 0 50000 i2c_2 0 0 0 25000000 0 0 50000 avs 0 0 0 25000000 0 0 50000 TBG-B-S 2 2 0 1000000000 0 0 50000 cpu 0 0 0 200000000 0 0 50000 usb32_ss_sys 1 1 0 125000000 0 0 50000 usb32_usb2_sys 0 0 0 100000000 0 0 50000 gbe_125 0 0 0 125000000 0 0 50000 gbe0_125 0 0 0 125000000 0 0 50000 gbe1_125 0 0 0 125000000 0 0 50000 gbe_core 1 1 0 250000000 0 0 50000 gbe_bm 0 0 0 250000000 0 0 50000 gbe0_core 1 1 0 250000000 0 0 50000 gbe1_core 0 0 0 250000000 0 0 50000 eip97 0 0 0 500000000 0 0 50000 trace 0 0 0 1000000000 0 0 50000 ddr_fclk 0 0 0 125000000 0 0 50000 pwm 0 0 0 50000000 0 0 50000 tscem_tmx 0 0 0 1000000000 0 0 50000 sec_dap 0 0 0 100000000 0 0 50000 sec_at 0 0 0 200000000 0 0 50000 TBG-A-S 0 0 0 1600000000 0 0 50000 sdio 0 0 0 400000000 0 0 50000 ddr_phy 0 0 0 400000000 0 0 50000 mmc 0 0 0 400000000 0 0 50000 TBG-B-P 0 0 0 500000000 0 0 50000 gbe_50 0 0 0 50000000 0 0 50000 gbe0_50 0 0 0 50000000 0 0 50000 gbe1_50 0 0 0 50000000 0 0 50000 TBG-A-P 0 1 0 800000000 0 0 50000 counter 0 0 0 160000000 0 0 50000 sqf 0 1 0 200000000 0 0 50000 tscem 0 0 0 200000000 0 0 50000 sata_host 0 0 0 200000000 0 0 50000 root@localhost ~ # > Also, during this week-end, Christian suggested that the issue might > come from the AVS support. > > Could you disable it and check you still have the issue? > > For this, you just have to remove the avs node in > arch/arm64/boot/dts/marvell/armada-37xx.dtsi and rebuild the dtb. Sure. You'll have to wait for a week though. Currently on a trip. I'll run that once i return Cheers /Ilias _______________________________________________ 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] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks 2019-03-18 12:21 ` Ilias Apalodimas @ 2019-04-24 9:30 ` Viresh Kumar -1 siblings, 0 replies; 30+ messages in thread From: Viresh Kumar @ 2019-04-24 9:30 UTC (permalink / raw) To: Gregory CLEMENT, Ilias Apalodimas Cc: Christian Neubert, Stephen Boyd, Mike Turquette, linux-clk, linux-kernel, Rafael J. Wysocki, linux-pm, Vincent Guittot, Jason Cooper, Andrew Lunn, Sebastian Hesselbarth, Thomas Petazzoni, linux-arm-kernel, Antoine Tenart, Miquèl Raynal, Maxime Chevallier, stable On 18-03-19, 14:21, Ilias Apalodimas wrote: > Hi Gregory, > > > >> > > > [ 14.909524] clk_pm_cpu_set_parent old=28004840 -> val=0x6000000 load_level=0 > > > [ 14.916424] clk_pm_cpu_set_parent old=2E004840 -> val=0x600 load_level=1 > > > [ 14.923315] clk_pm_cpu_set_parent old=8880A8C0 -> val=0x6000000 load_level=2 > > > [ 14.930572] clk_pm_cpu_set_parent old=8E80A8C0 -> val=0x600 load_level=3 > > > > > > > > > Let me know if you need anything else > > > > Could you show me the output of "cat /sys/kernel/debug/clk/clk_summary" > > > root@localhost ~ # cat /sys/kernel/debug/clk/clk_summary > enable prepare protect duty > clock count count count rate accuracy phase cycle > --------------------------------------------------------------------------------------------- > xtal 2 3 0 25000000 0 0 50000 > i2c_1 0 0 0 25000000 0 0 50000 > i2c_2 0 0 0 25000000 0 0 50000 > avs 0 0 0 25000000 0 0 50000 > TBG-B-S 2 2 0 1000000000 0 0 50000 > cpu 0 0 0 200000000 0 0 50000 > usb32_ss_sys 1 1 0 125000000 0 0 50000 > usb32_usb2_sys 0 0 0 100000000 0 0 50000 > gbe_125 0 0 0 125000000 0 0 50000 > gbe0_125 0 0 0 125000000 0 0 50000 > gbe1_125 0 0 0 125000000 0 0 50000 > gbe_core 1 1 0 250000000 0 0 50000 > gbe_bm 0 0 0 250000000 0 0 50000 > gbe0_core 1 1 0 250000000 0 0 50000 > gbe1_core 0 0 0 250000000 0 0 50000 > eip97 0 0 0 500000000 0 0 50000 > trace 0 0 0 1000000000 0 0 50000 > ddr_fclk 0 0 0 125000000 0 0 50000 > pwm 0 0 0 50000000 0 0 50000 > tscem_tmx 0 0 0 1000000000 0 0 50000 > sec_dap 0 0 0 100000000 0 0 50000 > sec_at 0 0 0 200000000 0 0 50000 > TBG-A-S 0 0 0 1600000000 0 0 50000 > sdio 0 0 0 400000000 0 0 50000 > ddr_phy 0 0 0 400000000 0 0 50000 > mmc 0 0 0 400000000 0 0 50000 > TBG-B-P 0 0 0 500000000 0 0 50000 > gbe_50 0 0 0 50000000 0 0 50000 > gbe0_50 0 0 0 50000000 0 0 50000 > gbe1_50 0 0 0 50000000 0 0 50000 > TBG-A-P 0 1 0 800000000 0 0 50000 > counter 0 0 0 160000000 0 0 50000 > sqf 0 1 0 200000000 0 0 50000 > tscem 0 0 0 200000000 0 0 50000 > sata_host 0 0 0 200000000 0 0 50000 > root@localhost ~ # > > > Also, during this week-end, Christian suggested that the issue might > > come from the AVS support. > > > > Could you disable it and check you still have the issue? > > > > For this, you just have to remove the avs node in > > arch/arm64/boot/dts/marvell/armada-37xx.dtsi and rebuild the dtb. > Sure. You'll have to wait for a week though. Currently on a trip. I'll run that > once i return @Ilias: Can you please try this now and confirm to Gregory ? -- viresh ^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks @ 2019-04-24 9:30 ` Viresh Kumar 0 siblings, 0 replies; 30+ messages in thread From: Viresh Kumar @ 2019-04-24 9:30 UTC (permalink / raw) To: Gregory CLEMENT, Ilias Apalodimas Cc: Andrew Lunn, Vincent Guittot, Jason Cooper, Christian Neubert, Antoine Tenart, Stephen Boyd, Mike Turquette, linux-pm, Rafael J. Wysocki, linux-kernel, stable, Maxime Chevallier, Thomas Petazzoni, Miquèl Raynal, linux-clk, linux-arm-kernel, Sebastian Hesselbarth On 18-03-19, 14:21, Ilias Apalodimas wrote: > Hi Gregory, > > > >> > > > [ 14.909524] clk_pm_cpu_set_parent old=28004840 -> val=0x6000000 load_level=0 > > > [ 14.916424] clk_pm_cpu_set_parent old=2E004840 -> val=0x600 load_level=1 > > > [ 14.923315] clk_pm_cpu_set_parent old=8880A8C0 -> val=0x6000000 load_level=2 > > > [ 14.930572] clk_pm_cpu_set_parent old=8E80A8C0 -> val=0x600 load_level=3 > > > > > > > > > Let me know if you need anything else > > > > Could you show me the output of "cat /sys/kernel/debug/clk/clk_summary" > > > root@localhost ~ # cat /sys/kernel/debug/clk/clk_summary > enable prepare protect duty > clock count count count rate accuracy phase cycle > --------------------------------------------------------------------------------------------- > xtal 2 3 0 25000000 0 0 50000 > i2c_1 0 0 0 25000000 0 0 50000 > i2c_2 0 0 0 25000000 0 0 50000 > avs 0 0 0 25000000 0 0 50000 > TBG-B-S 2 2 0 1000000000 0 0 50000 > cpu 0 0 0 200000000 0 0 50000 > usb32_ss_sys 1 1 0 125000000 0 0 50000 > usb32_usb2_sys 0 0 0 100000000 0 0 50000 > gbe_125 0 0 0 125000000 0 0 50000 > gbe0_125 0 0 0 125000000 0 0 50000 > gbe1_125 0 0 0 125000000 0 0 50000 > gbe_core 1 1 0 250000000 0 0 50000 > gbe_bm 0 0 0 250000000 0 0 50000 > gbe0_core 1 1 0 250000000 0 0 50000 > gbe1_core 0 0 0 250000000 0 0 50000 > eip97 0 0 0 500000000 0 0 50000 > trace 0 0 0 1000000000 0 0 50000 > ddr_fclk 0 0 0 125000000 0 0 50000 > pwm 0 0 0 50000000 0 0 50000 > tscem_tmx 0 0 0 1000000000 0 0 50000 > sec_dap 0 0 0 100000000 0 0 50000 > sec_at 0 0 0 200000000 0 0 50000 > TBG-A-S 0 0 0 1600000000 0 0 50000 > sdio 0 0 0 400000000 0 0 50000 > ddr_phy 0 0 0 400000000 0 0 50000 > mmc 0 0 0 400000000 0 0 50000 > TBG-B-P 0 0 0 500000000 0 0 50000 > gbe_50 0 0 0 50000000 0 0 50000 > gbe0_50 0 0 0 50000000 0 0 50000 > gbe1_50 0 0 0 50000000 0 0 50000 > TBG-A-P 0 1 0 800000000 0 0 50000 > counter 0 0 0 160000000 0 0 50000 > sqf 0 1 0 200000000 0 0 50000 > tscem 0 0 0 200000000 0 0 50000 > sata_host 0 0 0 200000000 0 0 50000 > root@localhost ~ # > > > Also, during this week-end, Christian suggested that the issue might > > come from the AVS support. > > > > Could you disable it and check you still have the issue? > > > > For this, you just have to remove the avs node in > > arch/arm64/boot/dts/marvell/armada-37xx.dtsi and rebuild the dtb. > Sure. You'll have to wait for a week though. Currently on a trip. I'll run that > once i return @Ilias: Can you please try this now and confirm to Gregory ? -- viresh _______________________________________________ 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] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks 2019-04-24 9:30 ` Viresh Kumar @ 2019-04-25 12:33 ` Ilias Apalodimas -1 siblings, 0 replies; 30+ messages in thread From: Ilias Apalodimas @ 2019-04-25 12:33 UTC (permalink / raw) To: Viresh Kumar Cc: Gregory CLEMENT, Christian Neubert, Stephen Boyd, Mike Turquette, linux-clk, linux-kernel, Rafael J. Wysocki, linux-pm, Vincent Guittot, Jason Cooper, Andrew Lunn, Sebastian Hesselbarth, Thomas Petazzoni, linux-arm-kernel, Antoine Tenart, Miquèl Raynal, Maxime Chevallier, stable Hi Viresh, > > > Also, during this week-end, Christian suggested that the issue might > > > come from the AVS support. > > > > > > Could you disable it and check you still have the issue? > > > > > > For this, you just have to remove the avs node in > > > arch/arm64/boot/dts/marvell/armada-37xx.dtsi and rebuild the dtb. > > Sure. You'll have to wait for a week though. Currently on a trip. I'll run that > > once i return > > @Ilias: Can you please try this now and confirm to Gregory ? I am more overloaded than usual and totally forgot about this. Apologies. I'll try finding some time and do this. Thanks /Ilias ^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks @ 2019-04-25 12:33 ` Ilias Apalodimas 0 siblings, 0 replies; 30+ messages in thread From: Ilias Apalodimas @ 2019-04-25 12:33 UTC (permalink / raw) To: Viresh Kumar Cc: Andrew Lunn, Vincent Guittot, Jason Cooper, Christian Neubert, Antoine Tenart, Rafael J. Wysocki, Gregory CLEMENT, linux-pm, Stephen Boyd, linux-kernel, stable, Maxime Chevallier, Mike Turquette, Thomas Petazzoni, Miquèl Raynal, linux-clk, linux-arm-kernel, Sebastian Hesselbarth Hi Viresh, > > > Also, during this week-end, Christian suggested that the issue might > > > come from the AVS support. > > > > > > Could you disable it and check you still have the issue? > > > > > > For this, you just have to remove the avs node in > > > arch/arm64/boot/dts/marvell/armada-37xx.dtsi and rebuild the dtb. > > Sure. You'll have to wait for a week though. Currently on a trip. I'll run that > > once i return > > @Ilias: Can you please try this now and confirm to Gregory ? I am more overloaded than usual and totally forgot about this. Apologies. I'll try finding some time and do this. Thanks /Ilias _______________________________________________ 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] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks 2019-04-25 12:33 ` Ilias Apalodimas @ 2019-05-20 11:20 ` Viresh Kumar -1 siblings, 0 replies; 30+ messages in thread From: Viresh Kumar @ 2019-05-20 11:20 UTC (permalink / raw) To: Ilias Apalodimas Cc: Gregory CLEMENT, Christian Neubert, Stephen Boyd, Mike Turquette, linux-clk, linux-kernel, Rafael J. Wysocki, linux-pm, Vincent Guittot, Jason Cooper, Andrew Lunn, Sebastian Hesselbarth, Thomas Petazzoni, linux-arm-kernel, Antoine Tenart, Miquèl Raynal, Maxime Chevallier, stable On 25-04-19, 15:33, Ilias Apalodimas wrote: > Hi Viresh, > > > > > Also, during this week-end, Christian suggested that the issue might > > > > come from the AVS support. > > > > > > > > Could you disable it and check you still have the issue? > > > > > > > > For this, you just have to remove the avs node in > > > > arch/arm64/boot/dts/marvell/armada-37xx.dtsi and rebuild the dtb. > > > Sure. You'll have to wait for a week though. Currently on a trip. I'll run that > > > once i return > > > > @Ilias: Can you please try this now and confirm to Gregory ? > I am more overloaded than usual and totally forgot about this. Apologies. > I'll try finding some time and do this. Ping Ilias. -- viresh ^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks @ 2019-05-20 11:20 ` Viresh Kumar 0 siblings, 0 replies; 30+ messages in thread From: Viresh Kumar @ 2019-05-20 11:20 UTC (permalink / raw) To: Ilias Apalodimas Cc: Andrew Lunn, Vincent Guittot, Jason Cooper, Christian Neubert, Antoine Tenart, Rafael J. Wysocki, Gregory CLEMENT, linux-pm, Stephen Boyd, linux-kernel, stable, Maxime Chevallier, Mike Turquette, Thomas Petazzoni, Miquèl Raynal, linux-clk, linux-arm-kernel, Sebastian Hesselbarth On 25-04-19, 15:33, Ilias Apalodimas wrote: > Hi Viresh, > > > > > Also, during this week-end, Christian suggested that the issue might > > > > come from the AVS support. > > > > > > > > Could you disable it and check you still have the issue? > > > > > > > > For this, you just have to remove the avs node in > > > > arch/arm64/boot/dts/marvell/armada-37xx.dtsi and rebuild the dtb. > > > Sure. You'll have to wait for a week though. Currently on a trip. I'll run that > > > once i return > > > > @Ilias: Can you please try this now and confirm to Gregory ? > I am more overloaded than usual and totally forgot about this. Apologies. > I'll try finding some time and do this. Ping Ilias. -- viresh _______________________________________________ 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] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks 2019-05-20 11:20 ` Viresh Kumar @ 2019-05-22 7:03 ` Ilias Apalodimas -1 siblings, 0 replies; 30+ messages in thread From: Ilias Apalodimas @ 2019-05-22 7:03 UTC (permalink / raw) To: Viresh Kumar Cc: Gregory CLEMENT, Christian Neubert, Stephen Boyd, Mike Turquette, linux-clk, linux-kernel, Rafael J. Wysocki, linux-pm, Vincent Guittot, Jason Cooper, Andrew Lunn, Sebastian Hesselbarth, Thomas Petazzoni, linux-arm-kernel, Antoine Tenart, Miquèl Raynal, Maxime Chevallier, stable Hi Viresh, Gregory On Mon, May 20, 2019 at 04:50:42PM +0530, Viresh Kumar wrote: > On 25-04-19, 15:33, Ilias Apalodimas wrote: > > Hi Viresh, > > > > > > > Also, during this week-end, Christian suggested that the issue might > > > > > come from the AVS support. > > > > > > > > > > Could you disable it and check you still have the issue? > > > > > > > > > > For this, you just have to remove the avs node in > > > > > arch/arm64/boot/dts/marvell/armada-37xx.dtsi and rebuild the dtb. > > > > Sure. You'll have to wait for a week though. Currently on a trip. I'll run that > > > > once i return > > > > > > @Ilias: Can you please try this now and confirm to Gregory ? > > I am more overloaded than usual and totally forgot about this. Apologies. > > I'll try finding some time and do this. > > Ping Ilias. Sorry for the huge delay. Applying this patch and removing tha 'avs' node from arch/arm64/boot/dts/marvell/armada-37xx.dtsi seems to work. Changing between governors does not freeze the board any more. I haven't checked the actual impact on the CPU speed but the values on /sys/devices/system/cpu/cpufreq/policy0/scaling_governor are correct Cheers /Ilias ^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks @ 2019-05-22 7:03 ` Ilias Apalodimas 0 siblings, 0 replies; 30+ messages in thread From: Ilias Apalodimas @ 2019-05-22 7:03 UTC (permalink / raw) To: Viresh Kumar Cc: Andrew Lunn, Vincent Guittot, Jason Cooper, Christian Neubert, Antoine Tenart, Rafael J. Wysocki, Gregory CLEMENT, linux-pm, Stephen Boyd, linux-kernel, stable, Maxime Chevallier, Mike Turquette, Thomas Petazzoni, Miquèl Raynal, linux-clk, linux-arm-kernel, Sebastian Hesselbarth Hi Viresh, Gregory On Mon, May 20, 2019 at 04:50:42PM +0530, Viresh Kumar wrote: > On 25-04-19, 15:33, Ilias Apalodimas wrote: > > Hi Viresh, > > > > > > > Also, during this week-end, Christian suggested that the issue might > > > > > come from the AVS support. > > > > > > > > > > Could you disable it and check you still have the issue? > > > > > > > > > > For this, you just have to remove the avs node in > > > > > arch/arm64/boot/dts/marvell/armada-37xx.dtsi and rebuild the dtb. > > > > Sure. You'll have to wait for a week though. Currently on a trip. I'll run that > > > > once i return > > > > > > @Ilias: Can you please try this now and confirm to Gregory ? > > I am more overloaded than usual and totally forgot about this. Apologies. > > I'll try finding some time and do this. > > Ping Ilias. Sorry for the huge delay. Applying this patch and removing tha 'avs' node from arch/arm64/boot/dts/marvell/armada-37xx.dtsi seems to work. Changing between governors does not freeze the board any more. I haven't checked the actual impact on the CPU speed but the values on /sys/devices/system/cpu/cpufreq/policy0/scaling_governor are correct Cheers /Ilias _______________________________________________ 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] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks 2019-05-22 7:03 ` Ilias Apalodimas @ 2019-05-22 7:06 ` Viresh Kumar -1 siblings, 0 replies; 30+ messages in thread From: Viresh Kumar @ 2019-05-22 7:06 UTC (permalink / raw) To: Ilias Apalodimas Cc: Gregory CLEMENT, Christian Neubert, Stephen Boyd, Mike Turquette, linux-clk, linux-kernel, Rafael J. Wysocki, linux-pm, Vincent Guittot, Jason Cooper, Andrew Lunn, Sebastian Hesselbarth, Thomas Petazzoni, linux-arm-kernel, Antoine Tenart, Miquèl Raynal, Maxime Chevallier, stable On 22-05-19, 10:03, Ilias Apalodimas wrote: > Hi Viresh, Gregory > On Mon, May 20, 2019 at 04:50:42PM +0530, Viresh Kumar wrote: > > On 25-04-19, 15:33, Ilias Apalodimas wrote: > > > Hi Viresh, > > > > > > > > > Also, during this week-end, Christian suggested that the issue might > > > > > > come from the AVS support. > > > > > > > > > > > > Could you disable it and check you still have the issue? > > > > > > > > > > > > For this, you just have to remove the avs node in > > > > > > arch/arm64/boot/dts/marvell/armada-37xx.dtsi and rebuild the dtb. > > > > > Sure. You'll have to wait for a week though. Currently on a trip. I'll run that > > > > > once i return > > > > > > > > @Ilias: Can you please try this now and confirm to Gregory ? > > > I am more overloaded than usual and totally forgot about this. Apologies. > > > I'll try finding some time and do this. > > > > Ping Ilias. > Sorry for the huge delay. > Applying this patch and removing tha 'avs' node from > arch/arm64/boot/dts/marvell/armada-37xx.dtsi seems to work. > Changing between governors does not freeze the board any more. I haven't checked > the actual impact on the CPU speed but the values on > /sys/devices/system/cpu/cpufreq/policy0/scaling_governor are correct Thanks for testing it out. Lets see what Gregory has to say now. -- viresh ^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks @ 2019-05-22 7:06 ` Viresh Kumar 0 siblings, 0 replies; 30+ messages in thread From: Viresh Kumar @ 2019-05-22 7:06 UTC (permalink / raw) To: Ilias Apalodimas Cc: Andrew Lunn, Vincent Guittot, Jason Cooper, Christian Neubert, Antoine Tenart, Rafael J. Wysocki, Gregory CLEMENT, linux-pm, Stephen Boyd, linux-kernel, stable, Maxime Chevallier, Mike Turquette, Thomas Petazzoni, Miquèl Raynal, linux-clk, linux-arm-kernel, Sebastian Hesselbarth On 22-05-19, 10:03, Ilias Apalodimas wrote: > Hi Viresh, Gregory > On Mon, May 20, 2019 at 04:50:42PM +0530, Viresh Kumar wrote: > > On 25-04-19, 15:33, Ilias Apalodimas wrote: > > > Hi Viresh, > > > > > > > > > Also, during this week-end, Christian suggested that the issue might > > > > > > come from the AVS support. > > > > > > > > > > > > Could you disable it and check you still have the issue? > > > > > > > > > > > > For this, you just have to remove the avs node in > > > > > > arch/arm64/boot/dts/marvell/armada-37xx.dtsi and rebuild the dtb. > > > > > Sure. You'll have to wait for a week though. Currently on a trip. I'll run that > > > > > once i return > > > > > > > > @Ilias: Can you please try this now and confirm to Gregory ? > > > I am more overloaded than usual and totally forgot about this. Apologies. > > > I'll try finding some time and do this. > > > > Ping Ilias. > Sorry for the huge delay. > Applying this patch and removing tha 'avs' node from > arch/arm64/boot/dts/marvell/armada-37xx.dtsi seems to work. > Changing between governors does not freeze the board any more. I haven't checked > the actual impact on the CPU speed but the values on > /sys/devices/system/cpu/cpufreq/policy0/scaling_governor are correct Thanks for testing it out. Lets see what Gregory has to say now. -- viresh _______________________________________________ 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] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks 2019-05-22 7:06 ` Viresh Kumar @ 2019-06-10 10:19 ` Viresh Kumar -1 siblings, 0 replies; 30+ messages in thread From: Viresh Kumar @ 2019-06-10 10:19 UTC (permalink / raw) To: Gregory CLEMENT, Ilias Apalodimas Cc: Christian Neubert, Stephen Boyd, Mike Turquette, linux-clk, linux-kernel, Rafael J. Wysocki, linux-pm, Vincent Guittot, Jason Cooper, Andrew Lunn, Sebastian Hesselbarth, Thomas Petazzoni, linux-arm-kernel, Antoine Tenart, Miquèl Raynal, Maxime Chevallier, stable On 22-05-19, 12:36, Viresh Kumar wrote: > On 22-05-19, 10:03, Ilias Apalodimas wrote: > > Hi Viresh, Gregory > > On Mon, May 20, 2019 at 04:50:42PM +0530, Viresh Kumar wrote: > > > On 25-04-19, 15:33, Ilias Apalodimas wrote: > > > > Hi Viresh, > > > > > > > > > > > Also, during this week-end, Christian suggested that the issue might > > > > > > > come from the AVS support. > > > > > > > > > > > > > > Could you disable it and check you still have the issue? > > > > > > > > > > > > > > For this, you just have to remove the avs node in > > > > > > > arch/arm64/boot/dts/marvell/armada-37xx.dtsi and rebuild the dtb. > > > > > > Sure. You'll have to wait for a week though. Currently on a trip. I'll run that > > > > > > once i return > > > > > > > > > > @Ilias: Can you please try this now and confirm to Gregory ? > > > > I am more overloaded than usual and totally forgot about this. Apologies. > > > > I'll try finding some time and do this. > > > > > > Ping Ilias. > > Sorry for the huge delay. > > Applying this patch and removing tha 'avs' node from > > arch/arm64/boot/dts/marvell/armada-37xx.dtsi seems to work. > > Changing between governors does not freeze the board any more. I haven't checked > > the actual impact on the CPU speed but the values on > > /sys/devices/system/cpu/cpufreq/policy0/scaling_governor are correct > > Thanks for testing it out. Lets see what Gregory has to say now. @Gregory: Do you have any further advice for Ilias ? -- viresh ^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks @ 2019-06-10 10:19 ` Viresh Kumar 0 siblings, 0 replies; 30+ messages in thread From: Viresh Kumar @ 2019-06-10 10:19 UTC (permalink / raw) To: Gregory CLEMENT, Ilias Apalodimas Cc: Andrew Lunn, Jason Cooper, Christian Neubert, Antoine Tenart, Stephen Boyd, Mike Turquette, linux-pm, Rafael J. Wysocki, linux-kernel, stable, Maxime Chevallier, Thomas Petazzoni, Miquèl Raynal, linux-clk, linux-arm-kernel, Sebastian Hesselbarth On 22-05-19, 12:36, Viresh Kumar wrote: > On 22-05-19, 10:03, Ilias Apalodimas wrote: > > Hi Viresh, Gregory > > On Mon, May 20, 2019 at 04:50:42PM +0530, Viresh Kumar wrote: > > > On 25-04-19, 15:33, Ilias Apalodimas wrote: > > > > Hi Viresh, > > > > > > > > > > > Also, during this week-end, Christian suggested that the issue might > > > > > > > come from the AVS support. > > > > > > > > > > > > > > Could you disable it and check you still have the issue? > > > > > > > > > > > > > > For this, you just have to remove the avs node in > > > > > > > arch/arm64/boot/dts/marvell/armada-37xx.dtsi and rebuild the dtb. > > > > > > Sure. You'll have to wait for a week though. Currently on a trip. I'll run that > > > > > > once i return > > > > > > > > > > @Ilias: Can you please try this now and confirm to Gregory ? > > > > I am more overloaded than usual and totally forgot about this. Apologies. > > > > I'll try finding some time and do this. > > > > > > Ping Ilias. > > Sorry for the huge delay. > > Applying this patch and removing tha 'avs' node from > > arch/arm64/boot/dts/marvell/armada-37xx.dtsi seems to work. > > Changing between governors does not freeze the board any more. I haven't checked > > the actual impact on the CPU speed but the values on > > /sys/devices/system/cpu/cpufreq/policy0/scaling_governor are correct > > Thanks for testing it out. Lets see what Gregory has to say now. @Gregory: Do you have any further advice for Ilias ? -- viresh _______________________________________________ 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] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks 2019-06-10 10:19 ` Viresh Kumar @ 2019-07-17 10:33 ` Viresh Kumar -1 siblings, 0 replies; 30+ messages in thread From: Viresh Kumar @ 2019-07-17 10:33 UTC (permalink / raw) To: Gregory CLEMENT, Ilias Apalodimas Cc: Christian Neubert, Stephen Boyd, Mike Turquette, linux-clk, linux-kernel, Rafael J. Wysocki, linux-pm, Vincent Guittot, Jason Cooper, Andrew Lunn, Sebastian Hesselbarth, Thomas Petazzoni, linux-arm-kernel, Antoine Tenart, Miquèl Raynal, Maxime Chevallier, stable On 10-06-19, 15:49, Viresh Kumar wrote: > On 22-05-19, 12:36, Viresh Kumar wrote: > > On 22-05-19, 10:03, Ilias Apalodimas wrote: > > > Hi Viresh, Gregory > > > On Mon, May 20, 2019 at 04:50:42PM +0530, Viresh Kumar wrote: > > > > On 25-04-19, 15:33, Ilias Apalodimas wrote: > > > > > Hi Viresh, > > > > > > > > > > > > > Also, during this week-end, Christian suggested that the issue might > > > > > > > > come from the AVS support. > > > > > > > > > > > > > > > > Could you disable it and check you still have the issue? > > > > > > > > > > > > > > > > For this, you just have to remove the avs node in > > > > > > > > arch/arm64/boot/dts/marvell/armada-37xx.dtsi and rebuild the dtb. > > > > > > > Sure. You'll have to wait for a week though. Currently on a trip. I'll run that > > > > > > > once i return > > > > > > > > > > > > @Ilias: Can you please try this now and confirm to Gregory ? > > > > > I am more overloaded than usual and totally forgot about this. Apologies. > > > > > I'll try finding some time and do this. > > > > > > > > Ping Ilias. > > > Sorry for the huge delay. > > > Applying this patch and removing tha 'avs' node from > > > arch/arm64/boot/dts/marvell/armada-37xx.dtsi seems to work. > > > Changing between governors does not freeze the board any more. I haven't checked > > > the actual impact on the CPU speed but the values on > > > /sys/devices/system/cpu/cpufreq/policy0/scaling_governor are correct > > > > Thanks for testing it out. Lets see what Gregory has to say now. > > @Gregory: Do you have any further advice for Ilias ? Ping. -- viresh ^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks @ 2019-07-17 10:33 ` Viresh Kumar 0 siblings, 0 replies; 30+ messages in thread From: Viresh Kumar @ 2019-07-17 10:33 UTC (permalink / raw) To: Gregory CLEMENT, Ilias Apalodimas Cc: Andrew Lunn, Jason Cooper, Christian Neubert, Antoine Tenart, Stephen Boyd, Mike Turquette, linux-pm, Rafael J. Wysocki, linux-kernel, stable, Maxime Chevallier, Thomas Petazzoni, Miquèl Raynal, linux-clk, linux-arm-kernel, Sebastian Hesselbarth On 10-06-19, 15:49, Viresh Kumar wrote: > On 22-05-19, 12:36, Viresh Kumar wrote: > > On 22-05-19, 10:03, Ilias Apalodimas wrote: > > > Hi Viresh, Gregory > > > On Mon, May 20, 2019 at 04:50:42PM +0530, Viresh Kumar wrote: > > > > On 25-04-19, 15:33, Ilias Apalodimas wrote: > > > > > Hi Viresh, > > > > > > > > > > > > > Also, during this week-end, Christian suggested that the issue might > > > > > > > > come from the AVS support. > > > > > > > > > > > > > > > > Could you disable it and check you still have the issue? > > > > > > > > > > > > > > > > For this, you just have to remove the avs node in > > > > > > > > arch/arm64/boot/dts/marvell/armada-37xx.dtsi and rebuild the dtb. > > > > > > > Sure. You'll have to wait for a week though. Currently on a trip. I'll run that > > > > > > > once i return > > > > > > > > > > > > @Ilias: Can you please try this now and confirm to Gregory ? > > > > > I am more overloaded than usual and totally forgot about this. Apologies. > > > > > I'll try finding some time and do this. > > > > > > > > Ping Ilias. > > > Sorry for the huge delay. > > > Applying this patch and removing tha 'avs' node from > > > arch/arm64/boot/dts/marvell/armada-37xx.dtsi seems to work. > > > Changing between governors does not freeze the board any more. I haven't checked > > > the actual impact on the CPU speed but the values on > > > /sys/devices/system/cpu/cpufreq/policy0/scaling_governor are correct > > > > Thanks for testing it out. Lets see what Gregory has to say now. > > @Gregory: Do you have any further advice for Ilias ? Ping. -- viresh _______________________________________________ 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] 30+ messages in thread
end of thread, other threads:[~2019-07-17 10:33 UTC | newest]
Thread overview: 30+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-03-13 16:35 [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks Gregory CLEMENT
2019-03-13 16:35 ` Gregory CLEMENT
2019-03-14 12:15 ` Ilias Apalodimas
2019-03-14 12:15 ` Ilias Apalodimas
[not found] ` <CAC5LXJcCs4nr-qFOWzUJpUBAJ9ngG-cgeTCVCFBKFc1SPzHMuQ@mail.gmail.com>
2019-03-14 13:44 ` Ilias Apalodimas
2019-03-14 13:44 ` Ilias Apalodimas
[not found] ` <CAC5LXJf=f2bZLroSXFkGiF2W=1XzAgb_wwtpvZY7O9xv28wbVA@mail.gmail.com>
2019-03-14 14:15 ` Ilias Apalodimas
2019-03-14 14:15 ` Ilias Apalodimas
2019-03-14 14:20 ` Gregory CLEMENT
2019-03-14 14:20 ` Gregory CLEMENT
2019-03-18 11:28 ` Ilias Apalodimas
2019-03-18 11:28 ` Ilias Apalodimas
2019-03-18 11:40 ` Gregory CLEMENT
2019-03-18 11:40 ` Gregory CLEMENT
2019-03-18 12:21 ` Ilias Apalodimas
2019-03-18 12:21 ` Ilias Apalodimas
2019-04-24 9:30 ` Viresh Kumar
2019-04-24 9:30 ` Viresh Kumar
2019-04-25 12:33 ` Ilias Apalodimas
2019-04-25 12:33 ` Ilias Apalodimas
2019-05-20 11:20 ` Viresh Kumar
2019-05-20 11:20 ` Viresh Kumar
2019-05-22 7:03 ` Ilias Apalodimas
2019-05-22 7:03 ` Ilias Apalodimas
2019-05-22 7:06 ` Viresh Kumar
2019-05-22 7:06 ` Viresh Kumar
2019-06-10 10:19 ` Viresh Kumar
2019-06-10 10:19 ` Viresh Kumar
2019-07-17 10:33 ` Viresh Kumar
2019-07-17 10:33 ` Viresh Kumar
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.