| commit | a587336d7ece5c4dab998d6335b10df990f3817a | [log] [tgz] |
|---|---|---|
| author | Tom Rini <trini@konsulko.com> | Wed May 10 15:20:14 2017 -0400 |
| committer | Tom Rini <trini@konsulko.com> | Fri May 12 08:37:38 2017 -0400 |
| tree | 1279626f8d5ae8807b72b9afb366efcf4a983b18 | |
| parent | e7f8aa1a8648ee140108fe5097173eba58784dc3 [diff] |
cmd/led.c: Remove unnecessary check on 'cmd' value We first check that if argc is less than 2 we return CMD_RET_USAGE. We then see if argc is greater than 2 and if so call get_led_cmd() to set 'cmd' and otherwise set it to LEDST_COUNT (which will always be positive as it's an enum with 0 already assigned). Therefore the test on if cmd is less than 0 will always be false and simply be omitted. Reported by clang-3.8. Signed-off-by: Tom Rini <trini@konsulko.com>