[U-Boot-Users] [PATCH 1/1] v2: coding style cleaning for sc520_spunk

Michal Simek Monstr at seznam.cz
Mon Jul 14 14:44:41 CEST 2008


Hi Jean,

> > From: Michal Simek <monstr at monstr.eu>
> > 
> > there are some long lines in config file
> > but this patches clean a lot of coding style issues
> > 
> > There could be small problem with Daniel name.
> > Please check it in merge
> > 
> > Signed-off-by: Michal Simek <monstr at monstr.eu>
> > ---
> >  include/configs/sc520_spunk.h |  138
> +++++++++++++++++++----------------------
> >  1 files changed, 63 insertions(+), 75 deletions(-)
> > 
> > diff --git a/include/configs/sc520_spunk.h b/include/configs/sc520_spunk.h
> > index 051b2e0..179ecdf 100644
> > --- a/include/configs/sc520_spunk.h
> > +++ b/include/configs/sc520_spunk.h
> > @@ -1,6 +1,6 @@
> >  /*
> >   * (C) Copyright 2002
> > - * Daniel Engstr?m, Omicron Ceti AB, daniel at omicron.se.
> > + * Daniel Engstr�m, Omicron Ceti AB, daniel at omicron.se.
> >   *
> >   * See file CREDITS for list of people who contributed to this
> >   * project.
> > @@ -12,7 +12,7 @@
> >   *
> >   * This program is distributed in the hope that it will be useful,
> >   * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.	 See the
> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> Please do not modify generic licence header

This is absurd tab in header which should be removed from all licenses in U-BOOT.


> >   * GNU General Public License for more details.
> >   *
> >   * You should have received a copy of the GNU General Public License
> > @@ -33,38 +33,36 @@
> >   * (easy to change)
> >   */
> >  #define CONFIG_BOOTDELAY	15
> > -#define CONFIG_BOOTARGS		"root=/dev/mtdblock1 console=ttyS0,9600 " \
> > -					"mtdparts=phys:7936k(root),256k(uboot) "
> > -#define CONFIG_BOOTCOMMAND	"setenv bootargs root=/dev/nfs ip=autoconf " \
> > -					"console=ttyS0,9600 " \
> > -					"mtdparts=phys:7808k(root),128k(env),256k(uboot);" \
> > -					"bootp;bootm"
> > +#define CONFIG_BOOTARGS		"root=/dev/mtdblock1 console=ttyS0,9600
> mtdparts=phys:7936k(root),256k(uboot) "
> > +#define CONFIG_BOOTCOMMAND	"setenv bootargs root=/dev/nfs ip=autoconf
> console=ttyS0,9600 mtdparts=phys:7808k(root),128k(env),256k(uboot); bootp;
> bootm"
> too long, please let the previous way

I know there are some long lines.

M


> >  
> >  #if defined(CONFIG_CMD_KGDB)
> >  #define CONFIG_KGDB_BAUDRATE	115200		/* speed to run kgdb serial port */
> >  #define CONFIG_KGDB_SER_INDEX	2		/* which serial port to use */
> >  #endif
> 
> Please be carreful about the 80 characters limits
> 
> Best Regards,
> J.
> 
> 
> 




More information about the U-Boot mailing list