Change in flashrom[master]: ene_lpc.c: Extract params check into a function

[Anastasia Klimchuk (Code Review)]

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

Change subject: ene_lpc.c: Extract params check into a function
......................................................................

ene_lpc.c: Extract params check into a function

This allows char *p to become a local variable in check_params,
and it is allocated and freed within check_params function.
Which means init function does not need char *p anymore,
in particular does not need to free it - and this makes cleanup
after failed init easier.

As a good side effect, init function becomes easier to read.

BUG=b:185191942
TEST=builds

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

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/18/52718/1

diff --git a/ene_lpc.c b/ene_lpc.c
index 1d045b8..c05fef8 100644
--- a/ene_lpc.c
+++ b/ene_lpc.c
@@ -513,11 +513,23 @@
    .write_256 = default_spi_write_256,
 };

+static int check_params(void)
+{
+	int ret = 0;
+	char *const p = extract_programmer_param("type");
+	if (p && strcmp(p, "ec")) {
+		msg_pdbg("ene_lpc only supports "ec" type devices\n");
+		ret = 1;
+	}
+
+	free(p);
+	return ret;
+}
+
 int ene_lpc_init()
 {
    uint8_t hwver, ediid, i;
    int ret = 0;
-	char *p = NULL;
    ene_lpc_data_t *ctx_data = NULL;

msg_pdbg("%s\n", __func__);
@@ -529,9 +541,7 @@
    }
    ctx_data->ec_state = EC_STATE_NORMAL;

-	p = extract_programmer_param("type");
-	if (p && strcmp(p, "ec")) {
-		msg_pdbg("ene_lpc only supports "ec" type devices\n");
+	if (check_params()) {
    	ret = 1;
    	goto ene_probe_spi_flash_exit;
    }
@@ -572,7 +582,6 @@
    msg_pdbg("%s: successfully initialized ene\n", __func__);

ene_probe_spi_flash_exit:
-	free(p);
    if (ret)
    	free(ctx_data);
    return ret;