diff options
author | Bill Richardson <wfrichar@chromium.org> | 2016-08-22 17:55:12 -0700 |
---|---|---|
committer | Bill Richardson <wfrichar@chromium.org> | 2016-08-24 16:30:10 +0000 |
commit | bb15561db56728b83ee7fe35e534da7577a7729d (patch) | |
tree | e586ee3b18c8fda464da38b42a06845f0d7f6786 /driver/temp_sensor | |
parent | f322e32a3ec052be3f0c226145448fed63aa7ef9 (diff) | |
download | chrome-ec-bb15561db56728b83ee7fe35e534da7577a7729d.tar.gz |
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>
Diffstat (limited to 'driver/temp_sensor')
-rw-r--r-- | driver/temp_sensor/g78x.c | 2 | ||||
-rw-r--r-- | driver/temp_sensor/tmp006.c | 6 | ||||
-rw-r--r-- | driver/temp_sensor/tmp432.c | 2 |
3 files changed, 4 insertions, 6 deletions
diff --git a/driver/temp_sensor/g78x.c b/driver/temp_sensor/g78x.c index 858ae38769..948569da1c 100644 --- a/driver/temp_sensor/g78x.c +++ b/driver/temp_sensor/g78x.c @@ -231,5 +231,5 @@ static int command_g78x(int argc, char **argv) DECLARE_CONSOLE_COMMAND(g78x, command_g78x, "[settemp|setbyte <offset> <value>] or [getbyte <offset>]. " "Temps in Celsius.", - "Print g781/g782 temp sensor status or set parameters.", NULL); + "Print g781/g782 temp sensor status or set parameters."); #endif diff --git a/driver/temp_sensor/tmp006.c b/driver/temp_sensor/tmp006.c index a816f20ec2..9801963ff5 100644 --- a/driver/temp_sensor/tmp006.c +++ b/driver/temp_sensor/tmp006.c @@ -436,8 +436,7 @@ static int command_sensor_info(int argc, char **argv) } DECLARE_CONSOLE_COMMAND(tmp006, command_sensor_info, "[ <index> ]", - "Print TMP006 sensors", - NULL); + "Print TMP006 sensors"); #endif /* Disable the t6cal command until/unless we have FP support in printf */ @@ -493,6 +492,5 @@ static int command_t6cal(int argc, char **argv) } DECLARE_CONSOLE_COMMAND(t6cal, command_t6cal, "[<index> <coeff_name> <radix>]", - "Set/print TMP006 calibration", - NULL); + "Set/print TMP006 calibration"); #endif diff --git a/driver/temp_sensor/tmp432.c b/driver/temp_sensor/tmp432.c index ea49fcb28b..f288f40ed5 100644 --- a/driver/temp_sensor/tmp432.c +++ b/driver/temp_sensor/tmp432.c @@ -330,7 +330,7 @@ DECLARE_CONSOLE_COMMAND(tmp432, command_tmp432, "[settemp|setbyte <offset> <value>] or [getbyte <offset>] or" "[power <on|off>]. " "Temps in Celsius.", - "Print tmp432 temp sensor status or set parameters.", NULL); + "Print tmp432 temp sensor status or set parameters."); #endif int tmp432_set_power(enum tmp432_power_state power_on) |