[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 02/20] libqos: Rename i2c_send and i2c_recv
From: |
Stefan Hajnoczi |
Subject: |
Re: [PATCH v4 02/20] libqos: Rename i2c_send and i2c_recv |
Date: |
Wed, 6 Nov 2019 16:17:03 +0100 |
User-agent: |
Mutt/1.12.1 (2019-06-15) |
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 <address@hidden>
signature.asc
Description: PGP signature
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [PATCH v4 02/20] libqos: Rename i2c_send and i2c_recv,
Stefan Hajnoczi <=