Re: [PATCH] PowerPC: Remove hardcoded BAT configuration of IMMR in CPM early debug console

2010-06-03 Thread Martyn Welch
Scott Wood wrote: On 06/02/2010 03:06 AM, Martyn Welch wrote: I think that's a more fundamental change to CPM early debug than I can handle right now. Is IMMRBASE on your board at some address that has a low likelihood of conflicting when treated as a kernel effective address? It's at

Re: [PATCH] PowerPC: Remove hardcoded BAT configuration of IMMR in CPM early debug console

2010-06-02 Thread Martyn Welch
Scott Wood wrote: On 06/01/2010 08:43 AM, Martyn Welch wrote: diff --git a/arch/powerpc/kernel/head_32.S b/arch/powerpc/kernel/head_32.S index e025e89..861cace 100644 --- a/arch/powerpc/kernel/head_32.S +++ b/arch/powerpc/kernel/head_32.S @@ -1194,12 +1194,13 @@ setup_disp_bat: #endif

Re: [PATCH] PowerPC: Remove hardcoded BAT configuration of IMMR in CPM early debug console

2010-06-02 Thread Scott Wood
On 06/02/2010 03:06 AM, Martyn Welch wrote: I think that's a more fundamental change to CPM early debug than I can handle right now. Is IMMRBASE on your board at some address that has a low likelihood of conflicting when treated as a kernel effective address? It's at 0x0f00, is seems ok,

Re: [PATCH] PowerPC: Remove hardcoded BAT configuration of IMMR in CPM early debug console

2010-06-01 Thread Martyn Welch
Scott Wood wrote: On 05/28/2010 10:18 AM, Martyn Welch wrote: The CPM early debug console hardcodes the BAT to cover the IMMR at 0xf000. The IMMR (on the mpc8270 at the very least) can be set to a number of locations with bootstrap configuration, which are outside the hardcoded BAT

Re: [PATCH] PowerPC: Remove hardcoded BAT configuration of IMMR in CPM early debug console

2010-06-01 Thread Martyn Welch
Martyn Welch wrote: Scott Wood wrote: On 05/28/2010 10:18 AM, Martyn Welch wrote: The CPM early debug console hardcodes the BAT to cover the IMMR at 0xf000. The IMMR (on the mpc8270 at the very least) can be set to a number of locations with bootstrap configuration, which are

Re: [PATCH] PowerPC: Remove hardcoded BAT configuration of IMMR in CPM early debug console

2010-06-01 Thread Scott Wood
On 06/01/2010 08:43 AM, Martyn Welch wrote: diff --git a/arch/powerpc/kernel/head_32.S b/arch/powerpc/kernel/head_32.S index e025e89..861cace 100644 --- a/arch/powerpc/kernel/head_32.S +++ b/arch/powerpc/kernel/head_32.S @@ -1194,12 +1194,13 @@ setup_disp_bat: #endif /* CONFIG_BOOTX_TEXT */

[PATCH] PowerPC: Remove hardcoded BAT configuration of IMMR in CPM early debug console

2010-05-28 Thread Martyn Welch
The CPM early debug console hardcodes the BAT to cover the IMMR at 0xf000. The IMMR (on the mpc8270 at the very least) can be set to a number of locations with bootstrap configuration, which are outside the hardcoded BAT configuration. This patch determines the correct location at which to

Re: [PATCH] PowerPC: Remove hardcoded BAT configuration of IMMR in CPM early debug console

2010-05-28 Thread Scott Wood
On 05/28/2010 10:18 AM, Martyn Welch wrote: The CPM early debug console hardcodes the BAT to cover the IMMR at 0xf000. The IMMR (on the mpc8270 at the very least) can be set to a number of locations with bootstrap configuration, which are outside the hardcoded BAT configuration. This patch

Re: [PATCH] PowerPC: Remove hardcoded BAT configuration of IMMR in CPM early debug console

2010-05-28 Thread Benjamin Herrenschmidt
On Fri, 2010-05-28 at 11:18 -0500, Scott Wood wrote: Only the physical address should depend on where IMMR is. We should use fixmap instead of an arbitrary address for the effective address. There's a existing FIX_EARLY_DEBUG_BASE, but it's only 128 KiB so we'll have to either grow it,