Re: [PATCH 2/2] drivers: replace printf with fprintf for debug functions

2024-07-02 Thread Hemant Agrawal
On 02-07-2024 15:01, David Marchand wrote: Hello Hemant, On Thu, Feb 16, 2023 at 10:28 AM Hemant Agrawal wrote: On 15-Feb-23 8:46 PM, Thomas Monjalon wrote: 15/02/2023 11:29, Hemant Agrawal: This patch replaces simple printf with fprintf for debug dump related functions for various NXP dpa

Re: [PATCH 2/2] drivers: replace printf with fprintf for debug functions

2024-07-02 Thread David Marchand
Hello Hemant, On Thu, Feb 16, 2023 at 10:28 AM Hemant Agrawal wrote: > On 15-Feb-23 8:46 PM, Thomas Monjalon wrote: > > 15/02/2023 11:29, Hemant Agrawal: > >> This patch replaces simple printf with fprintf for debug dump > >> related functions for various NXP dpaaX related drivers. > > Why replac

Re: [PATCH 2/2] drivers: replace printf with fprintf for debug functions

2023-02-16 Thread Hemant Agrawal
On 15-Feb-23 8:46 PM, Thomas Monjalon wrote: 15/02/2023 11:29, Hemant Agrawal: This patch replaces simple printf with fprintf for debug dump related functions for various NXP dpaaX related drivers. Why replacing with fprintf(stdout)? Would it be better to provide a FILE* parameter to the fun

Re: [PATCH 2/2] drivers: replace printf with fprintf for debug functions

2023-02-15 Thread Stephen Hemminger
On Wed, 15 Feb 2023 15:59:05 +0530 Hemant Agrawal wrote: > diff --git a/drivers/bus/dpaa/base/fman/netcfg_layer.c > b/drivers/bus/dpaa/base/fman/netcfg_layer.c > index 6a405c984d..69cee08fb9 100644 > --- a/drivers/bus/dpaa/base/fman/netcfg_layer.c > +++ b/drivers/bus/dpaa/base/fman/netcfg_layer.

Re: [PATCH 2/2] drivers: replace printf with fprintf for debug functions

2023-02-15 Thread Thomas Monjalon
15/02/2023 11:29, Hemant Agrawal: > This patch replaces simple printf with fprintf for debug dump > related functions for various NXP dpaaX related drivers. Why replacing with fprintf(stdout)? Would it be better to provide a FILE* parameter to the functions?

[PATCH 2/2] drivers: replace printf with fprintf for debug functions

2023-02-15 Thread Hemant Agrawal
This patch replaces simple printf with fprintf for debug dump related functions for various NXP dpaaX related drivers. Signed-off-by: Hemant Agrawal --- drivers/bus/dpaa/base/fman/netcfg_layer.c | 20 drivers/crypto/caam_jr/caam_jr.c| 4 +- drivers/crypto/caam_jr/caam_jr_