Re: [Outreachy kernel] [PATCH v2 1/2] Staging: rtl8188eu: Eliminate use of _set_timer

[Greg KH <gregkh@linuxfoundation.org>]

On Fri, Feb 27, 2015 at 11:48:03AM +0530, Vaishali Thakkar wrote:
> This patch introduces the use of API function mod_timer
> instead of driver specific function _set_timer as it is
> a more efficient and standard way to update the expire
> field of an active timer. Also, definition of function
> _set_timer is removed as it is no longer needed after
> this change.
> 
> Here, these cases are handled using Coccinelle and
> semantic patch used for this is as follows:
> 
> @@ expression x; expression y;@@
> 
> - _set_timer (&x, y);
> + mod_timer (&x, jiffies + msecs_to_jiffies (y));
> 
> Signed-off-by: Vaishali Thakkar <vthakkar1994@gmail.com>
> ---
> v1 : Here, this patch has some line over 80 characters as
>      limiting them to 80 characters does not make code look
>      good and readable.
> v2 : Change commit log
>    - Redo the patch against staging-testing

How about you do a v3 that includes:  Fixes build problems in previous
version.

ALWAYS test build your patches, it breaks my workflow when patches are
not tested and break the build :(

thanks,

greg k-h