Re: [PATCH 5/6] speedtch: don't abuse struct delayed_work

From: Greg KH
Date: Thu Dec 16 2010 - 16:40:33 EST


On Sun, Dec 12, 2010 at 05:48:18PM +0100, Tejun Heo wrote:
> speedtch directly uses the internal timer and work members of a struct
> delayed_work. Use a separate work item and timer instead.
>
> Cc: Greg Kroah-Hartman <gregkh@xxxxxxx>
> Cc: Duncan Sands <duncan.sands@xxxxxxx>
> Cc: linux-usb@xxxxxxxxxxxxxxx
> ---
> This is part of a series to remove flush_scheduled_work() usage to
> prepare for deprecation of flush_scheduled_work(). Patches in this
> series are self contained and mostly straight-forward.
>
> Please feel free to take it into the appropriate tree, or just ack it.
> In the latter case, I'll merge the patch through the workqueue tree
> during the next merge window.

You forgot to add a "Signed-off-by:" on these patches, so I can't take
them in my tree :(

But feel free to add:
Acked-by: Greg Kroah-Hartman <gregkh@xxxxxxx>

to both 5/6 and 6/6 of this series and take them through your tree.

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/