Dear "Reinhard Meyer (-VC)",
In message 4C08ED38.8030308@emk-elektronik.de you wrote:
This Patch adds support for the EMK TOP7000 CPU Module Part 2: modified files
Signed-off-by: Reinhard Meyer reinhard.meyer@emk-elektronik.de
MAINTAINERS | 6 ++++- Makefile | 3 ++ board/emk/common/vpd.c | 57 ++++++++++++++++++++++++++++++++++++++++----
Fix your mailer, it is wrapping lines!
@@ -928,6 +928,10 @@ Haavard Skinnemoen haavard.skinnemoen@atmel.com ATSTK1006 AT32AP7000 ATNGW100 AT32AP7000
+Reinhard Meyer reinhard.meyer@emk-elektronik.de
Please keep lists sorted - M < S
--- a/board/emk/common/vpd.c +++ b/board/emk/common/vpd.c @@ -1,6 +1,6 @@ /*
- (C) Copyright 2003
- Reinhard Meyer, EMK Elektronik GmbH, r.meyer@emk-elektronik.de
- (C) Copyright 2003-2010
- Reinhard Meyer, EMK Elektronik
Please add mail address
@@ -68,12 +69,56 @@ void read_factory_r (void) addr += p - buf; /*printf ("%s\n", buf); */ /* search for our specific entry */
if (!strncmp ((char *) buf, "[RLA/lan/Ethernet] ", 19)) {
if (!strncmp ((char *) buf, "[RLA/lan/Ethernet] ", 19)) setenv ("ethaddr", (char *)(buf + 19));
} else if (!strncmp ((char *) buf, "[BOARD/SERIAL] ", 15)) {+#if defined(CONFIG_TOP7000)
/* boards that have 2 LAN interfaces */else if (!strncmp ((char *) buf, "[RLA/lan2/Ethernet] ", 20))setenv ("eth1addr", (char *)(buf + 20));+#endif
Curly braces are needed for multiline statements.
...
- if (eeprom_write (CONFIG_SYS_I2C_FACT_ADDR, CONFIG_SYS_FACT_OFFSET,
buf, i+2)) {
printf ("cannot write factory configuration\n");
All previous comments apply here, too (too long lines, wrapped lines, no spaces after function name, ...)
Best regards,
Wolfgang Denk