Reviewed-by: Luben Tuikov <[email protected]>

Regards,
Luben

On 2023-04-05 05:45, Lijo Lazar wrote:
> Fix below warning due to incompatible types in conditional operator
> 
> ../pm/swsmu/smu13/smu_v13_0_6_ppt.c:315:17: sparse: sparse: incompatible
> types in conditional expression (different base types):
> 
> Signed-off-by: Lijo Lazar <[email protected]>
> Reported-by: kernel test robot <[email protected]>
> Link: 
> https://lore.kernel.org/oe-kbuild-all/[email protected]/
> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h 
> b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
> index bbac4239ceb3..376d14de7602 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu.h
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
> @@ -1241,7 +1241,7 @@ int emu_soc_asic_init(struct amdgpu_device *adev);
>       ((adev)->asic_funcs->flush_hdp ? (adev)->asic_funcs->flush_hdp((adev), 
> (r)) : (adev)->hdp.funcs->flush_hdp((adev), (r)))
>  #define amdgpu_asic_invalidate_hdp(adev, r) \
>       ((adev)->asic_funcs->invalidate_hdp ? 
> (adev)->asic_funcs->invalidate_hdp((adev), (r)) : \
> -      ((adev)->hdp.funcs->invalidate_hdp ? 
> (adev)->hdp.funcs->invalidate_hdp((adev), (r)) : 0))
> +      ((adev)->hdp.funcs->invalidate_hdp ? 
> (adev)->hdp.funcs->invalidate_hdp((adev), (r)) : (void)0))
>  #define amdgpu_asic_need_full_reset(adev) 
> (adev)->asic_funcs->need_full_reset((adev))
>  #define amdgpu_asic_init_doorbell_index(adev) 
> (adev)->asic_funcs->init_doorbell_index((adev))
>  #define amdgpu_asic_get_pcie_usage(adev, cnt0, cnt1) 
> ((adev)->asic_funcs->get_pcie_usage((adev), (cnt0), (cnt1)))

Reply via email to