tdx-cfg-block: fix off by one issue
authorMarcel Ziswiler <marcel.ziswiler@toradex.com>
Mon, 25 Mar 2019 16:18:29 +0000 (17:18 +0100)
committerStefano Babic <sbabic@denx.de>
Sat, 13 Apr 2019 18:30:10 +0000 (20:30 +0200)
Fix toradex_modules array off by one issue potentially leading to
spurious printout during boot e.g.

    Model: Toradex  V1.2A,

instead of

    Model: Toradex UNKNOWN MODULE V1.2A.

Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
Acked-by: Max Krummenacher <max.krummenacher@toradex.com>
Reviewed-by: Igor Opaniuk <igor.opaniuk@toradex.com>
board/toradex/common/tdx-cfg-block.c

index 2fcb998..b90077b 100644 (file)
@@ -261,7 +261,7 @@ int read_tdx_cfg_block(void)
        }
 
        /* Cap product id to avoid issues with a yet unknown one */
-       if (tdx_hw_tag.prodid > (sizeof(toradex_modules) /
+       if (tdx_hw_tag.prodid >= (sizeof(toradex_modules) /
                                  sizeof(toradex_modules[0])))
                tdx_hw_tag.prodid = 0;