From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Lezcano Subject: Re: [RFC][PATCH 0/7] OMAP4 cpuidle cleanup Date: Wed, 21 Mar 2012 10:51:33 +0100 Message-ID: <4F69A4A5.5020208@linaro.org> References: <1332322070-24577-1-git-send-email-daniel.lezcano@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linaro-dev-bounces-cunTk1MwBs8s++Sfvej+rw@public.gmane.org Errors-To: linaro-dev-bounces-cunTk1MwBs8s++Sfvej+rw@public.gmane.org To: "Shilimkar, Santosh" Cc: linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linaro-dev-cunTk1MwBs8s++Sfvej+rw@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org List-Id: linux-omap@vger.kernel.org T24gMDMvMjEvMjAxMiAxMDozNiBBTSwgU2hpbGlta2FyLCBTYW50b3NoIHdyb3RlOgo+IE9uIFdl ZCwgTWFyIDIxLCAyMDEyIGF0IDI6NTcgUE0sIERhbmllbCBMZXpjYW5vCj4gPGRhbmllbC5sZXpj YW5vQGxpbmFyby5vcmc+ICB3cm90ZToKPj4gVGhpcyBwYXRjaHNldCBpcyBhIHByb3Bvc2l0aW9u IHRvIGltcHJvdmUgYSBiaXQgdGhlIGNvZGUuCj4+IFRoZSBjaGFuZ2VzIGFyZSBjb2RlIGNsZWFu dXAgYW5kIGRvZXMgbm90IGNoYW5nZSB0aGUgYmVoYXZpb3Igb2YgdGhlCj4+IGRyaXZlciBpdHNl bGYuCj4+Cj4gVGhhbmtzLiBXaWxsIGhhdmUgYSBsb29rIGF0IHlvdXIgc2VyaWVzLgoKQ29vbCwg dGhhbmtzLgoKPj4gQSBjb3VwbGUgYSB0aGluZ3MgY2FsbCBteSBpbnRlbnRpb24uIFdoeSB0aGUg Y3B1aWRsZSBkZXZpY2UgaXMgc2V0IGZvciBjcHUwIG9ubHkKPiBCZWNhdXNlIHRoZSBtYWlubGlu ZSBjb2RlIENQVUlETEUgaXMgc3VwcG9ydGVkIGFsb25nIHdpdGggQ1BVaG90cGx1Zy4KPiBUaGlz IGlzIGdvaW5nIHRvIGNoYW5nZSB0aG91Z2ggd2l0aCBDb3VwbGUgQ1BVSURMRSBhbmQgY29ycmVz cG9uZGluZwo+IE9NQVAgdXBkYXRlcy4KCk9rLCB0aGFua3MgZm9yIHRoZSBpbmZvcm1hdGlvbi4g SSB3aWxsIGxvb2sgZGVlcGVyLiBXaGF0IGhhcHBlbnMgdG8gY3B1MSAKd2hlbiBpdCBpcyBvbmxp bmUgYW5kIGhhcyBub3RoaW5nIHRvIGRvID8KCj4+IGFuZCB3aHkgdGhlIFdGSSBpcyBub3QgdXNl ZCA/Cj4+Cj4gSSBkaWRuJ3QgZ2V0IHRoaXMgcXVlc3Rpb24uIERvIHlvdSBtZWFuIHRoZSBnZW5l cmljIFdGST8KCnllcy4KCj4gSWYgeWVzLCB0aGVuLCBpdCdzIG1haW5seSBiZWNhdXNlIE9NQVAg bmVlZCBhZGRpdGlvbmFsCj4gY3VzdG9tIGJhcnJpZXJzLgoKQWgsIG9rLiBJIGFtIG5vdCBzdXJl IGlmIGl0IGlzIHBvc3NpYmxlIGJ1dCB0aGF0IG1heSBiZSBjb29sIGlmIHdlIGNhbiAKY2FsbCBj cHVfZG9faWRsZSBpbnN0ZWFkIHdpdGggYWRkaXRpb25hbCBiYXJyaWVyLgoKCj4gLS0KPiBUbyB1 bnN1YnNjcmliZSBmcm9tIHRoaXMgbGlzdDogc2VuZCB0aGUgbGluZSAidW5zdWJzY3JpYmUgbGlu dXgtb21hcCIgaW4KPiB0aGUgYm9keSBvZiBhIG1lc3NhZ2UgdG8gbWFqb3Jkb21vQHZnZXIua2Vy bmVsLm9yZwo+IE1vcmUgbWFqb3Jkb21vIGluZm8gYXQgIGh0dHA6Ly92Z2VyLmtlcm5lbC5vcmcv bWFqb3Jkb21vLWluZm8uaHRtbAoKCi0tIAogIDxodHRwOi8vd3d3LmxpbmFyby5vcmcvPiBMaW5h cm8ub3JnIOKUgiBPcGVuIHNvdXJjZSBzb2Z0d2FyZSBmb3IgQVJNIFNvQ3MKCkZvbGxvdyBMaW5h cm86ICA8aHR0cDovL3d3dy5mYWNlYm9vay5jb20vcGFnZXMvTGluYXJvPiBGYWNlYm9vayB8Cjxo dHRwOi8vdHdpdHRlci5jb20vIyEvbGluYXJvb3JnPiBUd2l0dGVyIHwKPGh0dHA6Ly93d3cubGlu YXJvLm9yZy9saW5hcm8tYmxvZy8+IEJsb2cKCgpfX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fXwpsaW5hcm8tZGV2IG1haWxpbmcgbGlzdApsaW5hcm8tZGV2QGxp c3RzLmxpbmFyby5vcmcKaHR0cDovL2xpc3RzLmxpbmFyby5vcmcvbWFpbG1hbi9saXN0aW5mby9s aW5hcm8tZGV2Cg== From mboxrd@z Thu Jan 1 00:00:00 1970 From: daniel.lezcano@linaro.org (Daniel Lezcano) Date: Wed, 21 Mar 2012 10:51:33 +0100 Subject: [RFC][PATCH 0/7] OMAP4 cpuidle cleanup In-Reply-To: References: <1332322070-24577-1-git-send-email-daniel.lezcano@linaro.org> Message-ID: <4F69A4A5.5020208@linaro.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 03/21/2012 10:36 AM, Shilimkar, Santosh wrote: > On Wed, Mar 21, 2012 at 2:57 PM, Daniel Lezcano > wrote: >> This patchset is a proposition to improve a bit the code. >> The changes are code cleanup and does not change the behavior of the >> driver itself. >> > Thanks. Will have a look at your series. Cool, thanks. >> A couple a things call my intention. Why the cpuidle device is set for cpu0 only > Because the mainline code CPUIDLE is supported along with CPUhotplug. > This is going to change though with Couple CPUIDLE and corresponding > OMAP updates. Ok, thanks for the information. I will look deeper. What happens to cpu1 when it is online and has nothing to do ? >> and why the WFI is not used ? >> > I didn't get this question. Do you mean the generic WFI? yes. > If yes, then, it's mainly because OMAP need additional > custom barriers. Ah, ok. I am not sure if it is possible but that may be cool if we can call cpu_do_idle instead with additional barrier. > -- > To unsubscribe from this list: send the line "unsubscribe linux-omap" in > the body of a message to majordomo at vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Linaro.org ? Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog