Nico Huber submitted this change.

View Change

Approvals: build bot (Jenkins): Verified Edward O'Callaghan: Looks good to me, approved
opaque_master: Move shutdown function above opaque_master struct

This patch prepares opaque masters to use new API which allows to
register shutdown function in opaque_master struct. See also later
patch in this chain, where opaque masters are converted to new API.

BUG=b:185191942
TEST=builds and ninja test
Comparing flashrom binary before and after the patch,
make clean && make CONFIG_EVERYTHING=yes VERSION=none
binary is the same

Change-Id: I5000cfceeba859a76177a17c1cb7d1c1e9fc03fe
Signed-off-by: Anastasia Klimchuk <aklm@chromium.org>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/56824
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Edward O'Callaghan <quasisec@chromium.org>
---
M linux_mtd.c
M nicintel_eeprom.c
2 files changed, 25 insertions(+), 25 deletions(-)

diff --git a/linux_mtd.c b/linux_mtd.c
index 7780feb..42d569d 100644
--- a/linux_mtd.c
+++ b/linux_mtd.c
@@ -295,6 +295,17 @@
return 0;
}

+static int linux_mtd_shutdown(void *data)
+{
+ struct linux_mtd_data *mtd_data = data;
+ if (mtd_data->dev_fp != NULL) {
+ fclose(mtd_data->dev_fp);
+ }
+ free(data);
+
+ return 0;
+}
+
static const struct opaque_master linux_mtd_opaque_master = {
/* max_data_{read,write} don't have any effect for this programmer */
.max_data_read = MAX_DATA_UNSPECIFIED,
@@ -356,17 +367,6 @@
return ret;
}

-static int linux_mtd_shutdown(void *data)
-{
- struct linux_mtd_data *mtd_data = data;
- if (mtd_data->dev_fp != NULL) {
- fclose(mtd_data->dev_fp);
- }
- free(data);
-
- return 0;
-}
-
static int linux_mtd_init(void)
{
char *param;
diff --git a/nicintel_eeprom.c b/nicintel_eeprom.c
index ec0f04e..fd90250 100644
--- a/nicintel_eeprom.c
+++ b/nicintel_eeprom.c
@@ -395,20 +395,6 @@
return nicintel_ee_write_82580(flash, NULL, addr, len);
}

-static const struct opaque_master opaque_master_nicintel_ee_82580 = {
- .probe = nicintel_ee_probe_82580,
- .read = nicintel_ee_read,
- .write = nicintel_ee_write_82580,
- .erase = nicintel_ee_erase_82580,
-};
-
-static const struct opaque_master opaque_master_nicintel_ee_i210 = {
- .probe = nicintel_ee_probe_i210,
- .read = nicintel_ee_read,
- .write = nicintel_ee_write_i210,
- .erase = nicintel_ee_erase_i210,
-};
-
static int nicintel_ee_shutdown_i210(void *arg)
{
if (!done_i20_write)
@@ -455,6 +441,20 @@
return ret;
}

+static const struct opaque_master opaque_master_nicintel_ee_82580 = {
+ .probe = nicintel_ee_probe_82580,
+ .read = nicintel_ee_read,
+ .write = nicintel_ee_write_82580,
+ .erase = nicintel_ee_erase_82580,
+};
+
+static const struct opaque_master opaque_master_nicintel_ee_i210 = {
+ .probe = nicintel_ee_probe_i210,
+ .read = nicintel_ee_read,
+ .write = nicintel_ee_write_i210,
+ .erase = nicintel_ee_erase_i210,
+};
+
static int nicintel_ee_init(void)
{
if (rget_io_perms())

To view, visit change 56824. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I5000cfceeba859a76177a17c1cb7d1c1e9fc03fe
Gerrit-Change-Number: 56824
Gerrit-PatchSet: 2
Gerrit-Owner: Anastasia Klimchuk <aklm@chromium.org>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@mailbox.org>
Gerrit-MessageType: merged