[igt-dev] [PATCH i-g-t v2 02/13] lib/gpu_cmds: Remove prefix from gen7_fill_binding_table()
Lucas De Marchi
lucas.demarchi at intel.com
Wed Oct 4 03:10:12 UTC 2023
This function is used by pretty much all versions. It's odd to have a
gen7 function end up calling next-gen functions and it's error prone:
when adding something to e.g. a gen8 function to realize that behavior
breaks previous platforms.
Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>
---
lib/gpu_cmds.c | 9 ++++-----
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/lib/gpu_cmds.c b/lib/gpu_cmds.c
index bb35204fb..648451f74 100644
--- a/lib/gpu_cmds.c
+++ b/lib/gpu_cmds.c
@@ -259,8 +259,7 @@ gen11_fill_surface_state(struct intel_bb *ibb,
}
static uint32_t
-gen7_fill_binding_table(struct intel_bb *ibb,
- struct intel_buf *buf)
+fill_binding_table(struct intel_bb *ibb, struct intel_buf *buf)
{
uint32_t binding_table_offset;
uint32_t *binding_table;
@@ -321,7 +320,7 @@ gen7_fill_interface_descriptor(struct intel_bb *ibb,
uint32_t offset;
uint32_t binding_table_offset, kernel_offset;
- binding_table_offset = gen7_fill_binding_table(ibb, buf);
+ binding_table_offset = fill_binding_table(ibb, buf);
kernel_offset = gen7_fill_kernel(ibb, kernel, size);
intel_bb_ptr_align(ibb, 64);
@@ -357,7 +356,7 @@ gen8_fill_interface_descriptor(struct intel_bb *ibb,
uint32_t offset;
uint32_t binding_table_offset, kernel_offset;
- binding_table_offset = gen7_fill_binding_table(ibb, buf);
+ binding_table_offset = fill_binding_table(ibb, buf);
kernel_offset = gen7_fill_kernel(ibb, kernel, size);
intel_bb_ptr_align(ibb, 64);
@@ -803,7 +802,7 @@ xehp_fill_interface_descriptor(struct intel_bb *ibb,
{
uint32_t binding_table_offset, kernel_offset;
- binding_table_offset = gen7_fill_binding_table(ibb, dst);
+ binding_table_offset = fill_binding_table(ibb, dst);
kernel_offset = gen7_fill_kernel(ibb, kernel, size);
memset(idd, 0, sizeof(*idd));
--
2.40.1
More information about the igt-dev
mailing list