test: rockchip: add command description

=> rktest
Command: rktest [module] [args...]
  - module: timer|key|emmc|rknand|regulator|eth|ir|brom|vendor
  - args: depends on module, try 'rktest [module]' for test or more help

  - Enabled modules:
     -      timer: test timer and interrupt
     -       brom: enter bootrom download mode
     -        key: test board keys
     -       emmc: test emmc read/write speed
     -     rknand: test rknand read/write speed
     -  regulator: test regulator volatge set and show regulator status
     -     vendor: test vendor storage partition read/write

Change-Id: I1f58dd65f7ffb229970cb86cb2f568cbca933336
Signed-off-by: Joseph Chen <chenjh@rock-chips.com>

test/rockchip/test-rockchip.c

@@ -10,43 +10,90 @@
 
 typedef struct board_module {
 	char *name;
+	char *desc;
 	int (*test)(int argc, char * const argv[]);
 } board_module_t;
 
 static board_module_t g_board_modules[] = {
 #if defined(CONFIG_IRQ)
-	{ .name = "timer",	.test = board_timer_test },
+	{
+		.name = "timer",
+		.desc = "test timer and interrupt",
+		.test = board_timer_test
+	},
 #endif
-	{ .name = "brom",	.test = board_brom_dnl_test },
+	{
+		.name = "brom",
+		.desc = "enter bootrom download mode",
+		.test = board_brom_dnl_test
+	},
 
 #if defined(CONFIG_DM_KEY)
-	{ .name = "key",	.test = board_key_test },
+	{
+		.name = "key",
+		.desc = "test board keys",
+		.test = board_key_test
+	},
 #endif
 #if defined(CONFIG_MMC)
-	{ .name = "emmc",	.test = board_emmc_test },
+	{
-#endif
+		.name = "emmc",
-#if defined(CONFIG_DM_REGULATOR)
+		.desc = "test emmc read/write speed",
-	{ .name = "regulator",	.test = board_regulator_test },
+		.test = board_emmc_test
+	},
 #endif
 #if defined(CONFIG_RKNAND)
-	{ .name = "rknand",	.test = board_rknand_test },
+	{
+		.name = "rknand",
+		.desc = "test rknand read/write speed",
+		.test = board_rknand_test
+	},
+#endif
+
+#if defined(CONFIG_DM_REGULATOR)
+	{
+		.name = "regulator",
+		.desc = "test regulator volatge set and show regulator status",
+		.test = board_regulator_test
+	},
 #endif
 #if defined(CONFIG_GMAC_ROCKCHIP)
-	{ .name = "eth",        .test = board_eth_test },
+	{
+		.name = "eth",
+		.desc = "test ethernet",
+		.test = board_eth_test
+	},
 #endif
 #if defined(CONFIG_RK_IR)
-	{ .name = "ir",		.test = board_ir_test },
+	{
+		.name = "ir",
+		.desc = "test pwm ir remoter for box product",
+		.test = board_ir_test
+	},
 #endif
 #if defined(CONFIG_ROCKCHIP_VENDOR_PARTITION)
-	{ .name = "vendor",	.test = board_vendor_storage_test },
+	{
+		.name = "vendor",
+		.desc = "test vendor storage partition read/write",
+		.test = board_vendor_storage_test
+	},
 #endif
 };
 
 static void help(void)
 {
-	printf("Command: rktest [module] [args...]\n\n"
+	int i;
+
+	printf("Command: rktest [module] [args...]\n"
 	       "  - module: timer|key|emmc|rknand|regulator|eth|ir|brom|vendor\n"
-	       "  - args: depends on module\n");
+	       "  - args: depends on module, try 'rktest [module]' for test or more help\n\n");
+
+	printf("  - Enabled modules:\n");
+	for (i = 0; i < ARRAY_SIZE(g_board_modules); i++)
+		printf("     - %10s%s %s\n",
+		       g_board_modules[i].name,
+		       g_board_modules[i].desc ? ":" : "",
+		       g_board_modules[i].desc ? g_board_modules[i].desc : "");
 }
 
 static int do_rockchip_test(cmd_tbl_t *cmdtp, int flag,
@@ -68,7 +115,6 @@ static int do_rockchip_test(cmd_tbl_t *cmdtp, int flag,
 	if (!module_name)
 		return 0;
 
-
 	for (index = 0; index < ARRAY_SIZE(g_board_modules); index++) {
 		module = &g_board_modules[index];
 		if (module && !strcmp(module->name, module_name)) {