On 17 November 2015 at 17:07, Peter Maydell <peter.mayd...@linaro.org> wrote: > On 9 November 2015 at 01:11, Michael Davidsaver <mdavidsa...@gmail.com> wrote: >> This series grew from a previous incorrect patch attempting to fix >> some incorrect behavior. After spending some time going through the >> arch. ref. manual for v7-M I think I understand better how this should >> work and have made a number of changes which actually improve the situation. >> >> These changes have not yet been cross checked against real hardware, and >> I therefore don't consider them mergeable. It's gotten big enough though >> that I'd like to get some feedback.
> I'll reply to the various patches individually with comments. I think I've now done that at least for the earlier patches. There are probably some other finer details that I'll get to in a later round of patch review but hopefully you have enough to do some of the fixes and restructuring of this patchset for v2. I think the most important thing here is getting the structure of the changes into patches right so they're easy to review. The general principles here are: * each patch should aim to be self-contained and to do one thing, not several things (for instance, avoid making several bug fixes in one patch, avoid putting "restructure/refactor code" and "add new feature" in the same patch, and so on) * at each point in the patch series QEMU needs to still compile and run (this is particularly important to allow bisection of bugs later on where people will want to be able to narrow down which commit introduced a bug) If the structure of the patchset is right it should be fairly easy to review your improvements against the architecture manual. thanks -- PMM
