Re: [Qemu-devel] [PATCH 06/15] target-lm32: Clean includes

[Michael Walle <michael@walle.cc>]

Am Donnerstag 02 Februar 2012, 22:12:48 schrieb Stefan Weil:
> config.h was missing in cpu.h.
> 
> Cc: Michael Walle <michael@walle.cc>
> Signed-off-by: Stefan Weil <sw@weilnetz.de>
> ---
>  target-lm32/cpu.h       |    1 +
>  target-lm32/helper.c    |    5 -----
>  target-lm32/translate.c |    8 --------
>  3 files changed, 1 insertions(+), 13 deletions(-)
> 
> diff --git a/target-lm32/cpu.h b/target-lm32/cpu.h
> index 037ef52..b6b6b4c 100644
> --- a/target-lm32/cpu.h
> +++ b/target-lm32/cpu.h
> @@ -24,6 +24,7 @@
> 
>  #define CPUState struct CPULM32State
> 
> +#include "config.h"
>  #include "qemu-common.h"
>  #include "cpu-defs.h"
>  struct CPULM32State;
> diff --git a/target-lm32/helper.c b/target-lm32/helper.c
> index fc0b444..2637c03 100644
> --- a/target-lm32/helper.c
> +++ b/target-lm32/helper.c
> @@ -17,11 +17,6 @@
>   * License along with this library; if not, see
> <http://www.gnu.org/licenses/>. */
> 
> -#include <stdio.h>
> -#include <string.h>
> -#include <assert.h>
> -
> -#include "config.h"
>  #include "cpu.h"
>  #include "host-utils.h"
> 
> diff --git a/target-lm32/translate.c b/target-lm32/translate.c
> index 0be105d..c80e48b 100644
> --- a/target-lm32/translate.c
> +++ b/target-lm32/translate.c
> @@ -17,18 +17,10 @@
>   * License along with this library; if not, see
> <http://www.gnu.org/licenses/>. */
> 
> -#include <stdarg.h>
> -#include <stdlib.h>
> -#include <stdio.h>
> -#include <string.h>
> -#include <inttypes.h>
> -#include <assert.h>
> -
>  #include "cpu.h"
>  #include "disas.h"
>  #include "helper.h"
>  #include "tcg-op.h"
> -#include "qemu-common.h"
> 
>  #include "hw/lm32_pic.h"

Acked-by: Michael Walle <michael@walle.cc>

-- 
michael