diff options
author | Jason Ekstrand <[email protected]> | 2017-08-17 14:57:06 -0700 |
---|---|---|
committer | Jason Ekstrand <[email protected]> | 2017-08-17 18:09:05 -0700 |
commit | 1af8342b0c30d93feafdae99a69021cdb0caa8b5 (patch) | |
tree | 33b79692d286c543f2f798798a97455e22466497 /src/intel/isl/isl.c | |
parent | 2d68d27071b2d3133c6685fe3b83b29fb126a764 (diff) |
intel/isl: Replace switch statements of doom with a macro
Reviewed-by: Kenneth Graunke <[email protected]>
Diffstat (limited to 'src/intel/isl/isl.c')
-rw-r--r-- | src/intel/isl/isl.c | 135 |
1 files changed, 39 insertions, 96 deletions
diff --git a/src/intel/isl/isl.c b/src/intel/isl/isl.c index 133986782b7..3788f9c2ead 100644 --- a/src/intel/isl/isl.c +++ b/src/intel/isl/isl.c @@ -1742,6 +1742,42 @@ isl_surf_get_ccs_surf(const struct isl_device *dev, .tiling_flags = ISL_TILING_CCS_BIT); } +#define isl_genX_call(dev, func, ...) \ + switch (ISL_DEV_GEN(dev)) { \ + case 4: \ + /* G45 surface state is the same as gen5 */ \ + if (ISL_DEV_IS_G4X(dev)) { \ + isl_gen5_##func(__VA_ARGS__); \ + } else { \ + isl_gen4_##func(__VA_ARGS__); \ + } \ + break; \ + case 5: \ + isl_gen5_##func(__VA_ARGS__); \ + break; \ + case 6: \ + isl_gen6_##func(__VA_ARGS__); \ + break; \ + case 7: \ + if (ISL_DEV_IS_HASWELL(dev)) { \ + isl_gen75_##func(__VA_ARGS__); \ + } else { \ + isl_gen7_##func(__VA_ARGS__); \ + } \ + break; \ + case 8: \ + isl_gen8_##func(__VA_ARGS__); \ + break; \ + case 9: \ + isl_gen9_##func(__VA_ARGS__); \ + break; \ + case 10: \ + isl_gen10_##func(__VA_ARGS__); \ + break; \ + default: \ + assert(!"Unknown hardware generation"); \ + } + void isl_surf_fill_state_s(const struct isl_device *dev, void *state, const struct isl_surf_fill_state_info *restrict info) @@ -1765,74 +1801,14 @@ isl_surf_fill_state_s(const struct isl_device *dev, void *state, info->surf->logical_level0_px.array_len); } - switch (ISL_DEV_GEN(dev)) { - case 4: - if (ISL_DEV_IS_G4X(dev)) { - /* G45 surface state is the same as gen5 */ - isl_gen5_surf_fill_state_s(dev, state, info); - } else { - isl_gen4_surf_fill_state_s(dev, state, info); - } - break; - case 5: - isl_gen5_surf_fill_state_s(dev, state, info); - break; - case 6: - isl_gen6_surf_fill_state_s(dev, state, info); - break; - case 7: - if (ISL_DEV_IS_HASWELL(dev)) { - isl_gen75_surf_fill_state_s(dev, state, info); - } else { - isl_gen7_surf_fill_state_s(dev, state, info); - } - break; - case 8: - isl_gen8_surf_fill_state_s(dev, state, info); - break; - case 9: - isl_gen9_surf_fill_state_s(dev, state, info); - break; - case 10: - isl_gen10_surf_fill_state_s(dev, state, info); - break; - default: - assert(!"Cannot fill surface state for this gen"); - } + isl_genX_call(dev, surf_fill_state_s, dev, state, info); } void isl_buffer_fill_state_s(const struct isl_device *dev, void *state, const struct isl_buffer_fill_state_info *restrict info) { - switch (ISL_DEV_GEN(dev)) { - case 4: - case 5: - /* Gen 4-5 are all the same when it comes to buffer surfaces */ - isl_gen5_buffer_fill_state_s(state, info); - break; - case 6: - isl_gen6_buffer_fill_state_s(state, info); - break; - case 7: - if (ISL_DEV_IS_HASWELL(dev)) { - isl_gen75_buffer_fill_state_s(state, info); - } else { - isl_gen7_buffer_fill_state_s(state, info); - } - break; - case 8: - isl_gen8_buffer_fill_state_s(state, info); - break; - case 9: - isl_gen9_buffer_fill_state_s(state, info); - break; - case 10: - isl_gen10_buffer_fill_state_s(state, info); - break; - default: - assert(!"Cannot fill surface state for this gen"); - } + isl_genX_call(dev, buffer_fill_state_s, state, info); } void @@ -1869,40 +1845,7 @@ isl_emit_depth_stencil_hiz_s(const struct isl_device *dev, void *batch, } } - switch (ISL_DEV_GEN(dev)) { - case 4: - if (ISL_DEV_IS_G4X(dev)) { - /* G45 surface state is the same as gen5 */ - isl_gen5_emit_depth_stencil_hiz_s(dev, batch, info); - } else { - isl_gen4_emit_depth_stencil_hiz_s(dev, batch, info); - } - break; - case 5: - isl_gen5_emit_depth_stencil_hiz_s(dev, batch, info); - break; - case 6: - isl_gen6_emit_depth_stencil_hiz_s(dev, batch, info); - break; - case 7: - if (ISL_DEV_IS_HASWELL(dev)) { - isl_gen75_emit_depth_stencil_hiz_s(dev, batch, info); - } else { - isl_gen7_emit_depth_stencil_hiz_s(dev, batch, info); - } - break; - case 8: - isl_gen8_emit_depth_stencil_hiz_s(dev, batch, info); - break; - case 9: - isl_gen9_emit_depth_stencil_hiz_s(dev, batch, info); - break; - case 10: - isl_gen10_emit_depth_stencil_hiz_s(dev, batch, info); - break; - default: - assert(!"Cannot fill surface state for this gen"); - } + isl_genX_call(dev, emit_depth_stencil_hiz_s, dev, batch, info); } /** |