[U-Boot] [PATCH] - Add support for HP Jornada (710/720/728) board
Wolfgang Denk
wd at denx.de
Tue Oct 12 21:41:46 CEST 2010
Dear Kristoffer Ericson,
In message <20101006194103.GE896 at boggieman.bredbandsbolaget.se> you wrote:
>
> Content-Type: text/plain; charset=utf-8
> Content-Disposition: inline
>
> This patch add support for the HP Jornada 700-series with flashrom (manufact. 2008).
> With little effort it could also be made to support earlier flashroms (just a change of memory settings).
> The HP Jornada is an strongarm device (sa1110 + sa1111).
>
> Signed-off-by: Kristoffer Ericson <kristoffer.ericson at gmail.com>
Please send as plain text. Use git tools to prepare ("git
format-patch") and to submit ("git send-email") the patch. In any
case, please make sure that commit message, Signed-off-by line and
body of the patch are not split into separate MIME parts.
Entry to MAINTAINERS file missing.
...
> +int board_init(void)
> +{
> + gd->bd->bi_arch_number = 48;
> + gd->bd->bi_boot_params = 0xc0000100;
> +
> +
> + /* turn on flashing
> + * would be nice to have some protection but
> + * that would have to be implemented in the
> + * flash init function, which isnt possible yet. */
Incorrect multiline comment style. Please fix globally.
...
> +#define CONFIG_BOOTARGS "root=/dev/hda1 console=ttySA0,19200n8 console=tty1"
> +#define CONFIG_BOOTCOMMAND "run boot_kernel"
> +#define CONFIG_SYS_AUTOLOAD "n" /* No autoload */
> +#define CONFIG_SYS_LOAD_ADDR 0xc0000000
> +
> +/*
> + * Miscellaneous configurable options
> + */
> +#define CONFIG_SYS_LONGHELP /* undef to save memory */
> +#define CONFIG_SYS_PROMPT "HP Jornada# " /* Monitor Command Prompt */
> +#define CONFIG_SYS_CBSIZE 256 /* Console I/O Buffer Size */
> +#define CONFIG_SYS_PBSIZE (CONFIG_SYS_CBSIZE+sizeof(CONFIG_SYS_PROMPT)+16)/* Print Buffer Size */
> +#define CONFIG_SYS_MAXARGS 16 /* max number of command args */
> +#define CONFIG_SYS_BARGSIZE 256 /* Boot Argument Buffer Size */
> +#define CONFIG_SYS_MEMTEST_START 0xc0040000 /* memtest works on */
> +#define CONFIG_SYS_MEMTEST_END 0xc2000000 /* 4 ... 128 MB in SDRAM */
> +#define CONFIG_SYS_HZ 3686400 /* incrementer freq: 3.6864 MHz */
> +#define CONFIG_SYS_CPUSPEED 0x0a /* set core clock to 206MHz */
> +#define CONFIG_SYS_BAUDRATE_TABLE { 9600, 19200, 38400, 57600, 115200 } /* valid baudrates */
Lines too long. Please fix globally.
Please clean up and resubmit.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Never underestimate the bandwidth of a station wagon full of tapes.
-- Dr. Warren Jackson, Director, UTCS
More information about the U-Boot
mailing list