diff options
| author | Philippe Schenker <philippe.schenker@toradex.com> | 2022-06-09 16:58:51 +0200 |
|---|---|---|
| committer | Philippe Schenker <philippe.schenker@toradex.com> | 2022-06-14 10:34:37 +0200 |
| commit | cc59807f744f4c63f1edb9e51a1c2a75f3e53cd8 (patch) | |
| tree | 9a2923a7bc10ceeb32d427ffcc178b36d9860ac8 | |
| parent | 718d0fa76f0adc545cf147f12d794605008dbd6f (diff) | |
toradex: tdx-cfg-block: use defines for string length
[ backport from https://lists.denx.de/pipermail/u-boot/2022-June/486349.html ]
to make it easily extendable and code is implicitly documented
Signed-off-by: Philippe Schenker <philippe.schenker@toradex.com>
| -rw-r--r-- | board/toradex/common/tdx-common.c | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/board/toradex/common/tdx-common.c b/board/toradex/common/tdx-common.c index 46f89b4fec..8f099723e5 100644 --- a/board/toradex/common/tdx-common.c +++ b/board/toradex/common/tdx-common.c @@ -14,13 +14,17 @@ #define TORADEX_OUI 0x00142dUL +#define SERIAL_STR_LEN 8 +#define MODULE_VER_STR_LEN 4 // V1.1 +#define MODULE_REV_STR_LEN 1 // [A-Z] + #ifdef CONFIG_TDX_CFG_BLOCK -static char tdx_serial_str[9]; -static char tdx_board_rev_str[6]; +static char tdx_serial_str[SERIAL_STR_LEN + 1]; +static char tdx_board_rev_str[MODULE_VER_STR_LEN + MODULE_REV_STR_LEN + 1]; #ifdef CONFIG_TDX_CFG_BLOCK_EXTRA -static char tdx_car_serial_str[9]; -static char tdx_car_rev_str[6]; +static char tdx_car_serial_str[SERIAL_STR_LEN + 1]; +static char tdx_car_rev_str[MODULE_VER_STR_LEN + MODULE_REV_STR_LEN + 1]; static char *tdx_carrier_board_name; #endif |