[PATCH] drm/amd/powerplay: use pr_debug to print function not implemented message

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

 



Reviewed-by: Edward O'Callaghan <funfunctor at folklore1984.net>

On 12/08/2016 06:28 AM, Alex Deucher wrote:
> From: Arindam Nath <arindam.nath at amd.com>
> 
> For CZ, some function pointers like display_configuration_changed
> are not set. So when HW manager tries to configure display, we
> end up with messages like
> 
> "[ powerplay ] this function not implement!"
> 
> in the logs. This message is informational, but lacks details
> on which function is not implemented and why.
> 
> Rather than using KERN_INFO to print the message everytime the
> system boots, we rather use pr_debug so that it is only printed
> when debug prints are enabled at runtime.
> 
> Reviewed-by: Huang Rui <ray.huang at amd.com>
> Signed-off-by: Arindam Nath <arindam.nath at amd.com>
> Signed-off-by: Alex Deucher <alexander.deucher at amd.com>
> ---
>  drivers/gpu/drm/amd/powerplay/hwmgr/functiontables.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/functiontables.c b/drivers/gpu/drm/amd/powerplay/hwmgr/functiontables.c
> index a6abe81..71822ae 100644
> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/functiontables.c
> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/functiontables.c
> @@ -35,7 +35,7 @@ static int phm_run_table(struct pp_hwmgr *hwmgr,
>  	phm_table_function *function;
>  
>  	if (rt_table->function_list == NULL) {
> -		printk(KERN_INFO "[ powerplay ] this function not implement!\n");
> +		pr_debug("[ powerplay ] this function not implement!\n");
>  		return 0;
>  	}
>  
> 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20161208/b06d4adf/attachment.sig>


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

  Powered by Linux