提交 182519f4 编写于 作者: M Michael Krufky 提交者: Mauro Carvalho Chehab

V4L/DVB (6927): tda18271: improve printk macros

Signed-off-by: NMichael Krufky <mkrufky@linuxtv.org>
Signed-off-by: NMauro Carvalho Chehab <mchehab@infradead.org>
上级 2f27dfc9
...@@ -76,23 +76,23 @@ static void tda18271_dump_regs(struct dvb_frontend *fe) ...@@ -76,23 +76,23 @@ static void tda18271_dump_regs(struct dvb_frontend *fe)
struct tda18271_priv *priv = fe->tuner_priv; struct tda18271_priv *priv = fe->tuner_priv;
unsigned char *regs = priv->tda18271_regs; unsigned char *regs = priv->tda18271_regs;
dbg_reg("=== TDA18271 REG DUMP ===\n"); tda_reg("=== TDA18271 REG DUMP ===\n");
dbg_reg("ID_BYTE = 0x%02x\n", 0xff & regs[R_ID]); tda_reg("ID_BYTE = 0x%02x\n", 0xff & regs[R_ID]);
dbg_reg("THERMO_BYTE = 0x%02x\n", 0xff & regs[R_TM]); tda_reg("THERMO_BYTE = 0x%02x\n", 0xff & regs[R_TM]);
dbg_reg("POWER_LEVEL_BYTE = 0x%02x\n", 0xff & regs[R_PL]); tda_reg("POWER_LEVEL_BYTE = 0x%02x\n", 0xff & regs[R_PL]);
dbg_reg("EASY_PROG_BYTE_1 = 0x%02x\n", 0xff & regs[R_EP1]); tda_reg("EASY_PROG_BYTE_1 = 0x%02x\n", 0xff & regs[R_EP1]);
dbg_reg("EASY_PROG_BYTE_2 = 0x%02x\n", 0xff & regs[R_EP2]); tda_reg("EASY_PROG_BYTE_2 = 0x%02x\n", 0xff & regs[R_EP2]);
dbg_reg("EASY_PROG_BYTE_3 = 0x%02x\n", 0xff & regs[R_EP3]); tda_reg("EASY_PROG_BYTE_3 = 0x%02x\n", 0xff & regs[R_EP3]);
dbg_reg("EASY_PROG_BYTE_4 = 0x%02x\n", 0xff & regs[R_EP4]); tda_reg("EASY_PROG_BYTE_4 = 0x%02x\n", 0xff & regs[R_EP4]);
dbg_reg("EASY_PROG_BYTE_5 = 0x%02x\n", 0xff & regs[R_EP5]); tda_reg("EASY_PROG_BYTE_5 = 0x%02x\n", 0xff & regs[R_EP5]);
dbg_reg("CAL_POST_DIV_BYTE = 0x%02x\n", 0xff & regs[R_CPD]); tda_reg("CAL_POST_DIV_BYTE = 0x%02x\n", 0xff & regs[R_CPD]);
dbg_reg("CAL_DIV_BYTE_1 = 0x%02x\n", 0xff & regs[R_CD1]); tda_reg("CAL_DIV_BYTE_1 = 0x%02x\n", 0xff & regs[R_CD1]);
dbg_reg("CAL_DIV_BYTE_2 = 0x%02x\n", 0xff & regs[R_CD2]); tda_reg("CAL_DIV_BYTE_2 = 0x%02x\n", 0xff & regs[R_CD2]);
dbg_reg("CAL_DIV_BYTE_3 = 0x%02x\n", 0xff & regs[R_CD3]); tda_reg("CAL_DIV_BYTE_3 = 0x%02x\n", 0xff & regs[R_CD3]);
dbg_reg("MAIN_POST_DIV_BYTE = 0x%02x\n", 0xff & regs[R_MPD]); tda_reg("MAIN_POST_DIV_BYTE = 0x%02x\n", 0xff & regs[R_MPD]);
dbg_reg("MAIN_DIV_BYTE_1 = 0x%02x\n", 0xff & regs[R_MD1]); tda_reg("MAIN_DIV_BYTE_1 = 0x%02x\n", 0xff & regs[R_MD1]);
dbg_reg("MAIN_DIV_BYTE_2 = 0x%02x\n", 0xff & regs[R_MD2]); tda_reg("MAIN_DIV_BYTE_2 = 0x%02x\n", 0xff & regs[R_MD2]);
dbg_reg("MAIN_DIV_BYTE_3 = 0x%02x\n", 0xff & regs[R_MD3]); tda_reg("MAIN_DIV_BYTE_3 = 0x%02x\n", 0xff & regs[R_MD3]);
} }
static void tda18271_read_regs(struct dvb_frontend *fe) static void tda18271_read_regs(struct dvb_frontend *fe)
...@@ -116,8 +116,7 @@ static void tda18271_read_regs(struct dvb_frontend *fe) ...@@ -116,8 +116,7 @@ static void tda18271_read_regs(struct dvb_frontend *fe)
tda18271_i2c_gate_ctrl(fe, 0); tda18271_i2c_gate_ctrl(fe, 0);
if (ret != 2) if (ret != 2)
printk("ERROR: %s: i2c_transfer returned: %d\n", tda_err("ERROR: i2c_transfer returned: %d\n", ret);
__FUNCTION__, ret);
if (tda18271_debug & DBG_REG) if (tda18271_debug & DBG_REG)
tda18271_dump_regs(fe); tda18271_dump_regs(fe);
...@@ -147,8 +146,7 @@ static void tda18271_write_regs(struct dvb_frontend *fe, int idx, int len) ...@@ -147,8 +146,7 @@ static void tda18271_write_regs(struct dvb_frontend *fe, int idx, int len)
tda18271_i2c_gate_ctrl(fe, 0); tda18271_i2c_gate_ctrl(fe, 0);
if (ret != 1) if (ret != 1)
printk(KERN_WARNING "ERROR: %s: i2c_transfer returned: %d\n", tda_err("ERROR: i2c_transfer returned: %d\n", ret);
__FUNCTION__, ret);
} }
/*---------------------------------------------------------------------*/ /*---------------------------------------------------------------------*/
...@@ -158,7 +156,8 @@ static int tda18271_init_regs(struct dvb_frontend *fe) ...@@ -158,7 +156,8 @@ static int tda18271_init_regs(struct dvb_frontend *fe)
struct tda18271_priv *priv = fe->tuner_priv; struct tda18271_priv *priv = fe->tuner_priv;
unsigned char *regs = priv->tda18271_regs; unsigned char *regs = priv->tda18271_regs;
printk(KERN_INFO "tda18271: initializing registers\n"); tda_dbg("initializing registers for device @ %d-%04x\n",
i2c_adapter_id(priv->i2c_adap), priv->i2c_addr);
/* initialize registers */ /* initialize registers */
regs[R_ID] = 0x83; regs[R_ID] = 0x83;
...@@ -417,7 +416,7 @@ static int tda18271_tune(struct dvb_frontend *fe, ...@@ -417,7 +416,7 @@ static int tda18271_tune(struct dvb_frontend *fe,
tda18271_init(fe); tda18271_init(fe);
dbg_info("freq = %d, ifc = %d\n", freq, ifc); tda_dbg("freq = %d, ifc = %d\n", freq, ifc);
/* RF tracking filter calibration */ /* RF tracking filter calibration */
...@@ -602,8 +601,7 @@ static int tda18271_set_params(struct dvb_frontend *fe, ...@@ -602,8 +601,7 @@ static int tda18271_set_params(struct dvb_frontend *fe,
sgIF = 4000000; sgIF = 4000000;
break; break;
default: default:
printk(KERN_WARNING "%s: modulation not set!\n", tda_warn("modulation not set!\n");
__FUNCTION__);
return -EINVAL; return -EINVAL;
} }
#if 0 #if 0
...@@ -629,13 +627,11 @@ static int tda18271_set_params(struct dvb_frontend *fe, ...@@ -629,13 +627,11 @@ static int tda18271_set_params(struct dvb_frontend *fe,
sgIF = 4300000; sgIF = 4300000;
break; break;
default: default:
printk(KERN_WARNING "%s: bandwidth not set!\n", tda_warn("bandwidth not set!\n");
__FUNCTION__);
return -EINVAL; return -EINVAL;
} }
} else { } else {
printk(KERN_WARNING "%s: modulation type not supported!\n", tda_warn("modulation type not supported!\n");
__FUNCTION__);
return -EINVAL; return -EINVAL;
} }
...@@ -690,7 +686,7 @@ static int tda18271_set_analog_params(struct dvb_frontend *fe, ...@@ -690,7 +686,7 @@ static int tda18271_set_analog_params(struct dvb_frontend *fe,
if (params->mode == V4L2_TUNER_RADIO) if (params->mode == V4L2_TUNER_RADIO)
sgIF = 88; /* if frequency is 5.5 MHz */ sgIF = 88; /* if frequency is 5.5 MHz */
dbg_info("setting tda18271 to system %s\n", mode); tda_dbg("setting tda18271 to system %s\n", mode);
return tda18271_tune(fe, sgIF * 62500, params->frequency * 62500, return tda18271_tune(fe, sgIF * 62500, params->frequency * 62500,
0, std); 0, std);
...@@ -740,7 +736,7 @@ static int tda18271_get_id(struct dvb_frontend *fe) ...@@ -740,7 +736,7 @@ static int tda18271_get_id(struct dvb_frontend *fe)
break; break;
} }
dbg_info("%s detected @ %d-%04x%s\n", name, tda_info("%s detected @ %d-%04x%s\n", name,
i2c_adapter_id(priv->i2c_adap), priv->i2c_addr, i2c_adapter_id(priv->i2c_adap), priv->i2c_addr,
(0 == ret) ? "" : ", device not supported."); (0 == ret) ? "" : ", device not supported.");
......
...@@ -90,17 +90,23 @@ struct tda18271_priv { ...@@ -90,17 +90,23 @@ struct tda18271_priv {
extern int tda18271_debug; extern int tda18271_debug;
#define dprintk(level, fmt, arg...) do {\
if (tda18271_debug & level) \
printk(KERN_DEBUG "%s: " fmt, __FUNCTION__, ##arg); } while (0)
#define DBG_INFO 1 #define DBG_INFO 1
#define DBG_MAP 2 #define DBG_MAP 2
#define DBG_REG 4 #define DBG_REG 4
#define dbg_info(fmt, arg...) dprintk(DBG_INFO, fmt, ##arg) #define tda_printk(kern, fmt, arg...) \
#define dbg_map(fmt, arg...) dprintk(DBG_MAP, fmt, ##arg) printk(kern "%s: " fmt, __FUNCTION__, ##arg)
#define dbg_reg(fmt, arg...) dprintk(DBG_REG, fmt, ##arg)
#define dprintk(kern, lvl, fmt, arg...) do {\
if (tda18271_debug & lvl) \
tda_printk(kern, fmt, ##arg); } while (0)
#define tda_info(fmt, arg...) printk(KERN_INFO fmt, ##arg)
#define tda_warn(fmt, arg...) tda_printk(KERN_WARNING, fmt, ##arg)
#define tda_err(fmt, arg...) tda_printk(KERN_ERR, fmt, ##arg)
#define tda_dbg(fmt, arg...) dprintk(KERN_DEBUG, DBG_INFO, fmt, ##arg)
#define tda_map(fmt, arg...) dprintk(KERN_DEBUG, DBG_MAP, fmt, ##arg)
#define tda_reg(fmt, arg...) dprintk(KERN_DEBUG, DBG_REG, fmt, ##arg)
/*---------------------------------------------------------------------*/ /*---------------------------------------------------------------------*/
......
...@@ -290,7 +290,7 @@ int tda18271_lookup_pll_map(enum tda18271_map_type map_type, ...@@ -290,7 +290,7 @@ int tda18271_lookup_pll_map(enum tda18271_map_type map_type,
} }
if (!map) { if (!map) {
dbg_info("%s map is not set!\n", map_name); tda_warn("%s map is not set!\n", map_name);
return -EINVAL; return -EINVAL;
} }
...@@ -302,7 +302,7 @@ int tda18271_lookup_pll_map(enum tda18271_map_type map_type, ...@@ -302,7 +302,7 @@ int tda18271_lookup_pll_map(enum tda18271_map_type map_type,
*post_div = map[i].pd; *post_div = map[i].pd;
*div = map[i].d; *div = map[i].d;
dbg_map("%s: post div = 0x%02x, div = 0x%02x\n", tda_map("%s: post div = 0x%02x, div = 0x%02x\n",
map_name, *post_div, *div); map_name, *post_div, *div);
return 0; return 0;
...@@ -346,7 +346,7 @@ int tda18271_lookup_map(enum tda18271_map_type map_type, u32 *freq, u8 *val) ...@@ -346,7 +346,7 @@ int tda18271_lookup_map(enum tda18271_map_type map_type, u32 *freq, u8 *val)
} }
if (!map) { if (!map) {
dbg_info("%s map is not set!\n", map_name); tda_warn("%s map is not set!\n", map_name);
return -EINVAL; return -EINVAL;
} }
...@@ -357,7 +357,7 @@ int tda18271_lookup_map(enum tda18271_map_type map_type, u32 *freq, u8 *val) ...@@ -357,7 +357,7 @@ int tda18271_lookup_map(enum tda18271_map_type map_type, u32 *freq, u8 *val)
} }
*val = map[i].val; *val = map[i].val;
dbg_map("%s: 0x%02x\n", map_name, *val); tda_map("%s: 0x%02x\n", map_name, *val);
return 0; return 0;
} }
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册