On 06/15/2016 12:16 AM, Chen-Yu Tsai wrote: > Hi, > > On Tue, Jun 14, 2016 at 3:01 PM, <macro.wav...@gmail.com> wrote: >> From: Hongbo Zhang <hongbo.zh...@nxp.com> >> >> v5 changes: >> - Give up fixing the potential bug of PSCI stack overlap with secure text end >> when there is more CPUs in system. Because I just want to keep this series as >> simple as it could be: adding basic PSCI v1.0 support and adding more PSCI >> v1.0 implements of our platform. >> While too compplicated patches in assembly language won't call for effective >> reviews, even I think there is potential bug of PSCI stack, let's fix it in >> sepetated patch later. > > I've done some patches fixing the stack allocation issue: > > https://github.com/wens/u-boot-sunxi/commits/c-psci-part2 > > These patches are ready, but I want to add a secure data section still. > The data section will be for variables such as target PC, core/cluster > power status, context ID, etc.. IMHO this is better than putting stuff > at the top or bottom of the stack, and can also be referenced directly. >
Chenyu, I see you have been working on PSCI for ARMv7. Can you review Hongbo's patch set and give your comment or ack? York _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot