Re: [PATCH v2 1/2] MIPS: fix/enrich 34K APRP (APSP) functionalities

[Deng-Cheng Zhu <dczhu@mips.com>]

On 05/21/2012 05:32 AM, Maciej W. Rozycki wrote:
> On Fri, 18 May 2012, John Crispin wrote:
>
> > > > You could introduce a ARCH_HAS_APRP which any platform can then select ?
> > > Hmm... This is a good idea. Maybe the name could be SYS_SUPPORTS_APRP?
> > You are correct
>   What's so Malta-specific in the VPE loader anyway?  It's a CPU feature,
> not a board-specific one.
Well, first off, for VPE loader itself, when it comes to CPS we have
vpe_run() that derives from amon_cpu_start() in arch/mips/mti-malta/malta-
amon.c. There is no implementation of amon_cpu_start() on other platforms.
Secondly, I suppose VPE loader works uniquely for APRP, and part of APRP
(such as IRQ related stuff) depends on platform code. So it makes sense
(IMO) to impose the dependency of APRP on the root (VPE loader).


Thanks,

Deng-Cheng