diff options
| author | Mauro Carvalho Chehab <mchehab@redhat.com> | 2009-02-23 15:26:38 +0000 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2009-02-23 15:26:38 +0000 |
| commit | f1b096388308b69fdc4fe54e5952a7f7beb845bb (patch) | |
| tree | b38d26e402f75e70100185d80ff7b41f2b594dc5 /linux/drivers/media/dvb/frontends/cx24116.c | |
| parent | e24be1d8a703d1ebcc4b2027be8071d82446b2b3 (diff) | |
| download | mediapointer-dvb-s2-f1b096388308b69fdc4fe54e5952a7f7beb845bb.tar.gz mediapointer-dvb-s2-f1b096388308b69fdc4fe54e5952a7f7beb845bb.tar.bz2 | |
remove redundant memset after kzalloc
From: Matthias Schwarzott <zzam@gentoo.org>
Hi there!
While having a look at the allocation of struct dvb_frontend in *_attach
functions, I found some cases calling memset after kzalloc. This is
redundant, and the attached patch removes these calls.
I also changed one case calling kmalloc and memset to kzalloc.
Regards
Matthias
Signed-off-by: Matthias Schwarzott <zzam@gentoo.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'linux/drivers/media/dvb/frontends/cx24116.c')
| -rw-r--r-- | linux/drivers/media/dvb/frontends/cx24116.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/linux/drivers/media/dvb/frontends/cx24116.c b/linux/drivers/media/dvb/frontends/cx24116.c index 8301a9865..b5ff0b6a8 100644 --- a/linux/drivers/media/dvb/frontends/cx24116.c +++ b/linux/drivers/media/dvb/frontends/cx24116.c @@ -1112,13 +1112,10 @@ struct dvb_frontend *cx24116_attach(const struct cx24116_config *config, dprintk("%s\n", __func__); /* allocate memory for the internal state */ - state = kmalloc(sizeof(struct cx24116_state), GFP_KERNEL); + state = kzalloc(sizeof(struct cx24116_state), GFP_KERNEL); if (state == NULL) goto error1; - /* setup the state */ - memset(state, 0, sizeof(struct cx24116_state)); - state->config = config; state->i2c = i2c; |