Hi Greg Thanks for your comments. I still have to work through them. I am using OpenVirtualization in secure world, which then switches to a Linux kernel in non-secure world to test the patches. What about you?
Best, Fabian On 14 May 2014, at 15:55, Greg Bellows <greg.bell...@linaro.org<mailto:greg.bell...@linaro.org>> wrote: Hi Fabian, I too had been updating the core TZ patches provided by Samsung. From looking at your changes I see a lot of similarities in our code with the exception being the mechanism for banked register support. The difference being that your approach is a bit more explicit in the declaration of the banked registers. Whereas my approach was to update the banked registers once all the other registers were registered. Both approaches I believe work. I spoke with Peter M. and he and I are okay with your approach. I will be looking closer at your patches today and making comments. One thing that held me up from committing sooner was testing my changes. Do you have a good approach for testing the changes? Regards, Greg On 14 May 2014 03:58, Aggeler Fabian <aggel...@student.ethz.ch<mailto:aggel...@student.ethz.ch>> wrote: I see. What is Greg Bellows working on exactly? Also peripherals like TZASC, TZPC,...? My plan is to focus on them now if no one else is working on them. What do you suggest to minimize overlap? Thanks, Fabian ________________________________________ From: Peter Maydell [peter.mayd...@linaro.org<mailto:peter.mayd...@linaro.org>] Sent: Monday, May 12, 2014 10:39 PM To: Aggeler Fabian Cc: Edgar E. Iglesias; Rob Herring; Peter Crosthwaite; QEMU Developers; Alexander Graf; John Williams; Alex Bennée; Greg Bellows Subject: Re: [Qemu-devel] [PATCH v1 00/22] target-arm: Preparations for A64 EL2 and 3 On 12 May 2014 20:13, Aggeler Fabian <aggel...@student.ethz.ch<mailto:aggel...@student.ethz.ch>> wrote: > I’ve been reworking the Samsung patches as part of my Master thesis and I > wanted to > send them some time this week. I am currently rebasing them when I noticed > Edgar’s > patches. Is there some branch with the patches so I could rebase on them? Hmm, that makes about three lots of people trying to do similar things at this point. We should try to coordinate so we don't duplicate work. thanks -- PMM
