]> git.karo-electronics.de Git - karo-tx-uboot.git/commitdiff
fix pxa255_idp board
authorMarcel Ziswiler <marcel@ziswiler.com>
Thu, 18 Oct 2007 22:25:33 +0000 (00:25 +0200)
committerWolfgang Denk <wd@denx.de>
Tue, 23 Oct 2007 14:40:40 +0000 (16:40 +0200)
The pxa255_idp being an old unmaintained board showed several issues:
1. CONFIG_INIT_CRITICAL was still defined.
2. Neither CONFIG_MAC_PARTITION nor CONFIG_DOS_PARTITION was defined.
3. Symbol flash_addr was undeclared.
4. The boards lowlevel_init function was still called memsetup.
5. The TEXT_BASE was still 0xa3000000 rather than 0xa3080000.
6. Using -march=armv5 instead of -march=armv5te resulted in lots of
'target CPU does not support interworking' warnings on recent compilers.
7. The PXA's serial driver redefined FFUART, BTUART and STUART used as
indexes rather than the register definitions from the pxa-regs header
file. Renamed them to FFUART_INDEX, BTUART_INDEX and STUART_INDEX to
avoid any ambiguities.
8. There were several redefinition warnings concerning ICMR, OSMR3,
OSCR, OWER, OIER, RCSR and CCCR in the PXA's assembly start file.
9. The board configuration file was rather outdated.
10. The part header file defined the vendor, product and revision arrays
as unsigned chars instead of just chars in the block_dev_desc_t
structure.

Signed-off-by: Marcel Ziswiler <marcel@ziswiler.com>
board/pxa255_idp/Makefile
board/pxa255_idp/config.mk
board/pxa255_idp/lowlevel_init.S [moved from board/pxa255_idp/memsetup.S with 99% similarity]
board/pxa255_idp/u-boot.lds
cpu/pxa/config.mk
cpu/pxa/serial.c
cpu/pxa/start.S
include/configs/pxa255_idp.h
include/part.h

index a957dd39615ff8880b74eeb508444ca2901018c7..32399f0e94040a35156d5ecb5acaf0c230bca5d3 100644 (file)
@@ -27,7 +27,7 @@ include $(TOPDIR)/config.mk
 LIB    = $(obj)lib$(BOARD).a
 
 COBJS  := pxa_idp.o
-SOBJS  := memsetup.o
+SOBJS  := lowlevel_init.o
 
 SRCS   := $(SOBJS:.o=.S) $(COBJS:.o=.c)
 OBJS   := $(addprefix $(obj),$(COBJS))
index d2a2040e68927b843fcf47b4eab8b33086656c84..55c8b270a11280f6aaad8091289ef55d000ea689 100644 (file)
@@ -1,3 +1,3 @@
 #TEXT_BASE = 0xa1700000
-TEXT_BASE = 0xa3000000
+TEXT_BASE = 0xa3080000
 #TEXT_BASE = 0
similarity index 99%
rename from board/pxa255_idp/memsetup.S
rename to board/pxa255_idp/lowlevel_init.S
index 7e485a28a617fadf09381e3259a5a9da3aa3d820..aaa4d8eb93bd05e5f8efae691d8f122d5035a36f 100644 (file)
@@ -3,7 +3,7 @@
  *
  * NOTE: I haven't clean this up considerably, just enough to get it
  * running. See hal_platform_setup.h for the source. See
- * board/cradle/memsetup.S for another PXA250 setup that is
+ * board/cradle/lowlevel_init.S for another PXA250 setup that is
  * much cleaner.
  *
  * See file CREDITS for list of people who contributed to this
@@ -41,8 +41,8 @@ DRAM_SIZE:  .long   CFG_DRAM_SIZE
 /*
  *     Memory setup
  */
-.globl memsetup
-memsetup:
+.globl lowlevel_init
+lowlevel_init:
 
        mov      r10, lr
 
@@ -395,7 +395,7 @@ initclks:
 
        /* Save SDRAM size */
        ldr     r1, =DRAM_SIZE
-        str       r8, [r1]
+       str     r8, [r1]
 
        /* Interrupt init: Mask all interrupts                              */
        ldr     r0, =ICMR /* enable no sources */
@@ -426,7 +426,7 @@ initclks:
        bl      blink
 #endif
 
-endmemsetup:
+endlowlevel_init:
 
        mov     pc, r10
 
index 20ce1089389e94e13f815802c4b551039eb1a441..2facd832e9b56dc33d04d845f4b6c6c54c484415 100644 (file)
@@ -44,6 +44,7 @@ SECTIONS
        . = ALIGN(4);
        .got : { *(.got) }
 
+       . = .;
        __u_boot_cmd_start = .;
        .u_boot_cmd : { *(.u_boot_cmd) }
        __u_boot_cmd_end = .;
index fb810ca7c26fb88d95034e9545c25bde9b077e00..f0b86b7dc16c06c9223cd7870c4b6c25fe532296 100644 (file)
@@ -25,8 +25,7 @@
 PLATFORM_RELFLAGS += -fno-strict-aliasing  -fno-common -ffixed-r8 \
        -msoft-float
 
-#PLATFORM_CPPFLAGS += -mapcs-32 -march=armv4 -mtune=strongarm1100
-PLATFORM_CPPFLAGS += -march=armv5 -mtune=xscale
+PLATFORM_CPPFLAGS += -march=armv5te -mtune=xscale
 # =========================================================================
 #
 # Supply options according to compiler version
index 51e7f65887855a342cd7c1d1de7e1478255a88f2..9ba457e75aabfa97794a0689dbb6d0b3af191c2f 100644 (file)
 
 DECLARE_GLOBAL_DATA_PTR;
 
-#define FFUART 0
-#define BTUART 1
-#define STUART 2
+#define FFUART_INDEX   0
+#define BTUART_INDEX   1
+#define STUART_INDEX   2
 
 #ifndef CONFIG_SERIAL_MULTI
 #if defined (CONFIG_FFUART)
-#define UART_INDEX     FFUART
+#define UART_INDEX     FFUART_INDEX
 #elif defined (CONFIG_BTUART)
-#define UART_INDEX     BTUART
+#define UART_INDEX     BTUART_INDEX
 #elif defined (CONFIG_STUART)
-#define UART_INDEX     STUART
+#define UART_INDEX     STUART_INDEX
 #else
 #error "Bad: you didn't configure serial ..."
 #endif
