From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pali =?ISO-8859-1?Q?Roh=E1r?= Date: Sun, 29 Apr 2012 09:55:59 +0200 Subject: [U-Boot] [PATCH v2 11/11] New board support: Nokia RX-51 aka N900 In-Reply-To: <20120428213225.07D0D204A66@gemini.denx.de> References: <1335634011-9104-1-git-send-email-pali.rohar@gmail.com> <1335634011-9104-12-git-send-email-pali.rohar@gmail.com> <20120428213225.07D0D204A66@gemini.denx.de> Message-ID: <2499233.2JzaJFJkCD@pali> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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? 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? > > Best regards, > > Wolfgang Denk -- Pali Roh?r pali.rohar at gmail.com -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 198 bytes Desc: This is a digitally signed message part. URL: