[PATCH i-g-t] tools/Android.mk: Fix compile error in intel_reg.c

[PATCH i-g-t] tools/Android.mk: Fix compile error in intel_reg.c

[Derek Morton]

The patch "tools: install the register definition files" caused
a build error on android as it added 'PKGDATADIR' which was not
defined in the Android build environment. This patch adds that
define to tools/Android.mk. It also copies the files it points
to so they are actually in the target file system.
---
 tools/Android.mk | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/tools/Android.mk b/tools/Android.mk
index 0a196e4..aae7db5 100644
--- a/tools/Android.mk
+++ b/tools/Android.mk
@@ -34,11 +34,18 @@ define add_tool
                               libdrm        \
                               libdrm_intel
 
+    LOCAL_MODULE_PATH := $(TARGET_OUT_VENDOR)/intel/validation/core/igt/tools
+    LOCAL_CFLAGS += -DPKGDATADIR=\"/system/vendor/intel/validation/core/igt/tools\"
     include $(BUILD_EXECUTABLE)
 endef
 
 #================#
 
+# Copy the register files
+$(shell mkdir -p $(TARGET_OUT_VENDOR)/intel/validation/core/igt/tools/registers)
+$(shell cp $(LOCAL_PATH)/registers/* $(TARGET_OUT_VENDOR)/intel/validation/core/igt/tools/registers)
+
+
 skip_tools_list := \
     intel_framebuffer_dump \
     intel_reg_dumper \
-- 
1.9.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH i-g-t] tools/Android.mk: Fix compile error in intel_reg.c

[Thomas Wood]

On 9 September 2015 at 14:30, Derek Morton <derek.j.morton@intel.com> wrote:
> The patch "tools: install the register definition files" caused
> a build error on android as it added 'PKGDATADIR' which was not
> defined in the Android build environment. This patch adds that
> define to tools/Android.mk. It also copies the files it points
> to so they are actually in the target file system.

Missing Signed-off-by line.

> ---
>  tools/Android.mk | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/tools/Android.mk b/tools/Android.mk
> index 0a196e4..aae7db5 100644
> --- a/tools/Android.mk
> +++ b/tools/Android.mk
> @@ -34,11 +34,18 @@ define add_tool
>                                libdrm        \
>                                libdrm_intel
>
> +    LOCAL_MODULE_PATH := $(TARGET_OUT_VENDOR)/intel/validation/core/igt/tools

Is this part of the same fix?


> +    LOCAL_CFLAGS += -DPKGDATADIR=\"/system/vendor/intel/validation/core/igt/tools\"

This should have /registers on the end. Might be worth having a local
variable to avoid specifying the path explicitly each time.


>      include $(BUILD_EXECUTABLE)
>  endef
>
>  #================#
>
> +# Copy the register files
> +$(shell mkdir -p $(TARGET_OUT_VENDOR)/intel/validation/core/igt/tools/registers)
> +$(shell cp $(LOCAL_PATH)/registers/* $(TARGET_OUT_VENDOR)/intel/validation/core/igt/tools/registers)
> +
> +
>  skip_tools_list := \
>      intel_framebuffer_dump \
>      intel_reg_dumper \
> --
> 1.9.1
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH i-g-t] tools/Android.mk: Fix compile error in intel_reg.c

[Morton, Derek J]

-----Original Message-----
From: Thomas Wood [mailto:thomas.wood@intel.com] 
Sent: Wednesday, September 9, 2015 2:48 PM
To: Morton, Derek J
Cc: Intel Graphics Development
Subject: Re: [PATCH i-g-t] tools/Android.mk: Fix compile error in intel_reg.c
>
>On 9 September 2015 at 14:30, Derek Morton <derek.j.morton@intel.com> wrote:
>> The patch "tools: install the register definition files" caused a 
>> build error on android as it added 'PKGDATADIR' which was not defined 
>> in the Android build environment. This patch adds that define to 
>> tools/Android.mk. It also copies the files it points to so they are 
>> actually in the target file system.
>
>Missing Signed-off-by line.

Will add.

>
>> ---
>>  tools/Android.mk | 7 +++++++
>>  1 file changed, 7 insertions(+)
>>
>> diff --git a/tools/Android.mk b/tools/Android.mk index 
>> 0a196e4..aae7db5 100644
>> --- a/tools/Android.mk
>> +++ b/tools/Android.mk
>> @@ -34,11 +34,18 @@ define add_tool
>>                                libdrm        \
>>                                libdrm_intel
>>
>> +    LOCAL_MODULE_PATH := 
>> + $(TARGET_OUT_VENDOR)/intel/validation/core/igt/tools
>
>Is this part of the same fix?

This is putting all the tools with the other igt binaries so the register files then have somewhere logical to also be put.

>
>
>> +    LOCAL_CFLAGS += -DPKGDATADIR=\"/system/vendor/intel/validation/core/igt/tools\"
>
>This should have /registers on the end. Might be worth having a local variable to avoid specifying the path explicitly each time.

I don’t think it should. /registers is added in intel_reg.c
    path = PKGDATADIR"/registers";

I can create a local variable.

>
>
>>      include $(BUILD_EXECUTABLE)
>>  endef
>>
>>  #================#
>>
>> +# Copy the register files
>> +$(shell mkdir -p 
>> +$(TARGET_OUT_VENDOR)/intel/validation/core/igt/tools/registers)
>> +$(shell cp $(LOCAL_PATH)/registers/* 
>> +$(TARGET_OUT_VENDOR)/intel/validation/core/igt/tools/registers)
>> +
>> +
>>  skip_tools_list := \
>>      intel_framebuffer_dump \
>>      intel_reg_dumper \
>> --
>> 1.9.1
>>
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx