Thread (24 messages) 24 messages, 5 authors, 2011-11-07

[PATCH v2 2/8] arm/tegra: prepare early init for multiple tegra variants

From: Olof Johansson <hidden>
Date: 2011-11-04 15:09:45
Also in: linux-tegra, lkml

On Thu, Nov 3, 2011 at 3:02 AM, Russell King - ARM Linux
[off-list ref] wrote:
On Thu, Nov 03, 2011 at 11:21:16AM +0200, Peter De Schrijver wrote:
quoted
On Wed, Nov 02, 2011 at 07:30:43PM +0100, Colin Cross wrote:
quoted
On Wed, Nov 2, 2011 at 7:09 AM, Peter De Schrijver
[off-list ref] wrote:
quoted
This patch splits the early init code in a common and a tegra20 specific part.
L2 cache initialization is generalized and discovers the cache associativity
at runtime.

Signed-off-by: Peter De Schrijver <pdeschrijver@nvidia.com>
---
<snip>
quoted
-static __initdata struct tegra_clk_init_table common_clk_init_table[] = {
+void (*arch_reset)(char mode, const char *cmd) = tegra_assert_system_reset;
+
This is unrelated to the patch description?
This is a left over from the past. Will be removed from the next version.
Please can Tegra move over to hooking the restart via arm_pm_restart()
rather than via arch_reset()? ?That would leave arch_reset() for
software-based reboot methods - which I suspect Tegra doesn't use.

That would allow Tegra to dovetail into the arch_reset cleanup work much
better than it would otherwise do.
Sounds reasonable to me. Want a patch to put in your arch_reset
cleanup branch, I presume? I'll post something later today.


-Olof
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help