From: Olof Johansson <olof@lixom.net>
To: Stephen Warren <swarren@nvidia.com>
Cc: Peter De Schrijver <pdeschrijver@nvidia.com>,
Colin Cross <ccross@android.com>,
Russell King <linux@arm.linux.org.uk>,
"linux-tegra@vger.kernel.org" <linux-tegra@vger.kernel.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] ARM: tegra: initialize basic system clocks
Date: Wed, 18 Jan 2012 20:18:56 -0800 [thread overview]
Message-ID: <20120119041856.GC7145@quad.lixom.net> (raw)
In-Reply-To: <74CDBE0F657A3D45AFBB94109FB122FF177EE3A404@HQMAIL01.nvidia.com>
On Tue, Jan 10, 2012 at 09:23:52AM -0800, Stephen Warren wrote:
> Peter De Schrijver wrote at Tuesday, January 10, 2012 9:22 AM:
> > Initialize basic system clocks and provide a auxdata table to allow some
> > peripheral drivers to find their clocks.
> ...
> > This should fix the hang you observed when sdhci_add_host() fails.
>
> Yes, it does. This plus the Tegra30 clock patch series you posted stops
> I2C and SDHCI drivers complaining about a lack of clocks during boot,
> which is a great step forwards, even if SDHCI doesn't yet initialize
> successfully. Thanks.
>
> This patch and the clock series:
>
> Tested-by: Stephen Warren <swarren@nvidia.com>
> Acked-by: Stephen Warren <swarren@nvidia.com>
Applied.
Peter, in the future please provide at least a one-sentence description of the
patch in the patch description. "ARM: tegra: enable tegra30 clock framework"
had no actualy patch description body so I added a simple one. It's a trivial
patch but it's a good habit to always have something there.
Thanks!
-Olof
next prev parent reply other threads:[~2012-01-19 4:18 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-09 15:35 [RESEND PATCH v2 0/3] Tegra30 clockframework Peter De Schrijver
2012-01-09 15:35 ` [RESEND PATCH v2 1/3] ARM: tegra: add support for new clock framework features Peter De Schrijver
2012-01-09 15:35 ` [RESEND PATCH v2 2/3] ARM: tegra: implement basic tegra30 clock framework Peter De Schrijver
2012-01-09 15:35 ` [RESEND PATCH v2 3/3] ARM: tegra: enable " Peter De Schrijver
2012-01-09 21:30 ` [RESEND PATCH v2 0/3] Tegra30 clockframework Stephen Warren
2012-01-10 16:22 ` [PATCH] ARM: tegra: initialize basic system clocks Peter De Schrijver
2012-01-10 17:23 ` Stephen Warren
2012-01-19 4:18 ` Olof Johansson [this message]
2012-01-10 16:39 ` [RESEND PATCH v2 0/3] Tegra30 clockframework Peter De Schrijver
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=20120119041856.GC7145@quad.lixom.net \
--to=olof@lixom.net \
--cc=ccross@android.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-tegra@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=pdeschrijver@nvidia.com \
--cc=swarren@nvidia.com \
/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
Powered by JetHome