usb:g_dnl:f_thor: remove memset before memcpy
authorJeroen Hofstee <jeroen@myspectrum.nl>
Mon, 9 Jun 2014 13:28:58 +0000 (15:28 +0200)
committerMarek Vasut <marex@denx.de>
Wed, 25 Jun 2014 20:42:40 +0000 (22:42 +0200)
since ALLOC_CACHE_ALIGN_BUFFER defines a pointer and not a
buffer, the memset with sizeof(rqt) likely does something else
then intended. Since there is a memcpy directly after it with
the full size, drop the memset completely.

Cc: Lukasz Majewski <l.majewski@samsung.com>
Cc: Marek Vasut <marex@denx.de>
Signed-off-by: Jeroen Hofstee <jeroen@myspectrum.nl>
Acked-by: Lukasz Majewski <l.majewski@samsung.com>
drivers/usb/gadget/f_thor.c

index 28f215e07c6efa34e16ce2af151b23dc47e4fdc6..4e06273f7fb7e678b96a6bb3e11d6b8d6a4830a5 100644 (file)
@@ -306,7 +306,6 @@ static int process_data(void)
        ALLOC_CACHE_ALIGN_BUFFER(struct rqt_box, rqt, sizeof(struct rqt_box));
        int ret = -EINVAL;
 
-       memset(rqt, 0, sizeof(rqt));
        memcpy(rqt, thor_rx_data_buf, sizeof(struct rqt_box));
 
        debug("+RQT: %d, %d\n", rqt->rqt, rqt->rqt_data);