[PATCH] drm/amdgpu: delete set-but-not-read member has_uvd from amdgpu_device
Christian König
deathsimple at vodafone.de
Fri Mar 4 09:25:05 UTC 2016
Hi Nils,
Nice catch and yes that's just obviously a leftover from the radeon
code. In amdgpu we handle missing and present engines completely
differently.
Your patch looks good to me, but has some style problems. You should add
a commit message and a signed-of-by line (use git commit --amend -s).
Regards,
Christian.
Am 03.03.2016 um 21:17 schrieb Nils Wallménius:
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu.h | 1 -
> drivers/gpu/drm/amd/amdgpu/cik.c | 2 --
> drivers/gpu/drm/amd/amdgpu/vi.c | 4 ----
> 3 files changed, 7 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
> index 0c42a85..cfd35b0 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu.h
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
> @@ -2053,7 +2053,6 @@ struct amdgpu_device {
> struct amdgpu_sdma sdma;
>
> /* uvd */
> - bool has_uvd;
> struct amdgpu_uvd uvd;
>
> /* vce */
> diff --git a/drivers/gpu/drm/amd/amdgpu/cik.c b/drivers/gpu/drm/amd/amdgpu/cik.c
> index 6b1f053..5da14a3 100644
> --- a/drivers/gpu/drm/amd/amdgpu/cik.c
> +++ b/drivers/gpu/drm/amd/amdgpu/cik.c
> @@ -2025,8 +2025,6 @@ static int cik_common_early_init(void *handle)
>
> adev->asic_funcs = &cik_asic_funcs;
>
> - adev->has_uvd = true;
> -
> adev->rev_id = cik_get_rev_id(adev);
> adev->external_rev_id = 0xFF;
> switch (adev->asic_type) {
> diff --git a/drivers/gpu/drm/amd/amdgpu/vi.c b/drivers/gpu/drm/amd/amdgpu/vi.c
> index 1250035..9163f59 100644
> --- a/drivers/gpu/drm/amd/amdgpu/vi.c
> +++ b/drivers/gpu/drm/amd/amdgpu/vi.c
> @@ -1071,26 +1071,22 @@ static int vi_common_early_init(void *handle)
> adev->external_rev_id = 0xFF;
> switch (adev->asic_type) {
> case CHIP_TOPAZ:
> - adev->has_uvd = false;
> adev->cg_flags = 0;
> adev->pg_flags = 0;
> adev->external_rev_id = 0x1;
> break;
> case CHIP_FIJI:
> - adev->has_uvd = true;
> adev->cg_flags = 0;
> adev->pg_flags = 0;
> adev->external_rev_id = adev->rev_id + 0x3c;
> break;
> case CHIP_TONGA:
> - adev->has_uvd = true;
> adev->cg_flags = 0;
> adev->pg_flags = 0;
> adev->external_rev_id = adev->rev_id + 0x14;
> break;
> case CHIP_CARRIZO:
> case CHIP_STONEY:
> - adev->has_uvd = true;
> adev->cg_flags = 0;
> /* Disable UVD pg */
> adev->pg_flags = /* AMDGPU_PG_SUPPORT_UVD | */AMDGPU_PG_SUPPORT_VCE;
More information about the dri-devel
mailing list