diff options
| author | Angel Pons <th3fanbus@gmail.com> | 2021-05-03 14:01:56 +0200 |
|---|---|---|
| committer | Angel Pons <th3fanbus@gmail.com> | 2021-05-28 11:37:34 +0000 |
| commit | f1e8535794f48148fe93f60bcb2276b6b0e5140e (patch) | |
| tree | fc842fb92e7d87fc87a194706444f869bea2e334 /src/mainboard/prodrive/hermes/eeprom.c | |
| parent | 9bc780fc9fef78f5af8f2d93190ab4c5876cca60 (diff) | |
mb/prodrive/hermes: Simplify `read_write_config` signature
The `write_offset` parameter is always zero. Remove it.
Change-Id: Ib63cb25904ad6c1c7424a9c01d8bf1e84c08453b
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/52884
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/mainboard/prodrive/hermes/eeprom.c')
| -rw-r--r-- | src/mainboard/prodrive/hermes/eeprom.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/src/mainboard/prodrive/hermes/eeprom.c b/src/mainboard/prodrive/hermes/eeprom.c index a712156bc1..614f069c56 100644 --- a/src/mainboard/prodrive/hermes/eeprom.c +++ b/src/mainboard/prodrive/hermes/eeprom.c @@ -21,7 +21,7 @@ int check_signature(const size_t offset, const uint64_t signature) { u8 blob[8] = {0}; - if (!read_write_config(blob, offset, 0, ARRAY_SIZE(blob))) { + if (!read_write_config(blob, offset, ARRAY_SIZE(blob))) { /* Check signature */ if (*(uint64_t *)blob == signature) { printk(BIOS_DEBUG, "CFG EEPROM: Signature valid.\n"); @@ -41,7 +41,7 @@ static bool get_board_settings_from_eeprom(struct eeprom_board_settings *board_c { const size_t board_settings_offset = offsetof(struct eeprom_layout, BoardSettings); - if (read_write_config(board_cfg, board_settings_offset, 0, sizeof(*board_cfg))) { + if (read_write_config(board_cfg, board_settings_offset, sizeof(*board_cfg))) { printk(BIOS_ERR, "CFG EEPROM: Failed to read board settings\n"); return false; } @@ -79,7 +79,7 @@ struct eeprom_bmc_settings *get_bmc_settings(void) static int valid = 0; if (valid == 0) { - if (read_write_config(&bmc_cfg, bmc_settings_offset, 0, sizeof(bmc_cfg))) { + if (read_write_config(&bmc_cfg, bmc_settings_offset, sizeof(bmc_cfg))) { printk(BIOS_ERR, "CFG EEPROM: Failed to read BMC settings\n"); return NULL; } @@ -100,7 +100,7 @@ uint8_t get_bmc_hsi(void) } /* Read data from offset and write it to offset in UPD */ -bool read_write_config(void *blob, size_t read_offset, size_t write_offset, size_t size) +bool read_write_config(void *blob, size_t read_offset, size_t size) { int ret = 0; @@ -120,7 +120,7 @@ bool read_write_config(void *blob, size_t read_offset, size_t write_offset, size break; /* Write to UPD */ - uint8_t *writePointer = (uint8_t *)blob + write_offset + i; + uint8_t *writePointer = (uint8_t *)blob + i; writePointer[0] = tmp[0]; if (size - i > 1) writePointer[1] = tmp[1]; @@ -188,7 +188,7 @@ bool write_board_settings(const struct eeprom_board_layout *new_layout) bool changed = false; /* Read old settings */ - if (read_write_config(&old_layout, off, 0, sizeof(old_layout))) { + if (read_write_config(&old_layout, off, sizeof(old_layout))) { printk(BIOS_ERR, "CFG EEPROM: Read operation failed\n"); return true; } |