Robert Jarzmik <robert.jarz...@free.fr> writes:
> Felipe Balbi <ba...@ti.com> writes:
>
>> Hi,
>>
>> without any sort of logs it a bit difficult :-) Care to send some output
>> of the failure ? Are there any oopses or what exactly happens ?
> Ah well, the UDC is the only way to "speak" to the board (no UART), so I don't
> have any written feedback available. All I have are the logs displayed on the
> phone's screen in the framebuffer screen.
I can have logs if I kinda ... revert the patch, applying the diff in [1].
This enables the suspend/resume to work again, and I can gather the logs when
[1] is applied :
[ 63.649528] Suspending console(s) (use no_console_suspend to debug)
[ 63.688100] PM: suspend of devices complete after 37.275 msecs
[ 63.690351] PM: late suspend of devices complete after 2.191 msecs
[ 63.692595] PM: noirq suspend of devices complete after 2.196 msecs
[ 63.694387] PM: noirq resume of devices complete after 1.482 msecs
[ 63.696711] PM: early resume of devices complete after 1.533 msecs
[ 63.802930] PM: resume of devices complete after 106.122 msecs
[ 63.804976] Restarting tasks ...
[ 63.821371] pxa27x-udc pxa27x-udc: USB reset
[ 63.822988] done.
[ 63.933666] pxa27x-udc pxa27x-udc: USB reset
[ 64.064026] g_ether gadget: full-speed config #1: CDC Subset/SAFE
[ 64.069692] pxa27x-udc pxa27x-udc: ep3:pxa_ep_enable:usb_ep ep1out-bulk
already enabled, doing nothing
[ 64.075389] pxa27x-udc pxa27x-udc: ep3:pxa_ep_enable:usb_ep ep1out-bulk
already enabled, doing nothing
[ 64.082081] g_ether gadget: full-speed config #1: CDC Subset/SAFE
[ 64.087876] pxa27x-udc pxa27x-udc: ep4:pxa_ep_enable:usb_ep ep2in-bulk
already enabled, doing nothing
[ 64.093358] pxa27x-udc pxa27x-udc: ep3:pxa_ep_enable:usb_ep ep1out-bulk
already enabled, doing nothing
[ 64.099092] pxa27x-udc pxa27x-udc: ep4:pxa_ep_enable:usb_ep ep2in-bulk
already enabled, doing nothing
[ 64.104835] pxa27x-udc pxa27x-udc: ep3:pxa_ep_enable:usb_ep ep1out-bulk
already enabled, doing nothing
I don't know if that can help debug further ...
Cheers.
--
Robert
[1] Stupid diff to renable the suspend/resume to work on 4.4-rc1
rj@belgarion:~/mio_linux/kernel$ git diff
diff --git a/include/linux/usb/gadget.h b/include/linux/usb/gadget.h
index 3d583a10b926..2b8dcf546d03 100644
--- a/include/linux/usb/gadget.h
+++ b/include/linux/usb/gadget.h
@@ -267,8 +267,8 @@ static inline int usb_ep_enable(struct usb_ep *ep)
{
int ret;
- if (ep->enabled)
- return 0;
+ /* if (ep->enabled) */
+ /* return 0; */
ret = ep->ops->enable(ep, ep->desc);
if (ret)
@@ -295,8 +295,8 @@ static inline int usb_ep_disable(struct usb_ep *ep)
{
int ret;
- if (!ep->enabled)
- return 0;
+ /* if (!ep->enabled) */
+ /* return 0; */
ret = ep->ops->disable(ep);
if (ret)
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
