Thread (10 messages) flat view 10 messages, 5 authors, 2012-07-03
STALE5192d

[PATCH 2 2/4] net ethernet introduce mac_la_ap helper

From: arnd@arndb.de (Arnd Bergmann)
Date: 2012-07-03 08:05:03
Also in: linux-omap

On Tuesday 03 July 2012, Andy Green wrote:
quoted
I think it would be simpler to register the notifier from an
initcall and drop the mac_la_ap_started variable.
That was my first approach, to structure it as a real driver.  I had 
tried a few likely-looking initcall levels but the init of the helper 
was coming after the init of the network code.

I found this time that core_initcall works, so I used that, dropped the 
flag.  But isn't it a bit tricky with these to know if the prerequisite 
you're trying to ensure is already initialized might not actually be at 
the same initcall level and you're working by accident?
Well, you would only need to ensure that it's called before any of the
network devices are added, which should all happen very late in the
boot cycle compared to the platform init code.

I think core_initcall is fine here.

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