[U-Boot] [PATCH] board/BuR/common: incorrect check of dtb
Hannes Schmelzer
hannes at schmelzer.or.at
Thu May 4 10:35:37 UTC 2017
On 05/03/2017 11:44 PM, Heinrich Schuchardt wrote:
> The logical expression to check the dtb is incorrect in
> load_devicetree.
>
> The problem was indicated by cppcheck.
>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
> ---
> I do not have a board for testing.
> Please, review carefully.
> ---
> board/BuR/common/common.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/board/BuR/common/common.c b/board/BuR/common/common.c
> index 876150402c..c0316b9ebd 100644
> --- a/board/BuR/common/common.c
> +++ b/board/BuR/common/common.c
> @@ -265,7 +265,7 @@ static int load_devicetree(void)
> char *dtbname = getenv("dtb");
> char *dtbdev = getenv("dtbdev");
> char *dtppart = getenv("dtbpart");
> - if (!dtbdev || !dtbdev || !dtbname) {
> + if (!dtbdev || !dtbpart || !dtbname) {
> printf("%s: <dtbdev>/<dtbpart>/<dtb> missing.\n", __func__);
> return -1;
> }
Reviewed-by: Hannes Schmelzer <oe5hpm at oevsv.at>
Acked-by: Hannes Schmelzer <oe5hpm at oevsv.at>
Thanks!
More information about the U-Boot
mailing list