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; -- 2.54.0
