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

Marek Vasut
2014-08-05T07:58:10Z

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