Change in flashrom[master]: ene_lpc.c: Moving register_shutdown at the end of initialisation

[Anastasia Klimchuk (Code Review)]

Anastasia Klimchuk has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/52406 )

Change subject: ene_lpc.c: Moving register_shutdown at the end of initialisation
......................................................................

ene_lpc.c: Moving register_shutdown at the end of initialisation

A bit more details: since register_shutdown has moved a few lines
below, now ene_enter_flash_mode is called before the shutdown function
is registered. ene_enter_flash_mode can fail (at least in theory,
it has some return 1s), but its return value is not analyzed, so even if
it returns non-0, execution goes further and register_shutdown will
happen anyway.

It is a separate question whether the return value of
ene_enter_flash_mode needs to be analyzed or is it ok to ignore it,
but in this patch I plan to keep the same behaviour, and probably I
will get back to error handling later.

This unlocks API change which plans to move register_shutdown inside
register master API, see https://review.coreboot.org/c/flashrom/+/51761

TEST=builds
BUG=b:185191942

Change-Id: If89a758c91c77486adbac2779449bcd71ab8fc78
Signed-off-by: Anastasia Klimchuk 
---
M ene_lpc.c
1 file changed, 4 insertions(+), 5 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/06/52406/1

diff --git a/ene_lpc.c b/ene_lpc.c
index 56d6580..61f27b4 100644
--- a/ene_lpc.c
+++ b/ene_lpc.c
@@ -573,16 +573,15 @@
     * Compal - ec_command(0x41, 0xa1) returns 43 4f 4d 50 41 4c 9c
     */

+	ene_enter_flash_mode(ctx_data);
+
+	internal_buses_supported |= BUS_LPC;
+	spi_master_ene.data = ctx_data;

if (register_shutdown(ene_leave_flash_mode, ctx_data)) {
    	ret = 1;
    	goto ene_probe_spi_flash_exit;
    }
-
-	ene_enter_flash_mode(ctx_data);
-
-	internal_buses_supported |= BUS_LPC;
-	spi_master_ene.data = ctx_data;
    register_spi_master(&spi_master_ene);
    msg_pdbg("%s: successfully initialized ene\n", __func__);