Change in flashrom[master]: nicrealtek.c: Refactor singleton states into reentrant pattern

[Nico Huber (Code Review)]

Nico Huber has submitted this change. ( https://review.coreboot.org/c/flashrom/+/55108 )

Change subject: nicrealtek.c: Refactor singleton states into reentrant pattern
......................................................................

nicrealtek.c: Refactor singleton states into reentrant pattern

Move global singleton states into a struct and store within
the par_master data field for the life-time of the driver.

This is one of the steps on the way to move par_master data
memory management behind the initialisation API, for more
context see other patches under the same topic "register_master_api".

BUG=b:185191942
TEST=builds

Change-Id: If0cb0fefb53b2c6bb65a85f4c8dc6f323954dd0c
Signed-off-by: Anastasia Klimchuk 
Reviewed-on: https://review.coreboot.org/c/flashrom/+/55108
Tested-by: build bot (Jenkins) 
Reviewed-by: Edward O'Callaghan 
Reviewed-by: Nico Huber 
---
M nicrealtek.c
1 file changed, 31 insertions(+), 10 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Nico Huber: Looks good to me, but someone else must approve
  Edward O'Callaghan: Looks good to me, approved

diff --git a/nicrealtek.c b/nicrealtek.c
index 53d3c3d..81d305b 100644
--- a/nicrealtek.c
+++ b/nicrealtek.c
@@ -24,8 +24,11 @@
 #define PCI_VENDOR_ID_REALTEK	0x10ec
 #define PCI_VENDOR_ID_SMC1211	0x1113

-static uint32_t io_base_addr = 0;
-static int bios_rom_addr, bios_rom_data;
+struct nicrealtek_data {
+	uint32_t io_base_addr;
+	int bios_rom_addr;
+	int bios_rom_data;
+};

static const struct dev_entry nics_realtek[] = {
    {0x10ec, 0x8139, OK, "Realtek", "RTL8139/8139C/8139C+"},
@@ -37,38 +40,41 @@

static void nicrealtek_chip_writeb(const struct flashctx *flash, uint8_t val, chipaddr addr)
 {
+	struct nicrealtek_data *data = flash->mst->par.data;
+
    /* Output addr and data, set WE to 0, set OE to 1, set CS to 0,
     * enable software access.
     */
    OUTL(((uint32_t)addr & 0x01FFFF) | 0x0A0000 | (val << 24),
-	     io_base_addr + bios_rom_addr);
+	     data->io_base_addr + data->bios_rom_addr);
    /* Output addr and data, set WE to 1, set OE to 1, set CS to 1,
     * enable software access.
     */
    OUTL(((uint32_t)addr & 0x01FFFF) | 0x1E0000 | (val << 24),
-	     io_base_addr + bios_rom_addr);
+	     data->io_base_addr + data->bios_rom_addr);
 }

static uint8_t nicrealtek_chip_readb(const struct flashctx *flash, const chipaddr addr)
 {
+	struct nicrealtek_data *data = flash->mst->par.data;
    uint8_t val;

/* FIXME: Can we skip reading the old data and simply use 0? */
    /* Read old data. */
-	val = INB(io_base_addr + bios_rom_data);
+	val = INB(data->io_base_addr + data->bios_rom_data);
    /* Output new addr and old data, set WE to 1, set OE to 0, set CS to 0,
     * enable software access.
     */
    OUTL(((uint32_t)addr & 0x01FFFF) | 0x060000 | (val << 24),
-	     io_base_addr + bios_rom_addr);
+	     data->io_base_addr + data->bios_rom_addr);

/* Read new data. */
-	val = INB(io_base_addr + bios_rom_data);
+	val = INB(data->io_base_addr + data->bios_rom_data);
    /* Output addr and new data, set WE to 1, set OE to 1, set CS to 1,
     * enable software access.
     */
    OUTL(((uint32_t)addr & 0x01FFFF) | 0x1E0000 | (val << 24),
-	     io_base_addr + bios_rom_addr);
+	     data->io_base_addr + data->bios_rom_addr);

return val;
 }
@@ -87,12 +93,16 @@
 static int nicrealtek_shutdown(void *data)
 {
    /* FIXME: We forgot to disable software access again. */
+	free(data);
    return 0;
 }

static int nicrealtek_init(void)
 {
    struct pci_dev *dev = NULL;
+	uint32_t io_base_addr = 0;
+	int bios_rom_addr;
+	int bios_rom_data;

if (rget_io_perms())
    	return 1;
@@ -119,10 +129,21 @@
    	break;
    }

-	if (register_shutdown(nicrealtek_shutdown, NULL))
+	struct nicrealtek_data *data = calloc(1, sizeof(*data));
+	if (!data) {
+		msg_perr("Unable to allocate space for PAR master data\n");
    	return 1;
+	}
+	data->io_base_addr = io_base_addr;
+	data->bios_rom_addr = bios_rom_addr;
+	data->bios_rom_data = bios_rom_data;

-	register_par_master(&par_master_nicrealtek, BUS_PARALLEL, NULL);
+	if (register_shutdown(nicrealtek_shutdown, data)) {
+		free(data);
+		return 1;
+	}
+
+	register_par_master(&par_master_nicrealtek, BUS_PARALLEL, data);

return 0;
 }