summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJason Ekstrand <jason.ekstrand@intel.com>2018-05-11 15:02:13 -0700
committerJason Ekstrand <jason.ekstrand@intel.com>2018-05-14 10:41:26 -0700
commit18f8200a994440faa9fb9e80e99e8140ea912993 (patch)
tree96ddd164fb33365717afad30e5f538a313ee187a
parentf944a59996287de85d4c6d9b7b000d25f41b1d79 (diff)
intel/blorp: Use linear formats for CCS_E clear colors in copies
It's clear that the original code meant to do this and there is even a 10-line comment explaining why. Originally, we had a simple function for packing the clear colors which was unaware of sRGB. However, in a6b66a7b26ae1, when we started using ISL to do the packing, the wrong format was used. Fixes: a6b66a7b26 "intel/blorp: Use ISL instead of bitcast_color..." Reviewed-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
-rw-r--r--src/intel/blorp/blorp_blit.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/intel/blorp/blorp_blit.c b/src/intel/blorp/blorp_blit.c
index e825862d71..26bf4426c0 100644
--- a/src/intel/blorp/blorp_blit.c
+++ b/src/intel/blorp/blorp_blit.c
@@ -2562,7 +2562,7 @@ blorp_copy(struct blorp_batch *batch,
params.src.view.format));
uint32_t packed[4];
isl_color_value_pack(&params.src.clear_color,
- params.src.surf.format, packed);
+ linear_src_format, packed);
isl_color_value_unpack(&params.src.clear_color,
params.src.view.format, packed);
}
@@ -2576,7 +2576,7 @@ blorp_copy(struct blorp_batch *batch,
params.dst.view.format));
uint32_t packed[4];
isl_color_value_pack(&params.dst.clear_color,
- params.dst.surf.format, packed);
+ linear_dst_format, packed);
isl_color_value_unpack(&params.dst.clear_color,
params.dst.view.format, packed);
}