>-----Original Message-----
>From: Stephen Hemminger <step...@networkplumber.org>
>Sent: Wednesday, November 15, 2023 1:16 AM
>To: Tomasz Duszynski <tduszyn...@marvell.com>
>Cc: dev@dpdk.org; Jakub Palider <jpali...@marvell.com>; Jerin Jacob
>Kollanukkaran
><jer...@marvell.com>
>Subject: [EXT] Re: [PATCH] raw/cnxk_bphy: switch to dynamic logging
>
>External Email
>
>----------------------------------------------------------------------
>On Tue, 14 Nov 2023 09:04:46 +0100
>Tomasz Duszynski <tduszyn...@marvell.com> wrote:
>
>> Dynamically allocated log type is a standard approach among all drivers.
>> Switch to it.
>>
>> Signed-off-by: Tomasz Duszynski <tduszyn...@marvell.com>
>> ---
>> drivers/raw/cnxk_bphy/cnxk_bphy.c | 32 +++++++++++++---------
>> drivers/raw/cnxk_bphy/cnxk_bphy_cgx.c | 4 ++-
>> drivers/raw/cnxk_bphy/cnxk_bphy_cgx.h | 7 +++++
>> drivers/raw/cnxk_bphy/cnxk_bphy_cgx_test.c | 32
>> ++++++++++------------
>> 4 files changed, 44 insertions(+), 31 deletions(-)
>
>Good to see but there seems to be a lot of other places calling plt_err(),
>plt_info(),
>plt_warn(), plt_print()
Yeah, apparently my script did not catch everything. Thanks for pointing this
out.
