diff options
| author | John Törnblom <john.tornblom@gmail.com> | 2012-12-17 12:53:54 +0100 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2012-12-27 21:24:16 +0100 |
| commit | 1ca6ae8de8a563f69eebe114d023855b4f0bcb1b (patch) | |
| tree | c351474e9df9ae9511ecf9dab8f5e1210d2192c1 /drivers/media/pci/bt8xx | |
| parent | [media] omap3isp: ispqueue: Fix uninitialized variable compiler warnings (diff) | |
| download | linux-1ca6ae8de8a563f69eebe114d023855b4f0bcb1b.tar.xz linux-1ca6ae8de8a563f69eebe114d023855b4f0bcb1b.zip | |
[media] bttv: avoid flooding the kernel log when i2c debugging is disabled
When the bttv driver is running without i2c_debug being set, the kernel
log is being flooded with the string ">". This string is really a part of
a debug message that is logged using several substrings protected by a
conditional check.
This patch adds the same conditional check to the leaked substring.
Signed-off-by: John Törnblom <john.tornblom@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/pci/bt8xx')
| -rw-r--r-- | drivers/media/pci/bt8xx/bttv-i2c.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/pci/bt8xx/bttv-i2c.c b/drivers/media/pci/bt8xx/bttv-i2c.c index 580c8e682392..da400dbe4a10 100644 --- a/drivers/media/pci/bt8xx/bttv-i2c.c +++ b/drivers/media/pci/bt8xx/bttv-i2c.c @@ -173,7 +173,7 @@ bttv_i2c_sendbytes(struct bttv *btv, const struct i2c_msg *msg, int last) if (i2c_debug) pr_cont(" %02x", msg->buf[cnt]); } - if (!(xmit & BT878_I2C_NOSTOP)) + if (i2c_debug && !(xmit & BT878_I2C_NOSTOP)) pr_cont(">\n"); return msg->len; |