Commit 6752c797 authored by Mauro Carvalho Chehab's avatar Mauro Carvalho Chehab Committed by Mauro Carvalho Chehab

media: lgdt330x: use pr_foo() macros

Cleanup the usecases of dprintk() by using pr_fmt() and replace
printk by pr_foo().
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab+samsung@kernel.org>
parent 467845a1
...@@ -29,6 +29,8 @@ ...@@ -29,6 +29,8 @@
* *
*/ */
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
#include <linux/kernel.h> #include <linux/kernel.h>
#include <linux/module.h> #include <linux/module.h>
#include <linux/init.h> #include <linux/init.h>
...@@ -48,9 +50,11 @@ ...@@ -48,9 +50,11 @@
static int debug; static int debug;
module_param(debug, int, 0644); module_param(debug, int, 0644);
MODULE_PARM_DESC(debug, "Turn on/off lgdt330x frontend debugging (default:off)."); MODULE_PARM_DESC(debug, "Turn on/off lgdt330x frontend debugging (default:off).");
#define dprintk(args...) do { \
if (debug) \ #define dprintk(fmt, arg...) do { \
printk(KERN_DEBUG "lgdt330x: " args); \ if (debug) \
printk(KERN_DEBUG pr_fmt("%s: " fmt), \
__func__, ##arg); \
} while (0) } while (0)
struct lgdt330x_state { struct lgdt330x_state {
...@@ -85,8 +89,8 @@ static int i2c_write_demod_bytes(struct lgdt330x_state *state, ...@@ -85,8 +89,8 @@ static int i2c_write_demod_bytes(struct lgdt330x_state *state,
for (i = 0; i < len - 1; i += 2) { for (i = 0; i < len - 1; i += 2) {
err = i2c_transfer(state->i2c, &msg, 1); err = i2c_transfer(state->i2c, &msg, 1);
if (err != 1) { if (err != 1) {
printk(KERN_WARNING "lgdt330x: %s error (addr %02x <- %02x, err = %i)\n", pr_warn("%s: error (addr %02x <- %02x, err = %i)\n",
__func__, msg.buf[0], msg.buf[1], err); __func__, msg.buf[0], msg.buf[1], err);
if (err < 0) if (err < 0)
return err; return err;
else else
...@@ -122,8 +126,8 @@ static int i2c_read_demod_bytes(struct lgdt330x_state *state, ...@@ -122,8 +126,8 @@ static int i2c_read_demod_bytes(struct lgdt330x_state *state,
ret = i2c_transfer(state->i2c, msg, 2); ret = i2c_transfer(state->i2c, msg, 2);
if (ret != 2) { if (ret != 2) {
printk(KERN_WARNING "lgdt330x: %s: addr 0x%02x select 0x%02x error (ret == %i)\n", pr_warn("%s: addr 0x%02x select 0x%02x error (ret == %i)\n",
__func__, state->config->demod_address, reg, ret); __func__, state->config->demod_address, reg, ret);
if (ret >= 0) if (ret >= 0)
ret = -EIO; ret = -EIO;
} else { } else {
...@@ -295,7 +299,7 @@ static int lgdt330x_init(struct dvb_frontend *fe) ...@@ -295,7 +299,7 @@ static int lgdt330x_init(struct dvb_frontend *fe)
printk(KERN_WARNING "Only LGDT3302 and LGDT3303 are supported chips.\n"); printk(KERN_WARNING "Only LGDT3302 and LGDT3303 are supported chips.\n");
err = -ENODEV; err = -ENODEV;
} }
dprintk("%s entered as %s\n", __func__, chip_name); dprintk("entered as %s\n", chip_name);
if (err < 0) if (err < 0)
return err; return err;
return lgdt330x_sw_reset(state); return lgdt330x_sw_reset(state);
...@@ -378,7 +382,7 @@ static int lgdt330x_set_parameters(struct dvb_frontend *fe) ...@@ -378,7 +382,7 @@ static int lgdt330x_set_parameters(struct dvb_frontend *fe)
if (state->current_modulation != p->modulation) { if (state->current_modulation != p->modulation) {
switch (p->modulation) { switch (p->modulation) {
case VSB_8: case VSB_8:
dprintk("%s: VSB_8 MODE\n", __func__); dprintk("VSB_8 MODE\n");
/* Select VSB mode */ /* Select VSB mode */
top_ctrl_cfg[1] = 0x03; top_ctrl_cfg[1] = 0x03;
...@@ -395,7 +399,7 @@ static int lgdt330x_set_parameters(struct dvb_frontend *fe) ...@@ -395,7 +399,7 @@ static int lgdt330x_set_parameters(struct dvb_frontend *fe)
break; break;
case QAM_64: case QAM_64:
dprintk("%s: QAM_64 MODE\n", __func__); dprintk("QAM_64 MODE\n");
/* Select QAM_64 mode */ /* Select QAM_64 mode */
top_ctrl_cfg[1] = 0x00; top_ctrl_cfg[1] = 0x00;
...@@ -412,7 +416,7 @@ static int lgdt330x_set_parameters(struct dvb_frontend *fe) ...@@ -412,7 +416,7 @@ static int lgdt330x_set_parameters(struct dvb_frontend *fe)
break; break;
case QAM_256: case QAM_256:
dprintk("%s: QAM_256 MODE\n", __func__); dprintk("QAM_256 MODE\n");
/* Select QAM_256 mode */ /* Select QAM_256 mode */
top_ctrl_cfg[1] = 0x01; top_ctrl_cfg[1] = 0x01;
...@@ -428,13 +432,13 @@ static int lgdt330x_set_parameters(struct dvb_frontend *fe) ...@@ -428,13 +432,13 @@ static int lgdt330x_set_parameters(struct dvb_frontend *fe)
} }
break; break;
default: default:
printk(KERN_WARNING "lgdt330x: %s: Modulation type(%d) UNSUPPORTED\n", pr_warn("%s: Modulation type(%d) UNSUPPORTED\n",
__func__, p->modulation); __func__, p->modulation);
return -1; return -1;
} }
if (err < 0) if (err < 0)
printk(KERN_WARNING "lgdt330x: %s: error blasting bytes to lgdt3303 for modulation type(%d)\n", pr_warn("%s: error blasting bytes to lgdt3303 for modulation type(%d)\n",
__func__, p->modulation); __func__, p->modulation);
/* /*
* select serial or parallel MPEG hardware interface * select serial or parallel MPEG hardware interface
...@@ -488,7 +492,7 @@ static int lgdt3302_read_status(struct dvb_frontend *fe, ...@@ -488,7 +492,7 @@ static int lgdt3302_read_status(struct dvb_frontend *fe,
/* AGC status register */ /* AGC status register */
i2c_read_demod_bytes(state, AGC_STATUS, buf, 1); i2c_read_demod_bytes(state, AGC_STATUS, buf, 1);
dprintk("%s: AGC_STATUS = 0x%02x\n", __func__, buf[0]); dprintk("AGC_STATUS = 0x%02x\n", buf[0]);
if ((buf[0] & 0x0c) == 0x8) { if ((buf[0] & 0x0c) == 0x8) {
/* /*
* Test signal does not exist flag * Test signal does not exist flag
...@@ -505,8 +509,8 @@ static int lgdt3302_read_status(struct dvb_frontend *fe, ...@@ -505,8 +509,8 @@ static int lgdt3302_read_status(struct dvb_frontend *fe,
/* signal status */ /* signal status */
i2c_read_demod_bytes(state, TOP_CONTROL, buf, sizeof(buf)); i2c_read_demod_bytes(state, TOP_CONTROL, buf, sizeof(buf));
dprintk("%s: TOP_CONTROL = 0x%02x, IRO_MASK = 0x%02x, IRQ_STATUS = 0x%02x\n", dprintk("TOP_CONTROL = 0x%02x, IRO_MASK = 0x%02x, IRQ_STATUS = 0x%02x\n",
__func__, buf[0], buf[1], buf[2]); buf[0], buf[1], buf[2]);
/* sync status */ /* sync status */
if ((buf[2] & 0x03) == 0x01) if ((buf[2] & 0x03) == 0x01)
...@@ -520,7 +524,7 @@ static int lgdt3302_read_status(struct dvb_frontend *fe, ...@@ -520,7 +524,7 @@ static int lgdt3302_read_status(struct dvb_frontend *fe,
/* Carrier Recovery Lock Status Register */ /* Carrier Recovery Lock Status Register */
i2c_read_demod_bytes(state, CARRIER_LOCK, buf, 1); i2c_read_demod_bytes(state, CARRIER_LOCK, buf, 1);
dprintk("%s: CARRIER_LOCK = 0x%02x\n", __func__, buf[0]); dprintk("CARRIER_LOCK = 0x%02x\n", buf[0]);
switch (state->current_modulation) { switch (state->current_modulation) {
case QAM_256: case QAM_256:
case QAM_64: case QAM_64:
...@@ -533,8 +537,8 @@ static int lgdt3302_read_status(struct dvb_frontend *fe, ...@@ -533,8 +537,8 @@ static int lgdt3302_read_status(struct dvb_frontend *fe,
*status |= FE_HAS_CARRIER; *status |= FE_HAS_CARRIER;
break; break;
default: default:
printk(KERN_WARNING "lgdt330x: %s: Modulation set to unsupported value\n", pr_warn("%s: Modulation set to unsupported value\n",
__func__); __func__);
} }
return 0; return 0;
...@@ -554,7 +558,7 @@ static int lgdt3303_read_status(struct dvb_frontend *fe, ...@@ -554,7 +558,7 @@ static int lgdt3303_read_status(struct dvb_frontend *fe,
if (err < 0) if (err < 0)
return err; return err;
dprintk("%s: AGC_STATUS = 0x%02x\n", __func__, buf[0]); dprintk("AGC_STATUS = 0x%02x\n", buf[0]);
if ((buf[0] & 0x21) == 0x01) { if ((buf[0] & 0x21) == 0x01) {
/* /*
* Test input signal does not exist flag * Test input signal does not exist flag
...@@ -565,7 +569,7 @@ static int lgdt3303_read_status(struct dvb_frontend *fe, ...@@ -565,7 +569,7 @@ static int lgdt3303_read_status(struct dvb_frontend *fe,
/* Carrier Recovery Lock Status Register */ /* Carrier Recovery Lock Status Register */
i2c_read_demod_bytes(state, CARRIER_LOCK, buf, 1); i2c_read_demod_bytes(state, CARRIER_LOCK, buf, 1);
dprintk("%s: CARRIER_LOCK = 0x%02x\n", __func__, buf[0]); dprintk("CARRIER_LOCK = 0x%02x\n", buf[0]);
switch (state->current_modulation) { switch (state->current_modulation) {
case QAM_256: case QAM_256:
case QAM_64: case QAM_64:
...@@ -596,8 +600,8 @@ static int lgdt3303_read_status(struct dvb_frontend *fe, ...@@ -596,8 +600,8 @@ static int lgdt3303_read_status(struct dvb_frontend *fe,
} }
break; break;
default: default:
printk(KERN_WARNING "lgdt330x: %s: Modulation set to unsupported value\n", pr_warn("%s: Modulation set to unsupported value\n",
__func__); __func__);
} }
return 0; return 0;
} }
...@@ -673,7 +677,7 @@ static int lgdt3302_read_snr(struct dvb_frontend *fe, u16 *snr) ...@@ -673,7 +677,7 @@ static int lgdt3302_read_snr(struct dvb_frontend *fe, u16 *snr)
/* log10(688128)*2^24 and log10(696320)*2^24 */ /* log10(688128)*2^24 and log10(696320)*2^24 */
break; break;
default: default:
printk(KERN_ERR "lgdt330x: %s: Modulation set to unsupported value\n", pr_err("%s: Modulation set to unsupported value\n",
__func__); __func__);
return -EREMOTEIO; /* return -EDRIVER_IS_GIBBERED; */ return -EREMOTEIO; /* return -EDRIVER_IS_GIBBERED; */
} }
...@@ -681,7 +685,7 @@ static int lgdt3302_read_snr(struct dvb_frontend *fe, u16 *snr) ...@@ -681,7 +685,7 @@ static int lgdt3302_read_snr(struct dvb_frontend *fe, u16 *snr)
state->snr = calculate_snr(noise, c); state->snr = calculate_snr(noise, c);
*snr = (state->snr) >> 16; /* Convert from 8.24 fixed-point to 8.8 */ *snr = (state->snr) >> 16; /* Convert from 8.24 fixed-point to 8.8 */
dprintk("%s: noise = 0x%08x, snr = %d.%02d dB\n", __func__, noise, dprintk("noise = 0x%08x, snr = %d.%02d dB\n", noise,
state->snr >> 24, (((state->snr >> 8) & 0xffff) * 100) >> 16); state->snr >> 24, (((state->snr >> 8) & 0xffff) * 100) >> 16);
return 0; return 0;
...@@ -717,7 +721,7 @@ static int lgdt3303_read_snr(struct dvb_frontend *fe, u16 *snr) ...@@ -717,7 +721,7 @@ static int lgdt3303_read_snr(struct dvb_frontend *fe, u16 *snr)
/* log10(688128)*2^24 and log10(696320)*2^24 */ /* log10(688128)*2^24 and log10(696320)*2^24 */
break; break;
default: default:
printk(KERN_ERR "lgdt330x: %s: Modulation set to unsupported value\n", pr_err("%s: Modulation set to unsupported value\n",
__func__); __func__);
return -EREMOTEIO; /* return -EDRIVER_IS_GIBBERED; */ return -EREMOTEIO; /* return -EDRIVER_IS_GIBBERED; */
} }
...@@ -725,7 +729,7 @@ static int lgdt3303_read_snr(struct dvb_frontend *fe, u16 *snr) ...@@ -725,7 +729,7 @@ static int lgdt3303_read_snr(struct dvb_frontend *fe, u16 *snr)
state->snr = calculate_snr(noise, c); state->snr = calculate_snr(noise, c);
*snr = (state->snr) >> 16; /* Convert from 8.24 fixed-point to 8.8 */ *snr = (state->snr) >> 16; /* Convert from 8.24 fixed-point to 8.8 */
dprintk("%s: noise = 0x%08x, snr = %d.%02d dB\n", __func__, noise, dprintk("noise = 0x%08x, snr = %d.%02d dB\n", noise,
state->snr >> 24, (((state->snr >> 8) & 0xffff) * 100) >> 16); state->snr >> 24, (((state->snr >> 8) & 0xffff) * 100) >> 16);
return 0; return 0;
...@@ -817,7 +821,7 @@ struct dvb_frontend *lgdt330x_attach(const struct lgdt330x_config *config, ...@@ -817,7 +821,7 @@ struct dvb_frontend *lgdt330x_attach(const struct lgdt330x_config *config,
error: error:
kfree(state); kfree(state);
dprintk("%s: ERROR\n", __func__); dprintk("ERROR\n");
return NULL; return NULL;
} }
EXPORT_SYMBOL(lgdt330x_attach); EXPORT_SYMBOL(lgdt330x_attach);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment