[Crash-utility] [PATCH 0/5] [ppc32] Support E500 processor for FSL BOOKE

Toshikazu Nakayama nakayama.ts at ncos.nec.co.jp
Wed Mar 21 04:53:35 UTC 2012


(2012/03/20 15:52), Suzuki K. Poulose wrote:
> On 03/19/2012 03:17 PM, Toshikazu Nakayama wrote:
>> This patch add support fleescale ppce500mc in E500 processor chipset.
>> And make platform specific code shape-up so that new support can get easy.
>>
> Toshikazu Nakayama-san,
> 
> The patches look good to me.
> 
>> +	if (machdep->flags&  PAE)
>>   		pte = ULONGLONG(machdep->ptbl + PAGEOFFSET((ulong)page_table));
> May be we could introduce a macro to replace
> 
> (machdep->flags&  PAE)
> There are quite a lot of places where we use it now.
> 
> Something like,
> 
> #define IS_PAE()    (machdep->flags&  PAE)
> 
> 
> What do you think ?
> 
> Reviewed-by: Suzuki K. Poulose<suzuki at in.ibm.com>

Suzuki-san,

Thanks for your review and your proposal is good for me.
I'm going to add your suggestion in next patch set that
are also updating ppc32 MMU's implementation. 

Thanks,
Toshi

> Thanks
> Suzuki
> 
> 




More information about the Crash-utility mailing list