[Mesa-dev] [PATCH 7/7] i965: Remove brw_lower_mesa_image_format
Jason Ekstrand
jason at jlekstrand.net
Wed Nov 4 17:03:56 PST 2015
Now that the compiler is all switched over to brw_lower_image_format and
state-setup doesn't need it, we can get rid of the old helper.
---
src/mesa/drivers/dri/i965/brw_context.h | 2 -
src/mesa/drivers/dri/i965/brw_surface_formats.c | 109 -----------------------
src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 11 +--
3 files changed, 3 insertions(+), 119 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_context.h b/src/mesa/drivers/dri/i965/brw_context.h
index e075395..0a06cc9 100644
--- a/src/mesa/drivers/dri/i965/brw_context.h
+++ b/src/mesa/drivers/dri/i965/brw_context.h
@@ -1470,8 +1470,6 @@ void brw_upload_image_surfaces(struct brw_context *brw,
bool brw_render_target_supported(struct brw_context *brw,
struct gl_renderbuffer *rb);
uint32_t brw_depth_format(struct brw_context *brw, mesa_format format);
-mesa_format brw_lower_mesa_image_format(const struct brw_device_info *devinfo,
- mesa_format format);
/* brw_performance_monitor.c */
void brw_init_performance_monitors(struct brw_context *brw);
diff --git a/src/mesa/drivers/dri/i965/brw_surface_formats.c b/src/mesa/drivers/dri/i965/brw_surface_formats.c
index 97fff60..bf079da 100644
--- a/src/mesa/drivers/dri/i965/brw_surface_formats.c
+++ b/src/mesa/drivers/dri/i965/brw_surface_formats.c
@@ -900,112 +900,3 @@ brw_depth_format(struct brw_context *brw, mesa_format format)
unreachable("Unexpected depth format.");
}
}
-
-mesa_format
-brw_lower_mesa_image_format(const struct brw_device_info *devinfo,
- mesa_format format)
-{
- switch (format) {
- /* These are never lowered. Up to BDW we'll have to fall back to untyped
- * surface access for 128bpp formats.
- */
- case MESA_FORMAT_RGBA_UINT32:
- case MESA_FORMAT_RGBA_SINT32:
- case MESA_FORMAT_RGBA_FLOAT32:
- case MESA_FORMAT_R_UINT32:
- case MESA_FORMAT_R_SINT32:
- case MESA_FORMAT_R_FLOAT32:
- return format;
-
- /* From HSW to BDW the only 64bpp format supported for typed access is
- * RGBA_UINT16. IVB falls back to untyped.
- */
- case MESA_FORMAT_RGBA_UINT16:
- case MESA_FORMAT_RGBA_SINT16:
- case MESA_FORMAT_RGBA_FLOAT16:
- case MESA_FORMAT_RG_UINT32:
- case MESA_FORMAT_RG_SINT32:
- case MESA_FORMAT_RG_FLOAT32:
- return (devinfo->gen >= 9 ? format :
- devinfo->gen >= 8 || devinfo->is_haswell ?
- MESA_FORMAT_RGBA_UINT16 : MESA_FORMAT_RG_UINT32);
-
- /* Up to BDW no SINT or FLOAT formats of less than 32 bits per component
- * are supported. IVB doesn't support formats with more than one component
- * for typed access. For 8 and 16 bpp formats IVB relies on the
- * undocumented behavior that typed reads from R_UINT8 and R_UINT16
- * surfaces actually do a 32-bit misaligned read. The alternative would be
- * to use two surface state entries with different formats for each image,
- * one for reading (using R_UINT32) and another one for writing (using
- * R_UINT8 or R_UINT16), but that would complicate the shaders we generate
- * even more.
- */
- case MESA_FORMAT_RGBA_UINT8:
- case MESA_FORMAT_RGBA_SINT8:
- return (devinfo->gen >= 9 ? format :
- devinfo->gen >= 8 || devinfo->is_haswell ?
- MESA_FORMAT_RGBA_UINT8 : MESA_FORMAT_R_UINT32);
-
- case MESA_FORMAT_RG_UINT16:
- case MESA_FORMAT_RG_SINT16:
- case MESA_FORMAT_RG_FLOAT16:
- return (devinfo->gen >= 9 ? format :
- devinfo->gen >= 8 || devinfo->is_haswell ?
- MESA_FORMAT_RG_UINT16 : MESA_FORMAT_R_UINT32);
-
- case MESA_FORMAT_RG_UINT8:
- case MESA_FORMAT_RG_SINT8:
- return (devinfo->gen >= 9 ? format :
- devinfo->gen >= 8 || devinfo->is_haswell ?
- MESA_FORMAT_RG_UINT8 : MESA_FORMAT_R_UINT16);
-
- case MESA_FORMAT_R_UINT16:
- case MESA_FORMAT_R_FLOAT16:
- case MESA_FORMAT_R_SINT16:
- return (devinfo->gen >= 9 ? format : MESA_FORMAT_R_UINT16);
-
- case MESA_FORMAT_R_UINT8:
- case MESA_FORMAT_R_SINT8:
- return (devinfo->gen >= 9 ? format : MESA_FORMAT_R_UINT8);
-
- /* Neither the 2/10/10/10 nor the 11/11/10 packed formats are supported
- * by the hardware.
- */
- case MESA_FORMAT_R10G10B10A2_UINT:
- case MESA_FORMAT_R10G10B10A2_UNORM:
- case MESA_FORMAT_R11G11B10_FLOAT:
- return MESA_FORMAT_R_UINT32;
-
- /* No normalized fixed-point formats are supported by the hardware. */
- case MESA_FORMAT_RGBA_UNORM16:
- case MESA_FORMAT_RGBA_SNORM16:
- return (devinfo->gen >= 8 || devinfo->is_haswell ?
- MESA_FORMAT_RGBA_UINT16 : MESA_FORMAT_RG_UINT32);
-
- case MESA_FORMAT_R8G8B8A8_UNORM:
- case MESA_FORMAT_R8G8B8A8_SNORM:
- return (devinfo->gen >= 8 || devinfo->is_haswell ?
- MESA_FORMAT_RGBA_UINT8 : MESA_FORMAT_R_UINT32);
-
- case MESA_FORMAT_R16G16_UNORM:
- case MESA_FORMAT_R16G16_SNORM:
- return (devinfo->gen >= 8 || devinfo->is_haswell ?
- MESA_FORMAT_RG_UINT16 : MESA_FORMAT_R_UINT32);
-
- case MESA_FORMAT_R8G8_UNORM:
- case MESA_FORMAT_R8G8_SNORM:
- return (devinfo->gen >= 8 || devinfo->is_haswell ?
- MESA_FORMAT_RG_UINT8 : MESA_FORMAT_R_UINT16);
-
- case MESA_FORMAT_R_UNORM16:
- case MESA_FORMAT_R_SNORM16:
- return MESA_FORMAT_R_UINT16;
-
- case MESA_FORMAT_R_UNORM8:
- case MESA_FORMAT_R_SNORM8:
- return MESA_FORMAT_R_UINT8;
-
- default:
- unreachable("Unknown image format");
- }
-}
diff --git a/src/mesa/drivers/dri/i965/brw_wm_surface_state.c b/src/mesa/drivers/dri/i965/brw_wm_surface_state.c
index a305f11..719ec1f 100644
--- a/src/mesa/drivers/dri/i965/brw_wm_surface_state.c
+++ b/src/mesa/drivers/dri/i965/brw_wm_surface_state.c
@@ -1144,14 +1144,9 @@ get_image_format(struct brw_context *brw, mesa_format format, GLenum access)
(_mesa_get_format_bytes(format) >= 8 &&
(brw->gen == 7 && !brw->is_haswell)))
return BRW_SURFACEFORMAT_RAW;
- else {
- uint32_t format1 = brw_format_for_mesa_format(
- brw_lower_mesa_image_format(brw->intelScreen->devinfo, format));
- uint32_t format2 = brw_lower_image_format(
- brw->intelScreen->devinfo, brw_format_for_mesa_format(format));
- assert(format1 == format2);
- return format1;
- }
+ else
+ return brw_lower_image_format(brw->intelScreen->devinfo,
+ brw_format_for_mesa_format(format));
}
}
--
2.5.0.400.gff86faf
More information about the mesa-dev
mailing list