Hi Kumar, Anton,
Kumar Gala wrote:
On Aug 19, 2009, at 1:37 PM, Anton Vorontsov wrote:
Currently fdt_fixup_stdout() is using hard-coded CONFIG_CONS_INDEX constant. With multi-serial support, the CONS_INDEX may no longer represent actual console, so we should try to extract port number from the current stdio device name instead of always hard-coding the constant value.
Signed-off-by: Anton Vorontsov avorontsov@ru.mvista.com
common/fdt_support.c | 22 +++++++++++++++++++++- 1 files changed, 21 insertions(+), 1 deletions(-)
Should get some Ack from Jerry.
- k
Sorry, I was asleep at the wheel.
I'm OK with the patch as is, but it could be tightened up slightly by having the new function fill in the serial name in the non-multi situation. I renamed the function as well to be fdt_fill_sername() to remove the "multi" connotation.
Below is a *HAND MODIFIED* (i.e. probably broken) edit of Anton's patch reflecting my thoughts.
I'll leave it up to Anton and Kumar... I'm OK with Anton's original patch and you have my Acked-by: Gerald Van Baren vanbaren@cideas.com or you can verify and adopt my proposed change (more work :-() and add a Signed-off-by: Gerald Van Baren vanbaren@cideas.com to Anton's s-o-b.
Thanks, gvb ------------------------------------------------------------------------
Currently fdt_fixup_stdout() is using hard-coded CONFIG_CONS_INDEX constant. With multi-serial support, the CONS_INDEX may no longer represent actual console, so we should try to extract port number from the current stdio device name instead of always hard-coding the constant value.
Signed-off-by: Anton Vorontsov avorontsov@ru.mvista.com Signed-off-by: Gerald Van Baren vanbaren@cideas.com --- common/fdt_support.c | 22 +++++++++++++++++++++- 1 files changed, 21 insertions(+), 1 deletions(-)
diff --git a/common/fdt_support.c b/common/fdt_support.c index 89164a1..e01303a 100644 --- a/common/fdt_support.c +++ b/common/fdt_support.c @@ -22,6 +22,7 @@ */
#include <common.h> +#include <stdio_dev.h> #include <linux/ctype.h> #include <linux/types.h> #include <asm/global_data.h> @@ -90,6 +91,26 @@ int fdt_find_and_setprop(void *fdt, const char *node, const char *prop, }
#ifdef CONFIG_OF_STDOUT_VIA_ALIAS + +#ifdef CONFIG_SERIAL_MULTI +static void fdt_fill_sername(char *sername, size_t maxlen) +{ + const char *outname = stdio_devices[stdout]->name; + + if (strcmp(outname, "serial") > 0) + strncpy(sername, outname, maxlen); + + /* eserial? */ + if (strcmp(outname + 1, "serial") > 0) + strncpy(sername, outname + 1, maxlen); +} +#else +static inline void fdt_fill_sername(char *sername, size_t maxlen) +{ + sprintf(sername, "serial%d", CONFIG_CONS_INDEX - 1); +} +#endif /* CONFIG_SERIAL_MULTI */ + static int fdt_fixup_stdout(void *fdt, int chosenoff) { int err = 0; @@ -98,7 +116,7 @@ static int fdt_fixup_stdout(void *fdt, int chosenoff) char sername[9] = { 0 }; const char *path;
- sprintf(sername, "serial%d", CONFIG_CONS_INDEX - 1); + fdt_fill_sername(sername, sizeof(sername) - 1);
err = node = fdt_path_offset(fdt, "/aliases"); if (node >= 0) {