Reviewed-by: Alex Hung <[email protected]>
On 6/2/26 12:59, Aurabindo Pillai wrote:
force_frl_rate, select_ffe and limit_ffe in dc_debug_options carry
non-negative configuration values: an FRL link-rate enum (0..0xF), an
FFE level selector and an FFE level limit. They are only ever compared
against 0/0xF, assigned, or cast to uint8_t before being written to
hardware. No call site relies on signed semantics.
Make the types unsigned int to match how the values are actually used
and to silence MISRA-style signedness warnings on internal builds.
Signed-off-by: Aurabindo Pillai <[email protected]>
---
drivers/gpu/drm/amd/display/dc/dc.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/amd/display/dc/dc.h
b/drivers/gpu/drm/amd/display/dc/dc.h
index c6db021a61b0..82d02ebbd829 100644
--- a/drivers/gpu/drm/amd/display/dc/dc.h
+++ b/drivers/gpu/drm/amd/display/dc/dc.h
@@ -1155,10 +1155,10 @@ struct dc_debug_options {
bool force_vrr;
bool force_fva;
int max_frl_rate;
- int force_frl_rate;
+ unsigned int force_frl_rate;
bool ignore_ffe;
- int select_ffe;
- int limit_ffe;
+ unsigned int select_ffe;
+ unsigned int limit_ffe;
bool force_frl_always;
bool force_frl_dsc;
bool force_frl_max;
