diff options
author | Alexander Goncharov <chat@joursoir.net> | 2022-06-24 16:58:16 +0300 |
---|---|---|
committer | Anastasia Klimchuk <aklm@chromium.org> | 2022-08-09 23:19:15 +0000 |
commit | 02f43e89bae6bcdd7505af0ccd361e6eb3d3c74c (patch) | |
tree | a077f609bbc914d96d19c689a4ff64617be09c77 | |
parent | db026ea281bc36972c3a4fdc6009407180bf28d3 (diff) | |
download | flashrom-02f43e89bae6bcdd7505af0ccd361e6eb3d3c74c.tar.gz flashrom-02f43e89bae6bcdd7505af0ccd361e6eb3d3c74c.tar.bz2 flashrom-02f43e89bae6bcdd7505af0ccd361e6eb3d3c74c.zip |
atahpt: restore flash access state explicitly
Instead of using reversible write (rpci_write_long) that relies on
global state, do it manually. Save original PCI config space
register contents to programmer's structure during initialization
and restore it in programmer's shutdown.
TOPIC=reduce_global_pci_state
TEST=builds
Change-Id: I9996bb4d71801034e66ba0c233846e19fa29224d
Signed-off-by: Alexander Goncharov <chat@joursoir.net>
Ticket: https://ticket.coreboot.org/issues/389
Reviewed-on: https://review.coreboot.org/c/flashrom/+/65386
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Felix Singer <felixsinger@posteo.net>
Reviewed-by: Anastasia Klimchuk <aklm@chromium.org>
Reviewed-by: Thomas Heijligen <src@posteo.de>
-rw-r--r-- | atahpt.c | 19 |
1 files changed, 13 insertions, 6 deletions
@@ -25,11 +25,14 @@ #define BIOS_ROM_DATA 0x94 #define REG_FLASH_ACCESS 0x58 +#define BIT_FLASH_ACCESS BIT(24) #define PCI_VENDOR_ID_HPT 0x1103 struct atahpt_data { + struct pci_dev *dev; uint32_t io_base_addr; + uint32_t flash_access; }; static const struct dev_entry ata_hpt[] = { @@ -60,6 +63,11 @@ static uint8_t atahpt_chip_readb(const struct flashctx *flash, static int atahpt_shutdown(void *par_data) { + struct atahpt_data *data = par_data; + + /* Restore original flash access state. */ + pci_write_long(data->dev, REG_FLASH_ACCESS, data->flash_access); + free(par_data); return 0; } @@ -80,7 +88,6 @@ static int atahpt_init(void) { struct pci_dev *dev = NULL; uint32_t io_base_addr; - uint32_t reg32; if (rget_io_perms()) return 1; @@ -93,18 +100,18 @@ static int atahpt_init(void) if (!io_base_addr) return 1; - /* Enable flash access. */ - reg32 = pci_read_long(dev, REG_FLASH_ACCESS); - reg32 |= (1 << 24); - rpci_write_long(dev, REG_FLASH_ACCESS, reg32); - struct atahpt_data *data = calloc(1, sizeof(*data)); if (!data) { msg_perr("Unable to allocate space for PAR master data\n"); return 1; } + data->dev = dev; data->io_base_addr = io_base_addr; + /* Enable flash access. */ + data->flash_access = pci_read_long(dev, REG_FLASH_ACCESS); + pci_write_long(dev, REG_FLASH_ACCESS, data->flash_access | BIT_FLASH_ACCESS); + return register_par_master(&par_master_atahpt, BUS_PARALLEL, data); } |