[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PATCH 02/13] target: arm: Remove unused headers
From: |
Samuel Ortiz |
Subject: |
Re: [Qemu-arm] [PATCH 02/13] target: arm: Remove unused headers |
Date: |
Wed, 14 Nov 2018 00:28:32 +0100 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
Hi Philippe,
On Tue, Nov 13, 2018 at 07:02:57PM +0100, Philippe Mathieu-Daudé wrote:
> On 13/11/18 18:01, Peter Maydell wrote:
> > On 13 November 2018 at 16:52, Samuel Ortiz <address@hidden> wrote:
> > > From: Philippe Mathieu-Daudé <address@hidden>
> > >
> > > Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> > > Reviewed-by: Robert Bradford <address@hidden>
> > > Reviewed-by: Samuel Ortiz <address@hidden>
> > > ---
> > > target/arm/helper.c | 3 ---
> > > 1 file changed, 3 deletions(-)
> > >
> > > diff --git a/target/arm/helper.c b/target/arm/helper.c
> > > index 3d4e9c5c8a..27d9285e1e 100644
> > > --- a/target/arm/helper.c
> > > +++ b/target/arm/helper.c
> > > @@ -12,13 +12,10 @@
> > > #include "internals.h"
> > > #include "exec/gdbstub.h"
> > > #include "exec/helper-proto.h"
> > > -#include "qemu/host-utils.h"
> >
> > This is for muldiv64().
>
> But it is already included by "cpu.h" -> "exec/cpu-defs.h"
>
> >
> > > #include "sysemu/arch_init.h"
> > > #include "sysemu/sysemu.h"
> > > -#include "qemu/bitops.h"
>
> "cpu.h" -> "cpu-qom.h" -> "qom/cpu.h" -> "qemu/bitmap.h"
>
> >
> > This is for extract32() and friends.
> >
> > > #include "qemu/crc32c.h"
> > > #include "exec/exec-all.h"
> > > -#include "exec/cpu_ldst.h"
> >
> > This is for cpu_stl_data().
>
> Included by "arm_ldst.h"
>
> >
> > > #include "arm_ldst.h"
> > > #include <zlib.h> /* For crc32 */
> > > #include "exec/semihost.h"
>
> So they are not "unused" but "unnecessary".
>
> I thought this would be better to clean this once, before Samuel split.
>
> Samuel: please drop this patch from your series.
Dropped. I updated my
https://github.com/intel/nemu/tree/topic/upstream/arm-tcg-disable branch
accordingly. I will wait for Richard's review before sending a v2.
Cheers,
Samuel.
- Re: [Qemu-arm] [PATCH 07/13] target: arm: Remove the LDST headers, (continued)
Re: [Qemu-arm] [Qemu-devel] [PATCH 00/13] Support disabling TCG on ARM, no-reply, 2018/11/14