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.
