cleanup: DECLARE_CONSOLE_COMMAND only needs 4 args

Since pretty much always, we've declared console commands to take
a "longhelp" argument with detailed explanations of what the
command does. But since almost as long, we've never actually used
that argument for anything - we just silently throw it away in
the macro. There's only one command (usbchargemode) that even
thinks it defines that argument.

We're never going to use this, let's just get rid of it.

BUG=none
BRANCH=none
CQ-DEPEND=CL:*279060
CQ-DEPEND=CL:*279158
CQ-DEPEND=CL:*279037
TEST=make buildall; tested on Cr50 hardware

Everything builds. Since we never used this arg anyway, there had
better not be any difference in the result.

Change-Id: Id3f71a53d02e3dc625cfcc12aa71ecb50e35eb9f
Signed-off-by: Bill Richardson <wfrichar@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/374163
Reviewed-by: Myles Watson <mylesgw@chromium.org>
Reviewed-by: Randall Spangler <rspangler@chromium.org>

35 files changed, 52 insertions, 101 deletions

diff --git a/chip/g/gpio.c b/chip/g/gpio.c
index a6ec1bdcb9..5fd8ed0982 100644
--- a/chip/g/gpio.c
+++ b/chip/g/gpio.c
@@ -437,8 +437,7 @@ static int command_pinmux(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(pinmux, command_pinmux,
 			"",
-			"Display pinmux info",
-			NULL);
+			"Display pinmux info");
 
 static const char * const int_str[] = {
 	"LOW", "FALLING", "HIGH", "RISING",
@@ -481,5 +480,4 @@ static int command_gpiocfg(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(gpiocfg, command_gpiocfg,
 			"",
-			"Display GPIO configs",
-			NULL);
+			"Display GPIO configs");
diff --git a/chip/g/idle.c b/chip/g/idle.c
index 88a2c14eaa..f34525d8db 100644
--- a/chip/g/idle.c
+++ b/chip/g/idle.c
@@ -50,8 +50,7 @@ static int command_idle(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(idle, command_idle,
 			"[w|s|d]",
-			"Set or show the idle action: wfi, sleep, deep sleep",
-			NULL);
+			"Set or show the idle action: wfi, sleep, deep sleep");
 
 static int utmi_wakeup_is_enabled(void)
 {
diff --git a/chip/g/jitter.c b/chip/g/jitter.c
index 1dd938cbf3..3a485d3d84 100644
--- a/chip/g/jitter.c
+++ b/chip/g/jitter.c
@@ -218,6 +218,5 @@ static int command_sof(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(sof, command_sof,
 			"",
-			"Display the SoF clock stuff",
-			NULL);
+			"Display the SoF clock stuff");
 #endif	/* DEBUG_ME */
diff --git a/chip/g/rdd.c b/chip/g/rdd.c
index 7bcd172827..3ef9bf14c6 100644
--- a/chip/g/rdd.c
+++ b/chip/g/rdd.c
@@ -88,4 +88,4 @@ static int command_test_rdd(int argc, char **argv)
 	GWRITE_FIELD(RDD, INT_TEST, INTR_DEBUG_STATE_DETECTED, 1);
 	return EC_SUCCESS;
 }
-DECLARE_CONSOLE_COMMAND(test_rdd, command_test_rdd, "", "", NULL);
+DECLARE_CONSOLE_COMMAND(test_rdd, command_test_rdd, "", "");
diff --git a/chip/g/sps.c b/chip/g/sps.c
index 923e39398c..a25e379bb6 100644
--- a/chip/g/sps.c
+++ b/chip/g/sps.c
@@ -520,6 +520,5 @@ static int command_sps(int argc, char **argv)
 
 DECLARE_CONSOLE_COMMAND(spstest, command_sps,
 			"<num of frames>",
-			"Loop back frames (10 by default) back to the host",
-			NULL);
+			"Loop back frames (10 by default) back to the host");
 #endif /* CONFIG_SPS_TEST */
