V4L/DVB (13045): Fix debug messaging for stv0900 demod.

Signed-off-by: Igor M. Liplianin <liplianin@netup.ru>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

3 files changed, 46 insertions, 45 deletions

diff --git a/drivers/media/dvb/frontends/stv0900_core.c b/drivers/media/dvb/frontends/stv0900_core.c
index 3bde3324a032..768a1611e8ce 100644
--- a/drivers/media/dvb/frontends/stv0900_core.c
+++ b/drivers/media/dvb/frontends/stv0900_core.c
@@ -34,7 +34,7 @@
 #include "stv0900_priv.h"
 #include "stv0900_init.h"
 
-static int stvdebug = 1;
+int stvdebug = 1;
 module_param_named(debug, stvdebug, int, 0644);
 
 /* internal params node */
@@ -146,7 +146,7 @@ void stv0900_write_reg(struct stv0900_internal *i_params, u16 reg_addr,
 
         ret = i2c_transfer(i_params->i2c_adap, &i2cmsg, 1);
         if (ret != 1)
-                dprintk(KERN_ERR "%s: i2c error %d\n", __func__, ret);
+                dprintk("%s: i2c error %d\n", __func__, ret);
 }
 
 u8 stv0900_read_reg(struct stv0900_internal *i_params, u16 reg)
@@ -170,7 +170,7 @@ u8 stv0900_read_reg(struct stv0900_internal *i_params, u16 reg)
 
         ret = i2c_transfer(i_params->i2c_adap, msg, 2);
         if (ret != 2)
-                dprintk(KERN_ERR "%s: i2c error %d, reg[0x%02x]\n",
+                dprintk("%s: i2c error %d, reg[0x%02x]\n",
                                 __func__, ret, reg);
 
         return buf;
@@ -281,7 +281,7 @@ u32 stv0900_get_mclk_freq(struct stv0900_internal *i_params, u32 ext_clk)
 
         mclk = (div + 1) * ext_clk / ad_div;
 
-        dprintk(KERN_INFO "%s: Calculated Mclk = %d\n", __func__, mclk);
+        dprintk("%s: Calculated Mclk = %d\n", __func__, mclk);
 
         return mclk;
 }
