From: Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
To: Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
Cc: Mike Turquette
<mturquette-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
Subject: Re: [PATCH V2] clk: tegra: defer application of init table
Date: Thu, 04 Apr 2013 17:33:51 -0600 [thread overview]
Message-ID: <515E0DDF.6050008@wwwdotorg.org> (raw)
In-Reply-To: <1365111340-2143-1-git-send-email-swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
On 04/04/2013 03:35 PM, Stephen Warren wrote:
> From: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>
> The Tegra clock driver is initialized during the ARM machine descriptor's
> .init_irq() hook. It can't be initialized earlier, since dynamic memory
> usage is required. It can't be initialized later, since the .init_timer()
> hook needs the clocks initialized. However, at this time, udelay()
> doesn't work.
>
> The Tegra clock initialization table may enable some PLLs. Enabling a PLL
> may require usage of udelay(). Hence, this can't happen right when the
> clock driver is initialized.
>
> To solve this, separate the clock driver initialization from the clock
> table processing, so they can execute at separate times.
>
> Signed-off-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> ---
> Mike, the Tegra114 CCF V9 series assume this is applied first. Hence, I'm
> looking for an ack on this too. Thanks!
I've applied this to Tegra's for-3.10/clk branch, since Mike already
acked the Tegra114 CCF V9, which requires this patch, and embodies use
of it anyway.
WARNING: multiple messages have this Message-ID (diff)
From: swarren@wwwdotorg.org (Stephen Warren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH V2] clk: tegra: defer application of init table
Date: Thu, 04 Apr 2013 17:33:51 -0600 [thread overview]
Message-ID: <515E0DDF.6050008@wwwdotorg.org> (raw)
In-Reply-To: <1365111340-2143-1-git-send-email-swarren@wwwdotorg.org>
On 04/04/2013 03:35 PM, Stephen Warren wrote:
> From: Stephen Warren <swarren@nvidia.com>
>
> The Tegra clock driver is initialized during the ARM machine descriptor's
> .init_irq() hook. It can't be initialized earlier, since dynamic memory
> usage is required. It can't be initialized later, since the .init_timer()
> hook needs the clocks initialized. However, at this time, udelay()
> doesn't work.
>
> The Tegra clock initialization table may enable some PLLs. Enabling a PLL
> may require usage of udelay(). Hence, this can't happen right when the
> clock driver is initialized.
>
> To solve this, separate the clock driver initialization from the clock
> table processing, so they can execute at separate times.
>
> Signed-off-by: Stephen Warren <swarren@nvidia.com>
> ---
> Mike, the Tegra114 CCF V9 series assume this is applied first. Hence, I'm
> looking for an ack on this too. Thanks!
I've applied this to Tegra's for-3.10/clk branch, since Mike already
acked the Tegra114 CCF V9, which requires this patch, and embodies use
of it anyway.
next prev parent reply other threads:[~2013-04-04 23:33 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-04 21:35 [PATCH V2] clk: tegra: defer application of init table Stephen Warren
2013-04-04 21:35 ` Stephen Warren
[not found] ` <1365111340-2143-1-git-send-email-swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2013-04-04 23:33 ` Stephen Warren [this message]
2013-04-04 23:33 ` Stephen Warren
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=515E0DDF.6050008@wwwdotorg.org \
--to=swarren-3lzwwm7+weoh9zmkesr00q@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=mturquette-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
--cc=swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.