<div dir="ltr"><div dir="ltr">Hi,<div>kind reminder for Review or Ack,</div><div><br></div><div>apart from Android makefile changes Makefile.sources was modified</div><div>to add gfx10_format_table.h generated file.</div><div><br></div><div>Mauro</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jul 10, 2019 at 9:13 AM Mauro Rossi <<a href="mailto:issor.oruam@gmail.com">issor.oruam@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">This patch adds gfx10_format_table.h in Makefile.sources<br>
and the rules for Android to fix following building errors:<br>
<br>
In file included from external/mesa/src/amd/vulkan/radv_debug.c:35:<br>
In file included from external/mesa/src/amd/vulkan/radv_debug.h:27:<br>
external/mesa/src/amd/vulkan/radv_private.h:95:10:<br>
fatal error: 'gfx10_format_table.h' file not found<br>
         ^~~~~~~~~~~~~~~~~~~~~~<br>
1 error generated.<br>
<br>
In file included from external/mesa/src/amd/vulkan/radv_android.c:31:<br>
external/mesa/src/amd/vulkan/radv_private.h:95:10:<br>
fatal error: 'gfx10_format_table.h' file not found<br>
         ^~~~~~~~~~~~~~~~~~~~~~<br>
1 error generated.<br>
<br>
Fixes: 3dc5ec5d16 ("radv/gfx10: generate gfx10_format_table.h")<br>
Signed-off-by: Mauro Rossi <<a href="mailto:issor.oruam@gmail.com" target="_blank">issor.oruam@gmail.com</a>><br>
---<br>
 src/amd/vulkan/Android.mk       | 15 +++++++++++++++<br>
 src/amd/vulkan/Makefile.sources |  3 ++-<br>
 2 files changed, 17 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/src/amd/vulkan/Android.mk b/src/amd/vulkan/Android.mk<br>
index 0725feacb5..23cebb1ec8 100644<br>
--- a/src/amd/vulkan/Android.mk<br>
+++ b/src/amd/vulkan/Android.mk<br>
@@ -83,6 +83,7 @@ LOCAL_GENERATED_SOURCES += $(intermediates)/radv_entrypoints.h<br>
 LOCAL_GENERATED_SOURCES += $(intermediates)/radv_extensions.c<br>
 LOCAL_GENERATED_SOURCES += $(intermediates)/radv_extensions.h<br>
 LOCAL_GENERATED_SOURCES += $(intermediates)/vk_format_table.c<br>
+LOCAL_GENERATED_SOURCES += $(intermediates)/gfx10_format_table.h<br>
<br>
 RADV_ENTRYPOINTS_SCRIPT := $(MESA_TOP)/src/amd/vulkan/radv_entrypoints_gen.py<br>
 RADV_EXTENSIONS_SCRIPT := $(MESA_TOP)/src/amd/vulkan/radv_extensions.py<br>
@@ -117,6 +118,20 @@ $(intermediates)/vk_format_table.c: $(VK_FORMAT_TABLE_SCRIPT) \<br>
        @mkdir -p $(dir $@)<br>
        $(MESA_PYTHON2) $(VK_FORMAT_TABLE_SCRIPT) $(vk_format_layout_csv) > $@<br>
<br>
+RADV_GEN10_FORMAT_TABLE_INPUTS := \<br>
+       $(MESA_TOP)/src/amd/vulkan/vk_format_layout.csv \<br>
+       $(MESA_TOP)/src/amd/registers/gfx10-rsrc.json<br>
+<br>
+RADV_GEN10_FORMAT_TABLE_DEP := \<br>
+       $(MESA_TOP)/src/amd/registers/regdb.py<br>
+<br>
+RADV_GEN10_FORMAT_TABLE := $(LOCAL_PATH)/gfx10_format_table.py<br>
+<br>
+$(intermediates)/gfx10_format_table.h: $(RADV_GEN10_FORMAT_TABLE) $(RADV_GEN10_FORMAT_TABLE_INPUTS) $(RADV_GEN10_FORMAT_TABLE_DEP)<br>
+       @mkdir -p $(dir $@)<br>
+       @echo "Gen Header: $(PRIVATE_MODULE) <= $(notdir $(@))"<br>
+       $(hide) $(MESA_PYTHON2) $(RADV_GEN10_FORMAT_TABLE) $(RADV_GEN10_FORMAT_TABLE_INPUTS) > $@ || ($(RM) $@; false)<br>
+<br>
 LOCAL_SHARED_LIBRARIES += $(RADV_SHARED_LIBRARIES)<br>
<br>
 LOCAL_EXPORT_C_INCLUDE_DIRS := \<br>
diff --git a/src/amd/vulkan/Makefile.sources b/src/amd/vulkan/Makefile.sources<br>
index df90c1150a..312cd0b1e9 100644<br>
--- a/src/amd/vulkan/Makefile.sources<br>
+++ b/src/amd/vulkan/Makefile.sources<br>
@@ -91,5 +91,6 @@ VULKAN_GENERATED_FILES := \<br>
        radv_entrypoints.h \<br>
        radv_extensions.c \<br>
        radv_extensions.h \<br>
-       vk_format_table.c<br>
+       vk_format_table.c \<br>
+       gfx10_format_table.h<br>
<br>
-- <br>
2.20.1<br>
<br>
</blockquote></div></div>