Re: [Xen-devel] [PATCH v4 01/16] xen/arm: move io.h as mmio.h to include folder

[Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>]

On Mon, 26 May 2014, vijay.kilari@xxxxxxxxx wrote:
> From: Vijaya Kumar K <Vijaya.Kumar@xxxxxxxxxxxxxxxxxx>
> 
> io.h is local to arch/arm folder. move this file as mmio.h
> file to include/asm-arm folder as it might be
> required for inclusion in other header files in future.
> 
> Signed-off-by: Vijaya Kumar K <Vijaya.Kumar@xxxxxxxxxxxxxxxxxx>

Acked-by: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>


>  xen/arch/arm/io.c                             |    3 +--
>  xen/arch/arm/traps.c                          |    2 +-
>  xen/arch/arm/vgic.c                           |    2 +-
>  xen/arch/arm/vuart.c                          |    2 +-
>  xen/{arch/arm/io.h => include/asm-arm/mmio.h} |    8 ++++----
>  5 files changed, 8 insertions(+), 9 deletions(-)
> 
> diff --git a/xen/arch/arm/io.c b/xen/arch/arm/io.c
> index a6db00b..ada1918 100644
> --- a/xen/arch/arm/io.c
> +++ b/xen/arch/arm/io.c
> @@ -19,8 +19,7 @@
>  #include <xen/config.h>
>  #include <xen/lib.h>
>  #include <asm/current.h>
> -
> -#include "io.h"
> +#include <asm/mmio.h>
>  
>  static const struct mmio_handler *const mmio_handlers[] =
>  {
> diff --git a/xen/arch/arm/traps.c b/xen/arch/arm/traps.c
> index a4bdaaa..9348147 100644
> --- a/xen/arch/arm/traps.c
> +++ b/xen/arch/arm/traps.c
> @@ -36,9 +36,9 @@
>  #include <asm/regs.h>
>  #include <asm/cpregs.h>
>  #include <asm/psci.h>
> +#include <asm/mmio.h>
>  
>  #include "decode.h"
> -#include "io.h"
>  #include "vtimer.h"
>  #include <asm/gic.h>
>  
> diff --git a/xen/arch/arm/vgic.c b/xen/arch/arm/vgic.c
> index 4869b87..4962e70 100644
> --- a/xen/arch/arm/vgic.c
> +++ b/xen/arch/arm/vgic.c
> @@ -27,7 +27,7 @@
>  
>  #include <asm/current.h>
>  
> -#include "io.h"
> +#include <asm/mmio.h>
>  #include <asm/gic.h>
>  
>  #define REG(n) (n/4)
> diff --git a/xen/arch/arm/vuart.c b/xen/arch/arm/vuart.c
> index c02a8a9..953cd46 100644
> --- a/xen/arch/arm/vuart.c
> +++ b/xen/arch/arm/vuart.c
> @@ -38,9 +38,9 @@
>  #include <xen/errno.h>
>  #include <xen/ctype.h>
>  #include <xen/serial.h>
> +#include <asm/mmio.h>
>  
>  #include "vuart.h"
> -#include "io.h"
>  
>  #define domain_has_vuart(d) ((d)->arch.vuart.info != NULL)
>  
> diff --git a/xen/arch/arm/io.h b/xen/include/asm-arm/mmio.h
> similarity index 91%
> rename from xen/arch/arm/io.h
> rename to xen/include/asm-arm/mmio.h
> index 8d252c0..5870985 100644
> --- a/xen/arch/arm/io.h
> +++ b/xen/include/asm-arm/mmio.h
> @@ -1,5 +1,5 @@
>  /*
> - * xen/arch/arm/io.h
> + * xen/include/asm-arm/mmio.h
>   *
>   * ARM I/O handlers
>   *
> @@ -16,8 +16,8 @@
>   * GNU General Public License for more details.
>   */
>  
> -#ifndef __ARCH_ARM_IO_H__
> -#define __ARCH_ARM_IO_H__
> +#ifndef __ASM_ARM_MMIO_H__
> +#define __ASM_ARM_MMIO_H__
>  
>  #include <xen/lib.h>
>  #include <asm/processor.h>
> @@ -45,7 +45,7 @@ extern const struct mmio_handler vuart_mmio_handler;
>  
>  extern int handle_mmio(mmio_info_t *info);
>  
> -#endif
> +#endif  /* __ASM_ARM_MMIO_H__ */
>  
>  /*
>   * Local variables:
> -- 
> 1.7.9.5
> 

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel