diff options
author | Antti Palosaari <crope@iki.fi> | 2013-12-18 16:38:42 +0100 |
---|---|---|
committer | Mauro Carvalho Chehab <m.chehab@samsung.com> | 2013-12-19 12:24:44 +0100 |
commit | 8a878dc4f6686ccb2f6e4d72e29825fe3427eeb0 (patch) | |
tree | cf71192334ffece5f50859b201ce1cd3dcb93ffe /drivers/media/dvb-frontends/m88ds3103.c | |
parent | [media] m88ds3103: fix possible i2c deadlock (diff) | |
download | linux-8a878dc4f6686ccb2f6e4d72e29825fe3427eeb0.tar.xz linux-8a878dc4f6686ccb2f6e4d72e29825fe3427eeb0.zip |
[media] m88ds3103: fix some style issues reported by checkpatch.pl
* remove Free Software Foundation postal address
* add one pair of parenthesis
* use sizeof(*foo), not sizeof(struct foo)
Signed-off-by: Antti Palosaari <crope@iki.fi>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Diffstat (limited to 'drivers/media/dvb-frontends/m88ds3103.c')
-rw-r--r-- | drivers/media/dvb-frontends/m88ds3103.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/drivers/media/dvb-frontends/m88ds3103.c b/drivers/media/dvb-frontends/m88ds3103.c index b5503ed02534..b8a7897e7bd8 100644 --- a/drivers/media/dvb-frontends/m88ds3103.c +++ b/drivers/media/dvb-frontends/m88ds3103.c @@ -12,10 +12,6 @@ * but WITHOUT ANY WARRANTY; without even the implied warranty of * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the * 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. */ #include "m88ds3103_priv.h" @@ -927,8 +923,9 @@ static int m88ds3103_read_snr(struct dvb_frontend *fe, u16 *snr) if (signal > noise) { tmp = signal / noise; *snr = 100ul * intlog10(tmp) / (1 << 24); - } else + } else { *snr = 0; + } break; default: dev_dbg(&priv->i2c->dev, "%s: invalid delivery_system\n", @@ -1190,7 +1187,7 @@ struct dvb_frontend *m88ds3103_attach(const struct m88ds3103_config *cfg, u8 chip_id, u8tmp; /* allocate memory for the internal priv */ - priv = kzalloc(sizeof(struct m88ds3103_priv), GFP_KERNEL); + priv = kzalloc(sizeof(*priv), GFP_KERNEL); if (!priv) { ret = -ENOMEM; dev_err(&i2c->dev, "%s: kzalloc() failed\n", KBUILD_MODNAME); |