RE: [PATCH 3/3] drm/amdgpu: add printing to indicate rpm completeness

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Re: [Quan, Evan] Maybe better with something like "asic/device is runtime suspended".

Sure, will update it like this when pushing.

Regards,
Guchun

-----Original Message-----
From: Quan, Evan <Evan.Quan@xxxxxxx> 
Sent: Wednesday, November 23, 2022 3:37 PM
To: Chen, Guchun <Guchun.Chen@xxxxxxx>; amd-gfx@xxxxxxxxxxxxxxxxxxxxx; Deucher, Alexander <Alexander.Deucher@xxxxxxx>; Zhang, Hawking <Hawking.Zhang@xxxxxxx>; Lazar, Lijo <Lijo.Lazar@xxxxxxx>
Subject: RE: [PATCH 3/3] drm/amdgpu: add printing to indicate rpm completeness

[AMD Official Use Only - General]



> -----Original Message-----
> From: Chen, Guchun <Guchun.Chen@xxxxxxx>
> Sent: Wednesday, November 23, 2022 9:44 AM
> To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx; Deucher, Alexander 
> <Alexander.Deucher@xxxxxxx>; Zhang, Hawking <Hawking.Zhang@xxxxxxx>; 
> Lazar, Lijo <Lijo.Lazar@xxxxxxx>; Quan, Evan <Evan.Quan@xxxxxxx>
> Cc: Chen, Guchun <Guchun.Chen@xxxxxxx>
> Subject: [PATCH 3/3] drm/amdgpu: add printing to indicate rpm 
> completeness
> 
> Add an explicit printing to tell when finishing rpm execution in amdgpu.
> 
> Signed-off-by: Guchun Chen <guchun.chen@xxxxxxx>
> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
> index bf2d50c8c92a..fa42c0fcf848 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
> @@ -2572,6 +2572,8 @@ static int amdgpu_pmops_runtime_suspend(struct
> device *dev)
>  		amdgpu_device_baco_enter(drm_dev);
>  	}
> 
> +	dev_info(&pdev->dev, "amdgpu finishes runtime suspend\n");
[Quan, Evan] Maybe better with something like "asic/device is runtime suspended". 
Anyway, the series is Reviewed-by: Evan Quan <evan.quan@xxxxxxx>

Evan
> +
>  	return 0;
>  }
> 
> --
> 2.25.1

<<attachment: winmail.dat>>


[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux