Mesa (staging/21.1): intel/isl: Fix isl_color_value_unpack to match the prototype
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Fri Apr 30 17:26:24 UTC 2021
Module: Mesa
Branch: staging/21.1
Commit: dd20621ac3647a7ffcc258884e6b37eb33df62c7
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=dd20621ac3647a7ffcc258884e6b37eb33df62c7
Author: Jason Ekstrand <jason at jlekstrand.net>
Date: Thu Apr 29 22:47:35 2021 -0500
intel/isl: Fix isl_color_value_unpack to match the prototype
The prototype uses a pointer and the actual function definition had an
array. For some reason, GCC never complained about this until GCC 11.
This fixes a compile warning when building with GCC 11.
Fixes: 09ced6542049 "intel/isl: Add format conversion code"
Reviewed-by: Tapani Pälli <tapani.palli at intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/10537>
(cherry picked from commit b80720acb13e1014aea89e6bd25f22d43df85356)
---
.pick_status.json | 2 +-
src/intel/isl/isl_format.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/.pick_status.json b/.pick_status.json
index 9b6f075d8cb..71a322989c3 100644
--- a/.pick_status.json
+++ b/.pick_status.json
@@ -4,7 +4,7 @@
"description": "intel/isl: Fix isl_color_value_unpack to match the prototype",
"nominated": true,
"nomination_type": 1,
- "resolution": 0,
+ "resolution": 1,
"master_sha": null,
"because_sha": "09ced6542049986f7fe52af8087aec9fc23d9f16"
},
diff --git a/src/intel/isl/isl_format.c b/src/intel/isl/isl_format.c
index 5443f6ea0c7..9864e69dc26 100644
--- a/src/intel/isl/isl_format.c
+++ b/src/intel/isl/isl_format.c
@@ -1318,7 +1318,7 @@ unpack_channel(union isl_color_value *value,
void
isl_color_value_unpack(union isl_color_value *value,
enum isl_format format,
- const uint32_t data_in[4])
+ const uint32_t *data_in)
{
const struct isl_format_layout *fmtl = isl_format_get_layout(format);
assert(fmtl->colorspace == ISL_COLORSPACE_LINEAR ||
More information about the mesa-commit
mailing list