On 9/20/18 12:31 PM, Roman Kapl wrote: >> Given that the previous (broken) version of this patch is already >> queued in >> ppc-for-3.1, would it be better to send a v2 whereby these changes are >> squashed into >> the original? >> > > I don't know if David Gibson will want to modify history in his branch > -- maybe other people depend on it? > > And anyway, Phillipe wants it even more granular. > > I don't have any strong opinions about it, David can either squash it or > tell me to split it, I just need some consensus. I thought that > splitting this was overdoing it a bit and does not have that many > advantages given how close the commits are.
> Fixes: ea8073c10d ("target/ppc: add external PID support") Your previous patch is now d12a22c5c7 in David Gibson's tree, so I think here squashing your fixes (once this patch got reviewed) is the best option. Regards, Phil.