[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