Message ID | 1427384162-4994-4-git-send-email-cornelia.huck@de.ibm.com |
---|---|
State | New |
Headers | show |
Am Thu, 26 Mar 2015 16:36:01 +0100 schrieb Cornelia Huck <cornelia.huck@de.ibm.com>: > From: Paolo Bonzini <pbonzini@redhat.com> > > ram_addr.h is an internal interface and it is not needed anyway by > hw/s390x/ipl.c. > > Cc: Christian Borntraeger <borntraeger@de.ibm.com> > Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> > Message-Id: <1427295389-5054-1-git-send-email-pbonzini@redhat.com> > Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com> > Signed-off-by: Cornelia Huck <cornelia.huck@de.ibm.com> > --- > hw/s390x/ipl.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/hw/s390x/ipl.c b/hw/s390x/ipl.c > index 54d0835..5c86613 100644 > --- a/hw/s390x/ipl.c > +++ b/hw/s390x/ipl.c > @@ -14,7 +14,6 @@ > #include "sysemu/sysemu.h" > #include "cpu.h" > #include "elf.h" > -#include "exec/ram_addr.h" > #include "hw/loader.h" > #include "hw/sysbus.h" > #include "hw/s390x/virtio-ccw.h" Uh, IIRC I temporarily played around with qemu_get_ram_ptr() when I recently improved the BIOS loading in this file - and later forgot to remove that include file again - sorry! So of course, it should not be here. Thanks for cleaning it up! Reviewed-by: Thomas Huth <huth@tuxfamily.org>
diff --git a/hw/s390x/ipl.c b/hw/s390x/ipl.c index 54d0835..5c86613 100644 --- a/hw/s390x/ipl.c +++ b/hw/s390x/ipl.c @@ -14,7 +14,6 @@ #include "sysemu/sysemu.h" #include "cpu.h" #include "elf.h" -#include "exec/ram_addr.h" #include "hw/loader.h" #include "hw/sysbus.h" #include "hw/s390x/virtio-ccw.h"