Re: [PATCH 12/13] clk: mvebu: add support for 1866MHz variants

[Ralph Sennhauser]

On Tue, 23 May 2017 14:15:14 +0200
Andrew Lunn  wrote:

> On Sun, May 21, 2017 at 02:49:01PM +0200, Ralph Sennhauser wrote:
> > The Linksys WRT3200ACM CPU is clocked at 1866MHz. Add 1866MHz to
> > the list of supported CPU frequencies.
> > 
> > Noticed by the following warning:
> > [0.00] Selected CPU frequency (16) unsupported  
> 
> Are the L2 and DDR clocks using sensible values?
>

Hi Andrew

As you expected more changes are needed to fully support the new
CPU frequency. Will send out a V2 later.

Thanks for catching this.
Ralph

Re: [PATCH 12/13] clk: mvebu: add support for 1866MHz variants

[Andrew Lunn]

On Sun, May 21, 2017 at 02:49:01PM +0200, Ralph Sennhauser wrote:
> The Linksys WRT3200ACM CPU is clocked at 1866MHz. Add 1866MHz to
> the list of supported CPU frequencies.
> 
> Noticed by the following warning:
> [0.00] Selected CPU frequency (16) unsupported

Are the L2 and DDR clocks using sensible values?

Andrew

[PATCH 12/13] clk: mvebu: add support for 1866MHz variants

[Ralph Sennhauser]

The Linksys WRT3200ACM CPU is clocked at 1866MHz. Add 1866MHz to
the list of supported CPU frequencies.

Noticed by the following warning:
[0.00] Selected CPU frequency (16) unsupported

Signed-off-by: Ralph Sennhauser 
---
 drivers/clk/mvebu/armada-38x.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/clk/mvebu/armada-38x.c b/drivers/clk/mvebu/armada-38x.c
index 8bccf4e..6a92eb6 100644
--- a/drivers/clk/mvebu/armada-38x.c
+++ b/drivers/clk/mvebu/armada-38x.c
@@ -49,7 +49,8 @@ static const u32 armada_38x_cpu_frequencies[] __initconst = {
0, 0, 0, 0,
1066 * 1000 * 1000, 0, 0, 0,
1332 * 1000 * 1000, 0, 0, 0,
-   1600 * 1000 * 1000,
+   1600 * 1000 * 1000, 0, 0, 0,
+   1866 * 1000 * 1000
 };
 
 static u32 __init armada_38x_get_cpu_freq(void __iomem *sar)
-- 
2.10.2