diff --git a/chip/g/usb.c b/chip/g/usb.c
index 751b4b131d..e2d3e15177 100644
--- a/chip/g/usb.c
+++ b/chip/g/usb.c
@@ -1404,5 +1404,4 @@ static int command_usb(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(usb, command_usb,
 			"[<BOOLEAN> | a | b]",
-			"Get/set the USB connection state and PHY selection",
-			NULL);
+			"Get/set the USB connection state and PHY selection");
diff --git a/chip/g/usb_hid.c b/chip/g/usb_hid.c
index c00d3d20cb..11c82eacbf 100644
--- a/chip/g/usb_hid.c
+++ b/chip/g/usb_hid.c
@@ -155,5 +155,4 @@ static int command_hid(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(hid, command_hid,
 			"[<HID keycode>]",
-			"test USB HID driver",
-			NULL);
+			"test USB HID driver");
diff --git a/chip/it83xx/clock.c b/chip/it83xx/clock.c
index a63a86ec0a..60549c81fe 100644
--- a/chip/it83xx/clock.c
+++ b/chip/it83xx/clock.c
@@ -438,8 +438,7 @@ static int command_idle_stats(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(idlestats, command_idle_stats,
 			"",
-			"Print last idle stats",
-			NULL);
+			"Print last idle stats");
 
 /**
  * Configure deep sleep clock settings.
@@ -486,6 +485,5 @@ DECLARE_CONSOLE_COMMAND(dsleep, command_dsleep,
 		"Use 'off' to allow deep sleep to use heavysleep whenever\n"
 		"conditions allow.\n"
 		"Give a timeout value for the console in use timeout.\n"
-		"See also 'sleepmask'.",
-		NULL);
+		"See also 'sleepmask'.");
 #endif /* CONFIG_LOW_POWER_IDLE */
diff --git a/chip/it83xx/peci.c b/chip/it83xx/peci.c
index 7682ddbcfe..b8e0b7c451 100644
--- a/chip/it83xx/peci.c
+++ b/chip/it83xx/peci.c
@@ -403,8 +403,7 @@ static int peci_cmd(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(peci, peci_cmd,
 			"addr wlen rlen cmd timeout(us)",
-			"PECI command",
-			NULL);
+			"PECI command");
 
 static int command_peci_temp(int argc, char **argv)
 {
@@ -418,5 +417,4 @@ static int command_peci_temp(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(pecitemp, command_peci_temp,
 			NULL,
-			"Print CPU temperature",
-			NULL);
+			"Print CPU temperature");
diff --git a/chip/it83xx/system.c b/chip/it83xx/system.c
index f4ce1d6be9..64d6c09157 100644
--- a/chip/it83xx/system.c
+++ b/chip/it83xx/system.c
@@ -356,5 +356,4 @@ static int command_rw_ec_reg(int argc, char **argv)
 DECLARE_CONSOLE_COMMAND(rwreg, command_rw_ec_reg,
 			"addr [value (byte)]",
 			"R/W EC/PNPCFG registers."
-			" addr 0xec21xxyy for R/W PNPCFG, xx is LDN yy is IDX",
-			NULL);
+			" addr 0xec21xxyy for R/W PNPCFG, xx is LDN yy is IDX");
diff --git a/chip/lm4/adc.c b/chip/lm4/adc.c
index 82f28153e4..278908151c 100644
--- a/chip/lm4/adc.c
+++ b/chip/lm4/adc.c
@@ -214,8 +214,7 @@ static int command_ectemp(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(ectemp, command_ectemp,
 			NULL,
-			"Print EC temperature",
-			NULL);
+			"Print EC temperature");
 #endif
 
 /*****************************************************************************/
diff --git a/chip/lm4/clock.c b/chip/lm4/clock.c
index b8029f5d27..bc4b865403 100644
--- a/chip/lm4/clock.c
+++ b/chip/lm4/clock.c
@@ -551,8 +551,7 @@ static int command_sleep(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(sleep, command_sleep,
 			"[level [clock] [sram pm] [flash pm]]",
-			"Drop into sleep",
-			NULL);
+			"Drop into sleep");
 #endif /* CONFIG_CMD_SLEEP */
 
 #ifdef CONFIG_CMD_PLL
@@ -594,8 +593,7 @@ static int command_pll(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(pll, command_pll,
 			"[ on | off | <div> ]",
-			"Get/set PLL state",
-			NULL);
+			"Get/set PLL state");
 
 #endif /* CONFIG_CMD_PLL */
 
@@ -681,8 +679,7 @@ static int command_clock_gating(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(clockgates, command_clock_gating,
 			"",
-			"Get state of the clock gating controls regs",
-			NULL);
+			"Get state of the clock gating controls regs");
 #endif /* CONFIG_CMD_CLOCKGATES */
 
 #ifdef CONFIG_LOW_POWER_IDLE
@@ -705,8 +702,7 @@ static int command_idle_stats(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(idlestats, command_idle_stats,
 			"",
-			"Print last idle stats",
-			NULL);
+			"Print last idle stats");
 
 /**
  * Configure deep sleep clock settings.
@@ -753,7 +749,6 @@ DECLARE_CONSOLE_COMMAND(dsleep, command_dsleep,
 			"allow deep sleep to auto-select using the low speed "
 			"clock.\n"
 			"Give a timeout value for the console in use timeout.\n"
-			"See also 'sleepmask'.",
-			NULL);
+			"See also 'sleepmask'.");
 #endif /* CONFIG_LOW_POWER_IDLE */
 
diff --git a/chip/lm4/eeprom.c b/chip/lm4/eeprom.c
index b8c45c23e2..81fb47465d 100644
--- a/chip/lm4/eeprom.c
+++ b/chip/lm4/eeprom.c
@@ -146,8 +146,7 @@ static int command_eeprom_info(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(eeinfo, command_eeprom_info,
 			NULL,
-			"Print EEPROM info",
-			NULL);
+			"Print EEPROM info");
 
 
 static int command_eeprom_read(int argc, char **argv)
@@ -178,8 +177,7 @@ static int command_eeprom_read(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(eeread, command_eeprom_read,
 			"block [offset]",
-			"Read a word of EEPROM",
-			NULL);
+			"Read a word of EEPROM");
 
 
 static int command_eeprom_write(int argc, char **argv)
@@ -207,8 +205,7 @@ static int command_eeprom_write(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(eewrite, command_eeprom_write,
 			"block offset value",
-			"Write a word of EEPROM",
-			NULL);
+			"Write a word of EEPROM");
 
 
 #ifdef CONSOLE_COMMAND_EEHIDE
@@ -229,8 +226,7 @@ static int command_eeprom_hide(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(eehide, command_eeprom_hide,
 			"block",
-			"Hide a block of EEPROM",
-			NULL);
+			"Hide a block of EEPROM");
 #endif
 
 
diff --git a/chip/lm4/peci.c b/chip/lm4/peci.c
index 44c1699d26..70f65b5837 100644
--- a/chip/lm4/peci.c
+++ b/chip/lm4/peci.c
@@ -149,5 +149,4 @@ static int command_peci_temp(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(pecitemp, command_peci_temp,
 			NULL,
-			"Print CPU temperature",
-			NULL);
+			"Print CPU temperature");
diff --git a/chip/lm4/spi.c b/chip/lm4/spi.c
index 60394bb936..3fd3bc96de 100644
--- a/chip/lm4/spi.c
+++ b/chip/lm4/spi.c
@@ -184,5 +184,4 @@ static int command_spirom(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(spirom, command_spirom,
 			NULL,
-			"Test reading SPI EEPROM",
-			NULL);
+			"Test reading SPI EEPROM");
diff --git a/chip/lm4/system.c b/chip/lm4/system.c
index 5fec905517..26cd0f2967 100644
--- a/chip/lm4/system.c
+++ b/chip/lm4/system.c
@@ -650,8 +650,7 @@ static int command_system_rtc(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(rtc, command_system_rtc,
 			"[set <seconds>]",
-			"Get/set real-time clock",
-			NULL);
+			"Get/set real-time clock");
 
 #ifdef CONFIG_CMD_RTC_ALARM
 /**
@@ -684,8 +683,7 @@ static int command_rtc_alarm_test(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(rtc_alarm, command_rtc_alarm_test,
 			"[seconds [microseconds]]",
-			"Test alarm",
-			NULL);
+			"Test alarm");
 #endif /* CONFIG_CMD_RTC_ALARM */
 #endif /* CONFIG_CMD_RTC */
 
