Skip to contentSkip to conversation

ARM: at91: clock: fix PLLA overclocked warning

PATCH New Jiri Prchal 2 participants 2 messages Jul 9, 2012

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

PATCH 2/2 ARM: at91: clock: fix PLLA overclocked warning

This patch fix false warning "PLLA overclocked". Maximum frequency for sam9260 is 210MHz as stated in atmel's datasheet. Originally there was plus 4.5%, so I add this 4.5% to new value too.

---
 arch/arm/mach-at91/clock.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
arch/arm/mach-at91/clock.c +1 -1
index de2ec6b..e004308 100644
@@ -707,7 +707,7 @@ static int __init at91_pmc_init(unsigned long main_clock)
 		if (plla.rate_hz > 800000000)
 			pll_overclock = true;
 	} else {
-		if (plla.rate_hz > 209000000)
+		if (plla.rate_hz > 219000000)
 			pll_overclock = true;
 	}
 	if (pll_overclock)
Signature · 1 line
1.7.7
#1 lore raw

PATCH 2/2 ARM: at91: clock: fix PLLA overclocked warning

Hi Jiri,

On 07/09/2012 11:16 AM, Jiri Prchal :

Quoted 3 lines

This patch fix false warning "PLLA overclocked". Maximum frequency for sam9260 is 210MHz as stated in atmel's datasheet. Originally there was plus 4.5%, so I add this 4.5% to new value too.

Well, I do not see this 4.5% in addition. It is true that Maximum frequency had been indicated as higher but current value is 210MHz if VDD core is @ 1.8V (for industrial temperature range).

Quoted 10 lines
---
 arch/arm/mach-at91/clock.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
arch/arm/mach-at91/clock.c +0 -0
index de2ec6b..e004308 100644

arch/arm/mach-at91/clock.c static int __init at91_pmc_init(unsigned long main_clock) · line 710

Quoted 5 lines
arch/arm/mach-at91/clock.c +1 -1
@@ -707,7 +707,7 @@ static int __init at91_pmc_init(unsigned long main_clock)
 		if (plla.rate_hz > 800000000)
 			pll_overclock = true;
 	} else {
-		if (plla.rate_hz > 209000000)
+		if (plla.rate_hz > 219000000)

Well, I rework this patch using 210 MHz as stated in datasheet. We shall keep this 209 MHz test because at91rm9200 is still using this value. Moreover, I add a new category for 240 MHz.

arch/arm/mach-at91/clock.c static int __init at91_pmc_init(unsigned long main_clock) · line 713

Quoted 3 lines
arch/arm/mach-at91/clock.c +0 -0
 			pll_overclock = true;
 	}
 	if (pll_overclock)

Thanks, best regards,

Signature · 1 line
Nicolas Ferre