Re: [U-Boot] [PATCH] cmd_mp: Add support for showing all CPU status by one command

2015-06-23 Thread Tom Rini
On Mon, Jun 22, 2015 at 11:17:39AM +0200, Michal Simek wrote:

 Use one command for showing overall CPU status than several without
 knowing how many cpus is available in the system.
 
 Signed-off-by: Michal Simek michal.si...@xilinx.com

Reviewed-by: Tom Rini tr...@konsulko.com

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] cmd_mp: Add support for showing all CPU status by one command

2015-06-22 Thread Michal Simek
Use one command for showing overall CPU status than several without
knowing how many cpus is available in the system.

Signed-off-by: Michal Simek michal.si...@xilinx.com
---

 common/cmd_mp.c | 22 ++
 1 file changed, 22 insertions(+)

diff --git a/common/cmd_mp.c b/common/cmd_mp.c
index 328b338068b8..a80c6421575e 100644
--- a/common/cmd_mp.c
+++ b/common/cmd_mp.c
@@ -7,11 +7,32 @@
 #include common.h
 #include command.h
 
+static int cpu_status_all(void)
+{
+   unsigned long cpuid;
+
+   for (cpuid = 0; ; cpuid++) {
+   if (!is_core_valid(cpuid)) {
+   if (cpuid == 0) {
+   printf(Core num: %lu is not valid\n, cpuid);
+   return 1;
+   }
+   break;
+   }
+   cpu_status(cpuid);
+   }
+
+   return 0;
+}
+
 static int
 cpu_cmd(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 {
unsigned long cpuid;
 
+   if (argc == 2  strncmp(argv[1], status, 6) == 0)
+ return cpu_status_all();
+
if (argc  3)
return CMD_RET_USAGE;
 
@@ -48,6 +69,7 @@ cpu_cmd(cmd_tbl_t *cmdtp, int flag, int argc, char * const 
argv[])
 #ifdef CONFIG_SYS_LONGHELP
 static char cpu_help_text[] =
num reset - Reset cpu num\n
+   cpu status  - Status of all cpus\n
cpu num status- Status of cpu num\n
cpu num disable   - Disable cpu num\n
cpu num release addr [args] - Release cpu num at addr with 
[args]
-- 
2.3.5

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot