[U-Boot] [PATCH 12/18] arm: mx6: add support for Compulab cm-fx6 CoM

Marek Vasut marex at denx.de
Tue Aug 5 09:58:10 CEST 2014


On Tuesday, August 05, 2014 at 09:36:27 AM, Nikita Kiryanov wrote:

[...]

> >>> imx_ddr_size() won't cut it here ?
> >> 
> >> It doesn't handle 4GB correctly (returns 0). I suppose
> >> I can make a patch which caps the return value of
> >> imx_ddr_size() for MX6 socs to 3840MB.
> >> What do you think?
> > 
> > That you should check the U-Boot ML, since that's what I did yesterday
> > ;-) But still, this is rather sad practice -- instead of fixing a bug in
> > code which you do know about, you implement such a workaround :-(
> 
> Actually, I only learned of this bug yesterday after you asked about
> imx_ddr_size(). Glad to hear you already fixed it; I'll look at it for
> the v2.

I did the capping, yep. HTH :)

Best regards,
Marek Vasut


More information about the U-Boot mailing list