Hi,

On 11/25/2014 09:55 AM, Ian Campbell wrote:
On Sun, 2014-11-23 at 14:43 +0100, Hans de Goede wrote:
It turns out that there is a too large spread between boards to handle this
with a default value, turn this into Kconfig options, and set the values
the factory images are using for the Colombus and Mele_M9 boards.

Note this changes the ZQ default when not overriden through defconfig from
120 to 123, as that is what most boards seem to actually use.

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

Acked-by: Ian Campbell <i...@hellion.org.uk>

With one minor query:

+config DRAM_CLK
+       int "sun6i dram clock speed"
+       default 312
+       ---help---
+       Set the dram clock speed, valid range 240 - 480, must be a multiple
+       of 24.
[...]
diff --git a/configs/Mele_M9_defconfig b/configs/Mele_M9_defconfig
index 40eabce..740b931 100644
--- a/configs/Mele_M9_defconfig
+++ b/configs/Mele_M9_defconfig
@@ -5,6 +5,8 @@ CONFIG_FDTFILE="sun6i-a31-m9.dtb"
  +S:CONFIG_ARCH_SUNXI=y
  +S:CONFIG_MACH_SUN6I=y
  +S:CONFIG_TARGET_MELE_M9=y
++S:CONFIG_DRAM_CLK=312

You are overriding this to be the default, is that deliberate/necessary?

I suspect it's just to keep both or neither of CLK and ZQ explicitly
give, which is a fine reason.

Yes, this is just here to make it easier to see what CLK and ZQ is used without
needing to look in the Kconfig, it is also here to make it a better template
for other sun6i defconfig-s

Regards,

Hans
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to