diff mbox

[1/2] cleanup obsolete typedef

Message ID 1331663327-29982-1-git-send-email-mark.langsdorf@calxeda.com
State New
Headers show

Commit Message

Mark Langsdorf March 13, 2012, 6:28 p.m. UTC 
From: Joshua Housh <joshua.housh@calxeda.com>

There are no users of i2c_slave.

Signed-off-by: Joshua Housh <joshua.housh@calxeda.com>
Signed-off-by: Mark Langsdorf <mark.langsdorf@calxeda.com>
---
 qemu-common.h |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

Comments

Peter Maydell March 13, 2012, 6:38 p.m. UTC | #1 
On 13 March 2012 18:28, Mark Langsdorf <mark.langsdorf@calxeda.com> wrote:
> From: Joshua Housh <joshua.housh@calxeda.com>
>
> There are no users of i2c_slave.
>
> Signed-off-by: Joshua Housh <joshua.housh@calxeda.com>
> Signed-off-by: Mark Langsdorf <mark.langsdorf@calxeda.com>

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

Looks like we missed this typedef in commit 9e07bdf8
which renamed i2c_slave to I2CSlave. (The new typedef
for I2CSlave is in i2c.h.)

-- PMM
Stefan Hajnoczi March 15, 2012, 10:48 a.m. UTC | #2 
On Tue, Mar 13, 2012 at 01:28:47PM -0500, Mark Langsdorf wrote:
> From: Joshua Housh <joshua.housh@calxeda.com>
> 
> There are no users of i2c_slave.
> 
> Signed-off-by: Joshua Housh <joshua.housh@calxeda.com>
> Signed-off-by: Mark Langsdorf <mark.langsdorf@calxeda.com>
> ---
>  qemu-common.h |    1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)

Thanks, applied to the trivial patches tree:
https://github.com/stefanha/qemu/commits/trivial-patches

Stefan
diff mbox

Patch

diff --git a/qemu-common.h b/qemu-common.h
index b0fdf5c..c9e96a8 100644
--- a/qemu-common.h
+++ b/qemu-common.h
@@ -236,7 +236,6 @@  typedef struct MACAddr MACAddr;
 typedef struct VLANState VLANState;
 typedef struct VLANClientState VLANClientState;
 typedef struct i2c_bus i2c_bus;
-typedef struct i2c_slave i2c_slave;
 typedef struct ISABus ISABus;
 typedef struct SMBusDevice SMBusDevice;
 typedef struct PCIHostState PCIHostState;