[U-Boot] [PATCH] PalmTC support
Wolfgang Denk
wd at denx.de
Sun Jul 5 02:01:17 CEST 2009
Dear Marek Vasut,
In message <200907021200.03012.marek.vasut at gmail.com> you wrote:
...
> Patch is included in this mail. Do you also need S-O-B line?
Please make sure to read the documentation before posting, for example
http://www.denx.de/wiki/U-Boot/Patches
> ---
> Makefile | 3 +
> board/palmtc/Makefile | 54 ++++++
> board/palmtc/config.mk | 3 +
> board/palmtc/lowlevel_init.S | 435 ++++++++++++++++++++++++++++++++++++++> ++++
> board/palmtc/palmtc.c | 72 +++++++
> board/palmtc/u-boot.lds | 56 ++++++
> cpu/pxa/pxafb.c | 34 ++++
> include/configs/palmtc.h | 238 +++++++++++++++++++++++
> 8 files changed, 895 insertions(+), 0 deletions(-)
> create mode 100644 board/palmtc/Makefile
> create mode 100644 board/palmtc/config.mk
> create mode 100644 board/palmtc/lowlevel_init.S
> create mode 100644 board/palmtc/palmtc.c
> create mode 100644 board/palmtc/u-boot.lds
> create mode 100644 include/configs/palmtc.h
Entries to MAINTAINERS and MAKEALL (ands eventually to README)
missing.
> diff --git a/Makefile b/Makefile
> index 24e6410..3fdb1c9 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -3041,6 +3041,9 @@ scpu_config: unconfig
> pxa255_idp_config: unconfig
> @$(MKCONFIG) $(@:_config=) arm pxa pxa255_idp
>
> +palmtc_config: unconfig
> + @$(MKCONFIG) $(@:_config=) arm pxa palmtc
> +
Please keep lists sorted.
> trizepsiv_config : unconfig
> @$(MKCONFIG) $(@:_config=) arm pxa trizepsiv
>
> diff --git a/board/palmtc/Makefile b/board/palmtc/Makefile
> new file mode 100644
> index 0000000..92e5288
> --- /dev/null
> +++ b/board/palmtc/Makefile
> @@ -0,0 +1,54 @@
> +
Delete this empty line.
> +#
> +# Copyright (C) 2009
> +# Marek Vasut <marek.vasut at gmail.com>
...
> diff --git a/board/palmtc/config.mk b/board/palmtc/config.mk
> new file mode 100644
> index 0000000..bcb42e7
> --- /dev/null
> +++ b/board/palmtc/config.mk
> @@ -0,0 +1,3 @@
> +#TEXT_BASE = 0xa1700000
> +TEXT_BASE = 0xa1000000
> +#TEXT_BASE = 0
Please delete dead code.
> diff --git a/board/palmtc/lowlevel_init.S b/board/palmtc/lowlevel_init.S
> new file mode 100644
> index 0000000..325bfbd
> --- /dev/null
> +++ b/board/palmtc/lowlevel_init.S
> @@ -0,0 +1,435 @@
...
> +/* wait for clock to stabilize */
> + .macro CLKWAIT
> + ldr r3, =OSCR /* reset the OS Timer Count to zero */
> + mov r2, #0
> + str r2, [r3]
> + ldr r4, =0x300 /* really 0x2E1 is about 200usec, */
> + /* so 0x300 should be plenty */
Please use TAB instead of spaces for indentation / vertical alignment.
> +1:
> + ldr r2, [r3]
> + cmp r4, r2
> + bgt 1b
> + .endm
...
> + ldr r0, =PSSR /* enable GPIO pins */
> + ldr r1, =CONFIG_SYS_PSSR_VAL
> + str r1, [r0]
> +
> + /* ---------------------------------------------------------------- */
> + /* Enable memory interface */
> + /* */
Please use TAB instead of spaces for indentation / vertical alignment.
[also for rest of file]
Also: incorrecxt multiline comment. Please clean up here and
everywhere else.
> + /* detailed in the Intel PXA250 Operating Systems Developers Guide, */
> + /* Chapter 10. */
> + /* ---------------------------------------------------------------- */
...
> diff --git a/include/configs/palmtc.h b/include/configs/palmtc.h
> new file mode 100644
> index 0000000..187a165
> --- /dev/null
> +++ b/include/configs/palmtc.h
...
> +/*
> + * High Level Configuration Options
> + * (easy to change)
> + */
> +#define CONFIG_PXA250 1 /* This is an PXA250 CPU */
> +#define CONFIG_PALMTC 1 /* Palm Tungsten|C board */
> +#define BOARD_LATE_INIT 1
> +#define CONFIG_BOOT_PALMOS 1 /* Dont disable memory we are running from */
Line too long. Please fix here and everywhere else.
> +#define CONFIG_SHOW_BOOT_PROGRESS 1
> +#define CONFIG_MMC 1 /* Enable MMC card */
> +#define CONFIG_LCD
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
Am I your nanny? The kernel is there to support user programs, but
it's a _resource_ handler, not a baby feeder. - Linus Torvalds in
<Pine.LNX.3.91.960425074845.22041C-100000 at linux.cs.Helsinki.FI>
More information about the U-Boot
mailing list