From b925c4be4ac2c71f572000b5cf2ab57aee29e4be Mon Sep 17 00:00:00 2001 From: Erik Faye-Lund Date: Mon, 8 Dec 2025 13:12:37 +0100 Subject: [PATCH] pan: make S8_UINT code behave like the rest There's no reason why the S8_UINT check should be written in a different way than the other checks here; let's make this consistent. Reviewed-by: Yiwei Zhang Reviewed-by: Eric R. Smith Tested-by: Yiwei Zhang Part-of: --- src/panfrost/lib/pan_afbc.h | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/src/panfrost/lib/pan_afbc.h b/src/panfrost/lib/pan_afbc.h index 9ddb2828b50..840e566cba9 100644 --- a/src/panfrost/lib/pan_afbc.h +++ b/src/panfrost/lib/pan_afbc.h @@ -567,6 +567,11 @@ pan_afbc_format(unsigned arch, enum pipe_format format, unsigned plane_idx) return PAN_AFBC_MODE_INVALID; else break; + case PIPE_FORMAT_S8_UINT: + if (arch < 9) + return PAN_AFBC_MODE_INVALID; + else + break; default: break; } @@ -592,9 +597,7 @@ pan_afbc_format(unsigned arch, enum pipe_format format, unsigned plane_idx) case PIPE_FORMAT_X24S8_UINT: return PAN_AFBC_MODE_R8G8B8A8; case PIPE_FORMAT_Z24_UNORM_PACKED: return PAN_AFBC_MODE_R8G8B8; - /* AFBC(S8) only supported on v9+ */ - case PIPE_FORMAT_S8_UINT: - return arch >= 9 ? PAN_AFBC_MODE_R8 : PAN_AFBC_MODE_INVALID; + case PIPE_FORMAT_S8_UINT: return PAN_AFBC_MODE_R8; case PIPE_FORMAT_R16_UNORM: return PAN_AFBC_MODE_R16; case PIPE_FORMAT_R16G16_UNORM: return PAN_AFBC_MODE_R16G16;