[U-Boot] ddr: fsl: incorrect logical constraint in populate_memctl_options

Tom Rini trini at konsulko.com
Wed Apr 19 13:04:08 UTC 2017


On Sat, Apr 15, 2017 at 03:23:49PM +0200, xypron.glpk at gmx.de wrote:

> (pdimm[0].data_width >= 32) || (pdimm[0].data_width <= 40)
> is always true.
> 
> We should use && here.
> 
> The problem was indicated by cppcheck.
> 
> Signed-off-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
> Reviewed-by: Tom Rini <trini at konsulko.com>
> Reviewed-by: York Sun <york.sun at nxp.com>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170419/729cd3a2/attachment.sig>


More information about the U-Boot mailing list