diff --git a/chip/lm4/uart.c b/chip/lm4/uart.c
index 6ef44f8dc4..21fdf70ec9 100644
--- a/chip/lm4/uart.c
+++ b/chip/lm4/uart.c
@@ -347,7 +347,6 @@ static int command_comxtest(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(comxtest, command_comxtest,
 			"[string]",
-			"Write test data to COMx uart",
-			NULL);
+			"Write test data to COMx uart");
 
 #endif /* CONFIG_CMD_COMXTEST */
diff --git a/chip/mec1322/clock.c b/chip/mec1322/clock.c
index a49d2362aa..0fcb411d23 100644
--- a/chip/mec1322/clock.c
+++ b/chip/mec1322/clock.c
@@ -434,8 +434,7 @@ static int command_idle_stats(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(idlestats, command_idle_stats,
 			"",
-			"Print last idle stats",
-			NULL);
+			"Print last idle stats");
 
 /**
  * Configure deep sleep clock settings.
@@ -481,6 +480,5 @@ DECLARE_CONSOLE_COMMAND(dsleep, command_dsleep,
 			"allow deep sleep to use heavysleep whenever conditions"
 			"allow.\n"
 			"Give a timeout value for the console in use timeout.\n"
-			"See also 'sleepmask'.",
-			NULL);
+			"See also 'sleepmask'.");
 #endif /* CONFIG_LOW_POWER_IDLE */
diff --git a/chip/mec1322/lpc.c b/chip/mec1322/lpc.c
index df40f563a9..298a161762 100644
--- a/chip/mec1322/lpc.c
+++ b/chip/mec1322/lpc.c
@@ -598,7 +598,7 @@ static int lpc_command_init(int argc, char **argv)
 	lpc_init();
 	return EC_SUCCESS;
 }
-DECLARE_CONSOLE_COMMAND(lpcinit, lpc_command_init, NULL, NULL, NULL);
+DECLARE_CONSOLE_COMMAND(lpcinit, lpc_command_init, NULL, NULL);
 
 /* Get protocol information */
 static int lpc_get_protocol_info(struct host_cmd_handler_args *args)
diff --git a/chip/npcx/clock.c b/chip/npcx/clock.c
index 05d49ae751..725473cccb 100644
--- a/chip/npcx/clock.c
+++ b/chip/npcx/clock.c
@@ -380,8 +380,7 @@ static int command_idle_stats(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(idlestats, command_idle_stats,
 		"",
-		"Print last idle stats",
-		NULL);
+		"Print last idle stats");
 
 /**
  * Configure deep sleep clock settings.
@@ -428,6 +427,5 @@ DECLARE_CONSOLE_COMMAND(dsleep, command_dsleep,
 		"allow deep sleep to auto-select using the low speed "
 		"clock.\n"
 		"Give a timeout value for the console in use timeout.\n"
-		"See also 'sleepmask'.",
-		NULL);
+		"See also 'sleepmask'.");
 #endif /* CONFIG_LOW_POWER_IDLE */
