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

York Sun york.sun at nxp.com
Sat Apr 15 16:43:11 UTC 2017


On 04/15/2017 06:24 AM, Heinrich Schuchardt 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>
> ---
>  drivers/ddr/fsl/options.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/ddr/fsl/options.c b/drivers/ddr/fsl/options.c
> index d6a8fcb216..cee97fe232 100644
> --- a/drivers/ddr/fsl/options.c
> +++ b/drivers/ddr/fsl/options.c
> @@ -916,7 +916,7 @@ unsigned int populate_memctl_options(const common_timing_params_t *common_dimm,
>  		if ((pdimm[0].data_width >= 64) && \
>  			(pdimm[0].data_width <= 72))
>  			popts->data_bus_width = 0;
> -		else if ((pdimm[0].data_width >= 32) || \
> +		else if ((pdimm[0].data_width >= 32) && \
>  			(pdimm[0].data_width <= 40))
>  			popts->data_bus_width = 1;
>  		else {
>

Thanks for catching this.

Reviewed-by: York Sun <york.sun at nxp.com>


More information about the U-Boot mailing list