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 effectiv

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-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 @

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 */

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

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 BA

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

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 d

[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 con