Re: [PATCH v4 1/5] printk: nbcon: Export console_is_usable

[Konstantin Ryabitsev]

On Wed, Sep 17, 2025 at 09:21:48AM -0300, Marcos Paulo de Souza wrote:
> > Signed-off-by: Marcos Paulo de Souza <mpdesouza@xxxxxxxx>
> > Reviewed-by: Petr Mladek <pmladek@xxxxxxxx>
> > 
> > It means that Macros developed a patch and Petr reviewed it.
> > The patch was not longer modified.

That's not entirely correct. The signed-off trailer is used as the boundary
indicating who was the person responsible for collecting the trailers. When
the trailers are collected by the original author as part of their iteration
cycles, their signed-off-by trailer goes below any trailers they have
received.

When the trailers are collected by the maintainer, any trailers they retrieve
will go below the original author's S-o-b, but above their own.

> > 
> > Note the Reviewed-by tag might be preserved even when Marcos
> > later did some cosmetic changes, e.g. fixed a typo, formatting,
> > or rebased.
> 
> Got it. I used b4 prep --update-trailers, and it added the Reviewed-by
> before my Sign-off. But still, this patch didn't change since the last
> submission, so I thought that it would be ok.

That is the correct behaviour.

-K