[U-Boot] [PATCH v2] mx6dl: Add initial support for Wandboard dual lite

Wolfgang Denk wd at denx.de
Sun Mar 10 08:51:40 CET 2013


Dear Fabio Estevam,

In message <1362873704-23598-1-git-send-email-festevam at gmail.com> you wrote:
> 
> Add initial support for Wandboard dual lite.
...

> Changes since v1:
> - Use get_ram_size
> - Remove ARP_TIMEOUT
> - Remove unneeded "1u"

Thanks.  I have still a question:

> +int dram_init(void)
> +{
> +	gd->ram_size = get_ram_size((void *)PHYS_SDRAM, PHYS_SDRAM_SIZE);
> +
> +	return 0;
> +}

I see this new code does not reference imx_ddr_size() any more.  If
it's so easy to switch to get_ram_size(), should we not then also
convert the existing MX53 and MX6 boards to using get_ram_size(), and
remove the then unneeded imx_ddr_size() code?

Or would there be any drawbacks?

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
As far as the laws of mathematics refer to reality, they are not cer-
tain, and as far as they are certain, they do not refer  to  reality.
                                                   -- Albert Einstein


More information about the U-Boot mailing list