On Fri, Jun 05, 2015 at 03:51:11PM +0530, Lokesh Vutla wrote:

> From: Tom Rini <tr...@ti.com>
> 
> The recent changes for hw leveling on am33xx were not intended for
> DDR2 boards, only DDR3. Update emif_sdram_type to take a sdram_config
> value to check against. This lets us pass in the value we would use to
> configure, when we have not yet configured the board yet.  In other cases
> update the call to be as functional as before and check an already
> programmed value in.
> 
> Tested-by: Yan Liu <yan-...@ti.com>
> Signed-off-by: Tom Rini <tr...@ti.com>
> Signed-off-by: Lokesh Vutla <lokeshvu...@ti.com>

OK, I had to re-read this a few times and re-think.  The changes are
fine but the description is a little too vague.  The case that is broken
today is am43xx (which we handle under the am33xx code) and DDR2.
AM335x is still fine befoer and after.  If there's no other comments
I'll re-word things a bit more.

-- 
Tom

Attachment: signature.asc
Description: Digital signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to