Hi, Kees Cook writes: > The .data assignment appears to be redundant to the WORK_STOP bit for > stopping the timer. Also, it appears this timer is entirely unused > as it is only ever started under #define VERBOSE, which is explicitly > undefined. > > Cc: Felipe Balbi > Cc: Greg Kroah-Hartman > Cc: linux-usb@vger.kernel.org > Cc: linux-omap@vger.kernel.org > Signed-off-by: Kees Cook Are you merging these as part of a larger series altogether, or should I pick the patches that concern me? If you're taking them all together: Acked-by: Felipe Balbi -- balbi