On Tue, Sep 03, 2013 at 11:45:44PM -0300, Fabio Estevam wrote:
> On Tue, Sep 3, 2013 at 10:16 PM, Peter Chen <peter.chen@xxxxxxxxxxxxx> wrote:
> > On Tue, Sep 03, 2013 at 08:25:43AM -0300, Fabio Estevam wrote:
> >> On Tue, Sep 3, 2013 at 6:26 AM, Peter Chen <peter.chen@xxxxxxxxxxxxx> wrote:
> >>
> >> > +static const struct dev_pm_ops ci_hdrc_imx_pm_ops = {
> >> > + SET_SYSTEM_SLEEP_PM_OPS(ci_hdrc_imx_suspend, ci_hdrc_imx_resume)
> >>
> >> What about using the macro SIMPLE_DEV_PM_OPS instead?
> >>
> >
> > I will add runtime pm later, it is more suitable using current way.
>
> No, what I meant was to replace:
>
> static const struct dev_pm_ops ci_hdrc_imx_pm_ops = {
> SET_SYSTEM_SLEEP_PM_OPS(ci_hdrc_imx_suspend, ci_hdrc_imx_resume)
> };
>
> with:
>
> static SIMPLE_DEV_PM_OPS(ci_hdrc_imx_pm_ops, ci_hdrc_imx_suspend,
> ci_hdrc_imx_resume);
>
Yes, I got your meaning. When I add runtime pm it will delete
SIMPLE_DEV_PM_OPS if using your way.
--
Best Regards,
Peter Chen
--
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
