]> git.baikalelectronics.ru Git - kernel.git/commit
usb: musb: core: use ->resume instead ->resume_noirq
authorSebastian Andrzej Siewior <bigeasy@linutronix.de>
Mon, 27 Oct 2014 09:49:42 +0000 (10:49 +0100)
committerFelipe Balbi <balbi@ti.com>
Mon, 3 Nov 2014 16:01:23 +0000 (10:01 -0600)
commit9c706b52ca025d3bfaf5c246fe2aa559b9b38fbe
tree4adef13b58d5048bac3cfb70c9f2fc2865031b46
parent3563de232959766808d69826d8b68397a949386b
usb: musb: core: use ->resume instead ->resume_noirq

In commit 6eb1223b38 ("USB: musb: fix possible panic while resuming")
musb_resume() became musb_resume_early() in order to enable the clocks
early on resume. This piece of the resume code was removed later in
commit 90dc06240 ("usb: musb: move clock handling to glue layer").
In between the function was renamed from musb_resume_early() to
musb_resume_noirq() by commit commit c1fd5b65a5 ("USB: Rework musb
suspend()/resume_early()").

Now I see that first musb_core is resumed followed by phy and glue
layer and I ask myself is this really what we intend to do? This kind
of revoked the purpose of the first commit (6eb1223b38).
Because of this and because it looks wrong to resume (core) before the
glue layer I push it to the ->resume callback.

Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: Felipe Balbi <balbi@ti.com>
drivers/usb/musb/musb_core.c