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

Stephen Hemminger stephen at networkplumber.org
Wed Feb 15 18:17:59 CET 2023


On Wed, 15 Feb 2023 15:59:05 +0530
Hemant Agrawal <hemant.agrawal at nxp.com> 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.c
> @@ -33,33 +33,33 @@ dump_netcfg(struct netcfg_info *cfg_ptr)
>  {
>  	int i;
>  
> -	printf("..........  DPAA Configuration  ..........\n\n");
> +	fprintf(stdout, "..........  DPAA Configuration  ..........\n\n");
>  

If you only do "printf -> fprintf(stdout," then it doesn't make sense.
Ideally change debug interface to take a user supplied output FILE *
like other dump routines.


More information about the dev mailing list