[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-2.3 3/4] s390x: do not include ram_addr.h
From: |
Thomas Huth |
Subject: |
Re: [Qemu-devel] [PATCH for-2.3 3/4] s390x: do not include ram_addr.h |
Date: |
Fri, 27 Mar 2015 14:04:00 +0100 |
Am Thu, 26 Mar 2015 16:36:01 +0100
schrieb Cornelia Huck <address@hidden>:
> From: Paolo Bonzini <address@hidden>
>
> ram_addr.h is an internal interface and it is not needed anyway by
> hw/s390x/ipl.c.
>
> Cc: Christian Borntraeger <address@hidden>
> Signed-off-by: Paolo Bonzini <address@hidden>
> Message-Id: <address@hidden>
> Signed-off-by: Christian Borntraeger <address@hidden>
> Signed-off-by: Cornelia Huck <address@hidden>
> ---
> 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 <address@hidden>