From mboxrd@z Thu Jan  1 00:00:00 1970
From: Heiko Schocher <hs@denx.de>
Date: Tue, 14 Apr 2015 08:21:19 +0200
Subject: [U-Boot] [RESEND PATCH v4 2/3] mtd,
 nand: move common functions from cmd_nand.c to common place
In-Reply-To: <1428957158.22867.635.camel@freescale.com>
References: <1428826337-31738-1-git-send-email-hs@denx.de>
 <1428826337-31738-3-git-send-email-hs@denx.de>
 <1428957158.22867.635.camel@freescale.com>
Message-ID: <552CB1DF.7040301@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hello Scott,

Am 13.04.2015 22:32, schrieb Scott Wood:
> On Sun, 2015-04-12 at 10:12 +0200, Heiko Schocher wrote:
>> @@ -595,7 +491,10 @@ static int do_nand(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
>>   		printf("\nNAND %s: ", cmd);
>>   		/* skip first two or three arguments, look for offset and size */
>>   		if (arg_off_size(argc - o, argv + o, &dev, &off, &size,
>> -				 &maxsize) != 0)
>> +		    &maxsize, MTD_DEV_TYPE_NAND, nand_info[dev].size) != 0)
>
> Please keep the continuation line aligned with the arguments, not the
> function name (here and elsewhere).

Good catch! Fixed. Wait for more comments before posting v5 ...

bye,
Heiko
-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany