summaryrefslogtreecommitdiff
path: root/chip/it83xx/peci.c
diff options
context:
space:
mode:
authorBill Richardson <wfrichar@chromium.org>2016-08-22 17:55:12 -0700
committerBill Richardson <wfrichar@chromium.org>2016-08-24 16:30:10 +0000
commitbb15561db56728b83ee7fe35e534da7577a7729d (patch)
treee586ee3b18c8fda464da38b42a06845f0d7f6786 /chip/it83xx/peci.c
parentf322e32a3ec052be3f0c226145448fed63aa7ef9 (diff)
downloadchrome-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 'chip/it83xx/peci.c')
-rw-r--r--chip/it83xx/peci.c6
1 files changed, 2 insertions, 4 deletions
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");