Commit 47e76c5c authored by Michael Krufky's avatar Michael Krufky Committed by Mauro Carvalho Chehab

V4L/DVB (5252): Qt1010: use ARRAY_SIZE macro when appropriate

Use ARRAY_SIZE macro already defined in kernel.h
Signed-off-by: default avatarMichael Krufky <mkrufky@linuxtv.org>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@infradead.org>
parent b79ea694
...@@ -233,7 +233,7 @@ static int qt1010_set_params(struct dvb_frontend *fe, struct dvb_frontend_parame ...@@ -233,7 +233,7 @@ static int qt1010_set_params(struct dvb_frontend *fe, struct dvb_frontend_parame
rd[10].val, rd[13].val, rd[14].val, rd[15].val, rd[35].val, \ rd[10].val, rd[13].val, rd[14].val, rd[15].val, rd[35].val, \
rd[40].val, rd[41].val, rd[43].val, rd[45].val); rd[40].val, rd[41].val, rd[43].val, rd[45].val);
for (i = 0; i < sizeof(rd) / sizeof(*rd); i++) { for (i = 0; i < ARRAY_SIZE(rd); i++) {
if (rd[i].oper == QT1010_WR) { if (rd[i].oper == QT1010_WR) {
err = qt1010_writereg(priv, rd[i].reg, rd[i].val); err = qt1010_writereg(priv, rd[i].reg, rd[i].val);
} else { /* read is required to proper locking */ } else { /* read is required to proper locking */
...@@ -263,7 +263,7 @@ static int qt1010_init_meas1(struct qt1010_priv *priv, u8 oper, u8 reg, u8 reg_i ...@@ -263,7 +263,7 @@ static int qt1010_init_meas1(struct qt1010_priv *priv, u8 oper, u8 reg, u8 reg_i
{ QT1010_RD, reg, 0xff } { QT1010_RD, reg, 0xff }
}; };
for (i = 0; i < sizeof(i2c_data) / sizeof(*i2c_data); i++) { for (i = 0; i < ARRAY_SIZE(i2c_data); i++) {
if (i2c_data[i].oper == QT1010_WR) { if (i2c_data[i].oper == QT1010_WR) {
err = qt1010_writereg(priv, i2c_data[i].reg, i2c_data[i].val); err = qt1010_writereg(priv, i2c_data[i].reg, i2c_data[i].val);
} else { } else {
...@@ -296,7 +296,7 @@ static u8 qt1010_init_meas2(struct qt1010_priv *priv, u8 reg_init_val, u8 *retva ...@@ -296,7 +296,7 @@ static u8 qt1010_init_meas2(struct qt1010_priv *priv, u8 reg_init_val, u8 *retva
{ QT1010_WR, 0x1e, 0x00 }, { QT1010_WR, 0x1e, 0x00 },
{ QT1010_WR, 0x22, 0xff } { QT1010_WR, 0x22, 0xff }
}; };
for (i = 0; i < sizeof(i2c_data) / sizeof(*i2c_data); i++) { for (i = 0; i < ARRAY_SIZE(i2c_data); i++) {
if (i2c_data[i].oper == QT1010_WR) { if (i2c_data[i].oper == QT1010_WR) {
err = qt1010_writereg(priv, i2c_data[i].reg, i2c_data[i].val); err = qt1010_writereg(priv, i2c_data[i].reg, i2c_data[i].val);
} else { } else {
...@@ -355,7 +355,7 @@ static int qt1010_init(struct dvb_frontend *fe) ...@@ -355,7 +355,7 @@ static int qt1010_init(struct dvb_frontend *fe)
if (fe->ops.i2c_gate_ctrl) if (fe->ops.i2c_gate_ctrl)
fe->ops.i2c_gate_ctrl(fe, 1); /* open i2c_gate */ fe->ops.i2c_gate_ctrl(fe, 1); /* open i2c_gate */
for (i = 0; i < sizeof(i2c_data) / sizeof(*i2c_data); i++) { for (i = 0; i < ARRAY_SIZE(i2c_data); i++) {
switch (i2c_data[i].oper) { switch (i2c_data[i].oper) {
case QT1010_WR: case QT1010_WR:
err = qt1010_writereg(priv, i2c_data[i].reg, i2c_data[i].val); err = qt1010_writereg(priv, i2c_data[i].reg, i2c_data[i].val);
......
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