@@ -71,7 +71,7 @@ void pxa_setbrg_dev (unsigned int uart_index)
                hang ();
 
        switch (uart_index) {
-               case FFUART:
+               case FFUART_INDEX:
 #ifdef CONFIG_CPU_MONAHANS
                        CKENA |= CKENA_22_FFUART;
 #else
@@ -90,7 +90,7 @@ void pxa_setbrg_dev (unsigned int uart_index)
                        FFIER = IER_UUE;        /* Enable FFUART */
                break;
 
-               case BTUART:
+               case BTUART_INDEX:
 #ifdef CONFIG_CPU_MONAHANS
                        CKENA |= CKENA_21_BTUART;
 #else
@@ -110,7 +110,7 @@ void pxa_setbrg_dev (unsigned int uart_index)
 
                break;
 
-               case STUART:
+               case STUART_INDEX:
 #ifdef CONFIG_CPU_MONAHANS
                        CKENA |= CKENA_23_STUART;
 #else
@@ -154,20 +154,20 @@ int pxa_init_dev (unsigned int uart_index)
 void pxa_putc_dev (unsigned int uart_index,const char c)
 {
        switch (uart_index) {
-               case FFUART:
+               case FFUART_INDEX:
                /* wait for room in the tx FIFO on FFUART */
                        while ((FFLSR & LSR_TEMT) == 0)
                                WATCHDOG_RESET ();      /* Reset HW Watchdog, if needed */
                        FFTHR = c;
                        break;
 
-               case BTUART:
+               case BTUART_INDEX:
                        while ((BTLSR & LSR_TEMT ) == 0 )
                                WATCHDOG_RESET ();      /* Reset HW Watchdog, if needed */
                        BTTHR = c;
                        break;
 
-               case STUART:
+               case STUART_INDEX:
                        while ((STLSR & LSR_TEMT ) == 0 )
                                WATCHDOG_RESET ();      /* Reset HW Watchdog, if needed */
                        STTHR = c;
@@ -187,11 +187,11 @@ void pxa_putc_dev (unsigned int uart_index,const char c)
 int pxa_tstc_dev (unsigned int uart_index)
 {
        switch (uart_index) {
-               case FFUART:
+               case FFUART_INDEX:
                        return FFLSR & LSR_DR;
-               case BTUART:
+               case BTUART_INDEX:
                        return BTLSR & LSR_DR;
-               case STUART:
+               case STUART_INDEX:
                        return STLSR & LSR_DR;
        }
        return -1;
@@ -205,16 +205,16 @@ int pxa_tstc_dev (unsigned int uart_index)
 int pxa_getc_dev (unsigned int uart_index)
 {
        switch (uart_index) {
-               case FFUART:
+               case FFUART_INDEX:
                        while (!(FFLSR & LSR_DR))
                        WATCHDOG_RESET ();      /* Reset HW Watchdog, if needed */
                        return (char) FFRBR & 0xff;
 
-               case BTUART:
+               case BTUART_INDEX:
                        while (!(BTLSR & LSR_DR))
                        WATCHDOG_RESET ();      /* Reset HW Watchdog, if needed */
                        return (char) BTRBR & 0xff;
-               case STUART:
+               case STUART_INDEX:
                        while (!(STLSR & LSR_DR))
                        WATCHDOG_RESET ();      /* Reset HW Watchdog, if needed */
                        return (char) STRBR & 0xff;
@@ -233,32 +233,32 @@ pxa_puts_dev (unsigned int uart_index,const char *s)
 #if defined (CONFIG_FFUART)
 static int ffuart_init(void)
 {
-       return pxa_init_dev(FFUART);
+       return pxa_init_dev(FFUART_INDEX);
 }
 
 static void ffuart_setbrg(void)
 {
-       return pxa_setbrg_dev(FFUART);
+       return pxa_setbrg_dev(FFUART_INDEX);
 }
 
 static void ffuart_putc(const char c)
 {
-       return pxa_putc_dev(FFUART,c);
+       return pxa_putc_dev(FFUART_INDEX,c);
 }
 
 static void ffuart_puts(const char *s)
 {
-       return pxa_puts_dev(FFUART,s);
+       return pxa_puts_dev(FFUART_INDEX,s);
 }
 
 static int ffuart_getc(void)
 {
-       return pxa_getc_dev(FFUART);
+       return pxa_getc_dev(FFUART_INDEX);
 }
 
 static int ffuart_tstc(void)
 {
-       return pxa_tstc_dev(FFUART);
+       return pxa_tstc_dev(FFUART_INDEX);
 }
 
 struct serial_device serial_ffuart_device =
@@ -277,32 +277,32 @@ struct serial_device serial_ffuart_device =
 #if defined (CONFIG_BTUART)
 static int btuart_init(void)
 {
-       return pxa_init_dev(BTUART);
+       return pxa_init_dev(BTUART_INDEX);
 }
 
 static void btuart_setbrg(void)
 {
-       return pxa_setbrg_dev(BTUART);
+       return pxa_setbrg_dev(BTUART_INDEX);
 }
 
 static void btuart_putc(const char c)
 {
-       return pxa_putc_dev(BTUART,c);
+       return pxa_putc_dev(BTUART_INDEX,c);
 }
 
 static void btuart_puts(const char *s)
 {
-       return pxa_puts_dev(BTUART,s);
+       return pxa_puts_dev(BTUART_INDEX,s);
 }
 
 static int btuart_getc(void)
 {
-       return pxa_getc_dev(BTUART);
+       return pxa_getc_dev(BTUART_INDEX);
 }
 
 static int btuart_tstc(void)
 {
-       return pxa_tstc_dev(BTUART);
+       return pxa_tstc_dev(BTUART_INDEX);
 }
 
 struct serial_device serial_btuart_device =
@@ -321,32 +321,32 @@ struct serial_device serial_btuart_device =
 #if defined (CONFIG_STUART)
 static int stuart_init(void)
 {
-       return pxa_init_dev(STUART);
+       return pxa_init_dev(STUART_INDEX);
 }
 
 static void stuart_setbrg(void)
 {
-       return pxa_setbrg_dev(STUART);
+       return pxa_setbrg_dev(STUART_INDEX);
 }
 
 static void stuart_putc(const char c)
 {
-       return pxa_putc_dev(STUART,c);
+       return pxa_putc_dev(STUART_INDEX,c);
 }
 
 static void stuart_puts(const char *s)
 {
-       return pxa_puts_dev(STUART,s);
+       return pxa_puts_dev(STUART_INDEX,s);
 }
 
 static int stuart_getc(void)
 {
-       return pxa_getc_dev(STUART);
+       return pxa_getc_dev(STUART_INDEX);
 }
 
 static int stuart_tstc(void)
 {
-       return pxa_tstc_dev(STUART);
+       return pxa_tstc_dev(STUART_INDEX);
 }
 
 struct serial_device serial_stuart_device =
index ffaa30fdc585417e35c6a234ee8e437ff11533ad..b922485ed33a796926a7d27495d0daa42890f056 100644 (file)
@@ -166,13 +166,17 @@ _start_armboot: .word start_armboot
 /*                                                                         */
 /****************************************************************************/
 /* mk@tbd: Fix this! */
-#ifdef CONFIG_CPU_MONAHANS
+#if defined(CONFIG_PXA250) || defined(CONFIG_CPU_MONAHANS)
 #undef ICMR
 #undef OSMR3
 #undef OSCR
 #undef OWER
 #undef OIER
 #endif
+#ifdef CONFIG_PXA250
+#undef RCSR
+#undef CCCR
+#endif
 
 /* Interrupt-Controller base address                                       */
 IC_BASE:          .word           0x40d00000
index 0e884fc114c9b8ea3c5fa545885ecc044424103d..4a9cadbc73858d4fbd44d388fd9825262dfa3720 100644 (file)
 #include <asm/arch/pxa-regs.h>
 
 /*
- * If we are developing, we might want to start armboot from ram
+ * If we are developing, we might want to start U-Boot from RAM
  * so we MUST NOT initialize critical regs like mem-timing ...
  */
-#define CONFIG_INIT_CRITICAL                   /* undef for developing */
+#undef CONFIG_SKIP_LOWLEVEL_INIT                       /* define for developing */
+#undef CONFIG_SKIP_RELOCATE_UBOOT                      /* define for developing */
 
 /*
  * define the following to enable debug blinks.  A debug blink function
@@ -62,6 +63,7 @@
 #endif
 
 #define CONFIG_MMC             1
+#define CONFIG_DOS_PARTITION   1
 #define BOARD_LATE_INIT                1
 
 #undef CONFIG_USE_IRQ                  /* we don't need IRQ/FIQ stuff */
 #define CONFIG_CMD_FAT
 #define CONFIG_CMD_DHCP
 
-
 #define CONFIG_BOOTDELAY       3
 #define CONFIG_BOOTCOMMAND     "bootm 40000"
 #define CONFIG_BOOTARGS                "root=/dev/mtdblock2 rootfstype=cramfs console=ttyS0,115200"
 #define CFG_FLASH_CFI_DRIVER   1
 
 #define CFG_MONITOR_BASE       0
-#define CFG_MONITOR_LEN                0x40000
+#define CFG_MONITOR_LEN                PHYS_FLASH_SECT_SIZE
 
 #define CFG_MAX_FLASH_BANKS    1       /* max number of memory banks           */
 #define CFG_MAX_FLASH_SECT     128  /* max number of sectors on one chip    */
 #define CFG_ENV_IS_IN_FLASH    1
  /* Addr of Environment Sector */
 #define CFG_ENV_ADDR           (PHYS_FLASH_1 + PHYS_FLASH_SIZE - 0x40000)
-#define CFG_ENV_SIZE           0x40000 /* Total Size of Environment Sector     */
-#define        CFG_ENV_SECT_SIZE       0x40000
+#define CFG_ENV_SIZE           PHYS_FLASH_SECT_SIZE    /* Total Size of Environment Sector     */
+#define        CFG_ENV_SECT_SIZE       (PHYS_FLASH_SECT_SIZE / 16)
 
 #endif /* __CONFIG_H */
index 29c03205f2dc2950f98aa03252eb3a86e348b0f7..37b2b6845bc9acea3d3a907a805a693c252c6f09 100644 (file)
@@ -38,9 +38,9 @@ typedef struct block_dev_desc {
 #endif
        lbaint_t                lba;            /* number of blocks */
        unsigned long   blksz;          /* block size */
-       unsigned char   vendor [40+1];  /* IDE model, SCSI Vendor */
-       unsigned char   product[20+1];  /* IDE Serial no, SCSI product */
-       unsigned char   revision[8+1];  /* firmware revision */
+       char            vendor [40+1];  /* IDE model, SCSI Vendor */
+       char            product[20+1];  /* IDE Serial no, SCSI product */
+       char            revision[8+1];  /* firmware revision */
        unsigned long   (*block_read)(int dev,
                                      unsigned long start,
                                      lbaint_t blkcnt,