Hello.

Kalliguddi, Hema wrote:

Hema,

-----Original Message-----
From: [email protected] [mailto:linux-omap-
[email protected]] On Behalf Of Hema HK
Sent: Friday, August 06, 2010 10:57 PM
To: [email protected]; [email protected]
Cc: Kalliguddi, Hema; Mankad, Maulik Ojas; Felipe Balbi; Tony Lindgren;
Kevin Hilman
Subject: [PATCH V2 6/8] usb: musb: Offmode fix for idle path

From: Hema HK  <[email protected]>

With OMAP core-off support musb was not functional as context was getting
lost after wakeup from core-off. And also musb was blocking the core-off
after loading the gadget driver even with no cable connected sometimes.

Added the conext save/restore api in the platform layer which will
be called in the idle and wakeup path.

Changed the usb sysconfig settings as per the usbotg functional spec.
When the device is not active, configure to force idle and force standby
mode.
When it is being used, configure in smart standby and smart idle mode.
So while attempting to coreoff the usb is configured to force standby and
force idle mode, after wakeup configured in smart idle and smart standby.

Signed-off-by: Hema HK <[email protected]>
Signed-off-by: Maulik Mankad <[email protected]>

Cc: Felipe Balbi <[email protected]>
Cc: Tony Lindgren <[email protected]>
Cc: Kevin Hilman <[email protected]>

Index: linux-omap-pm/drivers/usb/musb/omap2430.c
===================================================================
--- linux-omap-pm.orig/drivers/usb/musb/omap2430.c      2010-08-06
09:24:21.069863329 -0400
+++ linux-omap-pm/drivers/usb/musb/omap2430.c   2010-08-06
10:44:30.093914217 -0400
[...]
@@ -259,15 +273,42 @@
 void musb_platform_save_context(struct musb *musb,
                struct musb_context_registers *musb_context)
 {
-       musb_context->otg_sysconfig = musb_readl(musb->mregs, OTG_SYSCONFIG);
-       musb_context->otg_forcestandby = musb_readl(musb->mregs, 
OTG_FORCESTDBY);
+       /*
+        * As per the omap-usbotg specification, configure it to forced standby
+        * and  force idle mode when no activity on usb.
+        */
+       void __iomem *musb_base = musb->mregs;
+

Just to clarify, have you already taken care of ioremap() / request_mem_region() for musb_base?

This is already done.

   Not quite. MUSB driver doesn't call request_mem_region().

WBR, Sergei
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to