On Mon, Jun 22, 2015 at 4:02 PM, Nanley Chery <nanleych...@gmail.com> wrote:
> From: Nanley Chery <nanley.g.ch...@intel.com>
>
> Intel surface formats default to LDR unless there is hardware
> support for HDR and the texture is able to be processed in HDR mode.
>
> v2: remove extra newlines.
> v3: follow existing coding style in translate_tex_format().
>
> Signed-off-by: Nanley Chery <nanley.g.ch...@intel.com>
> ---
> src/mesa/drivers/dri/i965/brw_defines.h | 32 ++++++++++
> src/mesa/drivers/dri/i965/brw_surface_formats.c | 80
> +++++++++++++++++++++++++
> 2 files changed, 112 insertions(+)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_defines.h
> b/src/mesa/drivers/dri/i965/brw_defines.h
> index bfcc442..da5d434 100644
> --- a/src/mesa/drivers/dri/i965/brw_defines.h
> +++ b/src/mesa/drivers/dri/i965/brw_defines.h
> @@ -504,6 +504,38 @@
> #define BRW_SURFACEFORMAT_R8G8B8_UINT 0x1C8
> #define BRW_SURFACEFORMAT_R8G8B8_SINT 0x1C9
> #define BRW_SURFACEFORMAT_RAW 0x1FF
> +
> +#define GEN9_SURFACE_ASTC_HDR_FORMAT_BIT 0x100
> +
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_4x4_U8sRGB 0x200
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_5x4_U8sRGB 0x208
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_5x5_U8sRGB 0x209
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_6x5_U8sRGB 0x211
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_6x6_U8sRGB 0x212
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_8x5_U8sRGB 0x221
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_8x6_U8sRGB 0x222
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_8x8_U8sRGB 0x224
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_10x5_U8sRGB 0x231
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_10x6_U8sRGB 0x232
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_10x8_U8sRGB 0x234
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_10x10_U8sRGB 0x236
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_12x10_U8sRGB 0x23E
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_12x12_U8sRGB 0x23F
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_4x4_FLT16 0x240
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_5x4_FLT16 0x248
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_5x5_FLT16 0x249
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_6x5_FLT16 0x251
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_6x6_FLT16 0x252
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_8x5_FLT16 0x261
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_8x6_FLT16 0x262
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_8x8_FLT16 0x264
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_10x5_FLT16 0x271
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_10x6_FLT16 0x272
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_10x8_FLT16 0x274
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_10x10_FLT16 0x276
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_12x10_FLT16 0x27E
> +#define BRW_SURFACEFORMAT_ASTC_LDR_2D_12x12_FLT16 0x27F
> +
> #define BRW_SURFACE_FORMAT_SHIFT 18
> #define BRW_SURFACE_FORMAT_MASK INTEL_MASK(26, 18)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_surface_formats.c
> b/src/mesa/drivers/dri/i965/brw_surface_formats.c
> index 0501606..a896b79 100644
> --- a/src/mesa/drivers/dri/i965/brw_surface_formats.c
> +++ b/src/mesa/drivers/dri/i965/brw_surface_formats.c
> @@ -307,6 +307,34 @@ const struct surface_format_info surface_formats[] = {
> SF( x, x, x, x, x, x, x, x, x, ETC2_EAC_SRGB8_A8)
> SF( x, x, x, x, x, x, x, x, x, R8G8B8_UINT)
> SF( x, x, x, x, x, x, x, x, x, R8G8B8_SINT)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_4x4_FLT16)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_5x4_FLT16)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_5x5_FLT16)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_6x5_FLT16)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_6x6_FLT16)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_8x5_FLT16)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_8x6_FLT16)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_8x8_FLT16)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_10x5_FLT16)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_10x6_FLT16)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_10x8_FLT16)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_10x10_FLT16)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_12x10_FLT16)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_12x12_FLT16)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_4x4_U8sRGB)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_5x4_U8sRGB)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_5x5_U8sRGB)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_6x5_U8sRGB)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_6x6_U8sRGB)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_8x5_U8sRGB)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_8x6_U8sRGB)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_8x8_U8sRGB)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_10x5_U8sRGB)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_10x6_U8sRGB)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_10x8_U8sRGB)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_10x10_U8sRGB)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_12x10_U8sRGB)
> + SF(90, 90, x, x, x, x, x, x, x, ASTC_LDR_2D_12x12_U8sRGB)
> };
As we discussed change 90 to 80.
> #undef x
> #undef Y
> @@ -503,6 +531,35 @@ brw_format_for_mesa_format(mesa_format mesa_format)
> [MESA_FORMAT_BPTC_RGB_SIGNED_FLOAT] = BRW_SURFACEFORMAT_BC6H_SF16,
> [MESA_FORMAT_BPTC_RGB_UNSIGNED_FLOAT] = BRW_SURFACEFORMAT_BC6H_UF16,
>
> + [MESA_FORMAT_ASTC_4x4_RGBA] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_4x4_FLT16,
> + [MESA_FORMAT_ASTC_5x4_RGBA] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_5x4_FLT16,
> + [MESA_FORMAT_ASTC_5x5_RGBA] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_5x5_FLT16,
> + [MESA_FORMAT_ASTC_6x5_RGBA] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_6x5_FLT16,
> + [MESA_FORMAT_ASTC_6x6_RGBA] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_6x6_FLT16,
> + [MESA_FORMAT_ASTC_8x5_RGBA] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_8x5_FLT16,
> + [MESA_FORMAT_ASTC_8x6_RGBA] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_8x6_FLT16,
> + [MESA_FORMAT_ASTC_8x8_RGBA] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_8x8_FLT16,
> + [MESA_FORMAT_ASTC_10x5_RGBA] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_10x5_FLT16,
> + [MESA_FORMAT_ASTC_10x6_RGBA] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_10x6_FLT16,
> + [MESA_FORMAT_ASTC_10x8_RGBA] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_10x8_FLT16,
> + [MESA_FORMAT_ASTC_10x10_RGBA] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_10x10_FLT16,
> + [MESA_FORMAT_ASTC_12x10_RGBA] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_12x10_FLT16,
> + [MESA_FORMAT_ASTC_12x12_RGBA] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_12x12_FLT16,
> + [MESA_FORMAT_ASTC_4x4_SRGB8_ALPHA8] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_4x4_U8sRGB,
> + [MESA_FORMAT_ASTC_5x4_SRGB8_ALPHA8] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_5x4_U8sRGB,
> + [MESA_FORMAT_ASTC_5x5_SRGB8_ALPHA8] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_5x5_U8sRGB,
> + [MESA_FORMAT_ASTC_6x5_SRGB8_ALPHA8] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_6x5_U8sRGB,
> + [MESA_FORMAT_ASTC_6x6_SRGB8_ALPHA8] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_6x6_U8sRGB,
> + [MESA_FORMAT_ASTC_8x5_SRGB8_ALPHA8] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_8x5_U8sRGB,
> + [MESA_FORMAT_ASTC_8x6_SRGB8_ALPHA8] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_8x6_U8sRGB,
> + [MESA_FORMAT_ASTC_8x8_SRGB8_ALPHA8] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_8x8_U8sRGB,
> + [MESA_FORMAT_ASTC_10x5_SRGB8_ALPHA8] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_10x5_U8sRGB,
> + [MESA_FORMAT_ASTC_10x6_SRGB8_ALPHA8] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_10x6_U8sRGB,
> + [MESA_FORMAT_ASTC_10x8_SRGB8_ALPHA8] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_10x8_U8sRGB,
> + [MESA_FORMAT_ASTC_10x10_SRGB8_ALPHA8] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_10x10_U8sRGB,
> + [MESA_FORMAT_ASTC_12x10_SRGB8_ALPHA8] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_12x10_U8sRGB,
> + [MESA_FORMAT_ASTC_12x12_SRGB8_ALPHA8] =
> BRW_SURFACEFORMAT_ASTC_LDR_2D_12x12_U8sRGB,
> +
> [MESA_FORMAT_A_SNORM8] = 0,
> [MESA_FORMAT_L_SNORM8] = 0,
> [MESA_FORMAT_L8A8_SNORM] = 0,
> @@ -768,6 +825,29 @@ translate_tex_format(struct brw_context *brw,
> }
> return brw_format_for_mesa_format(mesa_format);
>
> + case MESA_FORMAT_ASTC_4x4_RGBA:
> + case MESA_FORMAT_ASTC_5x4_RGBA:
> + case MESA_FORMAT_ASTC_5x5_RGBA:
> + case MESA_FORMAT_ASTC_6x5_RGBA:
> + case MESA_FORMAT_ASTC_6x6_RGBA:
> + case MESA_FORMAT_ASTC_8x5_RGBA:
> + case MESA_FORMAT_ASTC_8x6_RGBA:
> + case MESA_FORMAT_ASTC_8x8_RGBA:
> + case MESA_FORMAT_ASTC_10x5_RGBA:
> + case MESA_FORMAT_ASTC_10x6_RGBA:
> + case MESA_FORMAT_ASTC_10x8_RGBA:
> + case MESA_FORMAT_ASTC_10x10_RGBA:
> + case MESA_FORMAT_ASTC_12x10_RGBA:
> + case MESA_FORMAT_ASTC_12x12_RGBA: {
> + GLuint brw_fmt = brw_format_for_mesa_format(mesa_format);
> +
> + /* Process this texture using the Full Profile if possible. */
Just say enable HDR/Full profile bit on SKL+.
> + if (brw->gen >= 9)
> + brw_fmt |= GEN9_SURFACE_ASTC_HDR_FORMAT_BIT;
> +
> + return brw_fmt;
> + }
> +
> default:
> assert(brw_format_for_mesa_format(mesa_format) != 0);
> return brw_format_for_mesa_format(mesa_format);
> --
> 2.4.2
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Reviewed-by: Anuj Phogat <anuj.pho...@gmail.com>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
