mirror of
https://review.coreboot.org/flashrom.git
synced 2025-04-27 15:12:36 +02:00
realtek_mst_i2c_spi: Use i2c_open_from_programmer_params
This allows using `buspath` to specify which I2C device to use. Change-Id: Ibdf07a9fde0ddfcda1c0bfa35a3e7cde5c22cedb Signed-off-by: Angel Pons <th3fanbus@gmail.com> Reviewed-on: https://review.coreboot.org/c/flashrom/+/52831 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Edward O'Callaghan <quasisec@chromium.org> Reviewed-by: Peter Marheine <pmarheine@chromium.org>
This commit is contained in:
parent
db23295f8d
commit
b81dbc5233
@ -442,38 +442,11 @@ static int realtek_mst_i2c_spi_shutdown(void *data)
|
||||
return ret;
|
||||
}
|
||||
|
||||
static int get_params(int *i2c_bus, int *reset, int *enter_isp)
|
||||
static int get_params(int *reset, int *enter_isp)
|
||||
{
|
||||
char *bus_str = NULL, *reset_str = NULL, *isp_str = NULL;
|
||||
char *reset_str = NULL, *isp_str = NULL;
|
||||
int ret = SPI_GENERIC_ERROR;
|
||||
|
||||
bus_str = extract_programmer_param("bus");
|
||||
if (bus_str) {
|
||||
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_params_failed;
|
||||
}
|
||||
|
||||
if (bus < 0 || bus > 255) {
|
||||
msg_perr("%s: Value for 'bus' is out of range(0-255).\n", __func__);
|
||||
goto _get_params_failed;
|
||||
}
|
||||
|
||||
if (strlen(bus_suffix) > 0) {
|
||||
msg_perr("%s: Garbage following 'bus' value.\n", __func__);
|
||||
goto _get_params_failed;
|
||||
}
|
||||
|
||||
msg_pinfo("Using i2c bus %i.\n", bus);
|
||||
*i2c_bus = bus;
|
||||
ret = 0;
|
||||
} else {
|
||||
msg_perr("%s: Bus number not specified.\n", __func__);
|
||||
}
|
||||
|
||||
reset_str = extract_programmer_param("reset-mcu");
|
||||
if (reset_str) {
|
||||
if (reset_str[0] == '1') {
|
||||
@ -504,22 +477,18 @@ static int get_params(int *i2c_bus, int *reset, int *enter_isp)
|
||||
}
|
||||
free(isp_str);
|
||||
|
||||
_get_params_failed:
|
||||
if (bus_str)
|
||||
free(bus_str);
|
||||
|
||||
return ret;
|
||||
}
|
||||
|
||||
int realtek_mst_i2c_spi_init(void)
|
||||
{
|
||||
int ret = 0;
|
||||
int i2c_bus = 0, reset = 0, enter_isp = 0;
|
||||
int reset = 0, enter_isp = 0;
|
||||
|
||||
if (get_params(&i2c_bus, &reset, &enter_isp))
|
||||
if (get_params(&reset, &enter_isp))
|
||||
return SPI_GENERIC_ERROR;
|
||||
|
||||
int fd = i2c_open(i2c_bus, REGISTER_ADDRESS, 0);
|
||||
int fd = i2c_open_from_programmer_params(REGISTER_ADDRESS, 0);
|
||||
if (fd < 0)
|
||||
return fd;
|
||||
|
||||
|
Loading…
x
Reference in New Issue
Block a user