On Wed, Oct 30, 2019 at 02:49:49PM +0000, Oleinik, Alexander wrote:
> diff --git a/tests/libqos/i2c-imx.c b/tests/libqos/i2c-imx.c
> index f33ece55a3..42ebf8ba3a 100644
> --- a/tests/libqos/i2c-imx.c
> +++ b/tests/libqos/i2c-imx.c
> @@ -37,7 +37,7 @@ static void imx_i2c_set_slave_addr(IMXI2C *s, uint8_t addr,
> (addr << 1) | (direction == IMX_I2C_READ ? 1 : 0));
> }
>
> -static void imx_i2c_send(I2CAdapter *i2c, uint8_t addr,
> +static void qimx_i2c_send(I2CAdapter *i2c, uint8_t addr,
> const uint8_t *buf, uint16_t len)
> {
> IMXI2C *s = container_of(i2c, IMXI2C, parent);
> @@ -97,7 +97,7 @@ static void imx_i2c_send(I2CAdapter *i2c, uint8_t addr,
> g_assert((status & I2SR_IBB) == 0);
> }
>
> -static void imx_i2c_recv(I2CAdapter *i2c, uint8_t addr,
> +static void qimx_i2c_recv(I2CAdapter *i2c, uint8_t addr,
> uint8_t *buf, uint16_t len)
> {
> IMXI2C *s = container_of(i2c, IMXI2C, parent);
> @@ -202,8 +202,8 @@ void imx_i2c_init(IMXI2C *s, QTestState *qts, uint64_t
> addr)
>
> s->obj.get_driver = imx_i2c_get_driver;
>
> - s->parent.send = imx_i2c_send;
> - s->parent.recv = imx_i2c_recv;
> + s->parent.send = qimx_i2c_send;
> + s->parent.recv = qimx_i2c_recv;
Why do the imx static functions need to be renamed? Is this by
accident, maybe an aggressive sed command that touched all files?
> s->parent.qts = qts;
> }
>
> diff --git a/tests/libqos/i2c-omap.c b/tests/libqos/i2c-omap.c
> index 9ae8214fa8..5f4d79f87c 100644
> --- a/tests/libqos/i2c-omap.c
> +++ b/tests/libqos/i2c-omap.c
> @@ -50,7 +50,7 @@ static void omap_i2c_set_slave_addr(OMAPI2C *s, uint8_t
> addr)
> g_assert_cmphex(data, ==, addr);
> }
>
> -static void omap_i2c_send(I2CAdapter *i2c, uint8_t addr,
> +static void qomap_i2c_send(I2CAdapter *i2c, uint8_t addr,
> const uint8_t *buf, uint16_t len)
> {
> OMAPI2C *s = container_of(i2c, OMAPI2C, parent);
> @@ -94,7 +94,7 @@ static void omap_i2c_send(I2CAdapter *i2c, uint8_t addr,
> g_assert((data & OMAP_I2C_CON_STP) == 0);
> }
>
> -static void omap_i2c_recv(I2CAdapter *i2c, uint8_t addr,
> +static void qomap_i2c_recv(I2CAdapter *i2c, uint8_t addr,
> uint8_t *buf, uint16_t len)
> {
> OMAPI2C *s = container_of(i2c, OMAPI2C, parent);
> @@ -182,8 +182,8 @@ void omap_i2c_init(OMAPI2C *s, QTestState *qts, uint64_t
> addr)
> s->obj.get_driver = omap_i2c_get_driver;
> s->obj.start_hw = omap_i2c_start_hw;
>
> - s->parent.send = omap_i2c_send;
> - s->parent.recv = omap_i2c_recv;
> + s->parent.send = qomap_i2c_send;
> + s->parent.recv = qomap_i2c_recv;
Same here.
Otherwise:
Reviewed-by: Stefan Hajnoczi <stefa...@redhat.com>
signature.asc
Description: PGP signature
