[U-Boot-Users] [PATCH] Fix fdt boardsetup command parsing

The introduciton of the 'fdt bootcpu' broke parsing for 'fdt boardsetup'.
Signed-off-by: Kumar Gala galak@kernel.crashing.org --- common/cmd_fdt.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/common/cmd_fdt.c b/common/cmd_fdt.c index c31560b..26f09ad 100644 --- a/common/cmd_fdt.c +++ b/common/cmd_fdt.c @@ -324,7 +324,8 @@ int do_fdt (cmd_tbl_t * cmdtp, int flag, int argc, char *argv[]) /******************************************************************** * Set boot cpu id ********************************************************************/ - } else if ((argv[1][0] == 'b') && (argv[1][1] == 'o')) { + } else if ((argv[1][0] == 'b') && (argv[1][1] == 'o') && + (argv[1][0] == 'o')) { unsigned long tmp = simple_strtoul(argv[2], NULL, 16); fdt_set_boot_cpuid_phys(fdt, tmp);

On Wed, 20 Feb 2008 14:05:33 -0600 (CST) Kumar Gala galak@kernel.crashing.org wrote:
- } else if ((argv[1][0] == 'b') && (argv[1][1] == 'o') &&
(argv[1][0] == 'o')) {
don't you mean argv[1][2] here?
Kim

On Feb 20, 2008, at 2:28 PM, Kim Phillips wrote:
On Wed, 20 Feb 2008 14:05:33 -0600 (CST) Kumar Gala galak@kernel.crashing.org wrote:
- } else if ((argv[1][0] == 'b') && (argv[1][1] == 'o') &&
(argv[1][0] == 'o')) {
don't you mean argv[1][2] here?
gah.. just seeing if anyone is looking at these patches :)
- k
participants (2)
-
Kim Phillips
-
Kumar Gala