[U-Boot] at91: smartweb board fails with current HEAD

[Wenyou.Yang at microchip.com]

> -----Original Message-----
> From: Heiko Schocher [mailto:hs at denx.de]
> Sent: 2017年6月20日 22:05
> To: Wenyou Yang - A41535 <Wenyou.Yang at microchip.com>
> Cc: u-boot at lists.denx.de; Andreas Bießmann <biessmann at corscience.de>
> Subject: at91: smartweb board fails with current HEAD
> 
> Hello Wenyou,
> 
> since your patch:
> 
> commit f8b7fff1d5c5297d290acab1e398bd410b0c2e68
> Author: Wenyou Yang <wenyou.yang at atmel.com>
> Date:   Fri Apr 14 15:01:28 2017 +0800
> 
>      serial: atmel_usart: Add clk support
> 
> smartweb board fails booting (and I fear at last all other at91 based siemens
> boards also). Reverting your patch and U-Boot boots fine again.
> 
> First problem is, that I need now a bigger MALLOC_F_LEN space, but the value is
> for SPL and U-Boot ... but in SPL I cannot increase this value, as I have only 4k
> init ram.
> 
> So I need to configure a lower value in SPL as in U-Boot ... this is not possible at
> the moment... or?

After supporting the driver model and device tree, the size of SPL is beyond 4K.
SPL is not suitable in 4K internal SRAM

> 
> Second problem: init of the uart seems to go now wrong, I have to add this patch:
> 
> pollux:u-boot-at91sam9g45 hs [master] $ git diff diff --git
> a/arch/arm/dts/at91sam9260-smartweb.dts b/arch/arm/dts/at91sam9260-
> smartweb.dts
> index faed763..e59781b 100644
> --- a/arch/arm/dts/at91sam9260-smartweb.dts
> +++ b/arch/arm/dts/at91sam9260-smartweb.dts
> @@ -18,6 +18,7 @@
>          compatible = "atmel,at91sam9260", "atmel,at91sam9";
> 
>          chosen {
> +               u-boot,dm-pre-reloc;
>                  stdout-path = &dbgu;
>          };
> 
> @@ -48,6 +49,7 @@
>                          };
> 
>                          dbgu: serial at fffff200 {
> +                               u-boot,dm-pre-reloc;
>                                  status = "okay";
>                          };
> 
> diff --git a/configs/smartweb_defconfig b/configs/smartweb_defconfig index
> 48c8781..a769fc5 100644
> --- a/configs/smartweb_defconfig
> +++ b/configs/smartweb_defconfig
> @@ -7,6 +7,7 @@ CONFIG_TARGET_SMARTWEB=y
>   CONFIG_SPL_GPIO_SUPPORT=y
>   CONFIG_SPL_LIBCOMMON_SUPPORT=y
>   CONFIG_SPL_LIBGENERIC_SUPPORT=y
> +CONFIG_SYS_MALLOC_F_LEN=0x2000
>   CONFIG_SPL_NAND_SUPPORT=y
>   CONFIG_DEFAULT_DEVICE_TREE="at91sam9260-smartweb"
>   CONFIG_FIT=y
> @@ -33,6 +34,8 @@ CONFIG_CMD_FAT=y
>   # CONFIG_DOS_PARTITION is not set
>   CONFIG_OF_CONTROL=y
>   CONFIG_OF_EMBED=y
> +CONFIG_CLK=y
> +CONFIG_CLK_AT91=y
>   CONFIG_DFU_NAND=y
>   # CONFIG_MMC is not set
>   CONFIG_USB=y
> 
> May the other (none atmel) at91 based boards needs similiar fixes?

Yes, needs similar fixes.

> 
> Any hint is welcome.
> 
> bye,
> Heiko
> --
> DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany


Best Regards,
Wenyou Yang