diff options
| author | Edward O'Callaghan <quasisec@google.com> | 2023-03-06 09:01:54 +1100 |
|---|---|---|
| committer | Edward O'Callaghan <quasisec@chromium.org> | 2023-03-28 00:43:01 +0000 |
| commit | e7ff5f51fefa782edbe168ff339aa99e9b41c965 (patch) | |
| tree | 5b3dccee88a0e8de175d105d49aeda8de9f21dca | |
| parent | 50f812cfc705743a46132f54057b42e71d0f6350 (diff) | |
| download | flashrom-git-e7ff5f51fefa782edbe168ff339aa99e9b41c965.tar.gz | |
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 <quasisec@google.com>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/73453
Reviewed-by: Thomas Heijligen <src@posteo.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
| -rw-r--r-- | board_enable.c | 7 |
1 files changed, 1 insertions, 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; |