public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Manu Abraham <abraham.manu@gmail.com>
To: Mauro Carvalho Chehab <mchehab@infradead.org>
Cc: Artem Makhutov <artem@makhutov.org>, linux-media@vger.kernel.org
Subject: Re: [PATCH] Remove debug output from stb6100_cfg.h
Date: Fri, 27 Mar 2009 23:09:07 +0400	[thread overview]
Message-ID: <49CD2453.4090906@gmail.com> (raw)
In-Reply-To: <20090327142417.4d80f166@pedra.chehab.org>

Mauro Carvalho Chehab wrote:
> Manu,
> 
> If ok to you, please ack.
> 
> On Thu, 26 Mar 2009 10:45:53 +0100
> Artem Makhutov <artem@makhutov.org> wrote:
> 
> This patch removes the debug output from stb6100_cfg.h as it is flooding
> the syslog with tuning data during normal operation.
> 
> Signed-off-by: Artem Makhutov <artem@makhutov.org>
>

Looks okay,

Acked-by: Manu Abraham <abraham.manu@linuxtv.org>


> --- linux.old/drivers/media/dvb/frontends/stb6100_cfg.h	2009-03-26 10:28:57.000000000 +0100
> +++ linux/drivers/media/dvb/frontends/stb6100_cfg.h	2009-03-26 10:29:52.000000000 +0100
> @@ -36,7 +36,6 @@
>  			return err;
>  		}
>  		*frequency = t_state.frequency;
> -		printk("%s: Frequency=%d\n", __func__, t_state.frequency);
>  	}
>  	return 0;
>  }
> @@ -59,7 +58,6 @@
>  			return err;
>  		}
>  	}
> -	printk("%s: Frequency=%d\n", __func__, t_state.frequency);
>  	return 0;
>  }
>  
> @@ -81,7 +79,6 @@
>  		}
>  		*bandwidth = t_state.bandwidth;
>  	}
> -	printk("%s: Bandwidth=%d\n", __func__, t_state.bandwidth);
>  	return 0;
>  }
>  
> @@ -103,6 +100,5 @@
>  			return err;
>  		}
>  	}
> -	printk("%s: Bandwidth=%d\n", __func__, t_state.bandwidth);
>  	return 0;
>  }
> 

      reply	other threads:[~2009-03-27 19:09 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-26  9:45 [PATCH] Remove debug output from stb6100_cfg.h Artem Makhutov
2009-03-27 17:24 ` Mauro Carvalho Chehab
2009-03-27 19:09   ` Manu Abraham [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=49CD2453.4090906@gmail.com \
    --to=abraham.manu@gmail.com \
    --cc=artem@makhutov.org \
    --cc=linux-media@vger.kernel.org \
    --cc=mchehab@infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox