Hi, >-----Original Message----- >From: Balbi, Felipe >Sent: Friday, September 24, 2010 4:27 PM >To: Kalliguddi, Hema >Cc: Balbi, Felipe; Kevin Hilman; linux-omap@xxxxxxxxxxxxxxx; >linux-usb@xxxxxxxxxxxxxxx; Basak, Partha; Tony Lindgren; >Cousson, Benoit; Paul Walmsley >Subject: Re: [PATCH 8/9 v3] usb : musb: Using runtime pm apis for musb. > >Hi, > >On Fri, Sep 24, 2010 at 04:28:35AM -0500, Kalliguddi, Hema wrote: >>I just noticed one more think in the code. there are suspend/resume >>function pointers in platform_driver structure. Why can't we >use these >>for platform specific operations like enable/disable clocks, context >>save/restore? > >For doing that properly, we first need to split the HW glue layer to >another parent platform_device/driver. Otherwise, we will have >all sorts >of crazy stuff to change the function pointer depending on the ARCH. > Agreed, then for now I can think of using set_clok platform function to call the runtime pm apis, Is there any alternative you are thinking of? >-- >balbi >-- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html