From mboxrd@z Thu Jan 1 00:00:00 1970 From: santosh.shilimkar@ti.com (Shilimkar, Santosh) Date: Sun, 1 Apr 2012 11:23:44 +0530 Subject: [PATCH 1/3] ARM: OMAP2+: 32k-counter: Use hwmod lookup to check presence of 32k timer In-Reply-To: References: <1326983304-14619-1-git-send-email-hvaibhav@ti.com> <1326983304-14619-2-git-send-email-hvaibhav@ti.com> <87mx9ej8fp.fsf@ti.com> <79CD15C6BA57404B839C016229A409A83181EF3F@DBDE01.ent.ti.com> <4F672364.3020403@ti.com> <79CD15C6BA57404B839C016229A409A83182386C@DBDE01.ent.ti.com> <79CD15C6BA57404B839C016229A409A83183EA54@DBDE01.ent.ti.com> <79CD15C6BA57404B839C016229A409A83183EB47@DBDE01.ent.ti.com> <79CD15C6BA57404B839C016229A409A831840C2E@DBDE01.ent.ti.com> <79CD15C6BA57404B839C016229A409A831840E0C@DBDE01.ent.ti.com> <4F7570FC.8000907@ti.com> <79CD15C6BA57404B839C016229A409A831840F82@DBDE01.ent.ti.com> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Sun, Apr 1, 2012 at 7:09 AM, Ming Lei wrote: > On Sun, Apr 1, 2012 at 3:10 AM, Shilimkar, Santosh > wrote: >> On Sat, Mar 31, 2012 at 2:09 PM, Ming Lei wrote: >>> On Sat, Mar 31, 2012 at 2:30 PM, Shilimkar, Santosh >>>> Since you need to recompile the kernel, you can very much tweak the >>>> clocksource to use GPTIMER with sysl clock. Support for that is still >>>> in place. >>> >>> With current kernel, running 'make menuconfig' can do it, but after applying >>> Hiremath's patch[1], I have to edit the source code manually to get it, so looks >>> this kind of tweaking is not friendly enough, :-( >>> >> It's not friendly but doable. But above can be supported I guess. >> >> Since you are talking about doing menuconfig >> and rebuilding kernel so what can be done is when one disable CPUIDLE >> and PM, one can disabled 32K source as well. And then 32K clocksource >> init should fail and fallback on gptimer clock source. > > OK, it should work, but looks OMAP_32K_TIMER option has to be kept > for the usage, which may be a bit divergent to the purpose of the patch set. > > So how about introducing a kernel parameter to decide if bypassing > 32K source and using gptimer source directly, and let which depend > on PM? > We are just doing circles on this patch set. Till the dynamic clock-soucre switching supported for low power entry exit, the high precision clocksoucre selection is already broken and very much limited for use I don't personally like to add features which hardly anybody use and fundamentally broken with full kernel. I let Tony take final call on what's appropriate. Regards Santosh