diff options
author | Angel Pons <th3fanbus@gmail.com> | 2021-05-02 18:56:45 +0200 |
---|---|---|
committer | Edward O'Callaghan <quasisec@chromium.org> | 2021-05-09 03:34:05 +0000 |
commit | db23295f8d2a1ecc08f279c3f295558cc55fbeb5 (patch) | |
tree | 979f4304b82dba5a1d1650fa4ae30dbf260385c4 /lspcon_i2c_spi.c | |
parent | c7dd17062c1c0936d649f285c0013bb4a397e41c (diff) | |
download | flashrom-db23295f8d2a1ecc08f279c3f295558cc55fbeb5.tar.gz flashrom-db23295f8d2a1ecc08f279c3f295558cc55fbeb5.tar.bz2 flashrom-db23295f8d2a1ecc08f279c3f295558cc55fbeb5.zip |
lspcon_i2c_spi: Extract I2C bus parameter handling
Introduce the `i2c_open_from_programmer_params` function to avoid having
to duplicate parameter parsing code on all I2C programmers. This also
allows having the same programmer parameters on all I2C programmers.
Change-Id: I006b311c88feea37fe4b217f769b21ca1505def9
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/52830
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Edward O'Callaghan <quasisec@chromium.org>
Diffstat (limited to 'lspcon_i2c_spi.c')
-rw-r--r-- | lspcon_i2c_spi.c | 62 |
1 files changed, 1 insertions, 61 deletions
diff --git a/lspcon_i2c_spi.c b/lspcon_i2c_spi.c index e2c4a3c35..096fb7620 100644 --- a/lspcon_i2c_spi.c +++ b/lspcon_i2c_spi.c @@ -433,69 +433,9 @@ static int lspcon_i2c_spi_shutdown(void *data) return ret; } -/* TODO: remove this out of the specific SPI master implementation. */ -static int get_bus_number(void) -{ - char *bus_str = extract_programmer_param("bus"); - /* Return INVALID_ADDRESS if bus value was given but invalid, and GENERIC_ERROR - * if no value was provided. */ - int ret = SPI_INVALID_ADDRESS; - - if (bus_str == NULL) - return SPI_GENERIC_ERROR; - - char *bus_suffix; - errno = 0; - int bus = (int)strtol(bus_str, &bus_suffix, 10); - if (errno != 0 || bus_str == bus_suffix) { - msg_perr("%s: Could not convert 'bus'.\n", __func__); - goto get_bus_done; - } - - if (bus < 0 || bus > 255) { - msg_perr("%s: Value for 'bus' is out of range(0-255).\n", __func__); - goto get_bus_done; - } - - if (strlen(bus_suffix) > 0) { - msg_perr("%s: Garbage following 'bus' value.\n", __func__); - goto get_bus_done; - } - - msg_pinfo("Using i2c bus %i.\n", bus); - ret = bus; - -get_bus_done: - free(bus_str); - return ret; -} - int lspcon_i2c_spi_init(void) { - int fd = -1; - int bus_number = get_bus_number(); - if (bus_number == SPI_INVALID_ADDRESS) { - /* Bus was specified but unusable, bail out immediately */ - return SPI_GENERIC_ERROR; - } - char *device_path = extract_programmer_param("devpath"); - - if (device_path != NULL && bus_number >= 0) { - msg_perr("%s: only one of bus and devpath may be specified\n", __func__); - free(device_path); - return SPI_GENERIC_ERROR; - } else if (device_path == NULL && bus_number < 0) { - msg_perr("%s: one of bus and devpath must be specified\n", __func__); - return SPI_GENERIC_ERROR; - } - - if (device_path != NULL) { - fd = i2c_open_path(device_path, REGISTER_ADDRESS, 0); - free(device_path); - } else { - fd = i2c_open(bus_number, REGISTER_ADDRESS, 0); - } - + int fd = i2c_open_from_programmer_params(REGISTER_ADDRESS, 0); if (fd < 0) return fd; |