[PATCH] drm/amd/display: fix static checker warning

Harry Wentland harry.wentland at amd.com
Tue Nov 14 15:26:09 UTC 2017


On 2017-11-10 04:14 AM, S, Shirish wrote:
> From: Shirish S <shirish.s at amd.com>
> 
> This patch fixes static checker warning of
> "warn: cast after binop" introduced by
> 56087b31 drm/amd/display: fix high part address in dm_plane_helper_prepare_fb()
> 
> Signed-off-by: Shirish S <shirish.s at amd.com>

Reviewed-by: Harry Wentland <harry.wentland at amd.com>

Feel free to push to amd-staging-drm-next at your leisure.

Harry

> ---
>  drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> index ed8b7524..0537523e 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> @@ -2955,7 +2955,7 @@ static int dm_plane_helper_prepare_fb(struct drm_plane *plane,
>  							= lower_32_bits(afb->address);
>  			plane_state->address.video_progressive.luma_addr.high_part
>  							= upper_32_bits(afb->address);
> -			chroma_addr = afb->address + (u64)(awidth * new_state->fb->height);
> +			chroma_addr = afb->address + (u64)awidth * new_state->fb->height;
>  			plane_state->address.video_progressive.chroma_addr.low_part
>  							= lower_32_bits(chroma_addr);
>  			plane_state->address.video_progressive.chroma_addr.high_part
> 


More information about the amd-gfx mailing list