2019-02-02 09:01:17

by Tomasz Duszynski

[permalink] [raw]
Subject: [PATCH] iio: chemical: sps30: remove printk format specifier

pr_fmt is used by printk wrappers. There are not any in the driver
code so remove the format specifier.

Signed-off-by: Tomasz Duszynski <[email protected]>
---
drivers/iio/chemical/sps30.c | 2 --
1 file changed, 2 deletions(-)

diff --git a/drivers/iio/chemical/sps30.c b/drivers/iio/chemical/sps30.c
index 376fac41ecb5..e03a28a67146 100644
--- a/drivers/iio/chemical/sps30.c
+++ b/drivers/iio/chemical/sps30.c
@@ -7,8 +7,6 @@
* I2C slave address: 0x69
*/

-#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
-
#include <asm/unaligned.h>
#include <linux/crc8.h>
#include <linux/delay.h>
--
2.20.1



2019-02-02 17:09:26

by Jonathan Cameron

[permalink] [raw]
Subject: Re: [PATCH] iio: chemical: sps30: remove printk format specifier

On Sat, 2 Feb 2019 10:00:03 +0100
Tomasz Duszynski <[email protected]> wrote:

> pr_fmt is used by printk wrappers. There are not any in the driver
> code so remove the format specifier.
>
> Signed-off-by: Tomasz Duszynski <[email protected]>
Applied to the togreg branch of iio.git and pushed out
as testing for the autobuilders to mostly ignore it.

Thanks,

Jonathan

> ---
> drivers/iio/chemical/sps30.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/iio/chemical/sps30.c b/drivers/iio/chemical/sps30.c
> index 376fac41ecb5..e03a28a67146 100644
> --- a/drivers/iio/chemical/sps30.c
> +++ b/drivers/iio/chemical/sps30.c
> @@ -7,8 +7,6 @@
> * I2C slave address: 0x69
> */
>
> -#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
> -
> #include <asm/unaligned.h>
> #include <linux/crc8.h>
> #include <linux/delay.h>
> --
> 2.20.1
>