Re: [U-Boot] [PATCH 03/71] serial: Properly spell out the structure member names of serial_driver

2012-09-18 Thread Michal Simek

On 09/17/2012 01:20 AM, Marek Vasut wrote:

Properly spell out the whole structure member names when an initialized
varible is instantiated from the struct serial_driver. In case the
structure definition for struct serial_driver undergoes reordering,
there will be no impact on variables defined based on this structure.

Signed-off-by: Marek Vasut ma...@denx.de
Cc: Marek Vasut marek.va...@gmail.com
Cc: Tom Rini tr...@ti.com
Cc: Anatolij Gustschin ag...@denx.de
Cc: Stefan Roese s...@denx.de
Cc: Mike Frysinger vap...@gentoo.org
Cc: C Nauman cnau...@diagraph.com
Cc: Minkyu Kang mk7.k...@samsung.com
Cc: Michal Simek mon...@monstr.eu
---
  arch/powerpc/cpu/mpc512x/serial.c  |   18 +-
  arch/powerpc/cpu/mpc5xxx/serial.c  |   32 
  arch/powerpc/cpu/mpc8xx/serial.c   |   32 
  board/logicpd/zoom2/zoom2_serial.h |   16 
  drivers/serial/serial.c|   19 ++-
  drivers/serial/serial_pxa.c|   16 
  drivers/serial/serial_s3c24x0.c|   18 +-
  drivers/serial/serial_s5p.c|   19 ++-
  drivers/serial/serial_xuartlite.c  |   19 ++-
  9 files changed, 96 insertions(+), 93 deletions(-)


Uartlite part looks good to me.
Acked-by: Michal Simek mon...@monstr.eu

Thanks,
Michal


--
Michal Simek, Ing. (M.Eng)
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel 2.6 Microblaze Linux - http://www.monstr.eu/fdt/
Microblaze U-BOOT custodian
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 03/71] serial: Properly spell out the structure member names of serial_driver

2012-09-16 Thread Marek Vasut
Properly spell out the whole structure member names when an initialized
varible is instantiated from the struct serial_driver. In case the
structure definition for struct serial_driver undergoes reordering,
there will be no impact on variables defined based on this structure.

Signed-off-by: Marek Vasut ma...@denx.de
Cc: Marek Vasut marek.va...@gmail.com
Cc: Tom Rini tr...@ti.com
Cc: Anatolij Gustschin ag...@denx.de
Cc: Stefan Roese s...@denx.de
Cc: Mike Frysinger vap...@gentoo.org
Cc: C Nauman cnau...@diagraph.com
Cc: Minkyu Kang mk7.k...@samsung.com
Cc: Michal Simek mon...@monstr.eu
---
 arch/powerpc/cpu/mpc512x/serial.c  |   18 +-
 arch/powerpc/cpu/mpc5xxx/serial.c  |   32 
 arch/powerpc/cpu/mpc8xx/serial.c   |   32 
 board/logicpd/zoom2/zoom2_serial.h |   16 
 drivers/serial/serial.c|   19 ++-
 drivers/serial/serial_pxa.c|   16 
 drivers/serial/serial_s3c24x0.c|   18 +-
 drivers/serial/serial_s5p.c|   19 ++-
 drivers/serial/serial_xuartlite.c  |   19 ++-
 9 files changed, 96 insertions(+), 93 deletions(-)

diff --git a/arch/powerpc/cpu/mpc512x/serial.c 
b/arch/powerpc/cpu/mpc512x/serial.c
index 7c53346..f4f52ad 100644
--- a/arch/powerpc/cpu/mpc512x/serial.c
+++ b/arch/powerpc/cpu/mpc512x/serial.c
@@ -319,15 +319,15 @@ int serial_getcts_dev(unsigned int idx)
serial_puts_dev(port, s); \
}
 
