On 2016-09-15, Sandy Patterson wrote:
An alternative for you might be to disable OF_PLATDATA for firefly and enable BACK_TO_BROM.
I wasn't able to successfully build with OF_PLATDATA disabled; sounds like there are more patches needed for firefly for that to work.
You may not be getting any output because you didn't modify your burning process. See the README.rockchip section on CONFIG_ROCKCHIP_SPL_BACK_TO_BROM for details, but you have to dd your images slightly differently with this macro enabled.
Thanks! Unfortunately, even after using the burning process for CONFIG_ROCKCHIP_SPL_BACK_TO_BROM described in README.rockchip.
The patches (on top of what was submitted in this series) I've applied to v2016.09 are:
Index: u-boot/include/common.h =================================================================== --- u-boot.orig/include/common.h +++ u-boot/include/common.h @@ -95,6 +95,10 @@ typedef volatile unsigned char vu_char; #define CONFIG_SYS_SUPPORT_64BIT_DATA #endif
+#ifndef CONFIG_SPL_BUILD +#define DEBUG +#endif + #ifdef DEBUG #define _DEBUG 1 #else
Index: u-boot/arch/arm/mach-rockchip/rk3288/sdram_rk3288.c =================================================================== --- u-boot.orig/arch/arm/mach-rockchip/rk3288/sdram_rk3288.c +++ u-boot/arch/arm/mach-rockchip/rk3288/sdram_rk3288.c @@ -1039,7 +1039,7 @@ static int conv_of_platdata(struct udevi memcpy(&plat->phy_timing, of_plat->rockchip_phy_timing, sizeof(plat->phy_timing)); memcpy(&plat->base, of_plat->rockchip_sdram_params, sizeof(plat->base)); - plat->num_channels = of_plat->rockchip_num_channels; + plat->num_channels = 2; ret = regmap_init_mem_platdata(dev, of_plat->reg, ARRAY_SIZE(of_plat->reg) / 2, &plat->map);
Index: u-boot/configs/firefly-rk3288_defconfig =================================================================== --- u-boot.orig/configs/firefly-rk3288_defconfig +++ u-boot/configs/firefly-rk3288_defconfig @@ -70,3 +70,4 @@ CONFIG_USE_TINY_PRINTF=y CONFIG_CMD_DHRYSTONE=y CONFIG_ERRNO_STR=y # CONFIG_SPL_OF_LIBFDT is not set +CONFIG_ROCKCHIP_SPL_BACK_TO_BROM=y
I think I'll wait for an updated series to do more tests, so I'm not testing patches on patches on patches. Also reduces the liklihood of me misinterpreting what people are suggesting to try.
live well, vagrant