diff --git a/chip/npcx/espi.c b/chip/npcx/espi.c
index f998c7bb16..688030f948 100644
--- a/chip/npcx/espi.c
+++ b/chip/npcx/espi.c
@@ -1,4 +1,4 @@
-/* Copyright (c) 2016 The Chromium OS Authors. All rights reserved.
+/* Copyright 2016 The Chromium OS Authors. All rights reserved.
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
@@ -651,6 +651,5 @@ static int command_espi(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(espi, command_espi,
 			"cfg/vms/vsm/en/dis [channel]",
-			"eSPI configurations",
-			NULL);
+			"eSPI configurations");
 
diff --git a/chip/npcx/flash.c b/chip/npcx/flash.c
index e889163088..95b4796bb3 100644
--- a/chip/npcx/flash.c
+++ b/chip/npcx/flash.c
@@ -794,8 +794,7 @@ static int command_flash_spi_sel_lock(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(flash_spi_lock, command_flash_spi_sel_lock,
 			"[0 | 1]",
-			"Lock spi flash interface selection",
-			NULL);
+			"Lock spi flash interface selection");
 
 static int command_flash_tristate(int argc, char **argv)
 {
@@ -812,6 +811,5 @@ static int command_flash_tristate(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(flash_tristate, command_flash_tristate,
 			"[0 | 1]",
-			"Tristate spi flash pins",
-			NULL);
+			"Tristate spi flash pins");
 
diff --git a/chip/npcx/peci.c b/chip/npcx/peci.c
index 4cb926ef18..48920cdea2 100644
--- a/chip/npcx/peci.c
+++ b/chip/npcx/peci.c
@@ -295,5 +295,4 @@ static int command_peci_temp(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(pecitemp, command_peci_temp,
 		NULL,
-		"Print CPU temperature",
-		NULL);
+		"Print CPU temperature");
diff --git a/chip/npcx/spi.c b/chip/npcx/spi.c
index 04a05e6942..790f4c1e47 100644
--- a/chip/npcx/spi.c
+++ b/chip/npcx/spi.c
@@ -260,6 +260,5 @@ static int command_spirom(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(spirom, command_spirom,
 		NULL,
-		"Test reading SPI EEPROM",
-		NULL);
+		"Test reading SPI EEPROM");
 #endif
diff --git a/chip/npcx/system.c b/chip/npcx/system.c
index ec0847cb34..f25bf35fca 100644
--- a/chip/npcx/system.c
+++ b/chip/npcx/system.c
@@ -757,8 +757,7 @@ static int command_system_rtc(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(rtc, command_system_rtc,
 		"[set <seconds>]",
-		"Get/set real-time clock",
-		NULL);
+		"Get/set real-time clock");
 
 #ifdef CONFIG_CMD_RTC_ALARM
 /**
@@ -791,8 +790,7 @@ static int command_rtc_alarm_test(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(rtc_alarm, command_rtc_alarm_test,
 		"[seconds [microseconds]]",
-		"Test alarm",
-		NULL);
+		"Test alarm");
 #endif /* CONFIG_CMD_RTC_ALARM */
 #endif /* CONFIG_CMD_RTC */
 
