Re: [PATCH 1/2] drm/amd/display: widen dc_hdmi_frl_flags.force_frl_rate to unsigned int

[Alex Hung]

Reviewed-by: Alex Hung <alex.h...@amd.com>

On 6/2/26 13:21, Aurabindo Pillai wrote:

dc_hdmi_frl_flags.force_frl_rate mirrors dc_debug_options.force_frl_rate,
which was just widened to unsigned int. Match the type here too so the
assignment in link_hdmi_frl.c does not narrow from unsigned to signed.

All call sites in link_hdmi_frl.c only compare the value against 0, 0xF,
or an hdmi_frl_link_rate enum whose values are non-negative, so the
change is behaviour-preserving and does not introduce sign-compare
warnings.

Signed-off-by: Aurabindo Pillai <aurabindo.pil...@amd.com>
---
  drivers/gpu/drm/amd/display/dc/dc_hdmi_types.h | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/display/dc/dc_hdmi_types.h 
b/drivers/gpu/drm/amd/display/dc/dc_hdmi_types.h
index 0da03eb794aa..eb6e7f4043fd 100644
--- a/drivers/gpu/drm/amd/display/dc/dc_hdmi_types.h
+++ b/drivers/gpu/drm/amd/display/dc/dc_hdmi_types.h
@@ -266,7 +266,7 @@ struct dc_hdmi_frl_link_settings {
  };

struct dc_hdmi_frl_flags {

-       int  force_frl_rate;
+       unsigned int force_frl_rate;
        bool ignore_ffe;
        int  select_ffe;
        int  limit_ffe;