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 hdegoede@redhat.com
Acked-by: Ian Campbell ijc@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.
Ian.