From e7ff5f51fefa782edbe168ff339aa99e9b41c965 Mon Sep 17 00:00:00 2001 From: Edward O'Callaghan Date: Mon, 6 Mar 2023 09:01:54 +1100 Subject: board_enable.c: Clean up board_handle_phase() The board_enable_safetycheck() call already checks nullarity of the function pointer. Change-Id: I956961ee7204d3a6a9066ba5945f95af1411e700 Signed-off-by: Edward O'Callaghan Reviewed-on: https://review.coreboot.org/c/flashrom/+/73453 Reviewed-by: Thomas Heijligen Tested-by: build bot (Jenkins) --- board_enable.c | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/board_enable.c b/board_enable.c index 0531fab2..bf7897ba 100644 --- a/board_enable.c +++ b/board_enable.c @@ -2707,12 +2707,7 @@ static int board_enable_safetycheck(const struct board_match *board, bool force_ /* FIXME: Should this be identical to board_flash_enable? */ static int board_handle_phase(enum board_match_phase phase, bool force_boardenable) { - const struct board_match *board = NULL; - - board = board_match_pci_ids(phase); - - if (!board) - return 0; + const struct board_match *board = board_match_pci_ids(phase); if (board_enable_safetycheck(board, force_boardenable)) return 0; -- cgit v1.2.1