[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

From: BALATON Zoltan
Subject: Re: [PATCH] hw/isa/Kconfig: Add missing dependency VIA VT82C686 -> APM
Date: Wed, 10 Mar 2021 03:48:35 +0100 (CET)

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.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]