diff --git a/chip/nrf51/bluetooth_le.c b/chip/nrf51/bluetooth_le.c
index 831edd920c..7fe80ff8e1 100644
--- a/chip/nrf51/bluetooth_le.c
+++ b/chip/nrf51/bluetooth_le.c
@@ -469,8 +469,7 @@ static int command_ble_adv(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(ble_adv, command_ble_adv,
 			"type len [reps] [interval = 100000 (100ms)]",
-			"Send a BLE packet of type type of length len",
-			NULL);
+			"Send a BLE packet of type type of length len");
 
 static int command_ble_adv_scan(int argc, char **argv)
 {
@@ -534,6 +533,5 @@ static int command_ble_adv_scan(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(ble_scan, command_ble_adv_scan,
 			"chan [num] [addr0]",
-			"Scan for [num] BLE packets on channel chan",
-			NULL);
+			"Scan for [num] BLE packets on channel chan");
 
diff --git a/chip/stm32/clock-f.c b/chip/stm32/clock-f.c
index 02773dc678..1d28fa2168 100644
--- a/chip/stm32/clock-f.c
+++ b/chip/stm32/clock-f.c
@@ -203,7 +203,6 @@ static int command_rtc_alarm_test(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(rtc_alarm, command_rtc_alarm_test,
 			"[seconds [microseconds]]",
-			"Test alarm",
-			NULL);
+			"Test alarm");
 #endif /* CONFIG_CMD_RTC_ALARM */
 
diff --git a/chip/stm32/clock-stm32f0.c b/chip/stm32/clock-stm32f0.c
index 98c39694f2..b94641ef0a 100644
--- a/chip/stm32/clock-stm32f0.c
+++ b/chip/stm32/clock-stm32f0.c
@@ -412,7 +412,6 @@ static int command_idle_stats(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(idlestats, command_idle_stats,
 			"",
-			"Print last idle stats",
-			NULL);
+			"Print last idle stats");
 #endif /* CONFIG_CMD_IDLE_STATS */
 #endif
diff --git a/chip/stm32/clock-stm32l.c b/chip/stm32/clock-stm32l.c
index c8b8bc0683..709d53f902 100644
--- a/chip/stm32/clock-stm32l.c
+++ b/chip/stm32/clock-stm32l.c
@@ -373,5 +373,4 @@ static int command_clock(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(clock, command_clock,
 			"hsi | msi",
-			"Set clock frequency",
-			NULL);
+			"Set clock frequency");
diff --git a/chip/stm32/clock-stm32l4.c b/chip/stm32/clock-stm32l4.c
index 1e825e5c39..499c327baf 100644
--- a/chip/stm32/clock-stm32l4.c
+++ b/chip/stm32/clock-stm32l4.c
@@ -386,5 +386,4 @@ DECLARE_CONSOLE_COMMAND(clock, command_clock,
 			" | hse | pll"
 #endif
 			,
-			"Set clock frequency",
-			NULL);
+			"Set clock frequency");
diff --git a/chip/stm32/i2c-stm32l.c b/chip/stm32/i2c-stm32l.c
index cdca08a25a..136a9413ac 100644
--- a/chip/stm32/i2c-stm32l.c
+++ b/chip/stm32/i2c-stm32l.c
@@ -419,5 +419,4 @@ static int command_i2cdump(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(i2cdump, command_i2cdump,
 			NULL,
-			"Dump I2C regs",
-			NULL);
+			"Dump I2C regs");
diff --git a/chip/stm32/power_led.c b/chip/stm32/power_led.c
index 03a7f33a82..c6d4665e84 100644
--- a/chip/stm32/power_led.c
+++ b/chip/stm32/power_led.c
@@ -158,6 +158,5 @@ static int command_powerled(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(powerled, command_powerled,
 		"[off | on | suspend]",
-		"Change power LED state",
-		NULL);
+		"Change power LED state");
 #endif
diff --git a/chip/stm32/usart_info_command.c b/chip/stm32/usart_info_command.c
index 836dc6d84c..f6a76d99f3 100644
--- a/chip/stm32/usart_info_command.c
+++ b/chip/stm32/usart_info_command.c
@@ -40,5 +40,4 @@ static int command_usart_info(int argc, char **argv)
 DECLARE_CONSOLE_COMMAND(usart_info,
 			command_usart_info,
 			NULL,
-			"Display USART info",
-			NULL);
+			"Display USART info");
diff --git a/chip/stm32/usb.c b/chip/stm32/usb.c
index 82b0926702..a3de00f53a 100644
--- a/chip/stm32/usb.c
+++ b/chip/stm32/usb.c
@@ -515,6 +515,5 @@ static int command_serialno(int argc, char **argv)
 
 DECLARE_CONSOLE_COMMAND(serialno, command_serialno,
 	"load/set [value]",
-	"Read and write USB serial number",
-	NULL);
+	"Read and write USB serial number");
 #endif
diff --git a/chip/stm32/usb_hid.c b/chip/stm32/usb_hid.c
index 1f01ec8dc8..ddf1c3ccec 100644
--- a/chip/stm32/usb_hid.c
+++ b/chip/stm32/usb_hid.c
@@ -160,5 +160,4 @@ static int command_hid(int argc, char **argv)
 }
 DECLARE_CONSOLE_COMMAND(hid, command_hid,
 			"[<HID keycode>]",
-			"test USB HID driver",
-			NULL);
+			"test USB HID driver");