* [PATCH 1/3] m68knommu: clean up init code in ColdFire 523x startup
@ 2012-02-27 1:43 gerg
2012-02-27 1:43 ` [PATCH 2/3] m68knommu: clean up init code in ColdFire 528x startup gerg
2012-02-27 1:43 ` [PATCH 3/3] m68knommu: clean up init code in ColdFire 532x startup gerg
0 siblings, 2 replies; 3+ messages in thread
From: gerg @ 2012-02-27 1:43 UTC (permalink / raw)
To: linux-m68k, uclinux-dev; +Cc: Greg Ungerer
From: Greg Ungerer <gerg@uclinux.org>
We can move the QSPI init call to the more general config_BSP() code on
the 523x platorm setup code. Then we can remove the initcall code all
together.
We can also remove the un-needed include of mcfuart.h while we are
cleaning up here too.
Also I noticed that we are not calling the fec_init() code here, and we
should be doing that. Put that back in too.
Signed-off-by: Greg Ungerer <gerg@uclinux.org>
---
arch/m68k/platform/523x/config.c | 11 +----------
1 files changed, 1 insertions(+), 10 deletions(-)
diff --git a/arch/m68k/platform/523x/config.c b/arch/m68k/platform/523x/config.c
index 518c27b..7e39c69 100644
--- a/arch/m68k/platform/523x/config.c
+++ b/arch/m68k/platform/523x/config.c
@@ -19,7 +19,6 @@
#include <asm/machdep.h>
#include <asm/coldfire.h>
#include <asm/mcfsim.h>
-#include <asm/mcfuart.h>
/***************************************************************************/
@@ -66,18 +65,10 @@ static void m523x_cpu_reset(void)
void __init config_BSP(char *commandp, int size)
{
mach_reset = m523x_cpu_reset;
-}
-
-/***************************************************************************/
-
-static int __init init_BSP(void)
-{
+ m523x_fec_init();
#ifdef CONFIG_SPI_COLDFIRE_QSPI
m523x_qspi_init();
#endif
- return 0;
}
-arch_initcall(init_BSP);
-
/***************************************************************************/
--
1.7.0.4
^ permalink raw reply related [flat|nested] 3+ messages in thread* [PATCH 2/3] m68knommu: clean up init code in ColdFire 528x startup
2012-02-27 1:43 [PATCH 1/3] m68knommu: clean up init code in ColdFire 523x startup gerg
@ 2012-02-27 1:43 ` gerg
2012-02-27 1:43 ` [PATCH 3/3] m68knommu: clean up init code in ColdFire 532x startup gerg
1 sibling, 0 replies; 3+ messages in thread
From: gerg @ 2012-02-27 1:43 UTC (permalink / raw)
To: linux-m68k, uclinux-dev; +Cc: Greg Ungerer
From: Greg Ungerer <gerg@uclinux.org>
We can move all the init calls in the initcall code into the more general
arch setup code (which is config_BSP() here). That makes the 528x consistent
with other ColdFire CPUs setup cod. It means we can get rif of the initcall
setup here all together.
Signed-off-by: Greg Ungerer <gerg@uclinux.org>
---
arch/m68k/platform/528x/config.c | 9 ---------
1 files changed, 0 insertions(+), 9 deletions(-)
diff --git a/arch/m68k/platform/528x/config.c b/arch/m68k/platform/528x/config.c
index 7829123..038d13f 100644
--- a/arch/m68k/platform/528x/config.c
+++ b/arch/m68k/platform/528x/config.c
@@ -103,21 +103,12 @@ void __init config_BSP(char *commandp, int size)
#ifdef CONFIG_WILDFIREMOD
mach_halt = wildfiremod_halt;
#endif
-}
-
-/***************************************************************************/
-
-static int __init init_BSP(void)
-{
mach_reset = m528x_cpu_reset;
m528x_uarts_init();
m528x_fec_init();
#ifdef CONFIG_SPI_COLDFIRE_QSPI
m528x_qspi_init();
#endif
- return 0;
}
-arch_initcall(init_BSP);
-
/***************************************************************************/
--
1.7.0.4
^ permalink raw reply related [flat|nested] 3+ messages in thread* [PATCH 3/3] m68knommu: clean up init code in ColdFire 532x startup
2012-02-27 1:43 [PATCH 1/3] m68knommu: clean up init code in ColdFire 523x startup gerg
2012-02-27 1:43 ` [PATCH 2/3] m68knommu: clean up init code in ColdFire 528x startup gerg
@ 2012-02-27 1:43 ` gerg
1 sibling, 0 replies; 3+ messages in thread
From: gerg @ 2012-02-27 1:43 UTC (permalink / raw)
To: linux-m68k, uclinux-dev; +Cc: Greg Ungerer
From: Greg Ungerer <gerg@uclinux.org>
We can move all the init calls in the initcall code into the more general
arch setup code (which is config_BSP() here). That makes the 532x consistent
with other ColdFire CPUs setup code. It means we can get rid of the initcall
setup here all together. Also make sure we set the arch mach_reset function
pointer to get the local arch reset code called on reset.
Signed-off-by: Greg Ungerer <gerg@uclinux.org>
---
arch/m68k/platform/532x/config.c | 21 +++++++--------------
1 files changed, 7 insertions(+), 14 deletions(-)
diff --git a/arch/m68k/platform/532x/config.c b/arch/m68k/platform/532x/config.c
index ff20d3e..99c9e4a 100644
--- a/arch/m68k/platform/532x/config.c
+++ b/arch/m68k/platform/532x/config.c
@@ -82,6 +82,13 @@ void __init config_BSP(char *commandp, int size)
}
#endif
+ mach_reset = m532x_cpu_reset;
+ m532x_uarts_init();
+ m532x_fec_init();
+#ifdef CONFIG_SPI_COLDFIRE_QSPI
+ m532x_qspi_init();
+#endif
+
#ifdef CONFIG_BDM_DISABLE
/*
* Disable the BDM clocking. This also turns off most of the rest of
@@ -93,20 +100,6 @@ void __init config_BSP(char *commandp, int size)
}
/***************************************************************************/
-
-static int __init init_BSP(void)
-{
- m532x_uarts_init();
- m532x_fec_init();
-#ifdef CONFIG_SPI_COLDFIRE_QSPI
- m532x_qspi_init();
-#endif
- return 0;
-}
-
-arch_initcall(init_BSP);
-
-/***************************************************************************/
/* Board initialization */
/***************************************************************************/
/*
--
1.7.0.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2012-02-27 1:42 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-02-27 1:43 [PATCH 1/3] m68knommu: clean up init code in ColdFire 523x startup gerg
2012-02-27 1:43 ` [PATCH 2/3] m68knommu: clean up init code in ColdFire 528x startup gerg
2012-02-27 1:43 ` [PATCH 3/3] m68knommu: clean up init code in ColdFire 532x startup gerg
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox