[igt-dev] [Intel-xe] [PATCH i-g-t 03/14] lib/gpu_cmds: Reorder if/else ladder according to IP version
Matt Roper
matthew.d.roper at intel.com
Fri Sep 15 17:51:29 UTC 2023
On Fri, Sep 15, 2023 at 06:23:35AM -0700, Lucas De Marchi wrote:
> Make it easier to follow the flow by following the convention of "latest
> platform earlier".
>
> Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>
Reviewed-by: Matt Roper <matthew.d.roper at intel.com>
> ---
> lib/gpu_cmds.c | 12 +++++-------
> 1 file changed, 5 insertions(+), 7 deletions(-)
>
> diff --git a/lib/gpu_cmds.c b/lib/gpu_cmds.c
> index 648451f74..0fd8f8213 100644
> --- a/lib/gpu_cmds.c
> +++ b/lib/gpu_cmds.c
> @@ -270,17 +270,15 @@ fill_binding_table(struct intel_bb *ibb, struct intel_buf *buf)
> binding_table = intel_bb_ptr(ibb);
> intel_bb_ptr_add(ibb, 64);
>
> - if (IS_GEN7(devid))
> - binding_table[0] = gen7_fill_surface_state(ibb, buf,
> - SURFACEFORMAT_R8_UNORM, 1);
> -
> - else if (intel_graphics_ver(devid) >= IP_VER(12, 50))
> + if (intel_graphics_ver(devid) >= IP_VER(12, 50))
> binding_table[0] = xehp_fill_surface_state(ibb, buf,
> SURFACEFORMAT_R8_UNORM, 1);
> -
> - else
> + else if (intel_graphics_ver(devid) >= IP_VER(8, 0))
> binding_table[0] = gen8_fill_surface_state(ibb, buf,
> SURFACEFORMAT_R8_UNORM, 1);
> + else
> + binding_table[0] = gen7_fill_surface_state(ibb, buf,
> + SURFACEFORMAT_R8_UNORM, 1);
>
> return binding_table_offset;
> }
> --
> 2.40.1
>
--
Matt Roper
Graphics Software Engineer
Linux GPU Platform Enablement
Intel Corporation
More information about the igt-dev
mailing list