[PATCH 5/9] drm/amdgpu: cleanup UVD coding style

Alexandre Demers alexandre.f.demers at gmail.com
Fri Jul 8 01:57:20 UTC 2016


Reviewed-by: Alexandre Demers <alexandre.f.demers at gmail.com>

Alexandre Demers

On 2016-07-06 15:02, Christian König wrote:
> From: Christian König <christian.koenig at amd.com>
>
> Cleanup 80 chars limit.
>
> Signed-off-by: Christian König <christian.koenig at amd.com>
> ---
>   drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c | 7 +++----
>   1 file changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c
> index acdd0ba..5b4fd0b 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c
> @@ -40,7 +40,7 @@
>   #include "uvd/uvd_4_2_d.h"
>   
>   /* 1 second timeout */
> -#define UVD_IDLE_TIMEOUT_MS	1000
> +#define UVD_IDLE_TIMEOUT	msecs_to_jiffies(1000)
>   /* Polaris10/11 firmware version */
>   #define FW_1_66_16 ((1 << 24) | (66 << 16) | (16 << 8))
>   
> @@ -1109,8 +1109,7 @@ static void amdgpu_uvd_idle_work_handler(struct work_struct *work)
>   			amdgpu_asic_set_uvd_clocks(adev, 0, 0);
>   		}
>   	} else {
> -		schedule_delayed_work(&adev->uvd.idle_work,
> -				      msecs_to_jiffies(UVD_IDLE_TIMEOUT_MS));
> +		schedule_delayed_work(&adev->uvd.idle_work, UVD_IDLE_TIMEOUT);
>   	}
>   }
>   
> @@ -1118,7 +1117,7 @@ void amdgpu_uvd_note_usage(struct amdgpu_device *adev)
>   {
>   	bool set_clocks = !cancel_delayed_work_sync(&adev->uvd.idle_work);
>   	set_clocks &= schedule_delayed_work(&adev->uvd.idle_work,
> -					    msecs_to_jiffies(UVD_IDLE_TIMEOUT_MS));
> +					    UVD_IDLE_TIMEOUT);
>   
>   	if (set_clocks) {
>   		if (adev->pm.dpm_enabled) {



More information about the amd-gfx mailing list