-#define INIT_PSC_SERIAL_STRUCTURE(port, name) { \
-   name, \
-   serial##port##_init, \
-   serial##port##_uninit, \
-   serial##port##_setbrg, \
-   serial##port##_getc, \
-   serial##port##_tstc, \
-   serial##port##_putc, \
-   serial##port##_puts, \
+#define INIT_PSC_SERIAL_STRUCTURE(port, __name) {  \
+   .name   = __name,   \
+   .start  = serial##port##_init,  \
+   .stop   = serial##port##_uninit,\
+   .setbrg = serial##port##_setbrg,\
+   .getc   = serial##port##_getc,  \
+   .tstc   = serial##port##_tstc,  \
+   .putc   = serial##port##_putc,  \
+   .puts   = serial##port##_puts,  \
 }
 
 #if defined(CONFIG_SYS_PSC1)
diff --git a/arch/powerpc/cpu/mpc5xxx/serial.c 
b/arch/powerpc/cpu/mpc5xxx/serial.c
index aa09f67..9bebff8 100644
--- a/arch/powerpc/cpu/mpc5xxx/serial.c
+++ b/arch/powerpc/cpu/mpc5xxx/serial.c
@@ -338,14 +338,14 @@ int serial1_tstc(void)
 
 struct serial_device serial0_device =
 {
-   serial0,
-   serial0_init,
-   NULL,
-   serial0_setbrg,
-   serial0_getc,
-   serial0_tstc,
-   serial0_putc,
-   serial0_puts,
+   .name   = serial0,
+   .start  = serial0_init,
+   .stop   = NULL,
+   .setbrg = serial0_setbrg,
+   .getc   = serial0_getc,
+   .tstc   = serial0_tstc,
+   .putc   = serial0_putc,
+   .puts   = serial0_puts,
 };
 
 __weak struct serial_device *default_serial_console(void)
@@ -355,14 +355,14 @@ __weak struct serial_device *default_serial_console(void)
 
 struct serial_device serial1_device =
 {
-   serial1,
-   serial1_init,
-   NULL,
-   serial1_setbrg,
-   serial1_getc,
-   serial1_tstc,
-   serial1_putc,
-   serial1_puts,
+   .name   = serial1,
+   .start  = serial1_init,
+   .stop   = NULL,
+   .setbrg = serial1_setbrg,
+   .getc   = serial1_getc,
+   .tstc   = serial1_tstc,
+   .putc   = serial1_putc,
+   .puts   = serial1_puts,
 };
 #endif /* CONFIG_SERIAL_MULTI */
 
diff --git a/arch/powerpc/cpu/mpc8xx/serial.c b/arch/powerpc/cpu/mpc8xx/serial.c
index 9239b24..dc9b323 100644
--- a/arch/powerpc/cpu/mpc8xx/serial.c
+++ b/arch/powerpc/cpu/mpc8xx/serial.c
@@ -390,14 +390,14 @@ smc_tstc(void)
 
 struct serial_device serial_smc_device =
 {
-   serial_smc,
-   smc_init,
-   NULL,
-   smc_setbrg,
-   smc_getc,
-   smc_tstc,
-   smc_putc,
-   smc_puts,
+   .name   = serial_smc,
+   .start  = smc_init,
+   .stop   = NULL,
+   .setbrg = smc_setbrg,
+   .getc   = smc_getc,
+   .tstc   = smc_tstc,
+   .putc   = smc_putc,
+   .puts   = smc_puts,
 };
 
 #endif /* CONFIG_8xx_CONS_SMC1 || CONFIG_8xx_CONS_SMC2 */
@@ -660,14 +660,14 @@ scc_tstc(void)
 
 struct serial_device serial_scc_device =
 {
-   serial_scc,
-   scc_init,
-   NULL,
-   scc_setbrg,
-   scc_getc,
-   scc_tstc,
-   scc_putc,
-   scc_puts,
+   .name   = serial_scc,
+   .start  = scc_init,
+   .stop   = NULL,
+   .setbrg = scc_setbrg,
+   .getc   = scc_getc,
+   .tstc   = scc_tstc,
+   .putc   = scc_putc,
+   .puts   = scc_puts,
 };
 
 #endif /* CONFIG_8xx_CONS_SCCx */
diff --git a/board/logicpd/zoom2/zoom2_serial.h