Re: [PATCH v9 3/3] printk: fix double printing with earlycon

From: Aleksey Makarov
Date: Wed Apr 12 2017 - 02:24:27 EST




On 04/11/2017 10:43 AM, Petr Mladek wrote:
On Mon 2017-04-10 21:00:35, Aleksey Makarov wrote:


On 04/10/2017 05:22 PM, Petr Mladek wrote:
On Wed 2017-04-05 23:20:00, Aleksey Makarov wrote:

[..]

Thank you for review. Can you (or anybody else) ACK it?
I am going to resend the whole series without those empty lines.
May I add your Acked-by:?

Sure. Feel free to use:

Acked-by: Petr Mladek <pmladek@xxxxxxxx>

The meaning of the tags is a bit unclear. Acked-by means that
the maintainer agrees with the idea. But it does not necessarily
means that she reviewed the code in details. I agree with the idea
and did the review, so I used the Reviewed-by tag.

Also you do not need to resend the patchset just because the two
empty lines. Sergey agrees. I will wait day or two and push
all three patches into the printk.git if nobody complains
in the meantime. I could remove the two empty lines when doing so.

Ok, then I will not resend it.

Thanks a lot for the fix and patience.

Thank you for reviewing and acking it.
Aleksey Makarov