Hi Lengfeld, Am 31.10.2017 um 18:51 schrieb Stefan Lengfeld: > Hi Jorns, > > On Mon, Oct 30, 2017 at 11:34:17AM +0100, Enrico Jorns wrote: >> Signed-off-by: Enrico Jorns <ejo@xxxxxxxxxxxxxx> >> --- >> commands/nv.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/commands/nv.c b/commands/nv.c >> index 37cdb96647..1c5514dd32 100644 >> --- a/commands/nv.c >> +++ b/commands/nv.c >> @@ -56,6 +56,7 @@ static int do_nv(int argc, char *argv[]) >> argv += optind; >> >> if (argc < 1) >> + printf("Invalid usage: Missing argument"); >> return COMMAND_ERROR_USAGE; > > Missing curly brackets for if clause. Indentation does not matter in C > as it does in for example python. Now the function returns always > 'COMMAND_ERROR_USAGE'. ouh, you are right, must have lost them while merging the patches too quickly. But I tried my best to hide it, didn't I? Best regards, Enrico -- Pengutronix e.K. | Enrico Jörns | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-5080 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox