Thread (65 messages) 65 messages, 10 authors, 2015-03-18

[PATCH v2 05/18] reset: Add reset_controller_of_init() function

From: Rob Herring <hidden>
Date: 2015-03-10 21:31:12
Also in: linux-api, linux-arch, linux-devicetree, linux-gpio, linux-serial, lkml

On Tue, Mar 10, 2015 at 3:19 PM, Arnd Bergmann [off-list ref] wrote:
On Tuesday 10 March 2015 16:28:44 Maxime Coquelin wrote:
quoted
2015-03-10 16:00 GMT+01:00 Arnd Bergmann [off-list ref]:
quoted
On Friday 20 February 2015 19:01:04 Maxime Coquelin wrote:
quoted
Some platforms need to initialize the reset controller before the timers.

This patch introduces a reset_controller_of_init() function that can be
called before the timers intialization.

Signed-off-by: Maxime Coquelin <mcoquelin.stm32@gmail.com>
Not sure about this. It seems like the cleanest approach if we get
a lot of these, but then again it is probably very rare, and I'd
like to avoid adding such infrastructure if it's just for one
SoC. Could we add a hack in the machine initialization instead?
Sun6i also need to initialize the reset controller early. Today, they
hack the machine initialization.
With two SoCs having the same need, what should we do?
Good question, I'd like to hear some other opinions on this first.
2 is still far from the common case.
quoted
quoted
I think ideally this would be done in the boot loader before we
even start Linux, but I don't know if that's possible for you.
From what I understand, the only constraint is to perform it after the
clock is enabled.
So this should be possible to do it in the bootloader, but it means
also adding timers clocks ungating in the bootloader.
Ungating the timer clock input seems like a reasonable thing to
do for the bootloader, I think a lot of platforms rely on this
elsewhere (but enough of them don't, which is why we have the
early clk init).
+1

If the bootloader is u-boot, then you need a timer anyway.

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