Hi Cody,
On Sun, Jun 27, 2021 at 2:15 AM Cody Gray cody@codygray.com wrote:
The default implementation of checkboard() calls the nxp_board_rev_string() function to retrieve a character representing the revision number of the board. However, this attempt to retrieve the revision number may fail in certain situations or be otherwise undesirable.
There is already a configuration option to avoid retrieving the revision number of the board: CONFIG_NXP_BOARD_REVISION. In fact, if this option is enabled, the nxp_board_rev_string() function's definition will be omitted entirely, meaning that the previous implementation of checkboard() would result in a linker error.
This changeset makes the default implementation of checkboard() respect the CONFIG_NXP_BOARD_REVISION configuration option, only attempting to retrieve the board revision number if that option is defined.
Signed-off-by: Cody Gray cody@codygray.com
Please fix the checkpatch warnings below and submit a v2, thanks:
WARNING: Possible unwrapped commit description (prefer a maximum 75 chars per line) #72: attempt to retrieve the revision number may fail in certain situations or be
WARNING: Use 'if (IS_ENABLED(CONFIG...))' instead of '#if or #ifdef' where possible #100: FILE: board/freescale/mx6sxsabresd/mx6sxsabresd.c:323: +#ifdef CONFIG_NXP_BOARD_REVISION
WARNING: please, no spaces at the start of a line #103: FILE: board/freescale/mx6sxsabresd/mx6sxsabresd.c:326: + puts("Board: MX6SX SABRE SDB");$