Re: [PATCH] hw/isa/Kconfig: Add missing dependency VIA VT82C686 -> APM

[BALATON Zoltan]

On Wed, 10 Mar 2021, Philippe Mathieu-Daudé wrote:
> On 3/10/21 12:41 AM, BALATON Zoltan wrote:
> > On Tue, 9 Mar 2021, Philippe Mathieu-Daudé wrote:
> > > ping for review?
> >
> > This is included in my pegasos2 series as 6/8 replacing half of a
> > similar patch from my original version. Since I've reported it I don't
> > think I should be also reviewing it but it's quite trivial so may not
> > need that much review.
>
> Forwarding a patch doesn't mean you have reviewed it :/

I guess it does mean I agree with it and really don't see how another line 
with my name would make it any better when it already has my Suggested-by 
and Signed-off-by. An R-b tag probably really only helps for patches that 
are more complex than one line that the maintainter can easily check.

> If you can provide a formal R-b tag, then I can queue this or the
> one in your series.

The two are the same. If you commit this first my series can be easily 
rebased by dropping that patch or it will get committed as part of the 
series. At this point I think it's easiest if David takes the whole series 
than if you take a few of them then I have to rebase again and part of the 
series goes in via another tree. As the freeze is nearing I'd like 
somebody take the whole series so if you can ack the vt82c686 parts I 
think you don't need to do anything more which is what you said before 
you'd prefer.

I'll fix the errp you've noticed and repost. Adding any more to the series 
now would need starting review again which would risk missing the freeze. 
So unless it's needed to accept the series as it is I'd leave any other 
improvements for later after this pegasos2 series is merged.

Regards,
BALATON Zoltan

> Thanks,
>
> Phil.