On 13/03/2020 21:57, Philippe Mathieu-Daudé wrote:
On 3/12/20 5:54 PM, Liran Alon wrote:
No functional change. This is mere refactoring.

Suggested-by: Michael S. Tsirkin <m...@redhat.com>
Signed-off-by: Liran Alon <liran.a...@oracle.com>
---
  hw/i386/pc.c             |  1 +
  hw/i386/vmmouse.c        |  1 +
  hw/i386/vmport.c         |  1 +
  include/hw/i386/pc.h     | 13 -------------
  include/hw/i386/vmport.h | 16 ++++++++++++++++

What about moving it to hw/i386/vmport.h (no under include/)?

Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com>


Can you explain the logic that separates between hw/i386/*.h to include/hw/i386/*.h? If it makes sense, sure I will move it. I just don't know what is the convention here.

-Liran


Reply via email to