@@ -291,7 +291,7 @@ enum fe_stv0900_error stv0900_set_mclk(struct stv0900_internal *i_params, u32 mc
         enum fe_stv0900_error error = STV0900_NO_ERROR;
         u32 m_div, clk_sel;
 
-        dprintk(KERN_INFO "%s: Mclk set to %d, Quartz = %d\n", __func__, mclk,
+        dprintk("%s: Mclk set to %d, Quartz = %d\n", __func__, mclk,
                         i_params->quartz);
 
         if (i_params == NULL)
@@ -381,7 +381,7 @@ static void stv0900_set_ts_parallel_serial(struct stv0900_internal *i_params,
                                         enum fe_stv0900_clock_type path2_ts)
 {
 
-        dprintk(KERN_INFO "%s\n", __func__);
+        dprintk("%s\n", __func__);
 
         if (i_params->chip_id >= 0x20) {
                 switch (path1_ts) {
@@ -423,7 +423,7 @@ static void stv0900_set_ts_parallel_serial(struct stv0900_internal *i_params,
                         case STV0900_DVBCI_CLOCK:
                                 stv0900_write_reg(i_params,
                                                 R0900_TSGENERAL, 0x0A);
-                                dprintk(KERN_INFO "%s: 0x0a\n", __func__);
+                                dprintk("%s: 0x0a\n", __func__);
                                 break;
                         }
                         break;
@@ -469,7 +469,7 @@ static void stv0900_set_ts_parallel_serial(struct stv0900_internal *i_params,
                         case STV0900_DVBCI_CLOCK:
                                 stv0900_write_reg(i_params, R0900_TSGENERAL1X,
                                                         0x12);
-                                dprintk(KERN_INFO "%s: 0x12\n", __func__);
+                                dprintk("%s: 0x12\n", __func__);
                                 break;
                         }
 
@@ -584,7 +584,7 @@ static s32 stv0900_get_rf_level(struct stv0900_internal *i_params,
                 i,
                 rf_lvl = 0;
 
-        dprintk(KERN_INFO "%s\n", __func__);
+        dprintk("%s\n", __func__);
 
         if ((lookup != NULL) && lookup->size) {
                 switch (demod) {
@@ -622,7 +622,7 @@ static s32 stv0900_get_rf_level(struct stv0900_internal *i_params,
 
         }
 
-        dprintk(KERN_INFO "%s: RFLevel = %d\n", __func__, rf_lvl);
+        dprintk("%s: RFLevel = %d\n", __func__, rf_lvl);
 
         return rf_lvl;
 }
@@ -654,7 +654,7 @@ static s32 stv0900_carr_get_quality(struct dvb_frontend *fe,
                 noise_field1,
                 noise_field0;
 
-        dprintk(KERN_INFO "%s\n", __func__);
+        dprintk("%s\n", __func__);
 
         dmd_reg(lock_flag_field, F0900_P1_LOCK_DEFINITIF,
                                         F0900_P2_LOCK_DEFINITIF);
@@ -876,7 +876,7 @@ void stv0900_stop_all_s2_modcod(struct stv0900_internal *i_params,
         s32 regflist,
         i;
 
-        dprintk(KERN_INFO "%s\n", __func__);
+        dprintk("%s\n", __func__);
 
         dmd_reg(regflist, R0900_P1_MODCODLST0, R0900_P2_MODCODLST0);
 
@@ -893,7 +893,7 @@ void stv0900_activate_s2_modcode(struct stv0900_internal *i_params,
         reg_index,
         field_index;
 
-        dprintk(KERN_INFO "%s\n", __func__);
+        dprintk("%s\n", __func__);
 
         if (i_params->chip_id <= 0x11) {
                 msleep(5);
@@ -979,7 +979,7 @@ void stv0900_activate_s2_modcode_single(struct stv0900_internal *i_params,
 {
         u32 reg_index;
 
-        dprintk(KERN_INFO "%s\n", __func__);
+        dprintk("%s\n", __func__);
 
         switch (demod) {
         case STV0900_DEMOD_1:
@@ -1012,7 +1012,7 @@ static enum dvbfe_algo stv0900_frontend_algo(struct dvb_frontend *fe)
 static int stb0900_set_property(struct dvb_frontend *fe,
                                 struct dtv_property *tvp)
 {
-        dprintk(KERN_INFO "%s(..)\n", __func__);
+        dprintk("%s(..)\n", __func__);
 
         return 0;
 }
@@ -1020,7 +1020,7 @@ static int stb0900_set_property(struct dvb_frontend *fe,
 static int stb0900_get_property(struct dvb_frontend *fe,
                                 struct dtv_property *tvp)
 {
-        dprintk(KERN_INFO "%s(..)\n", __func__);
+        dprintk("%s(..)\n", __func__);
 
         return 0;
 }
@@ -1191,7 +1191,7 @@ u8 stv0900_get_optim_carr_loop(s32 srate, enum fe_stv0900_modcode modcode,
         s32     i;
         const struct stv0900_car_loop_optim *car_loop_s2;
 
-        dprintk(KERN_INFO "%s\n", __func__);
+        dprintk("%s\n", __func__);
 
         if (chip_id <= 0x12)
                 car_loop_s2 = FE_STV0900_S2CarLoop;
@@ -1294,7 +1294,7 @@ u8 stv0900_get_optim_short_carr_loop(s32 srate, enum fe_stv0900_modulation modul
 
         u8 aclc_value = 0x0b;
 
-        dprintk(KERN_INFO "%s\n", __func__);
+        dprintk("%s\n", __func__);
 
         switch (modulation) {
         case STV0900_QPSK:
@@ -1351,7 +1351,7 @@ static enum fe_stv0900_error stv0900_st_dvbs2_single(struct stv0900_internal *i_
 {
         enum fe_stv0900_error error = STV0900_NO_ERROR;
 
-        dprintk(KERN_INFO "%s\n", __func__);
+        dprintk("%s\n", __func__);
 
         switch (LDPC_Mode) {
         case STV0900_DUAL:
@@ -1398,12 +1398,12 @@ static enum fe_stv0900_error stv0900_init_internal(struct dvb_frontend *fe,
         struct stv0900_inode *temp_int = find_inode(state->i2c_adap,
                                                 state->config->demod_address);
 
-        dprintk(KERN_INFO "%s\n", __func__);
+        dprintk("%s\n", __func__);
 
         if (temp_int != NULL) {
                 state->internal = temp_int->internal;
                 (state->internal->dmds_used)++;
-                dprintk(KERN_INFO "%s: Find Internal Structure!\n", __func__);
+                dprintk("%s: Find Internal Structure!\n", __func__);
                 return STV0900_NO_ERROR;
         } else {
                 state->internal = kmalloc(sizeof(struct stv0900_internal), GFP_KERNEL);
@@ -1413,7 +1413,7 @@ static enum fe_stv0900_error stv0900_init_internal(struct dvb_frontend *fe,
                 state->internal->i2c_addr = state->config->demod_address;
                 state->internal->clkmode = state->config->clkmode;
                 state->internal->errs = STV0900_NO_ERROR;
-                dprintk(KERN_INFO "%s: Create New Internal Structure!\n", __func__);
+                dprintk("%s: Create New Internal Structure!\n", __func__);
         }
 
         if (state->internal != NULL) {
@@ -1551,7 +1551,7 @@ static enum dvbfe_search stv0900_search(struct dvb_frontend *fe,
 
         enum fe_stv0900_error error = STV0900_NO_ERROR;
 
-        dprintk(KERN_INFO "%s: ", __func__);
+        dprintk("%s: ", __func__);
 
         p_result.locked = FALSE;
         p_search.path = state->demod;
@@ -1652,10 +1652,10 @@ static enum dvbfe_search stv0900_search(struct dvb_frontend *fe,
                 error = STV0900_BAD_PARAMETER;
 
         if ((p_result.locked == TRUE) && (error == STV0900_NO_ERROR)) {
-                dprintk(KERN_INFO "Search Success\n");
+                dprintk("Search Success\n");
                 return DVBFE_ALGO_SEARCH_SUCCESS;
         } else {
-                dprintk(KERN_INFO "Search Fail\n");
+                dprintk("Search Fail\n");
                 return DVBFE_ALGO_SEARCH_FAILED;
         }
 
@@ -1723,7 +1723,7 @@ static int stv0900_diseqc_init(struct dvb_frontend *fe)
 
 static int stv0900_init(struct dvb_frontend *fe)
 {
-        dprintk(KERN_INFO "%s\n", __func__);
+        dprintk("%s\n", __func__);
 
         stv0900_stop_ts(fe, 1);
         stv0900_diseqc_init(fe);
@@ -1865,7 +1865,7 @@ static int stv0900_set_tone(struct dvb_frontend *fe, fe_sec_tone_mode_t tone)
         enum fe_stv0900_demod_num demod = state->demod;
         s32 mode_field, reset_field;
 
-        dprintk(KERN_INFO "%s: %s\n", __func__, ((tone == 0) ? "Off" : "On"));
+        dprintk("%s: %s\n", __func__, ((tone == 0) ? "Off" : "On"));
 
         dmd_reg(mode_field, F0900_P1_DISTX_MODE, F0900_P2_DISTX_MODE);
         dmd_reg(reset_field, F0900_P1_DISEQC_RESET, F0900_P2_DISEQC_RESET);
@@ -1889,11 +1889,11 @@ static void stv0900_release(struct dvb_frontend *fe)
 {
         struct stv0900_state *state = fe->demodulator_priv;
 
-        dprintk(KERN_INFO "%s\n", __func__);
+        dprintk("%s\n", __func__);
 
         if ((--(state->internal->dmds_used)) <= 0) {
 
-                dprintk(KERN_INFO "%s: Actually removing\n", __func__);
+                dprintk("%s: Actually removing\n", __func__);
 
                 remove_inode(state->internal);
                 kfree(state->internal);
diff --git a/drivers/media/dvb/frontends/stv0900_priv.h b/drivers/media/dvb/frontends/stv0900_priv.h
index 5ed7a145c7d3..5f4414f8da20 100644
--- a/drivers/media/dvb/frontends/stv0900_priv.h
+++ b/drivers/media/dvb/frontends/stv0900_priv.h
@@ -60,7 +60,7 @@
                 } \
         } while (0)
 
-static int stvdebug;
+extern int stvdebug;
 
 #define dprintk(args...) \
         do { \
diff --git a/drivers/media/dvb/frontends/stv0900_sw.c b/drivers/media/dvb/frontends/stv0900_sw.c
index 962fde1437ce..8c83c8b306a6 100644
--- a/drivers/media/dvb/frontends/stv0900_sw.c
+++ b/drivers/media/dvb/frontends/stv0900_sw.c
@@ -527,7 +527,7 @@ static void stv0900_set_symbol_rate(struct stv0900_internal *i_params,
         s32 sfr_init_reg;
         u32 symb;
 
-        dprintk(KERN_INFO "%s: Mclk %d, SR %d, Dmd %d\n", __func__, mclk,
+        dprintk("%s: Mclk %d, SR %d, Dmd %d\n", __func__, mclk,
                                                         srate, demod);
 
         dmd_reg(sfr_init_reg, R0900_P1_SFRINIT1, R0900_P2_SFRINIT1);
@@ -952,7 +952,7 @@ static void stv0900_set_viterbi_tracq(struct stv0900_internal *i_params,
 
         s32 vth_reg;
 
-        dprintk(KERN_INFO "%s\n", __func__);
+        dprintk("%s\n", __func__);
 
         dmd_reg(vth_reg, R0900_P1_VTH12, R0900_P2_VTH12);
 
@@ -973,7 +973,7 @@ static void stv0900_set_viterbi_standard(struct stv0900_internal *i_params,
         s32 fecmReg,
         prvitReg;
 
-        dprintk(KERN_INFO "%s: ViterbiStandard = ", __func__);
+        dprintk("%s: ViterbiStandard = ", __func__);
 
         switch (demod) {
         case STV0900_DEMOD_1:
@@ -1055,7 +1055,7 @@ static void stv0900_track_optimization(struct dvb_frontend *fe)
         enum fe_stv0900_rolloff rolloff;
         enum fe_stv0900_modcode foundModcod;
 
-        dprintk(KERN_INFO "%s\n", __func__);
+        dprintk("%s\n", __func__);
 
         srate = stv0900_get_symbol_rate(i_params, i_params->mclk, demod);
         srate += stv0900_get_timing_offst(i_params, srate, demod);
@@ -1370,7 +1370,7 @@ static int stv0900_get_fec_lock(struct stv0900_internal *i_params, enum fe_stv09
 
         enum fe_stv0900_search_state dmd_state;
 
-        dprintk(KERN_INFO "%s\n", __func__);
+        dprintk("%s\n", __func__);
 
         dmd_reg(header_field, F0900_P1_HEADER_MODE, F0900_P2_HEADER_MODE);
         dmd_reg(pktdelin_field, F0900_P1_PKTDELIN_LOCK, F0900_P2_PKTDELIN_LOCK);
@@ -1414,7 +1414,7 @@ static int stv0900_wait_for_lock(struct stv0900_internal *i_params,
 
         s32 timer = 0, lock = 0, str_merg_rst_fld, str_merg_lock_fld;
 
-        dprintk(KERN_INFO "%s\n", __func__);
+        dprintk("%s\n", __func__);
 
         dmd_reg(str_merg_rst_fld, F0900_P1_RST_HWARE, F0900_P2_RST_HWARE);
         dmd_reg(str_merg_lock_fld, F0900_P1_TSFIFO_LINEOK, F0900_P2_TSFIFO_LINEOK);
@@ -1427,7 +1427,8 @@ static int stv0900_wait_for_lock(struct stv0900_internal *i_params,
         if (lock) {
                 lock = 0;
 
-                dprintk(KERN_INFO "%s: Timer = %d, time_out = %d\n", __func__, timer, fec_timeout);
+                dprintk("%s: Timer = %d, time_out = %d\n", __func__, timer,
+                                                                fec_timeout);
 
                 while ((timer < fec_timeout) && (lock == 0)) {
                         lock = stv0900_get_bits(i_params, str_merg_lock_fld);
@@ -1437,9 +1438,9 @@ static int stv0900_wait_for_lock(struct stv0900_internal *i_params,
         }
 
         if (lock)
-                dprintk(KERN_INFO "%s: DEMOD LOCK OK\n", __func__);
+                dprintk("%s: DEMOD LOCK OK\n", __func__);
         else
-                dprintk(KERN_INFO "%s: DEMOD LOCK FAIL\n", __func__);
+                dprintk("%s: DEMOD LOCK FAIL\n", __func__);
 
         if (lock)
                 return TRUE;
@@ -1456,7 +1457,7 @@ enum fe_stv0900_tracking_standard stv0900_get_standard(struct dvb_frontend *fe,
         s32 state_field,
         dss_dvb_field;
 
-        dprintk(KERN_INFO "%s\n", __func__);
+        dprintk("%s\n", __func__);
 
         dmd_reg(state_field, F0900_P1_HEADER_MODE, F0900_P2_HEADER_MODE);
         dmd_reg(dss_dvb_field, F0900_P1_DSS_DVB, F0900_P2_DSS_DVB);
@@ -1820,7 +1821,7 @@ static u16 stv0900_blind_check_agc2_min_level(struct stv0900_internal *i_params,
 
         s32 i, j, nb_steps, direction;
 
-        dprintk(KERN_INFO "%s\n", __func__);
+        dprintk("%s\n", __func__);
 
         switch (demod) {
         case STV0900_DEMOD_1:
@@ -2274,7 +2275,7 @@ static int stv0900_blind_search_algo(struct dvb_frontend *fe)
         u16 agc2_integr;
         u8 dstatus2;
 
-        dprintk(KERN_INFO "%s\n", __func__);
+        dprintk("%s\n", __func__);
 
         if (i_params->chip_id < 0x20) {
                 k_ref_tmg_max = 233;
@@ -2397,7 +2398,7 @@ static void stv0900_set_viterbi_acq(struct stv0900_internal *i_params,
 {
         s32 vth_reg;
 
-        dprintk(KERN_INFO "%s\n", __func__);
+        dprintk("%s\n", __func__);
 
         dmd_reg(vth_reg, R0900_P1_VTH12, R0900_P2_VTH12);
 
@@ -2415,7 +2416,7 @@ static void stv0900_set_search_standard(struct stv0900_internal *i_params,
 
         int sstndrd;
 
-        dprintk(KERN_INFO "%s\n", __func__);
+        dprintk("%s\n", __func__);
 
         sstndrd = i_params->dmd1_srch_standard;
         if (demod == 1)
@@ -2578,7 +2579,7 @@ enum fe_stv0900_signal_type stv0900_algo(struct dvb_frontend *fe)
         enum fe_stv0900_search_algo algo;
         int no_signal = FALSE;
 
-        dprintk(KERN_INFO "%s\n", __func__);
+        dprintk("%s\n", __func__);
 
         switch (demod) {
         case STV0900_DEMOD_1: