'Comming of Age'
cover 'Comming of Age'
Hello Everyone
It has been a while since I posted to the U-Boot list, but I have not been idle. This patch series represents a 'comming of age' for the x86 U-Boot port. I can now boot a Linux 2.6.33 kernel *very happy hacker*
I've spent a lot of effort organising and cleaning up this patch series for community comment. This is an RFC patchset only - Over the next few days/weeks I plan to rebuild from scratch at each patch and apply a few little tidy-ups but the content and numbering of this series should not change.
This patch series can be examined as several distinct functional changes
- Patches 1-3 are x86 specific build and core fixups
- Patches 4-11 are x86 specific functional improvements
- Patches 12-13 switch the x86 port to CONFIG_SERIAL_MULTI
- Patches 14-15 provide Linux boot support
- Patch 16 is an sc520 specific patch (this one can be delayed)
- Patches 17-21 are various eNET specific patches
I will draw your attentions to patches 12,13 and 19. These are the only patches which touch files outside the x86 tree. In particular patch 19 - I had to tweak how the eNET accessed the PCI Address Space (don't know if this is really the correct terminology)
Once this patchset is applied I believe the x86 port of U-Boot will be ready for more mainstream use
Wolfgang - If there is any other renewed interest in the x86 port, I would be more than happy to be an official custodian
RFC PATCH 1/21 Add unaligned.h
--- include/asm-i386/unaligned.h | 14 ++++++++++++++ 1 files changed, 14 insertions(+), 0 deletions(-) create mode 100644 include/asm-i386/unaligned.h
include/asm-i386/unaligned.h new file +14 -0
@@ -0,0 +1,14 @@ +#ifndef _ASM_ARM_UNALIGNED_H +#define _ASM_ARM_UNALIGNED_H + +/* +* The x86 can do unaligned accesses itself. +*/ + +#include <linux/unaligned/access_ok.h> +#include <linux/unaligned/generic.h> + +#define get_unaligned __get_unaligned_le +#define put_unaligned __put_unaligned_le + +#endif /* _ASM_ARM_UNALIGNED_H */
Signature · 1 line
1.7.0.2.182.ge007
RFC PATCH 1/21 Add unaligned.h
why dont you just use asm-generic/unaligned.h ? see asm-blackfin/ ... -mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20100325/bd0cd49f/attachment.pgp
RFC PATCH 2/21 #ifdef out getenv_IPaddr()
In reply to Graeme Russ---
I cannot recall exactly why this is needed but it is
lib_i386/board.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-)
lib_i386/board.c +2 -0
@@ -280,8 +280,10 @@ void board_init_r(gd_t *id, ulong ram_start)show_boot_progress (0x26 ); +#ifdef CONFIG_CMD_NET /* IP Address */ bd_data.bi_ip_addr =getenv_IPaddr ("ipaddr" ); +#endif #if defined(CONFIG_PCI )/*
Signature · 1 line
1.7.0.2.182.ge007
RFC PATCH 2/21 #ifdef out getenv_IPaddr()
Quoted 15 lines · Graeme Russ
---
I cannot recall exactly why this is needed but it is?lib_i386/board.c | ? ?2 ++
?1 files changed, 2 insertions(+), 0 deletions(-)lib_i386/board.c +1
-0@@ -280,8 +280,10 @@ void board_init_r(gd_t *id, ulong ram_start) ? ? ? ?show_boot_progress(0x26); +#ifdef CONFIG_CMD_NET
Quoted 4 lines
? ? ? ?/* IP Address */
? ? ? ?bd_data.bi_ip_addr =getenv_IPaddr("ipaddr"); +#endif
Some boards do not support CMD_NET interface. getenvIPaddr is not defined in that case. that's why
Quoted 10 lines
?#if
defined(CONFIG_PCI)
? ? ? ?/*
--
1.7.0.2.182.ge007_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
RFC PATCH 3/21 Fix MMCR Access
In reply to Graeme RussChange sc520 MMCR Access to use memory accessor functions
---
board/eNET/eNET.c | 86 +++++++++++++++++++++--------------------- cpu/i386/sc520/sc520.c | 33 +++++++++------- cpu/i386/sc520/sc520_pci.c | 24 +++++++---- cpu/i386/sc520/sc520_ssi.c | 27 +++++++------ cpu/i386/sc520/sc520_timer.c | 31 ++++++++------- 5 files changed, 106 insertions(+), 95 deletions(-)
board/eNET/eNET.c +43 -43
@@ -46,7 +46,7 @@ unsigned long monitor_flash_len = CONFIG_SYS_MONITOR_LEN;void init_sc520_enet (void ) {/* Set CPU Speed to 100MHz */ - sc520_mmcr->cpuctl =0x01 ; +writeb (0x01 , &sc520_mmcr->cpuctl);/* wait at least one millisecond */ asm("movl $0x2000,%%ecx\n"
@@ -55,7 +55,7 @@ void init_sc520_enet (void) "loop 0b\n": : : "ecx"); /* turn on the SDRAM write buffer */ - sc520_mmcr->dbctl =0x11 ; + writeb(0x11 , &sc520_mmcr->dbctl);/* turn on the cache and disable write through */ asm("movl %%cr0, %%eax\n"
@@ -70,51 +70,51 @@ int board_early_init_f(void) {init_sc520_enet (); - sc520_mmcr->gpcsrt =0x01 ;/* GP Chip Select Recovery Time */ - sc520_mmcr->gpcspw =0x07 ;/* GP Chip Select Pulse Width */ - sc520_mmcr->gpcsoff =0x00 ;/* GP Chip Select Offset */ - sc520_mmcr->gprdw =0x05 ;/* GP Read pulse width */ - sc520_mmcr->gprdoff =0x01 ;/* GP Read offset */ - sc520_mmcr->gpwrw =0x05 ;/* GP Write pulse width */ - sc520_mmcr->gpwroff =0x01 ;/* GP Write offset */ - - sc520_mmcr->piodata15_0 =0x0630 ;/* PIO15_PIO0 Data */ - sc520_mmcr->piodata31_16 =0x2000 ;/* PIO31_PIO16 Data */ - sc520_mmcr->piodir31_16 =0x2000 ;/* GPIO Direction */ - sc520_mmcr->piodir15_0 =0x87b5 ;/* GPIO Direction */ - sc520_mmcr->piopfs31_16 =0x0dfe ;/* GPIO pin function 31-16 reg */ - sc520_mmcr->piopfs15_0 =0x200a ;/* GPIO pin function 15-0 reg */ - sc520_mmcr->cspfs =0x00f8 ;/* Chip Select Pin Function Select */ - - sc520_mmcr->par[2 ] =0x200713f8 ;/* Uart A (GPCS0, 0x013f8, 8 Bytes) */ - sc520_mmcr->par[3 ] =0x2c0712f8 ;/* Uart B (GPCS3, 0x012f8, 8 Bytes) */ - sc520_mmcr->par[4 ] =0x300711f8 ;/* Uart C (GPCS4, 0x011f8, 8 Bytes) */ - sc520_mmcr->par[5 ] =0x340710f8 ;/* Uart D (GPCS5, 0x010f8, 8 Bytes) */ - sc520_mmcr->par[6 ] =0xe3ffc000 ;/* SDRAM (0x00000000, 128MB) */ - sc520_mmcr->par[7 ] =0xaa3fd000 ;/* StrataFlash (ROMCS1, 0x10000000, 16MB) */ - sc520_mmcr->par[8 ] =0xca3fd100 ;/* StrataFlash (ROMCS2, 0x11000000, 16MB) */ - sc520_mmcr->par[9 ] =0x4203d900 ;/* SRAM (GPCS0, 0x19000000, 1MB) */ - sc520_mmcr->par[10 ] =0x4e03d910 ;/* SRAM (GPCS3, 0x19100000, 1MB) */ - sc520_mmcr->par[11 ] =0x50018100 ;/* DP-RAM (GPCS4, 0x18100000, 4kB) */ - sc520_mmcr->par[12 ] =0x54020000 ;/* CFLASH1 (0x200000000, 4kB) */ - sc520_mmcr->par[13 ] =0x5c020001 ;/* CFLASH2 (0x200010000, 4kB) */ -/* sc520_mmcr->par14 = 0x8bfff800; */ /* BOOTCS at 0x18000000 */ -/* sc520_mmcr->par15 = 0x38201000; */ /* LEDs etc (GPCS6, 0x1000, 20 Bytes */ +writeb (0x01 , &sc520_mmcr->gpcsrt);/* GP Chip Select Recovery Time */ +writeb (0x07 , &sc520_mmcr->gpcspw);/* GP Chip Select Pulse Width */ +writeb (0x00 , &sc520_mmcr->gpcsoff);/* GP Chip Select Offset */ +writeb (0x05 , &sc520_mmcr->gprdw);/* GP Read pulse width */ +writeb (0x01 , &sc520_mmcr->gprdoff);/* GP Read offset */ +writeb (0x05 , &sc520_mmcr->gpwrw);/* GP Write pulse width */ +writeb (0x01 , &sc520_mmcr->gpwroff);/* GP Write offset */ + +writew (0x0630 , &sc520_mmcr->piodata15_0);/* PIO15_PIO0 Data */ +writew (0x2000 , &sc520_mmcr->piodata31_16);/* PIO31_PIO16 Data */ +writew (0x2000 , &sc520_mmcr->piodir31_16);/* GPIO Direction */ +writew (0x87b5 , &sc520_mmcr->piodir15_0);/* GPIO Direction */ +writew (0x0dfe , &sc520_mmcr->piopfs31_16);/* GPIO pin function 31-16 reg */ +writew (0x200a , &sc520_mmcr->piopfs15_0);/* GPIO pin function 15-0 reg */ +writeb (0xf8 , &sc520_mmcr->cspfs);/* Chip Select Pin Function Select */ + +writel (0x200713f8 , &sc520_mmcr->par[2 ]);/* Uart A (GPCS0, 0x013f8, 8 Bytes) */ +writel (0x2c0712f8 , &sc520_mmcr->par[3 ]);/* Uart B (GPCS3, 0x012f8, 8 Bytes) */ +writel (0x300711f8 , &sc520_mmcr->par[4 ]);/* Uart C (GPCS4, 0x011f8, 8 Bytes) */ +writel (0x340710f8 , &sc520_mmcr->par[5 ]);/* Uart D (GPCS5, 0x010f8, 8 Bytes) */ +writel (0xe3ffc000 , &sc520_mmcr->par[6 ]);/* SDRAM (0x00000000, 128MB) */ +writel (0xaa3fd000 , &sc520_mmcr->par[7 ]);/* StrataFlash (ROMCS1, 0x10000000, 16MB) */ +writel (0xca3fd100 , &sc520_mmcr->par[8 ]);/* StrataFlash (ROMCS2, 0x11000000, 16MB) */ +writel (0x4203d900 , &sc520_mmcr->par[9 ]);/* SRAM (GPCS0, 0x19000000, 1MB) */ +writel (0x4e03d910 , &sc520_mmcr->par[10 ]);/* SRAM (GPCS3, 0x19100000, 1MB) */ +writel (0x50018100 , &sc520_mmcr->par[11 ]);/* DP-RAM (GPCS4, 0x18100000, 4kB) */ +writel (0x54020000 , &sc520_mmcr->par[12 ]);/* CFLASH1 (0x200000000, 4kB) */ +writel (0x5c020001 , &sc520_mmcr->par[13 ]);/* CFLASH2 (0x200010000, 4kB) */ +/* writel(0x8bfff800, &sc520_mmcr->par14); */ /* BOOTCS at 0x18000000 */ +/* writel(0x38201000, &sc520_mmcr->par15); */ /* LEDs etc (GPCS6, 0x1000, 20 Bytes */ /* Disable Watchdog */ - sc520_mmcr->wdtmrctl =0x3333 ; - sc520_mmcr->wdtmrctl =0xcccc ; - sc520_mmcr->wdtmrctl =0x0000 ; +writew (0x3333 , &sc520_mmcr->wdtmrctl); +writew (0xcccc , &sc520_mmcr->wdtmrctl); +writew (0x0000 , &sc520_mmcr->wdtmrctl);/* Chip Select Configuration */ - sc520_mmcr->bootcsctl =0x0033 ; - sc520_mmcr->romcs1ctl =0x0615 ; - sc520_mmcr->romcs2ctl =0x0615 ; - - sc520_mmcr->adddecctl =0x02 ; - sc520_mmcr->uart1ctl =0x07 ; - sc520_mmcr->sysarbctl =0x06 ; - sc520_mmcr->sysarbmenb =0x0003 ; +writew (0x0033 , &sc520_mmcr->bootcsctl); +writew (0x0615 , &sc520_mmcr->romcs1ctl); +writew (0x0615 , &sc520_mmcr->romcs2ctl); + +writeb (0x02 , &sc520_mmcr->adddecctl); +writeb (0x07 , &sc520_mmcr->uart1ctl); +writeb (0x06 , &sc520_mmcr->sysarbctl); +writew (0x0003 , &sc520_mmcr->sysarbmenb);return 0 ; }
cpu/i386/sc520/sc520.c +18 -15
@@ -44,24 +44,24 @@ void init_sc520(void)/* Set the UARTxCTL register at it's slower, * baud clock giving us a 1.8432 MHz reference */ - sc520_mmcr->uart1ctl =0x07 ; - sc520_mmcr->uart2ctl =0x07 ; +writeb (0x07 , &sc520_mmcr->uart1ctl); +writeb (0x07 , &sc520_mmcr->uart2ctl);/* first set the timer pin mapping */ - sc520_mmcr->clksel =0x72 ;/* no clock frequency selected, use 1.1892MHz */ +writeb (0x72 , &sc520_mmcr->clksel);/* no clock frequency selected, use 1.1892MHz */ /* enable PCI bus arbitrer */ - sc520_mmcr->sysarbctl =0x02 ;/* enable concurrent mode */ +writeb (0x02 , &sc520_mmcr->sysarbctl);/* enable concurrent mode */ - sc520_mmcr->sysarbmenb =0x1f ;/* enable external grants */ - sc520_mmcr->hbctl =0x04 ;/* enable posted-writes */ +writeb (0x1f , &sc520_mmcr->sysarbmenb);/* enable external grants */ +writeb (0x04 , &sc520_mmcr->hbctl);/* enable posted-writes */ if (CONFIG_SYS_SC520_HIGH_SPEED ) { - sc520_mmcr->cpuctl =0x02 ;/* set it to 133 MHz and write back */ +writeb (0x02 , &sc520_mmcr->cpuctl);/* set it to 133 MHz and write back */ gd->cpu_clk =133000000 ;printf ("## CPU Speed set to 133MHz\n" ); }else { - sc520_mmcr->cpuctl =0x01 ;/* set it to 100 MHz and write back */ +writeb (0x01 , &sc520_mmcr->cpuctl);/* set it to 100 MHz and write back */ printf ("## CPU Speed set to 100MHz\n" ); gd->cpu_clk =100000000 ; }
@@ -74,7 +74,7 @@ void init_sc520(void) "loop 0b\n": : : "ecx"); /* turn on the SDRAM write buffer */ - sc520_mmcr->dbctl =0x11 ; + writeb(0x11 , &sc520_mmcr->dbctl);/* turn on the cache and disable write through */ asm("movl %%cr0, %%eax\n"
@@ -88,6 +88,7 @@ unsigned long init_sc520_dram(void)u32 dram_present=0 ;u32 dram_ctrl; +#ifdef CONFIG_SYS_SDRAM_DRCTMCTL /* these memory control registers are set up in the assember part, * in sc520_asm.S, during 'mem_init'. If we muck with them here,
@@ -97,7 +98,8 @@ unsigned long init_sc520_dram(void)* simply dictates it. */ #else -int val; +u8 tmp; +u8 val;int cas_precharge_delay =CONFIG_SYS_SDRAM_PRECHARGE_DELAY ;int refresh_rate =CONFIG_SYS_SDRAM_REFRESH_RATE ;
@@ -116,9 +118,10 @@ unsigned long init_sc520_dram(void) val =3 ;/* 62.4us */ } - sc520_mmcr->drcctl = (sc520_mmcr->drcctl &0xcf ) | (val<<4 ); + tmp = (readb (&sc520_mmcr->drcctl) &0xcf ) | (val<<4 ); +writeb (tmp, &sc520_mmcr->drcctl); - val = sc520_mmcr->drctmctl &0xf0 ; + val =readb (&sc520_mmcr->drctmctl) &0xf0 ;if (cas_precharge_delay==3 ) { val |=0x04 ;/* 3T */
@@ -133,12 +136,12 @@ unsigned long init_sc520_dram(void) } else { val |=1 ; } - sc520_mmcr->drctmctl = val; +writeb (val, &c520_mmcr->drctmctl);#endif /* We read-back the configuration of the dram * controller that the assembly code wrote */ - dram_ctrl = sc520_mmcr->drcbendadr; + dram_ctrl =readl (&sc520_mmcr->drcbendadr); bd->bi_dram[0 ].start =0 ;if (dram_ctrl &0x80 ) {
@@ -191,7 +194,7 @@ void reset_cpu(ulong addr) {printf ("Resetting using SC520 MMCR\n" );/* Write a '1' to the SYS_RST of the RESCFG MMCR */ - sc520_mmcr->rescfg =0x01 ; +writeb (0x01 , &sc520_mmcr->rescfg);/* NOTREACHED */ }
cpu/i386/sc520/sc520_pci.c +15 -9
@@ -25,6 +25,7 @@#include <common.h> #include <pci.h> +#include <asm/io.h> #include <asm/pci.h> #include <asm/ic/sc520.h>
@@ -63,6 +64,8 @@ int sc520_pci_ints[15] = {int pci_sc520_set_irq (int pci_pin,int irq) {int i; +u8 tmpb; +u16 tmpw;# if 1 printf ("set_irq(): map INT%c to IRQ%d\n" , pci_pin +'A' , irq);
@@ -80,31 +83,34 @@ int pci_sc520_set_irq(int pci_pin, int irq)/* PCI interrupt mapping (A through D)*/ for (i=0 ; i<=3 ;i++) { -if (sc520_mmcr->pci_int_map[i] == sc520_irq[irq].priority) - sc520_mmcr->pci_int_map[i] =SC520_IRQ_DISABLED ; +if (readb (&sc520_mmcr->pci_int_map[i]) == sc520_irq[irq].priority) +writeb (SC520_IRQ_DISABLED , &sc520_mmcr->pci_int_map[i]); }/* GP IRQ interrupt mapping */ for (i=0 ; i<=10 ;i++) { -if (sc520_mmcr->gp_int_map[i] == sc520_irq[irq].priority) - sc520_mmcr->gp_int_map[i] =SC520_IRQ_DISABLED ; +if (readb (&sc520_mmcr->gp_int_map[i]) == sc520_irq[irq].priority) +writeb (SC520_IRQ_DISABLED , &sc520_mmcr->gp_int_map[i]); }/* Set the trigger to level */ - sc520_mmcr->pic_mode[sc520_irq[irq].level_reg] = - sc520_mmcr->pic_mode[sc520_irq[irq].level_reg] | sc520_irq[irq].level_bit; + tmpb =readb (&sc520_mmcr->pic_mode[sc520_irq[irq].level_reg]); + tmpb |= sc520_irq[irq].level_bit; +writeb (tmpb, &sc520_mmcr->pic_mode[sc520_irq[irq].level_reg]);if (pci_pin <4 ) {/* PCI INTA-INTD */ /* route the interrupt */ - sc520_mmcr->pci_int_map[pci_pin] = sc520_irq[irq].priority; +writeb (sc520_irq[irq].priority, &sc520_mmcr->pci_int_map[pci_pin]); }else {/* GPIRQ0-GPIRQ10 used for additional PCI INTS */ - sc520_mmcr->gp_int_map[pci_pin -4 ] = sc520_irq[irq].priority; +writeb (sc520_irq[irq].priority, &sc520_mmcr->gp_int_map[pci_pin -4 ]);/* also set the polarity in this case */ - sc520_mmcr->intpinpol = sc520_mmcr->intpinpol | (1 << (pci_pin-4 )); + tmpw =readw (&sc520_mmcr->intpinpol); + tmpw |= (1 << (pci_pin-4 )); +writew (tmpw, &sc520_mmcr->intpinpol); }/* register the pin */
cpu/i386/sc520/sc520_ssi.c +14 -13
@@ -24,6 +24,7 @@/* stuff specific for the sc520, but independent of implementation */ #include <common.h> +#include <asm/io.h> #include <asm/ic/ssi.h> #include <asm/ic/sc520.h>
@@ -61,34 +62,34 @@ int ssi_set_interface(int freq, int lsb_first, int inv_clock, int inv_phase) temp |=PHS_INV_ENB ; } - sc520_mmcr->ssictl = temp; +writeb (temp, &sc520_mmcr->ssictl);return 0 ; }u8 ssi_txrx_byte (u8 data) { - sc520_mmcr->ssixmit = data; -while (sc520_mmcr->ssista &SSISTA_BSY ); - sc520_mmcr->ssicmd =SSICMD_CMD_SEL_XMITRCV ; -while (sc520_mmcr->ssista &SSISTA_BSY ); +writeb (data, &sc520_mmcr->ssixmit); +while (readb (&sc520_mmcr->ssista) &SSISTA_BSY ); +writeb (SSICMD_CMD_SEL_XMITRCV , &sc520_mmcr->ssicmd); +while (readb (&sc520_mmcr->ssista) &SSISTA_BSY ); -return sc520_mmcr->ssircv; +return readb (&sc520_mmcr->ssircv); }void ssi_tx_byte (u8 data) { - sc520_mmcr->ssixmit = data; -while (sc520_mmcr->ssista &SSISTA_BSY ); - sc520_mmcr->ssicmd =SSICMD_CMD_SEL_XMIT ; +writeb (data, &sc520_mmcr->ssixmit); +while (readb (&sc520_mmcr->ssista) &SSISTA_BSY ); +writeb (SSICMD_CMD_SEL_XMIT , &sc520_mmcr->ssicmd); }u8 ssi_rx_byte (void ) { -while (sc520_mmcr->ssista &SSISTA_BSY ); - sc520_mmcr->ssicmd =SSICMD_CMD_SEL_RCV ; -while (sc520_mmcr->ssista &SSISTA_BSY ); +while (readb (&sc520_mmcr->ssista) &SSISTA_BSY ); +writeb (SSICMD_CMD_SEL_RCV , &sc520_mmcr->ssicmd); +while (readb (&sc520_mmcr->ssista) &SSISTA_BSY ); -return sc520_mmcr->ssircv; +return readb (&sc520_mmcr->ssircv); }
cpu/i386/sc520/sc520_timer.c +16 -15
@@ -24,13 +24,14 @@/* stuff specific for the sc520, but independent of implementation */ #include <common.h> +#include <asm/io.h> #include <asm/interrupt.h> #include <asm/ic/sc520.h> void sc520_timer_isr (void ) {/* Ack the GP Timer Interrupt */ - sc520_mmcr->gptmrsta =0x02 ; +writeb (0x02 , &sc520_mmcr->gptmrsta); }int timer_init (void )
@@ -42,28 +43,28 @@ int timer_init(void)irq_install_handler (0 , timer_isr,NULL );/* Map GP Timer 1 to Master PIC IR0 */ - sc520_mmcr->gp_tmr_int_map[1 ] =0x01 ; +writeb (0x01 , &sc520_mmcr->gp_tmr_int_map[1 ]);/* Disable GP Timers 1 & 2 - Allow configuration writes */ - sc520_mmcr->gptmr1ctl =0x4000 ; - sc520_mmcr->gptmr2ctl =0x4000 ; +writew (0x4000 , &sc520_mmcr->gptmr1ctl); +writew (0x4000 , &sc520_mmcr->gptmr2ctl);/* Reset GP Timers 1 & 2 */ - sc520_mmcr->gptmr1cnt =0x0000 ; - sc520_mmcr->gptmr2cnt =0x0000 ; +writew (0x0000 , &sc520_mmcr->gptmr1cnt); +writew (0x0000 , &sc520_mmcr->gptmr2cnt);/* Setup GP Timer 2 as a 100kHz (10us) prescaler */ - sc520_mmcr->gptmr2maxcmpa =83 ; - sc520_mmcr->gptmr2ctl =0xc001 ; +writew (83 , &sc520_mmcr->gptmr2maxcmpa); +writew (0xc001 , &sc520_mmcr->gptmr2ctl);/* Setup GP Timer 1 as a 1000 Hz (1ms) interrupt generator */ - sc520_mmcr->gptmr1maxcmpa =100 ; - sc520_mmcr->gptmr1ctl =0xe009 ; +writew (100 , &sc520_mmcr->gptmr1maxcmpa); +writew (0xe009 , &sc520_mmcr->gptmr1ctl);unmask_irq (0 );/* Clear the GP Timer 1 status register to get the show rolling*/ - sc520_mmcr->gptmrsta =0x02 ; +writeb (0x02 , &sc520_mmcr->gptmrsta);return 0 ; }
@@ -74,11 +75,11 @@ void __udelay(unsigned long usec)long u;long temp; - temp = sc520_mmcr->swtmrmilli; - temp = sc520_mmcr->swtmrmicro; + temp =readw (&sc520_mmcr->swtmrmilli); + temp =readw (&sc520_mmcr->swtmrmicro);do { - m += sc520_mmcr->swtmrmilli; - u = sc520_mmcr->swtmrmicro + (m *1000 ); + m +=readw (&sc520_mmcr->swtmrmilli); + u =readw (&sc520_mmcr->swtmrmicro) + (m *1000 ); }while (u < usec); }
Signature · 1 line
1.7.0.2.182.ge007
RFC PATCH 4/21 Add register dump to crash handlers
In reply to Graeme RussShamelessly steal the Linux x86 crash handling code and shove it into U-Boot (cool - it fits). Be sure to include suitable attribution to Linus
---
This patch also fixed the annoyance of adding an extra 3 bytes per IRQ handler. I _really_ like this patch - It has probably been the single most useful patch as it adds the ability to use int3 to help debugging
cpu/i386/interrupts.c | 310 ++++++++++++++++++++++++++++++++++++------------- 1 files changed, 228 insertions(+), 82 deletions(-)
cpu/i386/interrupts.c +228 -82
@@ -5,6 +5,9 @@* (C) Copyright 2002 * Daniel Engstr?m, Omicron Ceti AB, daniel at omicron.se. * +* Portions of this file are derived from the Linux kernel source +* Copyright (C) 1991, 1992 Linus Torvalds +* * See file CREDITS for list of people who contributed to this * project. *
@@ -32,12 +35,112 @@ ".hidden irq_"#x"\n" \ ".type irq_"#x", @function\n" \ "irq_"#x":\n" \ - "pushl %ebp\n" \ - "movl %esp,%ebp\n" \ - "pusha\n" \ "pushl $"#x"\n" \ "jmp irq_common_entry\n" +/* +* Volatile isn't enough to prevent the compiler from reordering the +* read/write functions for the control registers and messing everything up. +* A memory clobber would solve the problem, but would prevent reordering of +* all loads stores around it, which can hurt performance. Solution is to +* use a variable and mimic reads and writes to it to enforce serialization +*/ +static unsigned long __force_order; + +static inline unsigned long read_cr0 (void ) +{ +unsigned long val; + asmvolatile ("mov %%cr0,%0\n\t" :"=r" (val),"=m" (__force_order)); +return val; +} + +static inline unsigned long read_cr2 (void ) +{ +unsigned long val; + asmvolatile ("mov %%cr2,%0\n\t" :"=r" (val),"=m" (__force_order)); +return val; +} + +static inline unsigned long read_cr3 (void ) +{ +unsigned long val; + asmvolatile ("mov %%cr3,%0\n\t" :"=r" (val),"=m" (__force_order)); +return val; +} + +static inline unsigned long read_cr4 (void ) +{ +unsigned long val; + asmvolatile ("mov %%cr4,%0\n\t" :"=r" (val),"=m" (__force_order)); +return val; +} + +static inline unsigned long get_debugreg (int regno) +{ +unsigned long val =0 ;/* Damn you, gcc! */ + +switch (regno) { +case 0 : + asm("mov %%db0, %0" :"=r" (val)); +break ; +case 1 : + asm("mov %%db1, %0" :"=r" (val)); +break ; +case 2 : + asm("mov %%db2, %0" :"=r" (val)); +break ; +case 3 : + asm("mov %%db3, %0" :"=r" (val)); +break ; +case 6 : + asm("mov %%db6, %0" :"=r" (val)); +break ; +case 7 : + asm("mov %%db7, %0" :"=r" (val)); +break ; +default : + val =0 ; + } +return val; +} + +void dump_regs (struct pt_regs *regs) +{ +unsigned long cr0 =0L , cr2 =0L , cr3 =0L , cr4 =0L ; +unsigned long d0, d1, d2, d3, d6, d7; + +printf ("EIP: %04x:[<%08lx>] EFLAGS: %08lx\n" , + (u16 )regs->xcs, regs->eip, regs->eflags); + +printf ("EAX: %08lx EBX: %08lx ECX: %08lx EDX: %08lx\n" , + regs->eax, regs->ebx, regs->ecx, regs->edx); +printf ("ESI: %08lx EDI: %08lx EBP: %08lx ESP: %08lx\n" , + regs->esi, regs->edi, regs->ebp, regs->esp); +printf (" DS: %04x ES: %04x FS: %04x GS: %04x SS: %04x\n" , + (u16 )regs->xds, (u16 )regs->xes, (u16 )regs->xfs, (u16 )regs->xgs, (u16 )regs->xss); + + cr0 =read_cr0 (); + cr2 =read_cr2 (); + cr3 =read_cr3 (); + cr4 =read_cr4 (); + +printf ("CR0: %08lx CR2: %08lx CR3: %08lx CR4: %08lx\n" , + cr0, cr2, cr3, cr4); + + d0 =get_debugreg (0 ); + d1 =get_debugreg (1 ); + d2 =get_debugreg (2 ); + d3 =get_debugreg (3 ); + +printf ("DR0: %08lx DR1: %08lx DR2: %08lx DR3: %08lx\n" , + d0, d1, d2, d3); + + d6 =get_debugreg (6 ); + d7 =get_debugreg (7 ); +printf ("DR6: %08lx DR7: %08lx\n" , + d6, d7); +} +struct idt_entry {u16 base_low;u16 selector;
@@ -122,7 +225,7 @@ int disable_interrupts(void) }/* IRQ Low-Level Service Routine */ -__isr__ irq_llsr (int ip,int seg,int irq) +__isr__ irq_llsr (struct pt_regs *regs) {/* * For detailed description of each exception, refer to:
@@ -131,74 +234,93 @@ __isr__ irq_llsr(int ip, int seg, int irq)* Order Number: 253665-029US, November 2008 * Table 6-1. Exceptions and Interrupts */ -switch (irq) { -case 0x00 : -printf ("Divide Error (Division by zero) at %04x:%08x\n" , seg, ip); -while (1 ); -break ; -case 0x01 : -printf ("Debug Interrupt (Single step) at %04x:%08x\n" , seg, ip); -break ; -case 0x02 : -printf ("NMI Interrupt at %04x:%08x\n" , seg, ip); -break ; -case 0x03 : -printf ("Breakpoint at %04x:%08x\n" , seg, ip); -break ; -case 0x04 : -printf ("Overflow at %04x:%08x\n" , seg, ip); -while (1 ); -break ; -case 0x05 : -printf ("BOUND Range Exceeded at %04x:%08x\n" , seg, ip); -while (1 ); -break ; -case 0x06 : -printf ("Invalid Opcode (UnDefined Opcode) at %04x:%08x\n" , seg, ip); -while (1 ); -break ; -case 0x07 : -printf ("Device Not Available (No Math Coprocessor) at %04x:%08x\n" , seg, ip); -while (1 ); -break ; -case 0x08 : -printf ("Double fault at %04x:%08x\n" , seg, ip); -while (1 ); -break ; -case 0x09 : -printf ("Co-processor segment overrun at %04x:%08x\n" , seg, ip); -while (1 ); -break ; -case 0x0a : -printf ("Invalid TSS at %04x:%08x\n" , seg, ip); -break ; -case 0x0b : -printf ("Segment Not Present at %04x:%08x\n" , seg, ip); -while (1 ); -break ; -case 0x0c : -printf ("Stack Segment Fault at %04x:%08x\n" , seg, ip); -while (1 ); -break ; -case 0x0d : -printf ("General Protection at %04x:%08x\n" , seg, ip); -break ; -case 0x0e : -printf ("Page fault at %04x:%08x\n" , seg, ip); -while (1 ); -break ; -case 0x0f : -printf ("Floating-Point Error (Math Fault) at %04x:%08x\n" , seg, ip); -break ; -case 0x10 : -printf ("Alignment check at %04x:%08x\n" , seg, ip); -break ; -case 0x11 : -printf ("Machine Check at %04x:%08x\n" , seg, ip); -break ; -case 0x12 : -printf ("SIMD Floating-Point Exception at %04x:%08x\n" , seg, ip); -break ; +switch (regs->orig_eax) { +case 0x00 : +printf ("Divide Error (Division by zero)\n" ); +dump_regs (regs); +while (1 ); +break ; +case 0x01 : +printf ("Debug Interrupt (Single step)\n" ); +dump_regs (regs); +break ; +case 0x02 : +printf ("NMI Interrupt\n" ); +dump_regs (regs); +break ; +case 0x03 : +printf ("Breakpoint\n" ); +dump_regs (regs); +break ; +case 0x04 : +printf ("Overflow\n" ); +dump_regs (regs); +while (1 ); +break ; +case 0x05 : +printf ("BOUND Range Exceeded\n" ); +dump_regs (regs); +while (1 ); +break ; +case 0x06 : +printf ("Invalid Opcode (UnDefined Opcode)\n" ); +dump_regs (regs); +while (1 ); +break ; +case 0x07 : +printf ("Device Not Available (No Math Coprocessor)\n" ); +dump_regs (regs); +while (1 ); +break ; +case 0x08 : +printf ("Double fault\n" ); +dump_regs (regs); +while (1 ); +break ; +case 0x09 : +printf ("Co-processor segment overrun\n" ); +dump_regs (regs); +while (1 ); +break ; +case 0x0a : +printf ("Invalid TSS\n" ); +dump_regs (regs); +break ; +case 0x0b : +printf ("Segment Not Present\n" ); +dump_regs (regs); +while (1 ); +break ; +case 0x0c : +printf ("Stack Segment Fault\n" ); +dump_regs (regs); +while (1 ); +break ; +case 0x0d : +printf ("General Protection\n" ); +dump_regs (regs); +break ; +case 0x0e : +printf ("Page fault\n" ); +dump_regs (regs); +while (1 ); +break ; +case 0x0f : +printf ("Floating-Point Error (Math Fault)\n" ); +dump_regs (regs); +break ; +case 0x10 : +printf ("Alignment check\n" ); +dump_regs (regs); +break ; +case 0x11 : +printf ("Machine Check\n" ); +dump_regs (regs); +break ; +case 0x12 : +printf ("SIMD Floating-Point Exception\n" ); +dump_regs (regs); +break ;case 0x13 :case 0x14 :case 0x15 :
@@ -212,12 +334,13 @@ __isr__ irq_llsr(int ip, int seg, int irq)case 0x1d :case 0x1e :case 0x1f : -printf ("Reserved Exception %d at %04x:%08x\n" , irq, seg, ip); +printf ("Reserved Exception\n" ); +dump_regs (regs);break ;default :/* Hardware or User IRQ */ -do_irq (irq); +do_irq (regs->orig_eax); } }
@@ -226,22 +349,45 @@ __isr__ irq_llsr(int ip, int seg, int irq)* fully relocatable code. * - The call to irq_llsr will be a relative jump * - The IRQ entries will be guaranteed to be in order -* It's a bit annoying that we need to waste 3 bytes per interrupt entry -* (total of 768 code bytes), but we MUST create a Stack Frame and this is -* the easiest way I could do it. Maybe it can be made better later. +* Interrupt entries are now very small (a push and a jump) but they are +* now slower (all registers pushed on stack which provides complete +* crash dumps in the low level handlers */ asm(".globl irq_common_entry\n" \".hidden irq_common_entry\n" \".type irq_common_entry, @function\n" \"irq_common_entry:\n" \ -"pushl $0\n" \ -"pushl $0\n" \ +"cld\n" \ +"pushl %gs\n" \ +"pushl %fs\n" \ +"pushl %es\n" \ +"pushl %ds\n" \ +"pushl %eax\n" \ +"pushl %ebp\n" \ +"pushl %edi\n" \ +"pushl %esi\n" \ +"pushl %edx\n" \ +"pushl %ecx\n" \ +"pushl %ebx\n" \ +"mov %esp, %eax\n" \ +"pushl %ebp\n" \ +"movl %esp,%ebp\n" \ +"pushl %eax\n" \"call irq_llsr\n" \"popl %eax\n" \ -"popl %eax\n" \ -"popl %eax\n" \ -"popa\n" \"leave\n" \ +"popl %ebx\n" \ +"popl %ecx\n" \ +"popl %edx\n" \ +"popl %esi\n" \ +"popl %edi\n" \ +"popl %ebp\n" \ +"popl %eax\n" \ +"popl %ds\n" \ +"popl %es\n" \ +"popl %fs\n" \ +"popl %gs\n" \ +"add $4, %esp\n" \"iret\n" \ DECLARE_INTERRUPT(0) \DECLARE_INTERRUPT (1 ) \
Signature · 1 line
1.7.0.2.182.ge007
RFC PATCH 5/21 Fix do_go_exec()
In reply to Graeme RussThis was broken a long time ago by a49864593e083a5d0779fb9ca98e5a0f2053183d which munged the NIOS and x86 do_go_exec()
---
PS - I think NIOS is still broken (can someone please check)
lib_i386/board.c | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-)
lib_i386/board.c +4 -4
@@ -422,10 +422,10 @@ void hang (void)unsigned long do_go_exec (ulong (*entry)(int ,char *[]),int argc,char *argv[]) {/* -* TODO: Test this function - changed to fix compiler error. -* Original code was: -* return (entry >> 1) (argc, argv); -* with a comment about Nios function pointers are address >> 1 +* x86 does not use a dedicated register to pass the pointer +* to the global_data */ + argv[-1 ] = (char *)gd; +return (entry) (argc, argv); }
Signature · 1 line
1.7.0.2.182.ge007
RFC PATCH 6/21 Fix sc520 memory size reporting
In reply to Graeme RussThere is an error in how the assembler version of the sc520 memory size reporting code works. As a result, it will only ever report at most the size of one bank of RAM
--- cpu/i386/sc520/sc520_asm.S | 35 +++++++++++++++++------------------ 1 files changed, 17 insertions(+), 18 deletions(-)
cpu/i386/sc520/sc520_asm.S +17 -18
@@ -503,38 +503,37 @@ dram_done: movl $DRCBENDADR, %edi /* DRAM ending address register */ - movl (%edi), %eax - movl %eax, %ecx - andl $0x80000000, %ecx - jz bank2 - andl $0x7f000000, %eax - shrl $2, %eax - movl %eax, %ebx - -bank2: movl (%edi), %eax +bank0: movl (%edi), %eax movl %eax, %ecx - andl $0x00800000, %ecx + andl $0x00000080, %ecx jz bank1 - andl $0x007f0000, %eax - shll $6, %eax + andl $0x0000007f, %eax + shll $22, %eax movl %eax, %ebx bank1: movl (%edi), %eax movl %eax, %ecx andl $0x00008000, %ecx - jz bank0 + jz bank2 andl $0x00007f00, %eax shll $14, %eax movl %eax, %ebx -bank0: movl (%edi), %eax +bank2: movl (%edi), %eax movl %eax, %ecx - andl $0x00000080, %ecx - jz done - andl $0x0000007f, %eax - shll $22, %eax + andl $0x00800000, %ecx + jz bank3 + andl $0x007f0000, %eax + shll $6, %eax movl %eax, %ebx +bank3: movl (%edi), %eax + movl %eax, %ecx + andl $0x80000000, %ecx + jz done + andl $0x7f000000, %eax + shrl $2, %eax + movl %eax, %ebx done: movl %ebx, %eax
Signature · 1 line
1.7.0.2.182.ge007
RFC PATCH 7/21 Split sc520 memory sizing versus reporting
In reply to Graeme RussThis patch allows the low-level assembler boot-strap to obtain the RAM size without calling the destructive 'sizer' routine. This allows boot-strapping from a U-Boot image loaded in RAM
--- cpu/i386/sc520/sc520_asm.S | 95 ++++++++++++++++++++++++++----------------- cpu/i386/start.S | 5 ++ 2 files changed, 62 insertions(+), 38 deletions(-)
cpu/i386/sc520/sc520_asm.S +57 -38
@@ -498,47 +498,21 @@ bad_ram: dram_done: - /* readback DRCBENDADR and return the number - * of available ram bytes in %eax */ - - movl $DRCBENDADR, %edi /* DRAM ending address register */ - -bank0: movl (%edi), %eax - movl %eax, %ecx - andl $0x00000080, %ecx - jz bank1 - andl $0x0000007f, %eax - shll $22, %eax - movl %eax, %ebx - -bank1: movl (%edi), %eax - movl %eax, %ecx - andl $0x00008000, %ecx - jz bank2 - andl $0x00007f00, %eax - shll $14, %eax - movl %eax, %ebx - -bank2: movl (%edi), %eax - movl %eax, %ecx - andl $0x00800000, %ecx - jz bank3 - andl $0x007f0000, %eax - shll $6, %eax - movl %eax, %ebx +#if CONFIG_SYS_SDRAM_ECC_ENABLE + /* + * We are in the middle of an existing 'call' - Need to store the + * existing return address before making another 'call' + */ + movl %ebp, %ebx -bank3: movl (%edi), %eax - movl %eax, %ecx - andl $0x80000000, %ecx - jz done - andl $0x7f000000, %eax - shrl $2, %eax - movl %eax, %ebx + /* Get the memory size */ + movl $init_ecc, %ebp + jmpl get_mem_size -done: - movl %ebx, %eax +init_ecc: + /* Restore the orignal return address */ + movl %ebx, %ebp -#if CONFIG_SYS_SDRAM_ECC_ENABLE /* A nominal memory test: just a byte at each address line */ movl %eax, %ecx shrl $0x1, %ecx
@@ -575,6 +549,51 @@ set_ecc: mov $0x05, %al movb %al, (%edi) #endif + out: + jmp *%ebp + +/* + * Read and decode the sc520 DRCBENDADR MMCR and return the number of + * available ram bytes in %eax + */ +.globl get_mem_size +get_mem_size: + movl $DRCBENDADR, %edi /* DRAM ending address register */ + +bank0: movl (%edi), %eax + movl %eax, %ecx + andl $0x00000080, %ecx + jz bank1 + andl $0x0000007f, %eax + shll $22, %eax + movl %eax, %ebx + +bank1: movl (%edi), %eax + movl %eax, %ecx + andl $0x00008000, %ecx + jz bank2 + andl $0x00007f00, %eax + shll $14, %eax + movl %eax, %ebx + +bank2: movl (%edi), %eax + movl %eax, %ecx + andl $0x00800000, %ecx + jz bank3 + andl $0x007f0000, %eax + shll $6, %eax + movl %eax, %ebx + +bank3: movl (%edi), %eax + movl %eax, %ecx + andl $0x80000000, %ecx + jz done + andl $0x7f000000, %eax + shrl $2, %eax + movl %eax, %ebx + +done: movl %ebx, %eax jmp *%ebp +
cpu/i386/start.S +5 -0
@@ -63,6 +63,11 @@ early_board_init_ret: jmp mem_init mem_init_ret: + /* fetch memory size (into %eax) */ + mov $get_mem_size_ret, %ebp + jmp get_mem_size +get_mem_size_ret: + /* Check we have enough memory for stack */ movl $CONFIG_SYS_STACK_SIZE, %ecx cmpl %ecx, %eax
Signature · 1 line
1.7.0.2.182.ge007
RFC PATCH 8/21 Add RAM bootstrap functionality
In reply to Graeme RussAdd a parameter to the 32-bit entry to indicate if entry is from Real Mode or not. If entry is from Real Mode, execute the destructive 'sizer' routine to determine memory size as we are booting cold and running in Flash. If not entering from Real Mode, we are executing a U-Boot image from RAM and therefore the memory size is already known (and running 'sizer' will destroy the running image)
There are now two 32-bit entry points. The first is the 'in RAM' entry point which exists at the start of the U-Boot binary image. As such, you can load u-boot.bin in RAM and jump directly to the load address without needing to calculate any offsets. The second entry point is used by the real-to-protected mode switch
This patch also changes TEXT_BASE to 0x6000000 (in RAM). You can load the resulting image at 0x6000000 and simple go 0x6000000 from the u-boot prompt
Hopefully a later patch will completely elliminate any dependency on TEXT_BASE like a relocatable linux kernel (perfect world)
--- board/eNET/config.mk | 2 +- board/eNET/u-boot.lds | 9 ++++----- cpu/i386/start.S | 34 ++++++++++++++++++++++++++++++++++ cpu/i386/start16.S | 7 +++++-- 4 files changed, 44 insertions(+), 8 deletions(-)
board/eNET/config.mk +1 -1
@@ -21,7 +21,7 @@ # MA 02111-1307 USA # -TEXT_BASE = 0x38040000 +TEXT_BASE = 0x06000000 CFLAGS_dlmalloc.o += -Wa,--no-warn -fno-strict-aliasing PLATFORM_RELFLAGS += -fvisibility=hidden PLATFORM_CPPFLAGS += -fno-dwarf2-cfi-asm
board/eNET/u-boot.lds +4 -5
@@ -27,7 +27,7 @@ ENTRY(_start) SECTIONS { - . = 0x38040000; /* Location of bootcode in flash */ + . = 0x06000000; /* Location of bootcode in flash */ _i386boot_text_start = .; .text : { *(.text); }
@@ -97,14 +97,13 @@ SECTIONS * at reset and the code have to fit. * The fff0 offset of resetvec is important, however. */ - . = 0xfffffe00; - .start32 : AT (0x3807fe00) { *(.start32); } + .start32 : AT (0x0603fe00) { *(.start32); } . = 0xf800; - .start16 : AT (0x3807f800) { *(.start16); } + .start16 : AT (0x0603f800) { *(.start16); } . = 0xfff0; - .resetvec : AT (0x3807fff0) { *(.resetvec); } + .resetvec : AT (0x0603fff0) { *(.resetvec); } _i386boot_end = (LOADADDR(.resetvec) + SIZEOF(.resetvec) ); }
cpu/i386/start.S +34 -0
@@ -33,7 +33,32 @@ .type _start, @function .globl _i386boot_start _i386boot_start: + /* + * This is the fail safe 32-bit bootstrap entry point. The + * following code is not executed from a cold-reset (actually, a + * lot of it is, but from real-mode after cold reset. It is + * repeated here to put the board into a state as close to cold + * reset as necessary) + */ + cli + cld + + /* Turn of cache (this might require a 486-class CPU) */ + movl %cr0, %eax + orl $0x60000000,%eax + movl %eax, %cr0 + wbinvd + + lidt blank_idt_ptr + + /* Tell 32-bit code it is being entered from */ + movw $0x0000, %bx + + /* Flush the prefetch queue */ + jmp _start _start: + /* This is the 32-bit cold-reset entry point */ + movl $0x18,%eax /* Load our segement registes, the * gdt have already been loaded by start16.S */ movw %ax,%fs
@@ -42,6 +67,15 @@ _start: movw %ax,%es movw %ax,%ss + /* + * Skip low-level board and memory initialization if not starting + * from cold-reset. This allows us to do a fail safe boot-strap + * into a new build of U-Boot from a known-good boot flash + */ + movw $0x0001, %ax + cmpw %ax, %bx + jne mem_init_ret + /* We call a few functions in the board support package * since we have no stack yet we'll have to use %ebp * to store the return address */
cpu/i386/start16.S +5 -2
@@ -45,7 +45,7 @@ board_init16_ret: wbinvd /* load the descriptor tables */ -o32 cs lidt idt_ptr +o32 cs lidt blank_idt_ptr o32 cs lgdt gdt_ptr
@@ -57,6 +57,8 @@ o32 cs lgdt gdt_ptr /* Flush the prefetch queue */ jmp ff ff: + /* Tell 32-bit code it is being entered from hard-reset */ + movw $0x0001, %bx /* Finally jump to the 32bit initialization code */ movw $code32start, %ax
@@ -68,7 +70,8 @@ code32start: .long _start /* offset */ .word 0x10 /* segment */ -idt_ptr: +.global blank_idt_ptr +blank_idt_ptr: .word 0 /* limit */ .long 0 /* base */
Signature · 1 line
1.7.0.2.182.ge007
RFC PATCH 9/21 Move GDT to a safe location in RAM
In reply to Graeme RussCurrently, the GDT is either located in FLASH or in the non-relocated U-Boot image in RAM. Both of these locations are unsafe as those locations can be erased during a U-Boot update. Move the GDT into the highest available memory location and relocate U-Boot to just below it
--- cpu/i386/start.S | 110 +++++++++++++++++++++++++++++++++++++++++++++++++--- cpu/i386/start16.S | 28 +++---------- 2 files changed, 111 insertions(+), 27 deletions(-)
cpu/i386/start.S +104 -6
@@ -49,13 +49,8 @@ _i386boot_start: movl %eax, %cr0 wbinvd - lidt blank_idt_ptr - - /* Tell 32-bit code it is being entered from */ + /* Tell 32-bit code it is being entered from an in-RAM copy */ movw $0x0000, %bx - - /* Flush the prefetch queue */ - jmp _start _start: /* This is the 32-bit cold-reset entry point */
@@ -67,6 +62,9 @@ _start: movw %ax,%es movw %ax,%ss + /* Clear the interupt vectors */ + lidt blank_idt_ptr + /* * Skip low-level board and memory initialization if not starting * from cold-reset. This allows us to do a fail safe boot-strap
@@ -102,6 +100,53 @@ mem_init_ret: jmp get_mem_size get_mem_size_ret: + /* + * We are now in 'Flat Protected Mode' and we know how much memory + * the board has. The (temporary) Global Descriptor Table is not + * in a 'Safe' place (it is either in Flash which can be erased or + * reprogrammed or in a fail-safe boot-strap image which could be + * over-ridden). + * + * Move the final gdt to a safe place (top of RAM) and load it. + * This is not a trivial excercise - the lgdt instruction does not + * have a register operand (memory only) and we may well be + * running from Flash, so self modifying code will not work here. + * To overcome this, we copy a stub into upper memory along with + * the GDT. + */ + + /* Reduce upper memory limit by (Stub + GDT Pointer + GDT) */ + subl $(end_gdt_setup - start_gdt_setup), %eax + + /* Copy the GDT and Stub */ + movl $start_gdt_setup, %esi + movl %eax, %edi + movl $(end_gdt_setup - start_gdt_setup), %ecx + shrl $2, %ecx + cld + rep movsl + + /* write the lgdt 'parameter' */ + subl $(jmp_instr - start_gdt_setup - 4), %ebp + addl %eax, %ebp + movl $(gdt_ptr - start_gdt_setup), %ebx + addl %eax, %ebx + movl %ebx, (%ebp) + + /* write the gdt address into the pointer */ + movl $(gdt_addr - start_gdt_setup), %ebp + addl %eax, %ebp + movl $(gdt - start_gdt_setup), %ebx + addl %eax, %ebx + movl %ebx, (%ebp) + + /* Save the return address */ + movl $load_gdt_ret, %ebp + + /* Load the new (safe) Global Descriptor Table */ + jmp *%eax + +load_gdt_ret: /* Check we have enough memory for stack */ movl $CONFIG_SYS_STACK_SIZE, %ecx cmpl %ecx, %eax
@@ -172,3 +217,56 @@ stack_ok: die: hlt jmp die hlt + +blank_idt_ptr: + .word 0 /* limit */ + .long 0 /* base */ + +.align 4 +start_gdt_setup: + lgdt gdt_ptr +jmp_instr: + jmp *%ebp + +.align 4 +gdt_ptr: + .word 0x30 /* limit (48 bytes = 6 GDT entries) */ +gdt_addr: + .long gdt /* base */ + + /* The GDT table ... + * + * Selector Type + * 0x00 NULL + * 0x08 Unused + * 0x10 32bit code + * 0x18 32bit data/stack + * 0x20 16bit code + * 0x28 16bit data/stack + */ + +.align 4 +gdt: + .word 0, 0, 0, 0 /* NULL */ + .word 0, 0, 0, 0 /* unused */ + + .word 0xFFFF /* 4Gb - (0x100000*0x1000 = 4Gb) */ + .word 0 /* base address = 0 */ + .word 0x9B00 /* code read/exec */ + .word 0x00CF /* granularity = 4096, 386 (+5th nibble of limit) */ + + .word 0xFFFF /* 4Gb - (0x100000*0x1000 = 4Gb) */ + .word 0x0 /* base address = 0 */ + .word 0x9300 /* data read/write */ + .word 0x00CF /* granularity = 4096, 386 (+5th nibble of limit) */ + + .word 0xFFFF /* 64kb */ + .word 0 /* base address = 0 */ + .word 0x9b00 /* data read/write */ + .word 0x0010 /* granularity = 1 (+5th nibble of limit) */ + + .word 0xFFFF /* 64kb */ + .word 0 /* base address = 0 */ + .word 0x9300 /* data read/write */ + .word 0x0010 /* granularity = 1 (+5th nibble of limit) */ +end_gdt_setup:
cpu/i386/start16.S +7 -21
@@ -44,11 +44,9 @@ board_init16_ret: movl %eax, %cr0 wbinvd - /* load the descriptor tables */ -o32 cs lidt blank_idt_ptr + /* load the temporary Global Descriptor Table */ o32 cs lgdt gdt_ptr - /* Now, we enter protected mode */ movl %cr0, %eax orl $1,%eax
@@ -70,13 +68,13 @@ code32start: .long _start /* offset */ .word 0x10 /* segment */ -.global blank_idt_ptr -blank_idt_ptr: - .word 0 /* limit */ - .long 0 /* base */ - +/* + * The following Global Descriptor Table is just enough to get us into + * 'Flat Protected Mode' - It will be discarded as soon as the final + * GDT is setup in a safe location in RAM + */ gdt_ptr: - .word 0x30 /* limit (48 bytes = 6 GDT entries) */ + .word 0x20 /* limit (32 bytes = 4 GDT entries) */ .long BOOT_SEG + gdt /* base */ /* The GDT table ...
@@ -86,8 +84,6 @@ gdt_ptr: * 0x08 Unused * 0x10 32bit code * 0x18 32bit data/stack - * 0x20 16bit code - * 0x28 16bit data/stack */ gdt:
@@ -103,13 +99,3 @@ gdt: .word 0x0 /* base address = 0 */ .word 0x9300 /* data read/write */ .word 0x00CF /* granularity = 4096, 386 (+5th nibble of limit) */ - - .word 0xFFFF /* 64kb */ - .word 0 /* base address = 0 */ - .word 0x9b00 /* data read/write */ - .word 0x0010 /* granularity = 1 (+5th nibble of limit) */ - - .word 0xFFFF /* 64kb */ - .word 0 /* base address = 0 */ - .word 0x9300 /* data read/write */ - .word 0x0010 /* granularity = 1 (+5th nibble of limit) */
Signature · 1 line
1.7.0.2.182.ge007
RFC PATCH 10/21 Pass relocation offset into Global Data
In reply to Graeme RussIn order to locate the 16-bit BIOS code, we need to know the reloaction offset.
---
This was a lot tricker than it should have been
--- lib_i386/board.c | 63 ++++++++++++++++++++++++++++++----------------------- 1 files changed, 36 insertions(+), 27 deletions(-)
lib_i386/board.c +36 -27
@@ -52,7 +52,9 @@ extern ulong _i386boot_rel_dyn_start;extern ulong _i386boot_rel_dyn_end;extern ulong _i386boot_bss_start;extern ulong _i386boot_bss_size; -void ram_bootstrap (void *); + +void ram_bootstrap (void *,ulong ); +const char version_string[] =U_BOOT_VERSION " (" U_BOOT_DATE " - " U_BOOT_TIME ")" ;
@@ -162,6 +164,7 @@ init_fnc_t *init_sequence[] = { NULL, }; +static gd_t gd_data;gd_t *gd;/*
@@ -174,21 +177,18 @@ void board_init_f (ulong stack_limit)Elf32_Rel *rel_dyn_start = (Elf32_Rel *)&_i386boot_rel_dyn_start;Elf32_Rel *rel_dyn_end = (Elf32_Rel *)&_i386boot_rel_dyn_end;void *bss_start = &_i386boot_bss_start; -void *bss_size = &_i386boot_bss_size; +ulong bss_size = (ulong)&_i386boot_bss_size; -size_t uboot_size; -void *ram_start; +ulong uboot_size; +void *dest_addr;ulong rel_offset;Elf32_Rel *re; -void (*start_func)(void *); - -/* compiler optimization barrier needed for GCC >= 3.4 */ - __asm__ __volatile__("" : : :"memory" ); +void (*start_func)(void *,ulong ); - uboot_size = (size_t )u_boot_cmd_end - (size_t )text_start; - ram_start = (void *)stack_limit - (uboot_size + (ulong )bss_size); - rel_offset = text_start - ram_start; + uboot_size = (ulong )u_boot_cmd_end - (ulong )text_start; + dest_addr = (void *)stack_limit - (uboot_size + (ulong )bss_size); + rel_offset = text_start - dest_addr; start_func = ram_bootstrap - rel_offset;/* First stage CPU initialization */
@@ -200,10 +200,10 @@ void board_init_f (ulong stack_limit)hang ();/* Copy U-Boot into RAM */ -memcpy (ram_start, text_start, (size_t )uboot_size); +memcpy (dest_addr, text_start, uboot_size);/* Clear BSS */ -memset (bss_start - rel_offset,0 , (size_t )bss_size); +memset (bss_start - rel_offset,0 , bss_size);/* Perform relocation adjustments */ for (re = rel_dyn_start; re < rel_dyn_end; re++)
@@ -213,27 +213,39 @@ void board_init_f (ulong stack_limit) *(ulong *)(re->r_offset - rel_offset) -= (Elf32_Addr )rel_offset; } -start_func (ram_start); - -/* NOTREACHED - relocate_code() does not return */ +/* Enter the relocated U-Boot! */ +start_func (dest_addr, rel_offset); +/* NOTREACHED - board_init_f() does not return */ while (1 ); }/* -* All attempts to jump straight from board_init_f() to board_init_r() -* have failed, hence this special 'bootstrap' function. +* We cannot initialize gd_data in board_init_f() because we would be +* attempting to write to flash (I have even tried using manual relocation +* adjustments on pointers but it just won't work) and board_init_r() does +* not have enough arguments to allow us to pass the relocation offset +* straight up. This bootstrap function (which runs in RAM) is used to +* setup gd_data in order to pass the relocation offset to the rest of +* U-Boot. +* +* TODO: The compiler optimization barrier is intended to stop GCC from +* optimizing this function into board_init_f(). It seems to work without +* it, but I've left it in to be sure. I think also that the barrier in +* board_init_r() is no longer needed, but left it in 'just in case' */ -void ram_bootstrap (void *ram_start) +void ram_bootstrap (void *dest_addr,ulong rel_offset) { -static gd_t gd_data; -/* compiler optimization barrier needed for GCC >= 3.4 */ __asm__ __volatile__("" : : :"memory" ); -board_init_r (&gd_data, (ulong )ram_start); +/* tell others: relocation done */ + gd_data.reloc_off = rel_offset; + gd_data.flags |=GD_FLG_RELOC ; + +board_init_r (&gd_data, (ulong )dest_addr); } -void board_init_r (gd_t *id,ulong ram_start) +void board_init_r (gd_t *id,ulong dest_addr) {char *s;int i;
@@ -247,16 +259,13 @@ void board_init_r(gd_t *id, ulong ram_start)/* compiler optimization barrier needed for GCC >= 3.4 */ __asm__ __volatile__("" : : :"memory" ); -memset (gd,0 ,sizeof (gd_t)); gd->bd = &bd_data;memset (gd->bd,0 ,sizeof (bd_t));show_boot_progress (0x22 ); gd->baudrate =CONFIG_BAUDRATE ; - gd->flags |=GD_FLG_RELOC ;/* tell others: relocation done */ - -mem_malloc_init ((((ulong )ram_start -CONFIG_SYS_MALLOC_LEN )+3 )&~3 , +mem_malloc_init ((((ulong )dest_addr -CONFIG_SYS_MALLOC_LEN )+3 )&~3 ,CONFIG_SYS_MALLOC_LEN );for (init_fnc_ptr = init_sequence, i=0 ; *init_fnc_ptr; ++init_fnc_ptr, i++) {
Signature · 1 line
1.7.0.2.182.ge007
RFC PATCH 11/21 Fix copying of Real-Mode code into RAM
In reply to Graeme RussNow that the relocation offset is in Global Data, we can locate the BIOS code in the U-Boot image
--- lib_i386/bios_setup.c | 2 +- lib_i386/realmode.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)
lib_i386/bios_setup.c +1 -1
@@ -141,7 +141,7 @@ static void setvector(int vector, u16 segment, void *handler)int bios_setup (void ) { -ulong i386boot_bios = (ulong)&_i386boot_bios; +ulong i386boot_bios = (ulong )&_i386boot_bios + gd->reloc_off;ulong i386boot_bios_size = (ulong)&_i386boot_bios_size;static int done=0 ;
lib_i386/realmode.c +1 -1
@@ -37,7 +37,7 @@ extern char realmode_enter;int realmode_setup (void ) { -ulong i386boot_realmode = (ulong)&_i386boot_realmode; +ulong i386boot_realmode = (ulong )&_i386boot_realmode + gd->reloc_off;ulong i386boot_realmode_size = (ulong)&_i386boot_realmode_size;/* copy the realmode switch code */
Signature · 1 line
1.7.0.2.182.ge007
RFC PATCH 12/21 Enable port-mapped access
In reply to Graeme RussThe x86 architecture exclusively uses Port-Mapped I/O (inb/outb) to access the 16550 UARTs. This patch mimics how Linux selects between Memory-Mapped and Port-Mapped I/O. This allows x86 boards to use CONFIG_SERIAL_MUTLI and drop the custom serial port driver
--- drivers/serial/ns16550.c | 69 ++++++++++++++++++++++++++-------------------- 1 files changed, 39 insertions(+), 30 deletions(-)
drivers/serial/ns16550.c +39 -30
@@ -7,6 +7,8 @@#include <config.h> #include <ns16550.h> #include <watchdog.h> +#include <linux/types.h> +#include <asm/io.h> #define UART_LCRVAL UART_LCR_8N1/* 8 data, 1 stop, no parity */ #define UART_MCRVAL (UART_MCR_DTR | \
@@ -14,28 +16,35 @@#define UART_FCRVAL (UART_FCR_FIFO_EN | \ UART_FCR_RXSR | \ UART_FCR_TXSR)/* Clear & enable FIFOs */ +#ifdef CONFIG_X86 +#define serial_out (x,y) outb(x,(ulong)y) +#define serial_in (y) inb((ulong)y) +#else +#define serial_out (x,y) writeb(x,y) +#define serial_in (y) readb(y) +#endif void NS16550_init (NS16550_t com_port,int baud_divisor) { - com_port->ier =0x00 ; +serial_out (0x00 , &com_port->ier);#if defined(CONFIG_OMAP ) && !defined(CONFIG_OMAP3_ZOOM2 ) - com_port->mdr1 =0x7 ;/* mode select reset TL16C750*/ +serial_out (0x7 , &com_port->mdr1);/* mode select reset TL16C750*/ #endif - com_port->lcr =UART_LCR_BKSE |UART_LCRVAL ; - com_port->dll =0 ; - com_port->dlm =0 ; - com_port->lcr =UART_LCRVAL ; - com_port->mcr =UART_MCRVAL ; - com_port->fcr =UART_FCRVAL ; - com_port->lcr =UART_LCR_BKSE |UART_LCRVAL ; - com_port->dll = baud_divisor &0xff ; - com_port->dlm = (baud_divisor >>8 ) &0xff ; - com_port->lcr =UART_LCRVAL ; +serial_out (UART_LCR_BKSE |UART_LCRVAL , (ulong)&com_port->lcr); +serial_out (0 , &com_port->dll); +serial_out (0 , &com_port->dlm); +serial_out (UART_LCRVAL , &com_port->lcr); +serial_out (UART_MCRVAL , &com_port->mcr); +serial_out (UART_FCRVAL , &com_port->fcr); +serial_out (UART_LCR_BKSE |UART_LCRVAL , &com_port->lcr); +serial_out (baud_divisor &0xff , &com_port->dll); +serial_out ((baud_divisor >>8 ) &0xff , &com_port->dlm); +serial_out (UART_LCRVAL , &com_port->lcr);#if defined(CONFIG_OMAP ) && !defined(CONFIG_OMAP3_ZOOM2 )#if defined(CONFIG_APTIX ) - com_port->mdr1 =3 ;/* /13 mode so Aptix 6MHz can hit 115200 */ +serial_out (3 , &com_port->mdr1);/* /13 mode so Aptix 6MHz can hit 115200 */ #else - com_port->mdr1 =0 ;/* /16 is proper to hit 115200 with 48MHz */ +serial_out (0 , &com_port->mdr1);/* /16 is proper to hit 115200 with 48MHz */ #endif #endif /* CONFIG_OMAP */ }
@@ -43,42 +52,42 @@ void NS16550_init (NS16550_t com_port, int baud_divisor)#ifndef CONFIG_NS16550_MIN_FUNCTIONS void NS16550_reinit (NS16550_t com_port,int baud_divisor) { - com_port->ier =0x00 ; - com_port->lcr =UART_LCR_BKSE |UART_LCRVAL ; - com_port->dll =0 ; - com_port->dlm =0 ; - com_port->lcr =UART_LCRVAL ; - com_port->mcr =UART_MCRVAL ; - com_port->fcr =UART_FCRVAL ; - com_port->lcr =UART_LCR_BKSE ; - com_port->dll = baud_divisor &0xff ; - com_port->dlm = (baud_divisor >>8 ) &0xff ; - com_port->lcr =UART_LCRVAL ; +serial_out (0x00 , &com_port->ier); +serial_out (UART_LCR_BKSE |UART_LCRVAL , &com_port->lcr); +serial_out (0 , &com_port->dll); +serial_out (0 , &com_port->dlm); +serial_out (UART_LCRVAL , &com_port->lcr); +serial_out (UART_MCRVAL , &com_port->mcr); +serial_out (UART_FCRVAL , &com_port->fcr); +serial_out (UART_LCR_BKSE , &com_port->lcr); +serial_out (baud_divisor &0xff , &com_port->dll); +serial_out ((baud_divisor >>8 ) &0xff , &com_port->dlm); +serial_out (UART_LCRVAL , &com_port->lcr); }#endif /* CONFIG_NS16550_MIN_FUNCTIONS */ void NS16550_putc (NS16550_t com_port,char c) { -while ((com_port->lsr &UART_LSR_THRE ) ==0 ); - com_port->thr = c; +while ((serial_in (&com_port->lsr) &UART_LSR_THRE ) ==0 ); +serial_out (c, &com_port->thr); }#ifndef CONFIG_NS16550_MIN_FUNCTIONS char NS16550_getc (NS16550_t com_port) { -while ((com_port->lsr &UART_LSR_DR ) ==0 ) { +while ((serial_in (&com_port->lsr) &UART_LSR_DR ) ==0 ) {#ifdef CONFIG_USB_TTY extern void usbtty_poll (void );usbtty_poll ();#endif WATCHDOG_RESET (); } -return (com_port->rbr); +return serial_in (&com_port->rbr); }int NS16550_tstc (NS16550_t com_port) { -return ((com_port->lsr &UART_LSR_DR ) !=0 ); +return ((serial_in (&com_port->lsr) &UART_LSR_DR ) !=0 ); }#endif /* CONFIG_NS16550_MIN_FUNCTIONS */
Signature · 1 line
1.7.0.2.182.ge007
RFC PATCH 13/21 Use CONFIG_SERIAL_MULTI
In reply to Graeme Russ--- board/eNET/eNET.c | 3 +- common/serial.c | 3 +- cpu/i386/Makefile | 2 +- cpu/i386/serial.c | 506 ------------------------------------------------ include/configs/eNET.h | 25 ++- include/serial.h | 3 +- lib_i386/board.c | 5 +- 7 files changed, 31 insertions(+), 516 deletions(-) delete mode 100644 cpu/i386/serial.c
board/eNET/eNET.c +2 -1
@@ -111,8 +111,9 @@ int board_early_init_f(void)writew (0x0615 , &sc520_mmcr->romcs1ctl);writew (0x0615 , &sc520_mmcr->romcs2ctl); -writeb (0x02 , &sc520_mmcr->adddecctl); +writeb (0x00 , &sc520_mmcr->adddecctl);writeb (0x07 , &sc520_mmcr->uart1ctl); +writeb (0x07 , &sc520_mmcr->uart2ctl);writeb (0x06 , &sc520_mmcr->sysarbctl);writew (0x0003 , &sc520_mmcr->sysarbmenb);
common/serial.c +2 -1
@@ -41,7 +41,8 @@ struct serial_device *__default_serial_console (void)#elif defined(CONFIG_405GP ) || defined(CONFIG_405CR ) || defined(CONFIG_440 ) \ || defined(CONFIG_405EP ) || defined(CONFIG_405EZ ) || defined(CONFIG_405EX ) \ || defined(CONFIG_MPC5xxx) || defined(CONFIG_MPC83xx) \ - || defined(CONFIG_MPC85xx) || defined(CONFIG_MPC86xx) + || defined(CONFIG_MPC85xx) || defined(CONFIG_MPC86xx) \ + || defined(CONFIG_SYS_SC520 )#if defined(CONFIG_CONS_INDEX ) && defined(CONFIG_SYS_NS16550_SERIAL )#if (CONFIG_CONS_INDEX ==1 )return &eserial1_device;
cpu/i386/Makefile +1 -1
@@ -29,7 +29,7 @@ include $(TOPDIR)/config.mk LIB = $(obj)lib$(CPU).a START = start.o start16.o resetvec.o -COBJS = serial.o interrupts.o cpu.o +COBJS = interrupts.o cpu.o SRCS := $(START:.o=.S) $(SOBJS:.o=.S) $(COBJS:.o=.c) OBJS := $(addprefix $(obj),$(SOBJS) $(COBJS))
cpu/i386/serial.c deleted +0 -506
506 lines not on this page · raw message
include/configs/eNET.h +20 -5
@@ -21,6 +21,7 @@* MA 02111-1307 USA */ +#include <asm/ibmpc.h> /* * board/config.h - configuration options, board specific */
@@ -55,6 +56,25 @@#undef CONFIG_HW_WATCHDOG/*----------------------------------------------------------------------- +* Serial Configuration +*/ +#define CONFIG_SERIAL_MULTI +#undef CONFIG_SERIAL_SOFTWARE_FIFO +#define CONFIG_CONS_INDEX 1 +#define CONFIG_SYS_NS16550 +#define CONFIG_SYS_NS16550_SERIAL +#define CONFIG_SYS_NS16550_REG_SIZE 1 +#define CONFIG_SYS_NS16550_CLK 1843200 +#define CONFIG_BAUDRATE 9600 +#define CONFIG_SYS_BAUDRATE_TABLE \ + {300, 600, 1200, 2400, 4800, 9600, 19200, 38400,115200} + +#define CONFIG_SYS_NS16550_COM1 UART0_BASE +#define CONFIG_SYS_NS16550_COM2 UART1_BASE +#define CONFIG_SYS_NS16550_COM3 (0x1000 + UART0_BASE) +#define CONFIG_SYS_NS16550_COM4 (0x1000 + UART1_BASE) + +/*----------------------------------------------------------------------- * Video Configuration */ #undef CONFIG_VIDEO/* No Video Hardware */
@@ -65,8 +85,6 @@*/ #define CONFIG_SYS_MALLOC_LEN (CONFIG_ENV_SIZE + 128*1024) -#define CONFIG_BAUDRATE 9600 -/*----------------------------------------------------------------------- * Command line configuration. */
@@ -123,9 +141,6 @@#define CONFIG_SYS_HZ 1024/* incrementer freq: 1kHz */ -/* valid baudrates */ -#define CONFIG_SYS_BAUDRATE_TABLE { 9600, 19200, 38400, 57600, 115200 } -/*----------------------------------------------------------------------- * SDRAM Configuration */
include/serial.h +2 -1
@@ -25,7 +25,8 @@ extern struct serial_device * default_serial_console (void);#if defined(CONFIG_405GP ) || defined(CONFIG_405CR ) || defined(CONFIG_440 ) || \ defined(CONFIG_405EP ) || defined(CONFIG_405EZ ) || defined(CONFIG_405EX ) || \ defined(CONFIG_MPC5xxx) || defined(CONFIG_MPC83xx) || \ - defined(CONFIG_MPC85xx) || defined(CONFIG_MPC86xx) + defined(CONFIG_MPC85xx) || defined(CONFIG_MPC86xx) || \ + defined(CONFIG_SYS_SC520 )extern struct serial_device serial0_device;extern struct serial_device serial1_device;#if defined(CONFIG_SYS_NS16550_SERIAL )
lib_i386/board.c +4 -1
@@ -37,6 +37,7 @@#include <malloc.h> #include <net.h> #include <ide.h> +#include <serial.h> #include <asm/u-boot-i386.h> #include <elf.h>
@@ -149,7 +150,6 @@ static void display_flash_config (ulong size)typedef int (init_fnc_t ) (void );init_fnc_t *init_sequence[] = { - serial_init, cpu_init_r,/* basic cpu dependent setup */ board_early_init_r,/* basic board dependent setup */ dram_init,/* configure available RAM banks */
@@ -277,6 +277,9 @@ void board_init_r(gd_t *id, ulong dest_addr) }show_boot_progress (0x23 ); +#ifdef CONFIG_SERIAL_MULTI +serial_initialize (); +#endif /* configure available FLASH banks */ size =flash_init ();display_flash_config (size);
Signature · 1 line
1.7.0.2.182.ge007
RFC PATCH 14/21 Provide weak PC/AT compatibility setup function
In reply to Graeme RussIt is possibly to setup x86 boards to use non-PC/AT configurations. For example, the sc520 is an x86 CPU with PC/AT and non-PC/AT peripherals. This function allows the board to set itself up for maximum PC/AT compatibility just before booting the Linux kernel (the Linux kernel 'just works' if everything is PC/AT compliant)
--- include/asm-i386/u-boot-i386.h | 2 ++ lib_i386/board.c | 8 ++++++++ 2 files changed, 10 insertions(+), 0 deletions(-)
include/asm-i386/u-boot-i386.h +2 -0
@@ -43,6 +43,8 @@ int cpu_init_interrupts(void);int board_init (void );int dram_init (void ); +void setup_pcat_compatibility (void ); +void isa_unmap_rom (u32 addr);u32 isa_map_rom (u32 bus_addr,int size);
lib_i386/board.c +8 -0
@@ -441,3 +441,11 @@ unsigned long do_go_exec (ulong (*entry)(int, char *[]), int argc, char *argv[])return (entry) (argc, argv); } + +void setup_pcat_compatibility (void ) + __attribute__((weak,alias ("__setup_pcat_compatibility" ))); + +void __setup_pcat_compatibility (void ) +{ +} +
Signature · 1 line
1.7.0.2.182.ge007
RFC PATCH 15/21 Fix support for booting bzImage
In reply to Graeme RussAdd support for newer (up to 2.6.33) kernels
Add zboot command which takes the address of a bzImage as its first argument and (optionally) the size of the bzImage as the second argument (the second argument is needed for older kernels which do not include the bzImage size in the header)
--- include/asm-generic/ioctl.h | 105 ++++++++++++++ include/asm-i386/bootparam.h | 123 +++++++++++++++++ include/asm-i386/e820.h | 155 +++++++++++++++++++++ include/asm-i386/ioctl.h | 1 + include/asm-i386/ist.h | 34 +++++ include/linux/apm_bios.h | 220 ++++++++++++++++++++++++++++++ include/linux/compiler-gcc.h | 87 ++++++++++++ include/linux/compiler-gcc4.h | 61 ++++++++ include/linux/compiler.h | 303 +++++++++++++++++++++++++++++++++++++++++ include/linux/edd.h | 194 ++++++++++++++++++++++++++ include/linux/ioctl.h | 7 + include/linux/ioport.h | 192 ++++++++++++++++++++++++++ include/linux/screen_info.h | 84 +++++++++++ lib_i386/zimage.c | 127 +++++++++++++----- 14 files changed, 1661 insertions(+), 32 deletions(-) create mode 100644 include/asm-generic/ioctl.h create mode 100644 include/asm-i386/bootparam.h create mode 100644 include/asm-i386/e820.h create mode 100644 include/asm-i386/ioctl.h create mode 100644 include/asm-i386/ist.h create mode 100644 include/linux/apm_bios.h create mode 100644 include/linux/compiler-gcc.h create mode 100644 include/linux/compiler-gcc4.h create mode 100644 include/linux/compiler.h create mode 100644 include/linux/edd.h create mode 100644 include/linux/ioctl.h create mode 100644 include/linux/ioport.h create mode 100644 include/linux/screen_info.h
include/asm-generic/ioctl.h new file +105 -0
@@ -0,0 +1,105 @@ +#ifndef _ASM_GENERIC_IOCTL_H +#define _ASM_GENERIC_IOCTL_H + +/* ioctl command encoding: 32 bits total, command in lower 16 bits, +* size of the parameter structure in the lower 14 bits of the +* upper 16 bits. +* Encoding the size of the parameter structure in the ioctl request +* is useful for catching programs compiled with old versions +* and to avoid overwriting user space outside the user buffer area. +* The highest 2 bits are reserved for indicating the ``access mode''. +* NOTE: This limits the max parameter size to 16kB -1 ! +*/ + +/* +* The following is for compatibility across the various Linux +* platforms. The generic ioctl numbering scheme doesn't really enforce +* a type field. De facto, however, the top 8 bits of the lower 16 +* bits are indeed used as a type field, so we might just as well make +* this explicit here. Please be sure to use the decoding macros +* below from now on. +*/ +#define _IOC_NRBITS 8 +#define _IOC_TYPEBITS 8 + +/* +* Let any architecture override either of the following before +* including this file. +*/ + +#ifndef _IOC_SIZEBITS +# define _IOC_SIZEBITS 14 +#endif + +#ifndef _IOC_DIRBITS +# define _IOC_DIRBITS 2 +#endif + +#define _IOC_NRMASK ((1 << _IOC_NRBITS)-1) +#define _IOC_TYPEMASK ((1 << _IOC_TYPEBITS)-1) +#define _IOC_SIZEMASK ((1 << _IOC_SIZEBITS)-1) +#define _IOC_DIRMASK ((1 << _IOC_DIRBITS)-1) + +#define _IOC_NRSHIFT 0 +#define _IOC_TYPESHIFT (_IOC_NRSHIFT+_IOC_NRBITS) +#define _IOC_SIZESHIFT (_IOC_TYPESHIFT+_IOC_TYPEBITS) +#define _IOC_DIRSHIFT (_IOC_SIZESHIFT+_IOC_SIZEBITS) + +/* +* Direction bits, which any architecture can choose to override +* before including this file. +*/ + +#ifndef _IOC_NONE +# define _IOC_NONE 0U +#endif + +#ifndef _IOC_WRITE +# define _IOC_WRITE 1U +#endif + +#ifndef _IOC_READ +# define _IOC_READ 2U +#endif + +#define _IOC (dir,type,nr,size) \ + (((dir) << _IOC_DIRSHIFT) | \ + ((type) << _IOC_TYPESHIFT) | \ + ((nr) << _IOC_NRSHIFT) | \ + ((size) << _IOC_SIZESHIFT)) + +#ifdef __KERNEL__ +/* provoke compile error for invalid uses of size argument */ +extern unsigned int __invalid_size_argument_for_IOC; +#define _IOC_TYPECHECK (t) \ + ((sizeof(t) == sizeof(t[1]) && \ + sizeof(t) < (1 << _IOC_SIZEBITS)) ? \ + sizeof(t) : __invalid_size_argument_for_IOC) +#else +#define _IOC_TYPECHECK (t) (sizeof(t)) +#endif + +/* used to create numbers */ +#define _IO (type,nr) _IOC(_IOC_NONE,(type),(nr),0) +#define _IOR (type,nr,size) _IOC(_IOC_READ,(type),(nr),(_IOC_TYPECHECK(size))) +#define _IOW (type,nr,size) _IOC(_IOC_WRITE,(type),(nr),(_IOC_TYPECHECK(size))) +#define _IOWR (type,nr,size) _IOC(_IOC_READ|_IOC_WRITE,(type),(nr),(_IOC_TYPECHECK(size))) +#define _IOR_BAD (type,nr,size) _IOC(_IOC_READ,(type),(nr),sizeof(size)) +#define _IOW_BAD (type,nr,size) _IOC(_IOC_WRITE,(type),(nr),sizeof(size)) +#define _IOWR_BAD (type,nr,size) _IOC(_IOC_READ|_IOC_WRITE,(type),(nr),sizeof(size)) + +/* used to decode ioctl numbers.. */ +#define _IOC_DIR (nr) (((nr) >> _IOC_DIRSHIFT) & _IOC_DIRMASK) +#define _IOC_TYPE (nr) (((nr) >> _IOC_TYPESHIFT) & _IOC_TYPEMASK) +#define _IOC_NR (nr) (((nr) >> _IOC_NRSHIFT) & _IOC_NRMASK) +#define _IOC_SIZE (nr) (((nr) >> _IOC_SIZESHIFT) & _IOC_SIZEMASK) + +/* ...and for the drivers/sound files... */ + +#define IOC_IN (_IOC_WRITE << _IOC_DIRSHIFT) +#define IOC_OUT (_IOC_READ << _IOC_DIRSHIFT) +#define IOC_INOUT ((_IOC_WRITE|_IOC_READ) << _IOC_DIRSHIFT) +#define IOCSIZE_MASK (_IOC_SIZEMASK << _IOC_SIZESHIFT) +#define IOCSIZE_SHIFT (_IOC_SIZESHIFT) + +#endif /* _ASM_GENERIC_IOCTL_H */
include/asm-i386/bootparam.h new file +123 -0
123 lines not on this page · raw message
include/asm-i386/e820.h new file +155 -0
155 lines not on this page · raw message
include/asm-i386/ioctl.h new file +1 -0
@@ -0,0 +1 @@ +#include <asm-generic/ioctl.h>
include/asm-i386/ist.h new file +34 -0
@@ -0,0 +1,34 @@ +#ifndef _ASM_X86_IST_H +#define _ASM_X86_IST_H + +/* +* Include file for the interface to IST BIOS +* Copyright 2002 Andy Grover <[email protected]> +* +* This program is free software; you can redistribute it and/or modify it +* under the terms of the GNU General Public License as published by the +* Free Software Foundation; either version 2, or (at your option) any +* later version. +* +* This program is distributed in the hope that it will be useful, but +* WITHOUT ANY WARRANTY; without even the implied warranty of +* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU +* General Public License for more details. +*/ + + +#include <linux/types.h> + +struct ist_info { +__u32 signature; +__u32 command; +__u32 event; +__u32 perf_level; +}; + +#ifdef __KERNEL__ + +extern struct ist_info ist_info; + +#endif /* __KERNEL__ */ +#endif /* _ASM_X86_IST_H */
include/linux/apm_bios.h new file +220 -0
220 lines not on this page · raw message
include/linux/compiler-gcc.h new file +87 -0
87 lines not on this page · raw message
include/linux/compiler-gcc4.h new file +61 -0
61 lines not on this page · raw message
include/linux/compiler.h new file +303 -0
303 lines not on this page · raw message
include/linux/edd.h new file +194 -0
194 lines not on this page · raw message
include/linux/ioctl.h new file +7 -0
@@ -0,0 +1,7 @@ +#ifndef _LINUX_IOCTL_H +#define _LINUX_IOCTL_H + +#include <asm/ioctl.h> + +#endif /* _LINUX_IOCTL_H */ +
include/linux/ioport.h new file +192 -0
192 lines not on this page · raw message
include/linux/screen_info.h new file +84 -0
84 lines not on this page · raw message
lib_i386/zimage.c +95 -32
215 lines not on this page · raw message
Signature · 1 line
1.7.0.2.182.ge007
RFC PATCH 16/21 Allow boards to override udelay
In reply to Graeme RussIf the board has a high precision mico-second timer, it maked sense to use it instead of the on-chip one
--- cpu/i386/sc520/sc520_timer.c | 4 ++++ include/asm-i386/ic/sc520.h | 1 + 2 files changed, 5 insertions(+), 0 deletions(-)
cpu/i386/sc520/sc520_timer.c +4 -0
@@ -69,7 +69,11 @@ int timer_init(void)return 0 ; } +/* Allow boards to override udelay implementation */ void __udelay (unsigned long usec) + __attribute__((weak,alias ("sc520_udelay" ))); + +void sc520_udelay (unsigned long usec) {int m =0 ;long u;
include/asm-i386/ic/sc520.h +1 -0
@@ -28,6 +28,7 @@void init_sc520 (void );unsigned long init_sc520_dram (void ); +void sc520_udelay (unsigned long usec);/* Memory mapped configuration registers */ typedef struct sc520_mmcr {
Signature · 1 line
1.7.0.2.182.ge007
RFC PATCH 17/21 Fix Flash Write
In reply to Graeme RussOnboard AMD Flash chip does not support buffered writes
--- include/configs/eNET.h | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
include/configs/eNET.h +1 -1
@@ -197,7 +197,7 @@CONFIG_SYS_FLASH_BASE_1 , \ CONFIG_SYS_FLASH_BASE_2}#define CONFIG_SYS_FLASH_EMPTY_INFO -#define CONFIG_SYS_FLASH_USE_BUFFER_WRITE +#undef CONFIG_SYS_FLASH_USE_BUFFER_WRITE#define CONFIG_SYS_MAX_FLASH_SECT 128/* max number of sectors on one chip */ #define CONFIG_SYS_FLASH_CFI_WIDTH FLASH_CFI_8BIT#define CONFIG_SYS_FLASH_LEGACY_512Kx8
Signature · 1 line
1.7.0.2.182.ge007
RFC PATCH 18/21 Fix CONFIG_SYS_HZ to be 1000
In reply to Graeme RussThe clock interupt has always been 1kHz as per timer_init() in /cpu/i386/sc520/sc520_timer.c
--- include/configs/eNET.h | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
include/configs/eNET.h +1 -1
@@ -139,7 +139,7 @@#define CONFIG_SYS_LOAD_ADDR 0x100000/* default load address */ -#define CONFIG_SYS_HZ 1024/* incrementer freq: 1kHz */ +#define CONFIG_SYS_HZ 1000/* incrementer freq: 1kHz */ /*----------------------------------------------------------------------- * SDRAM Configuration
Signature · 1 line
1.7.0.2.182.ge007
RFC PATCH 19/21 Add support for onboard RTL8100B (RTL8139) chips
In reply to Graeme Russ---
I simply do not know why PCI_BASE_ADDRESS_1 has to be changed to PCI_BASE_ADDRESS_0 - Please comment
board/eNET/eNET.c | 12 ++++++++++++ drivers/net/rtl8139.c | 2 +- include/asm-i386/ic/sc520.h | 6 +++--- include/configs/eNET.h | 9 ++++++++- 4 files changed, 24 insertions(+), 5 deletions(-)
board/eNET/eNET.c +12 -0
@@ -24,6 +24,8 @@#include <common.h> #include <asm/io.h> #include <asm/ic/sc520.h> +#include <net.h> +#include <netdev.h> #ifdef CONFIG_HW_WATCHDOG #include <watchdog.h>
@@ -173,3 +175,13 @@ ulong board_flash_get_legacy (ulong base, int banknum, flash_info_t * info) }else return 0 ; } + +int board_eth_init (bd_t *bis) +{ +int adapters; + adapters =pci_eth_init (bis); + +if (adapters >0 ) +eth_init (bis); + +return adapters;
drivers/net/rtl8139.c +1 -1
@@ -214,7 +214,7 @@ int rtl8139_initialize(bd_t *bis)if ((devno =pci_find_devices (supported, idx++)) <0 )break ; -pci_read_config_dword (devno,PCI_BASE_ADDRESS_1 , &iobase); +pci_read_config_dword (devno,PCI_BASE_ADDRESS_0 , &iobase); iobase &= ~0xf ;debug ("rtl8139: REALTEK RTL8139 @0x%x\n" , iobase);
include/asm-i386/ic/sc520.h +3 -3
@@ -307,14 +307,14 @@ extern volatile sc520_mmcr_t *sc520_mmcr;/* PCI I/O space from 0x1000 to 0xdfff * (make 0xe000-0xfdff available for stuff like PCCard boot) */ -#define SC520_PCI_IO_PHYS 0x00001000 -#define SC520_PCI_IO_BUS 0x00001000 +#define SC520_PCI_IO_PHYS 0x00002000 +#define SC520_PCI_IO_BUS 0x00002000#define SC520_PCI_IO_SIZE 0x0000d000/* system memory from 0x00000000 to 0x0fffffff */ #define SC520_PCI_MEMORY_PHYS 0x00000000#define SC520_PCI_MEMORY_BUS 0x00000000 -#define SC520_PCI_MEMORY_SIZE 0x10000000 +#define SC520_PCI_MEMORY_SIZE 0x08000000/* PCI bus memory from 0x10000000 to 0x26ffffff * (make 0x27000000 - 0x27ffffff available for stuff like PCCard boot) */
include/configs/eNET.h +8 -1
@@ -104,9 +104,10 @@#define CONFIG_CMD_LOADS /* loads */ #define CONFIG_CMD_MEMORY /* md mm nm mw cp cmp crc base loop mtest */ #define CONFIG_CMD_MISC /* Misc functions like sleep etc*/ -#undef CONFIG_CMD_NET/* bootp, tftpboot, rarpboot */ +#define CONFIG_CMD_NET /* bootp, tftpboot, rarpboot */ #undef CONFIG_CMD_NFS/* NFS support */ #define CONFIG_CMD_PCI /* PCI support */ +#define CONFIG_CMD_PING /* ICMP echo support */ #define CONFIG_CMD_RUN /* run command in env variable */ #define CONFIG_CMD_SAVEENV /* saveenv */ #define CONFIG_CMD_SETGETDCR /* DCR support on 4xx */
@@ -225,6 +226,12 @@#define CONFIG_SYS_THIRD_PCI_IRQ 11#define CONFIG_SYS_FORTH_PCI_IRQ 15 +/* +* Network device (TRL8100B) support +*/ +#define CONFIG_NET_MULTI +#define CONFIG_RTL8139 +/*----------------------------------------------------------------------- * Hardware watchdog configuration */
Signature · 1 line
1.7.0.2.182.ge007
RFC PATCH 19/21 Add support for onboard RTL8100B (RTL8139) chips
HI Graeme,
Quoted 26 lines · Graeme Russ
---
I simply do not know whyPCI_BASE_ADDRESS_1has to be changed toPCI_BASE_ADDRESS_0- Please commentboard/eNET/eNET.c | 12 ++++++++++++ drivers/net/rtl8139.c | 2 +- include/asm-i386/ic/sc520.h | 6 +++--- include/configs/eNET.h | 9 ++++++++- 4 files changed, 24 insertions(+), 5 deletions(-)board/eNET/eNET.c +2
-0@@ -24,6 +24,8 @@#include <common.h> #include <asm/io.h> #include <asm/ic/sc520.h> +#include <net.h> +#include <netdev.h> #ifdef CONFIG_HW_WATCHDOG #include <watchdog.h> @@ -173,3 +175,13 @@ ulong board_flash_get_legacy (ulong base, int banknum, flash_info_t * info) }else
board/eNET/eNET.c ulong board_flash_get_legacy (ulong base, int banknum, flash_info_t * info) · line 185
Quoted 10 lines
board/eNET/eNET.c +8
-0return 0 ; } + +int board_eth_init (bd_t *bis) +{ +int adapters; + adapters =pci_eth_init (bis); + +if (adapters>0 ) +eth_init (bis);
I'm not sure why you're calling eth_init() here. Are you sure that's what you want to do? Keep in mind that board_eth_init() gets called by eth_initialize() in the common net code.
drivers/net/rtl8139.c int rtl8139_initialize(bd_t *bis) · line 218
Quoted 13 lines
board/eNET/eNET.c +2
-0+ +return adapters;drivers/net/rtl8139.c +1
-1@@ -214,7 +214,7 @@ int rtl8139_initialize(bd_t *bis)if ((devno =pci_find_devices (supported, idx++))<0 )break ; -pci_read_config_dword (devno,PCI_BASE_ADDRESS_1 ,&iobase); +pci_read_config_dword (devno,PCI_BASE_ADDRESS_0 ,&iobase);
Are you fixing a bug here? This seems like a change that would have fairly wide-ranging implications beyond your board. <snip>
regards,
Ben
RFC PATCH 19/21 Add support for onboard RTL8100B (RTL8139) chips
Hi Ben,
Thanks for having a look at this
Quoted 7 lines · Ben Warren
HI Graeme,
Quoted 4 lines · Graeme Russ
---
I simply do not know whyPCI_BASE_ADDRESS_1has to be changed toPCI_BASE_ADDRESS_0- Please comment
<snip>
Quoted 6 lines
Quoted 6 lines
}
board/eNET/eNET.c +5
-0+ +int board_eth_init (bd_t *bis) +{ +int adapters; + adapters =pci_eth_init (bis);
Quoted 8 lines
board/eNET/eNET.c ulong board_flash_get_legacy (ulong base, int banknum, flash_info_t * info) · line 185
Quoted 4 lines
board/eNET/eNET.c +3
-0+ +if (adapters>0 ) +eth_init (bis);I'm not sure why you're calling
eth_init()here. Are you sure that's what you want to do? Keep in mind thatboard_eth_init()gets called byeth_initialize()in the common net code.
I'll have a look at this and respin this patch
Quoted 11 lines
Quoted 11 lines
(quoted diff) +2
-0+ + return adapters;drivers/net/rtl8139.c +0
-0@@ -214,7 +214,7 @@ int rtl8139_initialize(bd_t *bis)if ((devno =pci_find_devices (supported, idx++))<0 )break ;
Quoted 7 lines
drivers/net/rtl8139.c int rtl8139_initialize(bd_t *bis) · line 218
Quoted 3 lines
drivers/net/rtl8139.c +1
-1-pci_read_config_dword (devno,PCI_BASE_ADDRESS_1 ,&iobase); +pci_read_config_dword (devno,PCI_BASE_ADDRESS_0 ,&iobase);Are you fixing a bug here? This seems like a change that would have fairly wide-ranging implications beyond your board.
<snip>
If I use PCI_BASE_ADDRESS_1 I simply cannot get the Ethernet adapters to work (I get complete rubbish when accessing the registers). It took me a long time to figure this out and in the end I simply ran out of logical adjustments and started hacking around and this worked.
Maybe there is some special initialisation of the PCI host in the SC520 that would make this work. I really don't know the inner workings of talking to PCI devices and am taking a lot of what I am doing here on blind faith. If it gives you any more of an idea as to why this change is needed, I have to use PIO instead of MMIO for the Linux driver
Regards,
Graeme
RFC PATCH 20/21 Add PC/AT compatibility setup function
In reply to Graeme RussThe eNET uses the sc520 software timers rather than the PC/AT clones
Set all interrupts and timers up to be PC/AT compatible
--- board/eNET/eNET.c | 38 ++++++++++++++++++++++++++++++++++++++ 1 files changed, 38 insertions(+), 0 deletions(-)
board/eNET/eNET.c +38 -0
@@ -185,3 +185,41 @@ int board_eth_init(bd_t *bis)eth_init (bis);return adapters; + +void setup_pcat_compatibility () +{ +/* disable global interrupt mode */ +writeb (0x40 , &sc520_mmcr->picicr); + +/* set all irqs to edge */ +writeb (0x00 , &sc520_mmcr->pic_mode[0 ]); +writeb (0x00 , &sc520_mmcr->pic_mode[1 ]); +writeb (0x00 , &sc520_mmcr->pic_mode[2 ]); + +/* +* active low polarity on PIC interrupt pins, +* active high polarity on all other irq pins +*/ +writew (0x0000 ,&sc520_mmcr->intpinpol); + +/* Set PIT 0 -> IRQ0, RTC -> IRQ8, FP error -> IRQ13 */ +writeb (SC520_IRQ0 , &sc520_mmcr->pit_int_map[0 ]); +writeb (SC520_IRQ8 , &sc520_mmcr->rtcmap); +writeb (SC520_IRQ13 , &sc520_mmcr->ferrmap); + +/* Disable all other interrupt sources */ +writeb (SC520_IRQ_DISABLED , &sc520_mmcr->gp_tmr_int_map[0 ]); +writeb (SC520_IRQ_DISABLED , &sc520_mmcr->gp_tmr_int_map[1 ]); +writeb (SC520_IRQ_DISABLED , &sc520_mmcr->gp_tmr_int_map[2 ]); +writeb (SC520_IRQ_DISABLED , &sc520_mmcr->pit_int_map[1 ]); +writeb (SC520_IRQ_DISABLED , &sc520_mmcr->pit_int_map[2 ]); +writeb (SC520_IRQ_DISABLED , &sc520_mmcr->pci_int_map[0 ]);/* disable PCI INT A */ +writeb (SC520_IRQ_DISABLED , &sc520_mmcr->pci_int_map[1 ]);/* disable PCI INT B */ +writeb (SC520_IRQ_DISABLED , &sc520_mmcr->pci_int_map[2 ]);/* disable PCI INT C */ +writeb (SC520_IRQ_DISABLED , &sc520_mmcr->pci_int_map[3 ]);/* disable PCI INT D */ +writeb (SC520_IRQ_DISABLED , &sc520_mmcr->dmabcintmap);/* disable DMA INT */ +writeb (SC520_IRQ_DISABLED , &sc520_mmcr->ssimap); +writeb (SC520_IRQ_DISABLED , &sc520_mmcr->wdtmap); +writeb (SC520_IRQ_DISABLED , &sc520_mmcr->wpvmap); +writeb (SC520_IRQ_DISABLED , &sc520_mmcr->icemap); +}
Signature · 1 line
1.7.0.2.182.ge007
RFC PATCH 21/21 Use SC520 MMCR to reset eNET board
In reply to Graeme Russ--- include/configs/eNET.h | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
include/configs/eNET.h +1 -1
@@ -159,7 +159,7 @@* CPU Features */ #define CONFIG_SYS_SC520_HIGH_SPEED 0/* 100 or 133MHz */ -#undef CONFIG_SYS_SC520_RESET/* use SC520 MMCR's to reset cpu */ +#define CONFIG_SYS_SC520_RESET /* use SC520 MMCR's to reset cpu */ #define CONFIG_SYS_SC520_TIMER /* use SC520 swtimers */ #undef CONFIG_SYS_GENERIC_TIMER/* use the i8254 PIT timers */ #undef CONFIG_SYS_TSC_TIMER/* use the Pentium TSC timers */
Signature · 1 line
1.7.0.2.182.ge007
cover 'Comming of Age'
In reply to Graeme Russ< snip>
Quoted 5 lines
Once this patchset is applied I believe the x86 port of U-Boot will be ready for more mainstream use
Wolfgang - If there is any other renewed interest in the x86 port, I would be more than happy to be an official custodian
Hi Graeme,
Is there a quick summary about what is supported, how its used, etc? Eg could x86 U-Boot replace the BIOS on on chipset X, similar to coreboot? Or is it a 2nd stage bootloader like GRUB? Or a bootloader that is run from GRUB? How hard would it be to port to other x86 platform?
It seems like cool stuff, I'm just trying to wrap my head around the high-level concept of U-Boot on x86 and where/why it'd be used instead of the "standard" alternatives.
Thanks,
Peter
cover 'Comming of Age'
Quoted 5 lines · Peter Tyser
< snip>
Quoted 3 lines
Once this patchset is applied I believe the x86 port of U-Boot will be ready for more mainstream use
Quoted 8 lines
Quoted 2 lines
Wolfgang - If there is any other renewed interest in the x86 port, I would be more than happy to be an official custodian
Hi Graeme,
Is there a quick summary about what is supported, how its used, etc? Eg could x86 U-Boot replace the BIOS on on chipset X, similar to coreboot? Or is it a 2nd stage bootloader like GRUB? Or a bootloader that is run from GRUB? How hard would it be to port to other x86 platform?
The x86 U-Boot port, like all other U-Boot ports, is a primary boot loader for embedded systems. As such, it is designed to bring a board up from power-on/cold reset to a state where Linux (or other OS) can execute (e.g load FPGAs, configure hardware address and I/O structures etc). The closest 'in-the-wild' comparative software would be RedBoot.
The current implementation is designed around the AMD SC520 ELAN CPU, but porting to other x86 CPUs and boards should be fairly simple. Porting is made easier if the target is PC/AT centric. That said, the modular design should allow for very easy diversion from PC/AT.
It does not (and will not) replace the BIOS of a regular PC (Coreboot) nor will it be a second stage bootloader (GRUB). However, at a pinch, I imagine you could use it to boot a VIA mini-ITX board for example.
Quoted 3 lines
It seems like cool stuff, I'm just trying to wrap my head around the high-level concept of U-Boot on x86 and where/why it'd be used instead of the "standard" alternatives.
x86 is a poor second cousin to PPC and ARM in the embedded world. The reasons for this are many and varied, but generally speaking ARM and PPC are better suited to the embedded world due to lower power consumption and integrated peripherals. x86 has its advantages (no need for cross compiling, easier to simulate and a PC without using QEMU etc).
x86 U-Boot does do a few things though that I think are very special:
- It fully relocates itself to the highest available RAM (including
Read-Only data and special data structures like the GDT and IDT which are sometimes overlooked)
- It can be fail-safe boot-strapped from RAM (i.e. boot an old version
from power-up, tftp a new version, jump to the new version)
In the near future, I intend to make it 100% relocatable (i.e. does not need to be loaded at TEXT_BASE) and maybe even add the ability to specify the relocation address.
Quoted 4 lines
Thanks,
Peter
Your welcome :)
Graeme
cover 'Comming of Age'
Quoted 4 lines
In the near future, I intend to make it 100% relocatable (i.e. does not need to be loaded at
TEXT_BASE) and maybe even add the ability to specify the relocation address.
I did a few months ago for MPC83xx and sent it to the list. There was some discussion but in the end it wasn't applied because it was too invasive. PPC is especially bad as it needs to relocate string literals too.
Search for LINK_OFF if you want to see what I did
Jocke
cover 'Comming of Age'
Quoted 10 lines · Joakim Tjernlund
Quoted 3 lines
In the near future, I intend to make it 100% relocatable (i.e. does not need to be loaded at
TEXT_BASE) and maybe even add the ability to specify the relocation address.I did a few months ago for MPC83xx and sent it to the list. There was some discussion but in the end it wasn't applied because it was too invasive. PPC is especially bad as it needs to relocate string literals too.
Search for
LINK_OFFif you want to see what I did
Yes, I saw a bit of that thread and you patches an I would agree that they were too invasive. x86 is not that great for relocation either (all calls are absolute as well as references to .text). Have a look at board_init_f() in lib_i386\board.c - it has a very basic parsing of the ELF relocation information.
I tried to champion this as a cross-platform method of achieving full relocation and the killing off of all those ugly reloc_off hacks all over the place. Unfortunately, I think the code size and time penalty may have been too great in the end.
To implement full relocation, I simply need to add an additional parameter to this function which provides the 'link versus load' offset which will get applied to the 'load versus run' offset calculation
cover 'Comming of Age'
Graeme Russ <[email protected]> wrote on 2010/03/27 12:54:03:
Quoted 19 lines
Quoted 10 lines · Joakim Tjernlund
Quoted 3 lines
In the near future, I intend to make it 100% relocatable (i.e. does not need to be loaded at
TEXT_BASE) and maybe even add the ability to specify the relocation address.I did a few months ago for MPC83xx and sent it to the list. There was some discussion but in the end it wasn't applied because it was too invasive. PPC is especially bad as it needs to relocate string literals too.
Search for
LINK_OFFif you want to see what I didYes, I saw a bit of that thread and you patches an I would agree that they were too invasive. x86 is not that great for relocation either (all calls are absolute as well as references to .text). Have a look at
board_init_f()in lib_i386\board.c - it has a very basic parsing of the ELF relocation information.
Quoted 8 lines
I tried to champion this as a cross-platform method of achieving full relocation and the killing off of all those ugly
reloc_offhacks all over the place. Unfortunately, I think the code size and time penalty may have been too great in the end.To implement full relocation, I simply need to add an additional parameter to this function which provides the 'link versus load' offset which will get applied to the 'load versus run' offset calculation
Are we talking about the same thing? when you say "does not need to be loaded at TEXT_BASE" you need something like my LINK_OFF patch, otherwise I don't see how you can boot out of flash otherwise.
Jocke
cover 'Comming of Age'
Quoted 7 lines · Joakim Tjernlund
Graeme Russ <[email protected]> wrote on 2010/03/27 12:54:03:
Quoted 5 lines
Quoted 4 lines · Joakim Tjernlund
I did a few months ago for MPC83xx and sent it to the list. There was some discussion but in the end it wasn't applied because it was too invasive. PPC is especially bad as it needs to relocate string literals too.
Quoted 8 lines
Quoted 3 lines
To implement full relocation, I simply need to add an additional parameter to this function which provides the 'link versus load' offset which will get applied to the 'load versus run' offset calculation
Are we talking about the same thing? when you say "does not need to be loaded at
TEXT_BASE" you need something like myLINK_OFFpatch, otherwise I don't see how you can boot out of flash otherwise.
I realise now that we are not talking about the same thing. The x86 port always relocates into (and runs from) RAM. Only a very small stub is run from flash (SDRAM setup and sizing, copy to RAM, process relocation entries etc). This stub can be made fully relocatable while running from Flash (with a few tricks)
By adjusting all position dependent code after copying into RAM, LINK_OFF is no longer needed
Graeme
cover 'Comming of Age'
Graeme Russ <[email protected]> wrote on 2010/03/28 09:38:47:
Quoted 18 lines
Quoted 7 lines · Joakim Tjernlund
Graeme Russ <[email protected]> wrote on 2010/03/27 12:54:03:
Quoted 5 lines
Quoted 4 lines · Joakim Tjernlund
I did a few months ago for MPC83xx and sent it to the list. There was some discussion but in the end it wasn't applied because it was too invasive. PPC is especially bad as it needs to relocate string literals too.
Quoted 8 lines
Quoted 3 lines
To implement full relocation, I simply need to add an additional parameter to this function which provides the 'link versus load' offset which will get applied to the 'load versus run' offset calculation
Are we talking about the same thing? when you say "does not need to be loaded at
TEXT_BASE" you need something like myLINK_OFFpatch, otherwise I don't see how you can boot out of flash otherwise.
Quoted 5 lines
I realise now that we are not talking about the same thing. The x86 port always relocates into (and runs from) RAM. Only a very small stub is run from flash (SDRAM setup and sizing, copy to RAM, process relocation entries etc). This stub can be made fully relocatable while running from Flash (with a few tricks)
Have you tried that yet? This is exactly what LINK_OFF is for. Turns out that there is more code than you think that is executed before relocation into RAM. Perhaps your stub is very small and doesn't need the serial port or access to EEPROM via I2C. That won't work in the long run though.
Quoted 3 lines
By adjusting all position dependent code after copying into RAM,
LINK_OFFis no longer needed
You don't need LINK_OFF for this, PPC already have this feature since it can relocate into any RAM address.