Commit 947debb4 authored by Antti Palosaari's avatar Antti Palosaari Committed by Mauro Carvalho Chehab

[media] rtl2830: style related changes

Trivial changes proposed by checkpatch.pl and some more.
Signed-off-by: default avatarAntti Palosaari <crope@iki.fi>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@osg.samsung.com>
parent b8cb50d2
...@@ -13,16 +13,6 @@ ...@@ -13,16 +13,6 @@
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details. * GNU General Public License for more details.
* *
* You should have received a copy of the GNU General Public License along
* with this program; if not, write to the Free Software Foundation, Inc.,
* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
*/
/*
* Driver implements own I2C-adapter for tuner I2C access. That's since chip
* have unusual I2C-gate control which closes gate automatically after each
* I2C transfer. Using own I2C adapter we can workaround that.
*/ */
#include "rtl2830_priv.h" #include "rtl2830_priv.h"
...@@ -61,6 +51,7 @@ static int rtl2830_wr(struct i2c_client *client, u8 reg, const u8 *val, int len) ...@@ -61,6 +51,7 @@ static int rtl2830_wr(struct i2c_client *client, u8 reg, const u8 *val, int len)
ret, reg, len); ret, reg, len);
ret = -EREMOTEIO; ret = -EREMOTEIO;
} }
return ret; return ret;
} }
...@@ -90,6 +81,7 @@ static int rtl2830_rd(struct i2c_client *client, u8 reg, u8 *val, int len) ...@@ -90,6 +81,7 @@ static int rtl2830_rd(struct i2c_client *client, u8 reg, u8 *val, int len)
ret, reg, len); ret, reg, len);
ret = -EREMOTEIO; ret = -EREMOTEIO;
} }
return ret; return ret;
} }
...@@ -187,42 +179,42 @@ static int rtl2830_init(struct dvb_frontend *fe) ...@@ -187,42 +179,42 @@ static int rtl2830_init(struct dvb_frontend *fe)
struct rtl2830_dev *dev = i2c_get_clientdata(client); struct rtl2830_dev *dev = i2c_get_clientdata(client);
int ret, i; int ret, i;
struct rtl2830_reg_val_mask tab[] = { struct rtl2830_reg_val_mask tab[] = {
{ 0x00d, 0x01, 0x03 }, {0x00d, 0x01, 0x03},
{ 0x00d, 0x10, 0x10 }, {0x00d, 0x10, 0x10},
{ 0x104, 0x00, 0x1e }, {0x104, 0x00, 0x1e},
{ 0x105, 0x80, 0x80 }, {0x105, 0x80, 0x80},
{ 0x110, 0x02, 0x03 }, {0x110, 0x02, 0x03},
{ 0x110, 0x08, 0x0c }, {0x110, 0x08, 0x0c},
{ 0x17b, 0x00, 0x40 }, {0x17b, 0x00, 0x40},
{ 0x17d, 0x05, 0x0f }, {0x17d, 0x05, 0x0f},
{ 0x17d, 0x50, 0xf0 }, {0x17d, 0x50, 0xf0},
{ 0x18c, 0x08, 0x0f }, {0x18c, 0x08, 0x0f},
{ 0x18d, 0x00, 0xc0 }, {0x18d, 0x00, 0xc0},
{ 0x188, 0x05, 0x0f }, {0x188, 0x05, 0x0f},
{ 0x189, 0x00, 0xfc }, {0x189, 0x00, 0xfc},
{ 0x2d5, 0x02, 0x02 }, {0x2d5, 0x02, 0x02},
{ 0x2f1, 0x02, 0x06 }, {0x2f1, 0x02, 0x06},
{ 0x2f1, 0x20, 0xf8 }, {0x2f1, 0x20, 0xf8},
{ 0x16d, 0x00, 0x01 }, {0x16d, 0x00, 0x01},
{ 0x1a6, 0x00, 0x80 }, {0x1a6, 0x00, 0x80},
{ 0x106, dev->pdata->vtop, 0x3f }, {0x106, dev->pdata->vtop, 0x3f},
{ 0x107, dev->pdata->krf, 0x3f }, {0x107, dev->pdata->krf, 0x3f},
{ 0x112, 0x28, 0xff }, {0x112, 0x28, 0xff},
{ 0x103, dev->pdata->agc_targ_val, 0xff }, {0x103, dev->pdata->agc_targ_val, 0xff},
{ 0x00a, 0x02, 0x07 }, {0x00a, 0x02, 0x07},
{ 0x140, 0x0c, 0x3c }, {0x140, 0x0c, 0x3c},
{ 0x140, 0x40, 0xc0 }, {0x140, 0x40, 0xc0},
{ 0x15b, 0x05, 0x07 }, {0x15b, 0x05, 0x07},
{ 0x15b, 0x28, 0x38 }, {0x15b, 0x28, 0x38},
{ 0x15c, 0x05, 0x07 }, {0x15c, 0x05, 0x07},
{ 0x15c, 0x28, 0x38 }, {0x15c, 0x28, 0x38},
{ 0x115, dev->pdata->spec_inv, 0x01 }, {0x115, dev->pdata->spec_inv, 0x01},
{ 0x16f, 0x01, 0x07 }, {0x16f, 0x01, 0x07},
{ 0x170, 0x18, 0x38 }, {0x170, 0x18, 0x38},
{ 0x172, 0x0f, 0x0f }, {0x172, 0x0f, 0x0f},
{ 0x173, 0x08, 0x38 }, {0x173, 0x08, 0x38},
{ 0x175, 0x01, 0x07 }, {0x175, 0x01, 0x07},
{ 0x176, 0x00, 0xc0 }, {0x176, 0x00, 0xc0},
}; };
for (i = 0; i < ARRAY_SIZE(tab); i++) { for (i = 0; i < ARRAY_SIZE(tab); i++) {
...@@ -264,7 +256,9 @@ static int rtl2830_sleep(struct dvb_frontend *fe) ...@@ -264,7 +256,9 @@ static int rtl2830_sleep(struct dvb_frontend *fe)
{ {
struct i2c_client *client = fe->demodulator_priv; struct i2c_client *client = fe->demodulator_priv;
struct rtl2830_dev *dev = i2c_get_clientdata(client); struct rtl2830_dev *dev = i2c_get_clientdata(client);
dev->sleeping = true; dev->sleeping = true;
return 0; return 0;
} }
...@@ -343,8 +337,7 @@ static int rtl2830_set_frontend(struct dvb_frontend *fe) ...@@ -343,8 +337,7 @@ static int rtl2830_set_frontend(struct dvb_frontend *fe)
ret = fe->ops.tuner_ops.get_if_frequency(fe, &if_frequency); ret = fe->ops.tuner_ops.get_if_frequency(fe, &if_frequency);
else else
ret = -EINVAL; ret = -EINVAL;
if (ret)
if (ret < 0)
goto err; goto err;
num = if_frequency % dev->pdata->clk; num = if_frequency % dev->pdata->clk;
...@@ -507,6 +500,7 @@ static int rtl2830_read_status(struct dvb_frontend *fe, fe_status_t *status) ...@@ -507,6 +500,7 @@ static int rtl2830_read_status(struct dvb_frontend *fe, fe_status_t *status)
struct rtl2830_dev *dev = i2c_get_clientdata(client); struct rtl2830_dev *dev = i2c_get_clientdata(client);
int ret; int ret;
u8 tmp; u8 tmp;
*status = 0; *status = 0;
if (dev->sleeping) if (dev->sleeping)
...@@ -540,9 +534,9 @@ static int rtl2830_read_snr(struct dvb_frontend *fe, u16 *snr) ...@@ -540,9 +534,9 @@ static int rtl2830_read_snr(struct dvb_frontend *fe, u16 *snr)
#define CONSTELLATION_NUM 3 #define CONSTELLATION_NUM 3
#define HIERARCHY_NUM 4 #define HIERARCHY_NUM 4
static const u32 snr_constant[CONSTELLATION_NUM][HIERARCHY_NUM] = { static const u32 snr_constant[CONSTELLATION_NUM][HIERARCHY_NUM] = {
{ 70705899, 70705899, 70705899, 70705899 }, {70705899, 70705899, 70705899, 70705899},
{ 82433173, 82433173, 87483115, 94445660 }, {82433173, 82433173, 87483115, 94445660},
{ 92888734, 92888734, 95487525, 99770748 }, {92888734, 92888734, 95487525, 99770748},
}; };
if (dev->sleeping) if (dev->sleeping)
...@@ -605,6 +599,7 @@ static int rtl2830_read_ber(struct dvb_frontend *fe, u32 *ber) ...@@ -605,6 +599,7 @@ static int rtl2830_read_ber(struct dvb_frontend *fe, u32 *ber)
static int rtl2830_read_ucblocks(struct dvb_frontend *fe, u32 *ucblocks) static int rtl2830_read_ucblocks(struct dvb_frontend *fe, u32 *ucblocks)
{ {
*ucblocks = 0; *ucblocks = 0;
return 0; return 0;
} }
...@@ -630,7 +625,7 @@ static int rtl2830_read_signal_strength(struct dvb_frontend *fe, u16 *strength) ...@@ -630,7 +625,7 @@ static int rtl2830_read_signal_strength(struct dvb_frontend *fe, u16 *strength)
else else
if_agc = if_agc_raw; if_agc = if_agc_raw;
*strength = (u8) (55 - if_agc / 182); *strength = (u8)(55 - if_agc / 182);
*strength |= *strength << 8; *strength |= *strength << 8;
return 0; return 0;
...@@ -640,7 +635,7 @@ static int rtl2830_read_signal_strength(struct dvb_frontend *fe, u16 *strength) ...@@ -640,7 +635,7 @@ static int rtl2830_read_signal_strength(struct dvb_frontend *fe, u16 *strength)
} }
static struct dvb_frontend_ops rtl2830_ops = { static struct dvb_frontend_ops rtl2830_ops = {
.delsys = { SYS_DVBT }, .delsys = {SYS_DVBT},
.info = { .info = {
.name = "Realtek RTL2830 (DVB-T)", .name = "Realtek RTL2830 (DVB-T)",
.caps = FE_CAN_FEC_1_2 | .caps = FE_CAN_FEC_1_2 |
...@@ -723,7 +718,6 @@ static int rtl2830_select(struct i2c_adapter *adap, void *mux_priv, u32 chan_id) ...@@ -723,7 +718,6 @@ static int rtl2830_select(struct i2c_adapter *adap, void *mux_priv, u32 chan_id)
} }
return 0; return 0;
err: err:
dev_dbg(&client->dev, "failed=%d\n", ret); dev_dbg(&client->dev, "failed=%d\n", ret);
return ret; return ret;
...@@ -796,8 +790,8 @@ static int rtl2830_probe(struct i2c_client *client, ...@@ -796,8 +790,8 @@ static int rtl2830_probe(struct i2c_client *client,
pdata->get_i2c_adapter = rtl2830_get_i2c_adapter; pdata->get_i2c_adapter = rtl2830_get_i2c_adapter;
dev_info(&client->dev, "Realtek RTL2830 successfully attached\n"); dev_info(&client->dev, "Realtek RTL2830 successfully attached\n");
return 0;
return 0;
err_kfree: err_kfree:
kfree(dev); kfree(dev);
err: err:
...@@ -813,6 +807,7 @@ static int rtl2830_remove(struct i2c_client *client) ...@@ -813,6 +807,7 @@ static int rtl2830_remove(struct i2c_client *client)
i2c_del_mux_adapter(dev->adapter); i2c_del_mux_adapter(dev->adapter);
kfree(dev); kfree(dev);
return 0; return 0;
} }
......
...@@ -13,15 +13,11 @@ ...@@ -13,15 +13,11 @@
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details. * GNU General Public License for more details.
* *
* You should have received a copy of the GNU General Public License along
* with this program; if not, write to the Free Software Foundation, Inc.,
* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
*/ */
#ifndef RTL2830_H #ifndef RTL2830_H
#define RTL2830_H #define RTL2830_H
#include <linux/kconfig.h>
#include <linux/dvb/frontend.h> #include <linux/dvb/frontend.h>
struct rtl2830_platform_data { struct rtl2830_platform_data {
......
...@@ -13,9 +13,6 @@ ...@@ -13,9 +13,6 @@
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details. * GNU General Public License for more details.
* *
* You should have received a copy of the GNU General Public License along
* with this program; if not, write to the Free Software Foundation, Inc.,
* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
*/ */
#ifndef RTL2830_PRIV_H #ifndef RTL2830_PRIV_H
......
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