[U-Boot-Users] Re: Your patch NAND READ.JFFS2S
Wolfgang Denk
wd at denx.de
Wed Mar 16 16:37:23 CET 2005
[I'm not sure which message this is a Re: to, as I haven't seen any
message(s) with this subject (yet?)]
In message <42384865.5080100 at fsforth.de> you wrote:
>
> you are right, the } has to be moved also.
> So the correct patch is:
>
> else if (cmdtail && !strncmp(cmdtail, ".jffs2s", 2)) {
> cmd |= NANDRW_JFFS2; /* skip bad blocks (on read too) */
> if (cmd & NANDRW_READ)
> cmd |= NANDRW_JFFS2_SKIP; /* skip bad blocks (on
> read too) */
> + }
> + else if (cmdtail && !strncmp(cmdtail, ".jffs2", 2))
> + cmd |= NANDRW_JFFS2; /* skip bad blocks */
I woould reject this because it is violating the coding style.
['}' and 'else' on the same line; indentation by TAB].
More information about the U-Boot
mailing list