Re: [PATCH] intel_ips: quieten "power or thermal limit exceeded" messages

[Alan Cox <alan@lxorguk.ukuu.org.uk>]

On Thu, 26 Aug 2010 20:29:01 -0300
Cesar Eduardo Barros <cesarb@cesarb.net> wrote:

> On my Dell Inspiron N4010, one of these messages is printed every five
> seconds. Change both to dev_dbg to quieten them even more.

It's not the kind of thing you want to quieten since in most cases it is
stuff you want to know about.

> 
> Signed-off-by: Cesar Eduardo Barros <cesarb@cesarb.net>
> ---
>  drivers/platform/x86/intel_ips.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/platform/x86/intel_ips.c b/drivers/platform/x86/intel_ips.c
> index 9024480..7833adc 100644
> --- a/drivers/platform/x86/intel_ips.c
> +++ b/drivers/platform/x86/intel_ips.c
> @@ -607,7 +607,7 @@ static bool mcp_exceeded(struct ips_driver *ips)
>  	spin_unlock_irqrestore(&ips->turbo_status_lock, flags);
>  
>  	if (ret)
> -		dev_info(&ips->dev->dev,
> +		dev_dbg(&ips->dev->dev,
>  			 "MCP power or thermal limit exceeded\n");

Probably better to ratelimit it. Being told it occurs is important, being
told every five seconds is indeed excessive.

Alan