From: jmkrzyszt@gmail.com (Janusz Krzysztofik)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH-V7 0/3] ARM: OMAP: Make OMAP clocksource source selection runtime
Date: Thu, 10 May 2012 23:36:48 +0200 [thread overview]
Message-ID: <2430095.tlzScq1o2g@acer> (raw)
In-Reply-To: <1336031941-23651-1-git-send-email-hvaibhav@ti.com>
Dnia czwartek, 3 maja 2012 13:28:58 Vaibhav Hiremath pisze:
> Current OMAP code supports couple of clocksource options based
> on compilation flag (CONFIG_OMAP_32K_TIMER). The 32KHz sync-timer
> and a gptimer which can run on 32KHz or system clock (e.g 38.4 MHz)
>
> This patch series cleans up the existing 32k-sync timer
implementation,
> movind SoC init code to respective files (mach-omap1/timer32k.c and
> mach-omap2/timer.c) and uses kernel parameter to override the default
> clocksource of "counter_32k", also in order to support some OMAP based
> derivative SoCs like AM33XX which doesn't have 32K sync-timer hardware
IP,
> adds hwmod lookup for omap2+ devices, and if lookup fails then
> fall back to gp-timer.
>
> if(use_gptimer_clksrc == true)
> gptimer clocksource init;
> else if (counter_32 init == false)
> /* Fallback to gptimer */
> gptimer clocksource init(;
>
> With this, we should be able to support multi-omap boot
> including devices with/without 32k-sync timer.
>
> This patch-series has been boot tested on AM37xEVM platform, it
> would be helpful if somebody help me to validate it on OMAP1/2
> platforms.
>
> The patches are also available at (based on linux-omap/master) -
> https://github.com/hvaibhav/am335x-linux 32ksync-timer-cleanup
Tried to test this branch, merged into 3.4-rc6, and with my Amstrad
Delta fixes applied, using my custom Amstrad Delta config, but initially
failed because an unrelated issue rised to the surface:
| LD .tmp_vmlinux1
| arch/arm/mach-omap1/built-in.o: In function `omap1_usb_init':
| lcd_dma.c:(.init.text+0x1420): undefined reference to `ocpi_enable'
| make: *** [.tmp_vmlinux1] Error 1
Apparently introduced with commit
d3645d39ad0ed9f09535065676ea0ba114f93cdf, "ARM: OMAP1: OHCI: use
platform_data fn ptr to enable OCPI bus", looks like one or more symbols
exported by the not linked in arch/arm/mach-omap1/ocpi.o may be missing
for non-OMAP16xx configs.
Otherwise, if configured with OMAP16xx selected, builds and seems to work
correctly for me, to the extent possible to verify without physical
access to the hardware. However, please note that the Amstrad Delta
doesn't make use of the 32k timer, only mpu_timer.
Thanks,
Janusz
next prev parent reply other threads:[~2012-05-10 21:36 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-05-03 7:58 [PATCH-V7 0/3] ARM: OMAP: Make OMAP clocksource source selection runtime Vaibhav Hiremath
2012-05-03 7:58 ` [PATCH-V7 1/3] ARM: OMAP1: FIX: check possible error condition in timer_init Vaibhav Hiremath
2012-05-03 7:59 ` [PATCH-V7 2/3] ARM: OMAP2+: Replace space=>underscore in the name field of system timers Vaibhav Hiremath
2012-05-03 7:59 ` [PATCH-V7 3/3] ARM: OMAP: Make OMAP clocksource source selection using kernel param Vaibhav Hiremath
2012-05-09 17:33 ` [PATCH-V7 0/3] ARM: OMAP: Make OMAP clocksource source selection runtime Tony Lindgren
2012-05-10 21:36 ` Janusz Krzysztofik [this message]
2012-05-10 22:04 ` Paul Walmsley
2012-05-10 22:26 ` Tony Lindgren
2012-05-10 22:28 ` Paul Walmsley
2012-05-11 6:57 ` Hiremath, Vaibhav
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2430095.tlzScq1o2g@acer \
--to=jmkrzyszt@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).