| Date | Tue, 8 Jan 2013 15:30:23 +0100 | Subject | Re: [PATCH 02/25] ab8500_charger: don't use [delayed_]work_pending() | From | Linus Walleij <> |
| |
On Sat, Dec 22, 2012 at 2:56 AM, Tejun Heo <tj@kernel.org> wrote:
> There's no need to test whether a (delayed) work item in pending > before queueing, flushing or cancelling it. Most uses are unnecessary > and quite a few of them are buggy. > > Remove unnecessary pending tests from ab8500_charger. Only compile > tested. > > Signed-off-by: Tejun Heo <tj@kernel.org> > Cc: Srinidhi Kasagar <srinidhi.kasagar@stericsson.com> > Cc: Linus Walleij <linus.walleij@linaro.org>
Works like a charm for us, thanks! Reviewed-by: Linus Walleij <linus.walleij@linaro.org> Tested-by: Marcus Cooper <marcus.xm.cooper@stericsson.com>
Yours, Linus Walleij
|