Message ID | 20200724111225.12513-16-ovidiu.panait@windriver.com |
---|---|
State | Accepted |
Delegated to: | Tom Rini |
Headers | show |
Series | [v4,01/17] Kconfig: Introduce CONFIG_SYS_HAS_SRAM | expand |
On Fri, Jul 24, 2020 at 02:12:24PM +0300, Ovidiu Panait wrote: > Extend manual relocation of block_cache list pointers to all platforms that > enable CONFIG_NEEDS_MANUAL_RELOC. Remove m68k-specific checks and provide a > single implementation that adds gd->reloc_off to the pre-relocation > pointers. > > Acked-by: Angelo Dureghello <angelo.dureghello@timesys.com> > Tested-by: Angelo Dureghello <angelo.dureghello@timesys.com> > Reviewed-by: Simon Glass <sjg@chromium.org> > Reviewed-by: Eric Nelson <eric@nelint.com> > Signed-off-by: Ovidiu Panait <ovidiu.panait@windriver.com> Applied to u-boot/master, thanks!
diff --git a/common/board_r.c b/common/board_r.c index 79772135a4..63c53b7b96 100644 --- a/common/board_r.c +++ b/common/board_r.c @@ -836,7 +836,7 @@ static init_fnc_t init_sequence_r[] = { #if defined(CONFIG_PRAM) initr_mem, #endif -#if defined(CONFIG_M68K) && defined(CONFIG_BLOCK_CACHE) +#if defined(CONFIG_NEEDS_MANUAL_RELOC) && defined(CONFIG_BLOCK_CACHE) blkcache_init, #endif run_main_loop, diff --git a/drivers/block/blkcache.c b/drivers/block/blkcache.c index b6fc72fe98..d97ee99cf4 100644 --- a/drivers/block/blkcache.c +++ b/drivers/block/blkcache.c @@ -12,6 +12,8 @@ #include <linux/ctype.h> #include <linux/list.h> +DECLARE_GLOBAL_DATA_PTR; + struct block_cache_node { struct list_head lh; int iftype; @@ -22,21 +24,20 @@ struct block_cache_node { char *cache; }; -#ifndef CONFIG_M68K static LIST_HEAD(block_cache); -#else -static struct list_head block_cache; -#endif static struct block_cache_stats _stats = { .max_blocks_per_entry = 8, .max_entries = 32 }; -#ifdef CONFIG_M68K +#ifdef CONFIG_NEEDS_MANUAL_RELOC int blkcache_init(void) { - INIT_LIST_HEAD(&block_cache); + struct list_head *head = &block_cache; + + head->next = (uintptr_t)head->next + gd->reloc_off; + head->prev = (uintptr_t)head->prev + gd->reloc_off; return 0; }