public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] sunxi: Remove Orangepi PC RAM speed to 624 MHz
@ 2015-12-23 10:50 Hans de Goede
  2015-12-24 16:08 ` Ian Campbell
  2015-12-24 16:55 ` Peter Korsgaard
  0 siblings, 2 replies; 4+ messages in thread
From: Hans de Goede @ 2015-12-23 10:50 UTC (permalink / raw)
  To: u-boot

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(-)

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

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH] sunxi: Remove Orangepi PC RAM speed to 624 MHz
  2015-12-23 10:50 [U-Boot] [PATCH] sunxi: Remove Orangepi PC RAM speed to 624 MHz Hans de Goede
@ 2015-12-24 16:08 ` Ian Campbell
  2015-12-27 11:24   ` Hans de Goede
  2015-12-24 16:55 ` Peter Korsgaard
  1 sibling, 1 reply; 4+ messages in thread
From: Ian Campbell @ 2015-12-24 16:08 UTC (permalink / raw)
  To: u-boot

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>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH] sunxi: Remove Orangepi PC RAM speed to 624 MHz
  2015-12-23 10:50 [U-Boot] [PATCH] sunxi: Remove Orangepi PC RAM speed to 624 MHz Hans de Goede
  2015-12-24 16:08 ` Ian Campbell
@ 2015-12-24 16:55 ` Peter Korsgaard
  1 sibling, 0 replies; 4+ messages in thread
From: Peter Korsgaard @ 2015-12-24 16:55 UTC (permalink / raw)
  To: u-boot

>>>>> "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 at lists.denx.de
 > http://lists.denx.de/mailman/listinfo/u-boot


-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH] sunxi: Remove Orangepi PC RAM speed to 624 MHz
  2015-12-24 16:08 ` Ian Campbell
@ 2015-12-27 11:24   ` Hans de Goede
  0 siblings, 0 replies; 4+ messages in thread
From: Hans de Goede @ 2015-12-27 11:24 UTC (permalink / raw)
  To: u-boot

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2015-12-27 11:24 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-12-23 10:50 [U-Boot] [PATCH] sunxi: Remove Orangepi PC RAM speed to 624 MHz Hans de Goede
2015-12-24 16:08 ` Ian Campbell
2015-12-27 11:24   ` Hans de Goede
2015-12-24 16:55 ` Peter Korsgaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox