Re: [Qemu-devel] [PATCH] hw/integratorcp: Simplify flash remap code, fix sense of REMAP bit

[Avi Kivity]

On 12/20/2011 05:51 PM, Peter Maydell wrote:
> On 20 December 2011 11:00, Avi Kivity <address@hidden> wrote:
> > On 12/19/2011 06:45 PM, Peter Maydell wrote:
> >> Use the new memory mutator API to simplify the flash remap code;
> >> this allows us to drop the flash_mapped flag.
> >>
> >> This patch also fixes the sense of the REMAP bit, which was
> >> reversed.
> >>
> >
> > I'm surprised the word "also" doesn't cause the maintainers' scripts to
> > auto-reject the patch.  It makes the patch hard to review, and also
> > makes cherry picking for updating stable releases harder.
>
> Well, if you like I could split it into one patch which just
> changes the "if (flash) {" in integratorcm_do_remap() to
> "if (!flash) {", and then another patch which was exactly
> this one, but I don't think that makes this patch easier to
> review, because it's exactly the same size.

It's easier to review because the smaller patch just fixes a bug, and is
thus very visible.  When reviewing the larger patch you verify that it
doesn't change behaviour in any way, only simplifies the code.  This
helps people with limited cache in their brains.

-- 
error compiling committee.c: too many arguments to function