mirror of
https://github.com/Fishwaldo/u-boot.git
synced 2025-03-28 18:11:33 +00:00
ICU862: Fix incorrect flash handling
There is only one bank of NOR flash memory on ICU862 boards. Remove code that attempts to deal with a second bank. This also fixes build warnings: flash.c: In function 'flash_init': flash.c:120:12: warning: array subscript is above array bounds [-Warray-bounds] flash.c:121:12: warning: array subscript is above array bounds [-Warray-bounds] flash.c:124:12: warning: array subscript is above array bounds [-Warray-bounds] Signed-off-by: Wolfgang Denk <wd@denx.de>
This commit is contained in:
parent
bae65019d7
commit
13039eb1fa
1 changed files with 3 additions and 29 deletions
|
@ -52,13 +52,12 @@ unsigned long flash_init (void)
|
||||||
{
|
{
|
||||||
volatile immap_t *immap = (immap_t *)CONFIG_SYS_IMMR;
|
volatile immap_t *immap = (immap_t *)CONFIG_SYS_IMMR;
|
||||||
volatile memctl8xx_t *memctl = &immap->im_memctl;
|
volatile memctl8xx_t *memctl = &immap->im_memctl;
|
||||||
unsigned long size_b0, size_b1;
|
unsigned long size_b0;
|
||||||
int i;
|
int i;
|
||||||
|
|
||||||
/* Init: no FLASHes known */
|
/* Init: no FLASHes known */
|
||||||
for (i=0; i < CONFIG_SYS_MAX_FLASH_BANKS; ++i) {
|
for (i=0; i < CONFIG_SYS_MAX_FLASH_BANKS; ++i)
|
||||||
flash_info[i].flash_id = FLASH_UNKNOWN;
|
flash_info[i].flash_id = FLASH_UNKNOWN;
|
||||||
}
|
|
||||||
|
|
||||||
/* Static FLASH Bank configuration here - FIXME XXX */
|
/* Static FLASH Bank configuration here - FIXME XXX */
|
||||||
|
|
||||||
|
@ -70,27 +69,6 @@ unsigned long flash_init (void)
|
||||||
size_b0 >> 20);
|
size_b0 >> 20);
|
||||||
}
|
}
|
||||||
|
|
||||||
if (FLASH_BASE1_PRELIM != 0x0) {
|
|
||||||
size_b1 = flash_get_size((vu_long *)FLASH_BASE1_PRELIM, &flash_info[1]);
|
|
||||||
|
|
||||||
if (size_b1 > size_b0) {
|
|
||||||
printf ("## ERROR: Bank 1 (0x%08lx = %ld MB)"
|
|
||||||
" > Bank 0 (0x%08lx = %ld MB)\n",
|
|
||||||
size_b1, size_b1 >> 20,
|
|
||||||
size_b0, size_b0 >> 20);
|
|
||||||
|
|
||||||
flash_info[0].flash_id = FLASH_UNKNOWN;
|
|
||||||
flash_info[1].flash_id = FLASH_UNKNOWN;
|
|
||||||
flash_info[0].sector_count = -1;
|
|
||||||
flash_info[1].sector_count = -1;
|
|
||||||
flash_info[0].size = 0;
|
|
||||||
flash_info[1].size = 0;
|
|
||||||
return (0);
|
|
||||||
}
|
|
||||||
} else {
|
|
||||||
size_b1 = 0;
|
|
||||||
}
|
|
||||||
|
|
||||||
/* Remap FLASH according to real size */
|
/* Remap FLASH according to real size */
|
||||||
memctl->memc_or0 = CONFIG_SYS_OR_TIMING_FLASH | (-size_b0 & OR_AM_MSK);
|
memctl->memc_or0 = CONFIG_SYS_OR_TIMING_FLASH | (-size_b0 & OR_AM_MSK);
|
||||||
memctl->memc_br0 = (CONFIG_SYS_FLASH_BASE & BR_BA_MSK) | BR_MS_GPCM | BR_V;
|
memctl->memc_br0 = (CONFIG_SYS_FLASH_BASE & BR_BA_MSK) | BR_MS_GPCM | BR_V;
|
||||||
|
@ -117,13 +95,9 @@ unsigned long flash_init (void)
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
/* ICU862 Board has only one Flash Bank */
|
/* ICU862 Board has only one Flash Bank */
|
||||||
flash_info[1].flash_id = FLASH_UNKNOWN;
|
|
||||||
flash_info[1].sector_count = -1;
|
|
||||||
|
|
||||||
flash_info[0].size = size_b0;
|
flash_info[0].size = size_b0;
|
||||||
flash_info[1].size = size_b1;
|
|
||||||
|
|
||||||
return (size_b0 + size_b1);
|
return size_b0;
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Add table
Reference in a new issue