diff options
| author |   Srinivasan Shanmugam <srinivasan.shanmugam@amd.com> | 2023-08-12 20:17:24 +0530 |
|---|---|---|
| committer |   Alex Deucher <alexander.deucher@amd.com> | 2023-08-15 18:08:28 -0400 |
| commit | 44fd83e920e2ec0322ad82320ca575445b5e135e (patch) | |
| tree | fe4134fad1f911abda35a1ba720705a5d99722f6 /drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | |
| parent | drm/amd/display: Add some missing register definitions (diff) | |
| download | linux-rng-44fd83e920e2ec0322ad82320ca575445b5e135e.tar.xz linux-rng-44fd83e920e2ec0322ad82320ca575445b5e135e.zip | |
drm/amdgpu: Replace ternary operator with min() in 'amdgpu_iomem_read'
Fixes the following coccicheck:
drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c:2427:16-17: WARNING opportunity for min()
min() macro is defined in include/linux/minmax.h. It avoids multiple
evaluations of the arguments when non-constant and performs strict
type-checking.
Cc: Guchun Chen <guchun.chen@amd.com>
Cc: Christian König <christian.koenig@amd.com>
Cc: Alex Deucher <alexander.deucher@amd.com>
Cc: "Pan, Xinhui" <Xinhui.Pan@amd.com>
Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c')
| -rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c index dea848bb55c1..d9adc520f656 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c @@ -2420,7 +2420,7 @@ static ssize_t amdgpu_iomem_read(struct file *f, char __user *buf, struct page *p; void *ptr; - bytes = bytes < size ? bytes : size; + bytes = min(bytes, size); /* Translate the bus address to a physical address. If * the domain is NULL it means there is no IOMMU active |