lme2510_state exists for the entire duration of driver.

Move usb_buffer to lme2510_state removing the need for
lme2510_exit_int for removing the buffer.

Signed-off-by: Malcolm Priestley <tvbox...@gmail.com>
---
 drivers/media/usb/dvb-usb-v2/lmedm04.c | 35 +++-----------------------
 1 file changed, 3 insertions(+), 32 deletions(-)

diff --git a/drivers/media/usb/dvb-usb-v2/lmedm04.c 
b/drivers/media/usb/dvb-usb-v2/lmedm04.c
index 8b405e131439..8fb53b83c914 100644
--- a/drivers/media/usb/dvb-usb-v2/lmedm04.c
+++ b/drivers/media/usb/dvb-usb-v2/lmedm04.c
@@ -136,7 +136,7 @@ struct lme2510_state {
        u8 pid_off;
        void *buffer;
        struct urb *lme_urb;
-       void *usb_buffer;
+       u8 usb_buffer[64];
        /* Frontend original calls */
        int (*fe_read_status)(struct dvb_frontend *, enum fe_status *);
        int (*fe_read_signal_strength)(struct dvb_frontend *, u16 *);
@@ -169,18 +169,9 @@ static int lme2510_usb_talk(struct dvb_usb_device *d,
                u8 *wbuf, int wlen, u8 *rbuf, int rlen)
 {
        struct lme2510_state *st = d->priv;
-       u8 *buff;
+       u8 *buff = st->usb_buffer;
        int ret = 0;
 
-       if (st->usb_buffer == NULL) {
-               st->usb_buffer = kmalloc(64, GFP_KERNEL);
-               if (st->usb_buffer == NULL) {
-                       info("MEM Error no memory");
-                       return -ENOMEM;
-               }
-       }
-       buff = st->usb_buffer;
-
        ret = mutex_lock_interruptible(&d->usb_mutex);
 
        if (ret < 0)
@@ -1245,23 +1236,15 @@ static int lme2510_get_rc_config(struct dvb_usb_device 
*d,
        return 0;
 }
 
-static void *lme2510_exit_int(struct dvb_usb_device *d)
+static void lme2510_exit(struct dvb_usb_device *d)
 {
        struct lme2510_state *st = d->priv;
        struct dvb_usb_adapter *adap = &d->adapter[0];
-       void *buffer = NULL;
 
        if (adap != NULL) {
                lme2510_kill_urb(&adap->stream);
        }
 
-       if (st->usb_buffer != NULL) {
-               st->i2c_talk_onoff = 1;
-               st->signal_level = 0;
-               st->signal_sn = 0;
-               buffer = st->usb_buffer;
-       }
-
        if (st->lme_urb != NULL) {
                usb_kill_urb(st->lme_urb);
                usb_free_urb(st->lme_urb);
@@ -1269,18 +1252,6 @@ static void *lme2510_exit_int(struct dvb_usb_device *d)
                                  st->lme_urb->transfer_dma);
                info("Interrupt Service Stopped");
        }
-
-       return buffer;
-}
-
-static void lme2510_exit(struct dvb_usb_device *d)
-{
-       void *usb_buffer;
-
-       if (d != NULL) {
-               usb_buffer = lme2510_exit_int(d);
-               kfree(usb_buffer);
-       }
 }
 
 static struct dvb_usb_device_properties lme2510_props = {
-- 
2.19.1

Reply via email to