V4L/DVB (9401): Code Simplification

Use an enumeration for I/Q Swap rather than an int,
easier to spot the nasty ones

Signed-off-by: Manu Abraham <manu@linuxtv.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

diff --git a/drivers/media/dvb/frontends/stb0899_drv.h b/drivers/media/dvb/frontends/stb0899_drv.h
index 52c2ce17f856d3a0875443129eb33bdebc8eaf29..8cb320643f8e7b0be1aa27dfb8559a39c22cced3 100644 (file)
--- a/drivers/media/dvb/frontends/stb0899_drv.h
+++ b/drivers/media/dvb/frontends/stb0899_drv.h
@@ -44,6 +44,12 @@ struct stb0899_s2_reg {
        u32     data;
 };
 
+enum stb0899_inversion {
+       IQ_SWAP_OFF     = 0,
+       IQ_SWAP_ON,
+       IQ_SWAP_AUTO
+};
+
 struct stb0899_config {
        const struct stb0899_s1_reg     *init_dev;
        const struct stb0899_s2_reg     *init_s2_demod;
@@ -51,6 +57,8 @@ struct stb0899_config {
        const struct stb0899_s2_reg     *init_s2_fec;
        const struct stb0899_s1_reg     *init_tst;
 
+       enum stb0899_inversion          inversion;
+
        u32     xtal_freq;
 
        u8      demod_address;
@@ -65,7 +73,7 @@ struct stb0899_config {
        u8      data_fifo_mode;
        u8      out_rate_comp;
        u8      i2c_repeater;
-       int     inversion;
+//     int     inversion;
 
        u32     esno_ave;
        u32     esno_quant;

diff --git a/drivers/media/dvb/frontends/stb0899_priv.h b/drivers/media/dvb/frontends/stb0899_priv.h
index 47e533dd417d91b595fe2075e815daa3ddd73dc5..73790b4e345346e8d2590335adc7788b580da1ec 100644 (file)
--- a/drivers/media/dvb/frontends/stb0899_priv.h
+++ b/drivers/media/dvb/frontends/stb0899_priv.h
@@ -137,12 +137,6 @@ enum stb0899_frame {
        STB0899_SHORT_FRAME
 };
 
-enum stb0899_inversion {
-       IQ_SWAP_OFF     = 0,
-       IQ_SWAP_ON,
-       IQ_SWAP_AUTO
-};
-
 enum stb0899_alpha {
        RRC_20,
        RRC_25,

diff --git a/drivers/media/dvb/ttpci/budget-av.c b/drivers/media/dvb/ttpci/budget-av.c
index 9e7fab0e31e1c20ad85c612c6a63686c0105ae41..1b9bbdb36b828e8139a916b6a181992c75899a63 100644 (file)
--- a/drivers/media/dvb/ttpci/budget-av.c
+++ b/drivers/media/dvb/ttpci/budget-av.c
@@ -1451,7 +1451,7 @@ static struct stb0899_config knc1_dvbs2_config = {
 //     .ts_pfbit_toggle        = STB0899_MPEG_NORMAL,  /* DirecTV, MPEG toggling seq   */
 
        .xtal_freq              = 27000000,
-       .inversion              = 1,
+       .inversion              = IQ_SWAP_ON, /* 1 */
 
        .esno_ave               = KNC1_DVBS2_ESNO_AVE,
        .esno_quant             = KNC1_DVBS2_ESNO_QUANT,

diff --git a/drivers/media/dvb/ttpci/budget-ci.c b/drivers/media/dvb/ttpci/budget-ci.c
index 005c4596343d32f934bd445ae585afb8b73c36df..fc4cc491ab597f32d8f4102ae18956de7aca2849 100644 (file)
--- a/drivers/media/dvb/ttpci/budget-ci.c
+++ b/drivers/media/dvb/ttpci/budget-ci.c
@@ -1658,7 +1658,7 @@ static struct stb0899_config tt3200_config = {
        .demod_address          = 0x68,
 
        .xtal_freq              = 27000000,
-       .inversion              = 1,
+       .inversion              = IQ_SWAP_ON, /* 1 */
 
        .esno_ave               = TT3200_DVBS2_ESNO_AVE,
        .esno_quant             = TT3200_DVBS2_ESNO_QUANT,