diff mbox series

hw/audio/marvell: Don't include unnecessary i2c.h header file

Message ID 1546614146-10525-1-git-send-email-thuth@redhat.com (mailing list archive)
State New, archived
Headers show
Series hw/audio/marvell: Don't include unnecessary i2c.h header file | expand

Commit Message

Thomas Huth Jan. 4, 2019, 3:02 p.m. UTC
This device does not use I2C, so no need to include the header file here.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 hw/audio/marvell_88w8618.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Laurent Vivier Jan. 9, 2019, 10:18 a.m. UTC | #1
On 04/01/2019 16:02, Thomas Huth wrote:
> This device does not use I2C, so no need to include the header file here.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  hw/audio/marvell_88w8618.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/hw/audio/marvell_88w8618.c b/hw/audio/marvell_88w8618.c
> index 6600ab4..035dd76 100644
> --- a/hw/audio/marvell_88w8618.c
> +++ b/hw/audio/marvell_88w8618.c
> @@ -12,7 +12,6 @@
>  #include "qemu/osdep.h"
>  #include "hw/sysbus.h"
>  #include "hw/hw.h"
> -#include "hw/i2c/i2c.h"
>  #include "hw/audio/wm8750.h"
>  #include "audio/audio.h"
>  #include "qapi/error.h"
> 


Applied to my trivial-patches branch.

Thanks,
Laurent
Gerd Hoffmann Jan. 10, 2019, 8:52 a.m. UTC | #2
On Fri, Jan 04, 2019 at 04:02:26PM +0100, Thomas Huth wrote:
> This device does not use I2C, so no need to include the header file here.

Added to audio queue.

thanks,
  Gerd
diff mbox series

Patch

diff --git a/hw/audio/marvell_88w8618.c b/hw/audio/marvell_88w8618.c
index 6600ab4..035dd76 100644
--- a/hw/audio/marvell_88w8618.c
+++ b/hw/audio/marvell_88w8618.c
@@ -12,7 +12,6 @@ 
 #include "qemu/osdep.h"
 #include "hw/sysbus.h"
 #include "hw/hw.h"
-#include "hw/i2c/i2c.h"
 #include "hw/audio/wm8750.h"
 #include "audio/audio.h"
 #include "qapi/error.h"