Re: [PATCH] RDMA/bnxt_re: QPLIB: Add and use #define dev_fmt(fmt) "QPLIB: " fmt

From: Selvin Xavier
Date: Thu Aug 16 2018 - 06:30:37 EST


On Sat, Aug 11, 2018 at 12:12 AM Joe Perches <joe@xxxxxxxxxxx> wrote:
>
> Consistently use the "QPLIB: " prefix for dev_<level> logging.
>
> Miscellanea:
>
> o Add missing newlines to avoid possible message interleaving
> o Coalesce consecutive dev_<level> uses that emit a message header to
> avoid < 80 column lengths and mistakenly output on multiple lines
> o Reflow modified lines to use 80 columns where appropriate
> o Consistently use "%s: " where __func__ is output
> o QPLIB: is now always output immediately after the dev_<level> header
>
> Signed-off-by: Joe Perches <joe@xxxxxxxxxxx>

Thanks for the patch, Joe.

Acked-by: Selvin Xavier <selvin.xavier@xxxxxxxxxxxx>