diff mbox

[U-Boot] sunxi: Remove Orangepi PC RAM speed to 624 MHz

Message ID 1450867836-8759-1-git-send-email-hdegoede@redhat.com
State Accepted
Delegated to: Hans de Goede
Headers show

Commit Message

Hans de Goede Dec. 23, 2015, 10:50 a.m. UTC
There are some reports of stability issues at 672 MHz, see:
http://linux-sunxi.org/Orange_Pi_PC#DRAM_clock_speed_limit

So reduce the DRAM speed to 624MHz which seems to be reliable everywhere.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 configs/orangepi_pc_defconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Ian Campbell Dec. 24, 2015, 4:08 p.m. UTC | #1
On Wed, 2015-12-23 at 11:50 +0100, Hans de Goede wrote:
> There are some reports of stability issues at 672 MHz, see:
> http://linux-sunxi.org/Orange_Pi_PC#DRAM_clock_speed_limit

Looks like only one (in that table at least)? And do we expect this
data to remain in the wiki, I suppose so.

In any case, this...
> So reduce the DRAM speed to 624MHz which seems to be reliable
> everywhere.

...seems reasonable to me.

> Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Acked-by: Ian Campbell <ijc@hellion.org.uk>
Peter Korsgaard Dec. 24, 2015, 4:55 p.m. UTC | #2
>>>>> "Hans" == Hans de Goede <hdegoede@redhat.com> writes:

 > There are some reports of stability issues at 672 MHz, see:
 > http://linux-sunxi.org/Orange_Pi_PC#DRAM_clock_speed_limit

 > So reduce the DRAM speed to 624MHz which seems to be reliable everywhere.

With the subject s/Remove/Reduce/

Acked-by: Peter Korsgaard <peter@korsgaard.com>

 > Signed-off-by: Hans de Goede <hdegoede@redhat.com>
 > ---
 >  configs/orangepi_pc_defconfig | 2 +-
 >  1 file changed, 1 insertion(+), 1 deletion(-)

 > diff --git a/configs/orangepi_pc_defconfig b/configs/orangepi_pc_defconfig
 > index 75afca9..358caa5 100644
 > --- a/configs/orangepi_pc_defconfig
 > +++ b/configs/orangepi_pc_defconfig
 > @@ -1,7 +1,7 @@
 >  CONFIG_ARM=y
 >  CONFIG_ARCH_SUNXI=y
 >  CONFIG_MACH_SUN8I_H3=y
 > -CONFIG_DRAM_CLK=672
 > +CONFIG_DRAM_CLK=624
 >  CONFIG_DRAM_ZQ=3881979
 >  CONFIG_DRAM_ODT_EN=y
 >  # CONFIG_VIDEO is not set
 > -- 
 > 2.5.0

 > _______________________________________________
 > U-Boot mailing list
 > U-Boot@lists.denx.de
 > http://lists.denx.de/mailman/listinfo/u-boot
Hans de Goede Dec. 27, 2015, 11:24 a.m. UTC | #3
Hi,

On 24-12-15 17:08, Ian Campbell wrote:
> On Wed, 2015-12-23 at 11:50 +0100, Hans de Goede wrote:
>> There are some reports of stability issues at 672 MHz, see:
>> http://linux-sunxi.org/Orange_Pi_PC#DRAM_clock_speed_limit
>
> Looks like only one (in that table at least)?

Correct, but another 7 out of 13 are only just reliable at
672 and fail one speed step higher, showing that the manufacturer
default of 672 is cutting it awfully close to the failure point,
this feels like the factory default is actually an overclocked
setting.

> And do we expect this
> data to remain in the wiki, I suppose so.

Ack.

> In any case, this...
>> So reduce the DRAM speed to 624MHz which seems to be reliable
>> everywhere.
>
> ...seems reasonable to me.
>
>> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
>
> Acked-by: Ian Campbell <ijc@hellion.org.uk>

Thanks,

Hans
diff mbox

Patch

diff --git a/configs/orangepi_pc_defconfig b/configs/orangepi_pc_defconfig
index 75afca9..358caa5 100644
--- a/configs/orangepi_pc_defconfig
+++ b/configs/orangepi_pc_defconfig
@@ -1,7 +1,7 @@ 
 CONFIG_ARM=y
 CONFIG_ARCH_SUNXI=y
 CONFIG_MACH_SUN8I_H3=y
-CONFIG_DRAM_CLK=672
+CONFIG_DRAM_CLK=624
 CONFIG_DRAM_ZQ=3881979
 CONFIG_DRAM_ODT_EN=y
 # CONFIG_VIDEO is not set