[PATCH v2 2/2] staging: gasket: Use __func__ instead of hardcoded string - Style

Greg KH gregkh at linuxfoundation.org
Fri Jul 13 08:34:35 UTC 2018


On Fri, Jul 13, 2018 at 12:58:49AM +0200, Felix Siegel wrote:
> Changed logging statements to use %s and __func__ instead of hard coding
> the function name in a string.
> 
> Signed-off-by: Felix Siegel <felix.siegel at posteo.de>
> ---
>  drivers/staging/gasket/apex_driver.c       |  7 ++++---
>  drivers/staging/gasket/gasket_core.c       |  8 +++++---
>  drivers/staging/gasket/gasket_ioctl.c      |  3 ++-
>  drivers/staging/gasket/gasket_page_table.c | 13 ++++++++-----
>  4 files changed, 19 insertions(+), 12 deletions(-)
> 
> diff --git a/drivers/staging/gasket/apex_driver.c b/drivers/staging/gasket/apex_driver.c
> index 9218ad7..ee91137 100644
> --- a/drivers/staging/gasket/apex_driver.c
> +++ b/drivers/staging/gasket/apex_driver.c
> @@ -427,8 +427,9 @@ static int apex_device_cleanup(struct gasket_dev *gasket_dev)
>  
>  	gasket_log_info(
>  		gasket_dev,
> -		"apex_device_cleanup 0x%p hib_error 0x%llx scalar_error "
> +		"%s 0x%p hib_error 0x%llx scalar_error "
>  		"0x%llx.",

One nit, it's not your doing, but the string should be all on one line
here.  Something to think about for future patches :)

thanks,

greg k-h


More information about the devel mailing list