Reviewed-by: Alex Deucher <alexander.deucher@xxxxxxx> On Thu, Jun 29, 2023 at 12:47 AM Srinivasan Shanmugam <srinivasan.shanmugam@xxxxxxx> wrote: > > Fix the following warnings reported by checkpatch: > > WARNING: Missing a blank line after declarations > WARNING: Prefer 'unsigned int' to bare use of 'unsigned' > > Cc: Rodrigo Siqueira <Rodrigo.Siqueira@xxxxxxx> > Cc: Aurabindo Pillai <aurabindo.pillai@xxxxxxx> > Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam@xxxxxxx> > --- > drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c | 1 + > drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.c | 4 ++-- > drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_psr.c | 1 + > 3 files changed, 4 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c > index 46d0a8f57e55..95eefa6b4f2f 100644 > --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c > +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c > @@ -296,6 +296,7 @@ static int dm_dp_mst_get_modes(struct drm_connector *connector) > > if (!aconnector->edid) { > struct edid *edid; > + > edid = drm_dp_mst_get_edid(connector, &aconnector->mst_root->mst_mgr, aconnector->mst_output_port); > > if (!edid) { > diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.c > index 322668973747..de1c7026ffcd 100644 > --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.c > +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.c > @@ -164,7 +164,7 @@ static bool modifier_has_dcc(uint64_t modifier) > return IS_AMD_FMT_MOD(modifier) && AMD_FMT_MOD_GET(DCC, modifier); > } > > -static unsigned modifier_gfx9_swizzle_mode(uint64_t modifier) > +static unsigned int modifier_gfx9_swizzle_mode(uint64_t modifier) > { > if (modifier == DRM_FORMAT_MOD_LINEAR) > return 0; > @@ -581,7 +581,7 @@ static void add_gfx11_modifiers(struct amdgpu_device *adev, > int pkrs = 0; > u32 gb_addr_config; > u8 i = 0; > - unsigned swizzle_r_x; > + unsigned int swizzle_r_x; > uint64_t modifier_r_x; > uint64_t modifier_dcc_best; > uint64_t modifier_dcc_4k; > diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_psr.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_psr.c > index 4f61d4f257cd..08ce3bb8f640 100644 > --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_psr.c > +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_psr.c > @@ -166,6 +166,7 @@ bool amdgpu_dm_psr_enable(struct dc_stream_state *stream) > */ > if (vsync_rate_hz != 0) { > unsigned int frame_time_microsec = 1000000 / vsync_rate_hz; > + > num_frames_static = (30000 / frame_time_microsec) + 1; > } > > -- > 2.25.1 >