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) {
