It's not easy to understand PAGE_SIZE * 0x10 and PAGE_SIZE * 16 at the first glance.
Signed-off-by: Julien Grall <julien.gr...@citrix.com> Acked-by: Ian Campbell <ian.campb...@citrix.com> Cc: Zoltan Kiss <zoltan.k...@huawei.com> --- Changes in v2: - Add Ian's ack - Merge "xen/arm: gic-v2: Use SZ_64K rather than our custom value" and "xen/arm: gic-hip04: Use SZ_64K rather than our custom value" in a single patch. --- xen/arch/arm/gic-hip04.c | 6 +++--- xen/arch/arm/gic-v2.c | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/xen/arch/arm/gic-hip04.c b/xen/arch/arm/gic-hip04.c index 71cdba0..0ba15d1 100644 --- a/xen/arch/arm/gic-hip04.c +++ b/xen/arch/arm/gic-hip04.c @@ -29,6 +29,7 @@ #include <xen/list.h> #include <xen/device_tree.h> #include <xen/libfdt/libfdt.h> +#include <xen/sizes.h> #include <asm/p2m.h> #include <asm/domain.h> #include <asm/platform.h> @@ -470,7 +471,7 @@ static int hip04gicv_setup(struct domain *d) 2, paddr_to_pfn(gicv2.vbase + PAGE_SIZE)); else ret = map_mmio_regions(d, paddr_to_pfn(d->arch.vgic.cbase + PAGE_SIZE), - 2, paddr_to_pfn(gicv2.vbase + 16*PAGE_SIZE)); + 2, paddr_to_pfn(gicv2.vbase + SZ_64K)); return ret; } @@ -721,8 +722,7 @@ static int __init hip04gic_init(void) gicv2.map_cbase[0] = ioremap_nocache(gicv2.cbase, PAGE_SIZE); if ( platform_has_quirk(PLATFORM_QUIRK_GIC_64K_STRIDE) ) - gicv2.map_cbase[1] = ioremap_nocache(gicv2.cbase + PAGE_SIZE * 0x10, - PAGE_SIZE); + gicv2.map_cbase[1] = ioremap_nocache(gicv2.cbase + SZ_64K, PAGE_SIZE); else gicv2.map_cbase[1] = ioremap_nocache(gicv2.cbase + PAGE_SIZE, PAGE_SIZE); diff --git a/xen/arch/arm/gic-v2.c b/xen/arch/arm/gic-v2.c index cecb092..f49ecd8 100644 --- a/xen/arch/arm/gic-v2.c +++ b/xen/arch/arm/gic-v2.c @@ -28,6 +28,7 @@ #include <xen/list.h> #include <xen/device_tree.h> #include <xen/libfdt/libfdt.h> +#include <xen/sizes.h> #include <asm/p2m.h> #include <asm/domain.h> #include <asm/platform.h> @@ -460,7 +461,7 @@ static int gicv2v_setup(struct domain *d) 2, paddr_to_pfn(gicv2.vbase + PAGE_SIZE)); else ret = map_mmio_regions(d, paddr_to_pfn(d->arch.vgic.cbase + PAGE_SIZE), - 2, paddr_to_pfn(gicv2.vbase + 16*PAGE_SIZE)); + 2, paddr_to_pfn(gicv2.vbase + SZ_64K)); return ret; } @@ -707,8 +708,7 @@ static int __init gicv2_init(void) gicv2.map_cbase[0] = ioremap_nocache(gicv2.cbase, PAGE_SIZE); if ( platform_has_quirk(PLATFORM_QUIRK_GIC_64K_STRIDE) ) - gicv2.map_cbase[1] = ioremap_nocache(gicv2.cbase + PAGE_SIZE * 0x10, - PAGE_SIZE); + gicv2.map_cbase[1] = ioremap_nocache(gicv2.cbase + SZ_64K, PAGE_SIZE); else gicv2.map_cbase[1] = ioremap_nocache(gicv2.cbase + PAGE_SIZE, PAGE_SIZE); -- 2.1.4 _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel