summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMarek Vasut <marex@denx.de>2012-09-12 19:39:57 +0200
committerTom Rini <trini@ti.com>2012-10-15 11:53:49 -0700
commitb4980515f3a304b90f91a57dc6278f8d07b25995 (patch)
treedf4369c2f0eed31dd867af3d72d8019f1bdcf6f3
parent1fe5c11045c128d36cdc9b68b7d1199aacc7b383 (diff)
downloadu-boot-imx-b4980515f3a304b90f91a57dc6278f8d07b25995.zip
u-boot-imx-b4980515f3a304b90f91a57dc6278f8d07b25995.tar.gz
u-boot-imx-b4980515f3a304b90f91a57dc6278f8d07b25995.tar.bz2
serial: s5p: Move serial registration from serial_initialize()
Move the registration of s5p_serialN_device ports from default serial_initialize() into driver specific function called from serial_initialize(). This slims down the serial_initialize() call to a bare tracker of all possible serial port registration routines in U-Boot. The newly implemented s5p_serial_initialize() function, which is implemented inside of the serial_s5p driver allows encapsulation of s5p_serialN_device within the serial_s5p driver itself. Also, remove the exports of s5p_serialN_device from include/serial.h as they are no longer needed. This is simply because the implementation of default_serial_console() is wrapped into the serial_s5p driver and the default console is picked by CONFIG_SERIAL<N> macro in config file. Signed-off-by: Marek Vasut <marex@denx.de> Cc: Marek Vasut <marek.vasut@gmail.com> Cc: Tom Rini <trini@ti.com> Cc: Minkyu Kang <mk7.kang@samsung.com>
-rw-r--r--common/serial.c8
-rw-r--r--drivers/serial/serial_s5p.c8
-rw-r--r--include/serial.h7
3 files changed, 10 insertions, 13 deletions
diff --git a/common/serial.c b/common/serial.c
index 70f6261..bed907e 100644
--- a/common/serial.c
+++ b/common/serial.c
@@ -43,6 +43,7 @@ static void serial_null(void)
serial_initfunc(mpc8xx_serial_initialize);
serial_initfunc(pxa_serial_initialize);
serial_initfunc(s3c24xx_serial_initialize);
+serial_initfunc(s5p_serial_initialize);
void serial_register(struct serial_device *dev)
{
@@ -78,12 +79,7 @@ void serial_initialize(void)
#endif /* CONFIG_SYS_NS16550_SERIAL */
pxa_serial_initialize();
s3c24xx_serial_initialize();
-#if defined(CONFIG_S5P)
- serial_register(&s5p_serial0_device);
- serial_register(&s5p_serial1_device);
- serial_register(&s5p_serial2_device);
- serial_register(&s5p_serial3_device);
-#endif
+ s5p_serial_initialize();
#if defined(CONFIG_MPC512X)
#if defined(CONFIG_SYS_PSC1)
serial_register(&serial1_device);
diff --git a/drivers/serial/serial_s5p.c b/drivers/serial/serial_s5p.c
index 43cbc4f..3c41242 100644
--- a/drivers/serial/serial_s5p.c
+++ b/drivers/serial/serial_s5p.c
@@ -221,3 +221,11 @@ __weak struct serial_device *default_serial_console(void)
#error "CONFIG_SERIAL? missing."
#endif
}
+
+void s5p_serial_initialize(void)
+{
+ serial_register(&s5p_serial0_device);
+ serial_register(&s5p_serial1_device);
+ serial_register(&s5p_serial2_device);
+ serial_register(&s5p_serial3_device);
+}
diff --git a/include/serial.h b/include/serial.h
index 10fce02..ee7a53f 100644
--- a/include/serial.h
+++ b/include/serial.h
@@ -57,13 +57,6 @@ extern struct serial_device uartlite_serial2_device;
extern struct serial_device uartlite_serial3_device;
#endif
-#if defined(CONFIG_S5P)
-extern struct serial_device s5p_serial0_device;
-extern struct serial_device s5p_serial1_device;
-extern struct serial_device s5p_serial2_device;
-extern struct serial_device s5p_serial3_device;
-#endif
-
#if defined(CONFIG_OMAP3_ZOOM2)
extern struct serial_device zoom2_serial_device0;
extern struct serial_device zoom2_serial_device1;