From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marex@denx.de>
Date: Sun, 29 Apr 2012 11:18:48 +0200
Subject: [U-Boot] [PATCH v2 11/11] New board support: Nokia RX-51 aka
	N900
In-Reply-To: <2499233.2JzaJFJkCD@pali>
References: <1335634011-9104-1-git-send-email-pali.rohar@gmail.com>
	<20120428213225.07D0D204A66@gemini.denx.de>
	<2499233.2JzaJFJkCD@pali>
Message-ID: <201204291118.48185.marex@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

Dear Pali Roh?r,

> On Saturday 28 April 2012 23:32:25 Wolfgang Denk wrote:
> > Please stop sending base64 encoded patches!   Send plain text
> > only!
> 
> I used git format-patch and git send-email. Is there easy way how
> to tell this to git?
> 
> > While it is a good idea to provide documentation, this is the
> > wrong place for it.  Here in the commit message is about the
> > last place where most of the users will search for it.  Please
> > provide a README for this board - either in the board
> > directory, or in doc/
> 
> Is doc/README.nokia_rx51 correct place?
> 
> > BUt also keep in mind that this is a maintenance problem - who
> > will make sure that this documentation gets updated when
> > someone changes to code?
> > 
> > >  board/nokia/rx51/Makefile        |   46 +++++
> > >  board/nokia/rx51/lowlevel_init.S |   71 +++++++
> > >  board/nokia/rx51/rx51.c          |  380
> > >  ++++++++++++++++++++++++++++++++++++++
> > >  board/nokia/rx51/rx51.h          |  378
> > >  +++++++++++++++++++++++++++++++++++++ boards.cfg
> > >  
> > >             |    1 +
> > >  
> > >  include/configs/nokia_rx51.h     |  373
> > >  +++++++++++++++++++++++++++++++++++++ 6 files changed, 1249
> > >  insertions(+)
> > >  create mode 100644 board/nokia/rx51/Makefile
> > >  create mode 100644 board/nokia/rx51/lowlevel_init.S
> > >  create mode 100644 board/nokia/rx51/rx51.c
> > >  create mode 100644 board/nokia/rx51/rx51.h
> > >  create mode 100644 include/configs/nokia_rx51.h
> > 
> > Entry in MAINTAINERS missing.
> 
> Ok, I will add myself to MAINTAINERS.
> 
> > > +/*
> > > + * Size of malloc() pool
> > > + */
> > > +#define CONFIG_ENV_SIZE			(128 << 10)	/* 128 KiB
> 
> Sector */
> 
> > > +#define CONFIG_UBI_SIZE			(512 << 10)	/* 512 KiB
> 
> Sector */
> 
> > > +#define CONFIG_SYS_MALLOC_LEN		(CONFIG_ENV_SIZE +
> > > CONFIG_UBI_SIZE + \ +					(128 << 10))
> > 
> > Comment appears to be misplaced.
> 
> Ok, comment will be removed.
> 
> > > +/* Maemo kernel 2.6.28 will crash if u-boot enable usb tty
> > > */
> > > +/*
> > > +#define CONFIG_USB_TTY
> > > +*/
> > 
> > Please remove dead code.
> 
> usb tty is usefull for debuging u-boot. because this device does
> not have easy access to serial console, usb tty is only option
> how to connect u-boot terminal to pc. but default (and still only
> one working on this device) kernel crash when u-boot enable usb
> tty. I want to have usb tty option in u-boot and belive that one
> day will be some upstream linux kernel with fixed usb drivers
> working on nokia n900.
> 
> > > +/* OneNand support is disabled, because U-Boot image is too
> > > big */ +/* Uncomment next line to enable it */
> > > +/* #define ONENAND_SUPPORT */
> > 
> > Incorrect multiline comment / please remove dead code.
> 
> How to write correct comment?

/*
 * BLURB blurb BLurB ...
 * Blurb, hardware on fire
 * foo bar it doesn't work.
 */

> 
> Also onenand code working fine, but when is enabled u-boot binary
> is too big and cannot be flashed into this device. But for
> testing in qemu or booting u-boot with enabled onenand support
> stored in mmc via flashed u-boot (with disabled onenand support)
> working too.
> 
> Is there way how to decrease u-boot binary size?

Try compiling it in thumb mode? I think Tom Rini added support for that and 
tested it on omap.

> 
> > Best regards,
> > 
> > Wolfgang Denk