Re: [Qemu-devel] [PATCH v2 00/11] hw: Remove "hw/devices.h"

2019-04-08 Thread Markus Armbruster
Philippe Mathieu-Daudé  writes:

> Hi,
>
> As his first comment describes itself, the "hw/devices.h" contains
> declarations for "Devices that have nowhere better to go."
> This series remove it, creating new headers for devices covered there.
> MAINTAINERS is updated.
> I also included 2 cleanups while working on this, in "qemu/typedefs.h"
> and "hw/net/ne2000-isa.h" header guard.

Needs a rebase now.  Apart from that, series
Reviewed-by: Markus Armbruster 



Re: [Qemu-devel] [PATCH v2 00/11] hw: Remove "hw/devices.h"

2019-02-12 Thread Philippe Mathieu-Daudé
On 2/5/19 12:18 AM, Philippe Mathieu-Daudé wrote:
> Hi,
> 
> As his first comment describes itself, the "hw/devices.h" contains
> declarations for "Devices that have nowhere better to go."
> This series remove it, creating new headers for devices covered there.
> MAINTAINERS is updated.
> I also included 2 cleanups while working on this, in "qemu/typedefs.h"
> and "hw/net/ne2000-isa.h" header guard.
> 
> v2:
> - added R-b, T-b tags
> - addressed many review comments from Thomas
> 
> v1: https://lists.gnu.org/archive/html/qemu-devel/2019-01/msg00611.html

ping?

Since it is mostly touch (old) ARM boards, Peter can you do a quick
review and give your Ack-by, so it can go via the Trivial/Misc tree?
Half of the series is already reviewed by Thomas.

Thanks,

Phil.



[Qemu-devel] [PATCH v2 00/11] hw: Remove "hw/devices.h"

2019-02-04 Thread Philippe Mathieu-Daudé
Hi,

As his first comment describes itself, the "hw/devices.h" contains
declarations for "Devices that have nowhere better to go."
This series remove it, creating new headers for devices covered there.
MAINTAINERS is updated.
I also included 2 cleanups while working on this, in "qemu/typedefs.h"
and "hw/net/ne2000-isa.h" header guard.

v2:
- added R-b, T-b tags
- addressed many review comments from Thomas

v1: https://lists.gnu.org/archive/html/qemu-devel/2019-01/msg00611.html

Phil.

Philippe Mathieu-Daudé (11):
  hw/arm/aspeed: Use TYPE_TMP105/TYPE_PCA9552 instead of hardcoded
string
  hw: Remove unused 'hw/devices.h' include
  hw/devices: Move TC6393XB declarations into a new header
  hw/devices: Move Blizzard declarations into a new header
  hw/devices: Move CBus declarations into a new header
  hw/devices: Move Gamepad declarations into a new header
  hw/devices: Move TI touchscreen declarations into a new header
  hw/devices: Move LAN9118 declarations into a new header
  hw/net/ne2000-isa: Add guards to the header
  hw/net/lan9118: Export TYPE_LAN9118 and use it instead of hardcoded
string
  hw/devices: Move SMSC 91C111 declaration into a new header

 MAINTAINERS  |  7 +++
 hw/alpha/typhoon.c   |  1 -
 hw/arm/allwinner-a10.c   |  1 -
 hw/arm/aspeed.c  | 13 +++--
 hw/arm/collie.c  |  1 -
 hw/arm/cubieboard.c  |  1 -
 hw/arm/exynos4_boards.c  |  3 +-
 hw/arm/gumstix.c |  2 +-
 hw/arm/highbank.c|  1 -
 hw/arm/integratorcp.c|  2 +-
 hw/arm/kzm.c |  2 +-
 hw/arm/mainstone.c   |  2 +-
 hw/arm/mps2-tz.c |  4 +-
 hw/arm/mps2.c|  2 +-
 hw/arm/musicpal.c|  1 -
 hw/arm/nrf51_soc.c   |  1 -
 hw/arm/nseries.c |  4 +-
 hw/arm/palm.c|  2 +-
 hw/arm/realview.c|  3 +-
 hw/arm/spitz.c   |  1 -
 hw/arm/stellaris.c   |  2 +-
 hw/arm/tosa.c|  2 +-
 hw/arm/versatilepb.c |  2 +-
 hw/arm/vexpress.c|  2 +-
 hw/arm/virt.c|  1 -
 hw/arm/z2.c  |  1 -
 hw/display/blizzard.c|  2 +-
 hw/display/sm501.c   |  1 -
 hw/display/tc6393xb.c|  2 +-
 hw/hppa/dino.c   |  1 -
 hw/input/stellaris_input.c   |  2 +-
 hw/input/tsc2005.c   |  2 +-
 hw/input/tsc210x.c   |  4 +-
 hw/intc/allwinner-a10-pic.c  |  1 -
 hw/lm32/lm32_boards.c|  1 -
 hw/lm32/milkymist.c  |  1 -
 hw/microblaze/petalogix_ml605_mmu.c  |  1 -
 hw/microblaze/petalogix_s3adsp1800_mmu.c |  1 -
 hw/misc/cbus.c   |  2 +-
 hw/net/dp8393x.c |  1 -
 hw/net/lan9118.c |  3 +-
 hw/net/smc91c111.c   |  2 +-
 hw/ppc/virtex_ml507.c|  1 -
 hw/sh4/r2d.c |  1 -
 hw/tricore/tricore_testboard.c   |  1 -
 hw/usb/tusb6010.c|  1 -
 include/hw/arm/omap.h|  6 +--
 include/hw/devices.h | 63 
 include/hw/display/blizzard.h| 22 +
 include/hw/display/tc6393xb.h| 26 ++
 include/hw/input/gamepad.h   | 21 
 include/hw/input/tsc2xxx.h   | 38 ++
 include/hw/misc/cbus.h   | 32 
 include/hw/net/lan9118.h | 21 
 include/hw/net/ne2000-isa.h  |  6 +++
 include/hw/net/smc91c111.h   | 19 +++
 include/qemu/typedefs.h  |  1 -
 57 files changed, 230 insertions(+), 120 deletions(-)
 delete mode 100644 include/hw/devices.h
 create mode 100644 include/hw/display/blizzard.h
 create mode 100644 include/hw/display/tc6393xb.h
 create mode 100644 include/hw/input/gamepad.h
 create mode 100644 include/hw/input/tsc2xxx.h
 create mode 100644 include/hw/misc/cbus.h
 create mode 100644 include/hw/net/lan9118.h
 create mode 100644 include/hw/net/smc91c111.h

-- 
2.20.1