Re: [Xen-devel] [PATCH v5 17/17] tools/tests: Enable xen-access on ARM

[Tamas K Lengyel <tamas.lengyel@xxxxxxxxxxxx>]

On Thu, Sep 11, 2014 at 11:29 PM, Julien Grall <julien.grall@xxxxxxxxxx> wrote:

Hello Tamas,

On 10/09/14 06:28, Tamas K Lengyel wrote:

diff --git a/config/arm32.mk b/config/arm32.mk
index aa79d22..4a7c259 100644
--- a/config/arm32.mk
+++ b/config/arm32.mk
@@ -13,6 +13,7 @@ HAS_PL011 := y
  HAS_EXYNOS4210 := y
  HAS_OMAP := y
  HAS_NS16550 := y
+HAS_MEM_ACCESS := y

  # Use only if calling $(LD) directly.
  LDFLAGS_DIRECT += -EL
diff --git a/config/arm64.mk b/config/arm64.mk
index 15b57a4..ea6558d 100644
--- a/config/arm64.mk
+++ b/config/arm64.mk
@@ -8,6 +8,7 @@ CFLAGS += #-marm -march= -mcpu= etc

  HAS_PL011 := y
  HAS_NS16550 := y
+HAS_MEM_ACCESS := y

Why can't you move HAS_MEM_ACCESS directly here in patch #1?

It would make this patch simpler.

It would but having HAS_MEM_ACCESS in config/* is not really required up until this patch. I'm fine with it either way, I just thought this is the clean solution.

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