[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4E36943E.5020105@st.com>
Date: Mon, 1 Aug 2011 17:25:42 +0530
From: viresh kumar <viresh.kumar@...com>
To: Russell King - ARM Linux <linux@....linux.org.uk>
Cc: "linus.walleij@...aro.org" <linus.walleij@...aro.org>,
Pratyush ANAND <pratyush.anand@...com>,
Rajeev KUMAR <rajeev-dlh.kumar@...com>,
Bhupesh SHARMA <bhupesh.sharma@...com>,
Shiraz HASHIM <shiraz.hashim@...com>,
"vinod.koul@...el.com" <vinod.koul@...el.com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Vipin KUMAR <vipin.kumar@...com>,
Armando VISCONTI <armando.visconti@...com>,
Amit VIRDI <Amit.VIRDI@...com>,
Vipul Kumar SAMAR <vipulkumar.samar@...com>,
"viresh.linux@...il.com" <viresh.linux@...il.com>,
Deepak SIKRI <deepak.sikri@...com>,
"dan.j.williams@...el.com" <dan.j.williams@...el.com>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH V2 08/20] dmaengine/amba-pl08x: support runtime PM
On 08/01/2011 03:56 PM, Russell King - ARM Linux wrote:
> I think a primecell's probe function should look like this:
>
> primecell_probe()
> {
> ret = amba_request_regions(adev, NULL);
> if (ret)
> return ret;
>
> ... allocate stuff, don't access primecell though ...
>
> pm_runtime_set_active(&adev->dev);
> pm_runtime_enable(&adev->dev);
>
> ... get clocks and enable them, do rest of init ...
>
> pm_runtime_put_sync(&adev->dev);
> return 0;
> }
Ok. I didn't had much knowhow of pm_runtime* framework. I will
send V3 for this patch alone, please see if that one is fine or still
needs some updation.
Thanks for your help.
--
viresh
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists