On 07-03-17, 22:48, Michael Sartain wrote:
Fixes checkpatch warning: braces {} are not necessary for any arm of this statement
Well checkpatch isn't supposed to be run on userspace example code. But anyway, I am fine with the improvements.
Signed-off-by: Michael Sartain mikesart@fastmail.com
drivers/staging/greybus/Documentation/firmware/firmware.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-)
diff --git drivers/staging/greybus/Documentation/firmware/firmware.c drivers/staging/greybus/Documentation/firmware/firmware.c index 10ef86120815..70b7fcbba5ad 100644 --- drivers/staging/greybus/Documentation/firmware/firmware.c +++ drivers/staging/greybus/Documentation/firmware/firmware.c @@ -217,13 +217,12 @@ int main(int argc, char *argv[]) if (argc > 2) sscanf(argv[2], "%u", &fw_update_type);
- if (argc > 3) {
- if (argc > 3) firmware_tag = argv[3];
- } else if (!fw_update_type) {
- else if (!fw_update_type) firmware_tag = FW_TAG_INT_DEFAULT;
- } else {
- else firmware_tag = FW_TAG_BCND_DEFAULT;
- }
if (argc > 4) sscanf(argv[4], "%u", &fw_timeout);
Acked-by: Viresh Kumar viresh.kumar@linaro.org