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

Dear Fabio Estevam,

In message 1362873704-23598-1-git-send-email-festevam@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