[media] m88ds3103: remove unneeded AGC from inittab
authorAntti Palosaari <crope@iki.fi>
Mon, 2 Dec 2013 16:11:21 +0000 (13:11 -0300)
committerMauro Carvalho Chehab <m.chehab@samsung.com>
Thu, 19 Dec 2013 11:22:29 +0000 (09:22 -0200)
Optimal AGC is highly depended on used RF tuner and due to that
it is already included to chip configuration. However, inittab
has default AGC value, which was later replaced by one from config.

Add also comment to all chip configuration options about default
values and if those are needed to set or not.

Reported-by: David Howells <dhowells@redhat.com>
Signed-off-by: Antti Palosaari <crope@iki.fi>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
drivers/media/dvb-frontends/m88ds3103.h
drivers/media/dvb-frontends/m88ds3103_priv.h

index 287d62a4ba7ff0e65395ba6de5625616968c91d9..eaa5d10921d33135b3602f3ff2073c65d4b68142 100644 (file)
 struct m88ds3103_config {
        /*
         * I2C address
+        * Default: none, must set
         * 0x68, ...
         */
        u8 i2c_addr;
 
        /*
         * clock
+        * Default: none, must set
         * 27000000
         */
        u32 clock;
 
        /*
         * max bytes I2C provider is asked to write at once
-        * Note: Buffer is taken from the stack currently!
-        * Value must be set.
+        * Default: none, must set
         * 33, 65, ...
         */
        u16 i2c_wr_max;
 
        /*
         * TS output mode
+        * Default: M88DS3103_TS_SERIAL
         */
 #define M88DS3103_TS_SERIAL             0 /* TS output pin D0, normal */
 #define M88DS3103_TS_SERIAL_D7          1 /* TS output pin D7 */
@@ -58,16 +60,19 @@ struct m88ds3103_config {
 
        /*
         * spectrum inversion
+        * Default: 0
         */
        u8 spec_inv:1;
 
        /*
         * AGC polarity
+        * Default: 0
         */
        u8 agc_inv:1;
 
        /*
         * clock output
+        * Default: M88DS3103_CLOCK_OUT_DISABLED
         */
 #define M88DS3103_CLOCK_OUT_DISABLED        0
 #define M88DS3103_CLOCK_OUT_ENABLED         1
@@ -76,9 +81,14 @@ struct m88ds3103_config {
 
        /*
         * DiSEqC envelope mode
+        * Default: 0
         */
        u8 envelope_mode:1;
 
+       /*
+        * AGC configuration
+        * Default: none, must set
+        */
        u8 agc;
 };
 
index 322db4d59df360c93b194eaf91d913b9b17b94fd..80c5a250125077400f64d02a150891daedfa32fb 100644 (file)
@@ -57,7 +57,6 @@ static const struct m88ds3103_reg_val m88ds3103_dvbs_init_reg_vals[] = {
        {0x30, 0x08},
        {0x31, 0x40},
        {0x32, 0x32},
-       {0x33, 0x35},
        {0x35, 0xff},
        {0x3a, 0x00},
        {0x37, 0x10},
@@ -139,7 +138,6 @@ static const struct m88ds3103_reg_val m88ds3103_dvbs2_init_reg_vals[] = {
        {0x27, 0x31},
        {0x30, 0x08},
        {0x32, 0x32},
-       {0x33, 0x35},
        {0x35, 0xff},
        {0x3a, 0x00},
        {0x37, 0x10},