Commit 846fe83e authored by David Brownell's avatar David Brownell Committed by Tony Lindgren

mach-omap2: fix more arch_initcall() breakage

Remove more bogus arch_initcall() logic in mach-omap2/board-xyx.c files.
They broke a multi-OMAP build I did, at *RUN TIME* not build time, since
it tried to do the i2c init for every board linked in the kernel.

Remember, init_machine() entries run at arch_initcall() time; that's
where any board-specific init logic should normally go.  Any initcalls
in the mach-*/*c files should normally be guarded by tests to make sure
they only run on the relevant hardware (board, cpu).  Better yet, get
rid of the initcalls; init_machine() can *explicitly* call the right
version of that code, and pass in board-specific config data; and there
are hooks that can handle cpu-specific stuff too.

A quick glance suggests most of the remaining initcall logic in the
mach-omap2 directory is similarly broken... this patch gets rid of
one frequently-cloned idiom, it should help.
Signed-off-by: default avatarDavid Brownell <dbrownell@users.sourceforge.net>
Signed-off-by: default avatarTony Lindgren <tony@atomide.com>
parent edbfb2fa
...@@ -384,6 +384,8 @@ static int __init omap2430_i2c_init(void) ...@@ -384,6 +384,8 @@ static int __init omap2430_i2c_init(void)
static void __init omap_2430sdp_init(void) static void __init omap_2430sdp_init(void)
{ {
omap2430_i2c_init();
platform_add_devices(sdp2430_devices, ARRAY_SIZE(sdp2430_devices)); platform_add_devices(sdp2430_devices, ARRAY_SIZE(sdp2430_devices));
omap_board_config = sdp2430_config; omap_board_config = sdp2430_config;
omap_board_config_size = ARRAY_SIZE(sdp2430_config); omap_board_config_size = ARRAY_SIZE(sdp2430_config);
...@@ -408,8 +410,6 @@ static void __init omap_2430sdp_map_io(void) ...@@ -408,8 +410,6 @@ static void __init omap_2430sdp_map_io(void)
omap2_map_common_io(); omap2_map_common_io();
} }
arch_initcall(omap2430_i2c_init);
MACHINE_START(OMAP_2430SDP, "OMAP2430 sdp2430 board") MACHINE_START(OMAP_2430SDP, "OMAP2430 sdp2430 board")
/* Maintainer: Syed Khasim - Texas Instruments Inc */ /* Maintainer: Syed Khasim - Texas Instruments Inc */
.phys_io = 0x48000000, .phys_io = 0x48000000,
......
...@@ -346,6 +346,7 @@ extern void __init sdp3430_flash_init(void); ...@@ -346,6 +346,7 @@ extern void __init sdp3430_flash_init(void);
static void __init omap_3430sdp_init(void) static void __init omap_3430sdp_init(void)
{ {
omap3430_i2c_init();
platform_add_devices(sdp3430_devices, ARRAY_SIZE(sdp3430_devices)); platform_add_devices(sdp3430_devices, ARRAY_SIZE(sdp3430_devices));
omap_board_config = sdp3430_config; omap_board_config = sdp3430_config;
omap_board_config_size = ARRAY_SIZE(sdp3430_config); omap_board_config_size = ARRAY_SIZE(sdp3430_config);
...@@ -370,7 +371,6 @@ static void __init omap_3430sdp_map_io(void) ...@@ -370,7 +371,6 @@ static void __init omap_3430sdp_map_io(void)
omap2_set_globals_343x(); omap2_set_globals_343x();
omap2_map_common_io(); omap2_map_common_io();
} }
arch_initcall(omap3430_i2c_init);
MACHINE_START(OMAP_3430SDP, "OMAP3430 3430SDP board") MACHINE_START(OMAP_3430SDP, "OMAP3430 3430SDP board")
/* Maintainer: Syed Khasim - Texas Instruments Inc */ /* Maintainer: Syed Khasim - Texas Instruments Inc */
......
...@@ -215,6 +215,7 @@ static int __init omap_i2c_init(void) ...@@ -215,6 +215,7 @@ static int __init omap_i2c_init(void)
static void __init omap_ldp_init(void) static void __init omap_ldp_init(void)
{ {
omap_i2c_init();
platform_add_devices(ldp_devices, ARRAY_SIZE(ldp_devices)); platform_add_devices(ldp_devices, ARRAY_SIZE(ldp_devices));
omap_board_config = ldp_config; omap_board_config = ldp_config;
omap_board_config_size = ARRAY_SIZE(ldp_config); omap_board_config_size = ARRAY_SIZE(ldp_config);
...@@ -234,7 +235,6 @@ static void __init omap_ldp_map_io(void) ...@@ -234,7 +235,6 @@ static void __init omap_ldp_map_io(void)
omap2_set_globals_343x(); omap2_set_globals_343x();
omap2_map_common_io(); omap2_map_common_io();
} }
arch_initcall(omap_i2c_init);
MACHINE_START(OMAP_LDP, "OMAP LDP board") MACHINE_START(OMAP_LDP, "OMAP LDP board")
.phys_io = 0x48000000, .phys_io = 0x48000000,
......
...@@ -253,6 +253,8 @@ static struct platform_device *omap2_evm_devices[] __initdata = { ...@@ -253,6 +253,8 @@ static struct platform_device *omap2_evm_devices[] __initdata = {
static void __init omap2_evm_init(void) static void __init omap2_evm_init(void)
{ {
omap2_evm_i2c_init();
platform_add_devices(omap2_evm_devices, ARRAY_SIZE(omap2_evm_devices)); platform_add_devices(omap2_evm_devices, ARRAY_SIZE(omap2_evm_devices));
omap_board_config = omap2_evm_config; omap_board_config = omap2_evm_config;
omap_board_config_size = ARRAY_SIZE(omap2_evm_config); omap_board_config_size = ARRAY_SIZE(omap2_evm_config);
...@@ -267,8 +269,6 @@ static void __init omap2_evm_map_io(void) ...@@ -267,8 +269,6 @@ static void __init omap2_evm_map_io(void)
omap2_map_common_io(); omap2_map_common_io();
} }
arch_initcall(omap2_evm_i2c_init);
MACHINE_START(OMAP2EVM, "OMAP2EVM Board") MACHINE_START(OMAP2EVM, "OMAP2EVM Board")
/* Maintainer: Arun KS <arunks@mistralsolutions.com> */ /* Maintainer: Arun KS <arunks@mistralsolutions.com> */
.phys_io = 0x48000000, .phys_io = 0x48000000,
......
...@@ -215,6 +215,8 @@ static struct platform_device *omap3_evm_devices[] __initdata = { ...@@ -215,6 +215,8 @@ static struct platform_device *omap3_evm_devices[] __initdata = {
static void __init omap3_evm_init(void) static void __init omap3_evm_init(void)
{ {
omap3_evm_i2c_init();
platform_add_devices(omap3_evm_devices, ARRAY_SIZE(omap3_evm_devices)); platform_add_devices(omap3_evm_devices, ARRAY_SIZE(omap3_evm_devices));
omap_board_config = omap3_evm_config; omap_board_config = omap3_evm_config;
omap_board_config_size = ARRAY_SIZE(omap3_evm_config); omap_board_config_size = ARRAY_SIZE(omap3_evm_config);
...@@ -230,8 +232,6 @@ static void __init omap3_evm_init(void) ...@@ -230,8 +232,6 @@ static void __init omap3_evm_init(void)
ads7846_dev_init(); ads7846_dev_init();
} }
arch_initcall(omap3_evm_i2c_init);
static void __init omap3_evm_map_io(void) static void __init omap3_evm_map_io(void)
{ {
omap2_set_globals_343x(); omap2_set_globals_343x();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment