[PATCH] iio: adc: meson-saradc: use NULL instead of 0 for pointer

Fix sparse warning: Using plain integer as NULL pointer Signed-off-by: Paolo Cretaro --- drivers/iio/adc/meson_saradc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/iio/adc/meson_saradc.c b/drivers/iio/adc/meson_saradc.c index 81cd39a57fe3..fb3f67a9ae1f 100644

Re: [PATCH] iio: adc: meson-saradc: use NULL instead of 0 for pointer

On 28/05/2017 16:43, Jonathan Cameron wrote: > On Sun, 28 May 2017 13:24:38 +0200 > Paolo Cretaro wrote: > >> Fix sparse warning: Using plain integer as NULL pointer >> >> Signed-off-by: Paolo Cretaro > This looks fine to me, but ideally you should always try

[PATCH] vfio/mdev: add static modifier to add_mdev_supported_type

Set add_mdev_supported_type as static since it is only used within mdev_sysfs.c. This fixes -Wmissing-prototypes gcc warning. Signed-off-by: Paolo Cretaro --- drivers/vfio/mdev/mdev_sysfs.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/vfio/mdev/mdev_sysfs.c b

[PATCH] drivers/char/random.c: add missing include

Include linux/hw_random.h header file to silence the following Wmissing-prototypes gcc warning: drivers/char/random.c:2346:6: warning: no previous prototype for ‘add_hwgenerator_randomness’ [-Wmissing-prototypes] Signed-off-by: Paolo Cretaro --- drivers/char/random.c | 1 + 1 file changed, 1

[PATCH] staging: pi433: use IS_ERR to check kthread_run return value

Fix compiler warning: ordered comparison of pointer with integer zero Signed-off-by: Paolo Cretaro --- drivers/staging/pi433/pi433_if.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c index 1bc478a7f49e

Re: [PATCH] staging: pi433: use IS_ERR to check kthread_run return value

On 22/07/2017 12:27, Paolo Cretaro wrote: > Fix compiler warning: ordered comparison of pointer with integer zero Sorry, just noticed a similar patch has been submitted a few days ago, please ignore this patch. Paolo > > Signed-off-by: Paolo Cretaro > --- > drivers/staging/p

[PATCH] staging: vboxvideo: remove unused variables

Fix compiler warnings: vbox_mode.c:57:15: warning: variable ‘crtc_id’ set but not used vbox_mode.c:581:25: warning: variable ‘vbox_connector’ set but not used Signed-off-by: Paolo Cretaro --- drivers/staging/vboxvideo/vbox_mode.c | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers

[PATCH] iio: cros_ec: Remove unused variable

Fix gcc warning: cros_ec_baro.c:130:25: warning: variable ‘ec_device’ set but not used Signed-off-by: Paolo Cretaro --- drivers/iio/pressure/cros_ec_baro.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/iio/pressure/cros_ec_baro.c b/drivers/iio/pressure/cros_ec_baro.c index

[PATCH] iio: adc: mxs-lradc: fix non-static symbol warnings

_available iio_dev_attr_in_voltage12_scale_available iio_dev_attr_in_voltage13_scale_available iio_dev_attr_in_voltage14_scale_available iio_dev_attr_in_voltage15_scale_available Signed-off-by: Paolo Cretaro --- drivers/iio/adc/mxs-lradc-adc.c | 32 1 file changed, 16 insertions(+), 16 deletions(-) di

Re: [PATCH] iio: adc: meson-saradc: use NULL instead of 0 for pointer

May 2017 13:24:38 +0200 Paolo Cretaro wrote: Fix sparse warning: Using plain integer as NULL pointer Signed-off-by: Paolo Cretaro This looks fine to me, but ideally you should always try to include the driver author - particularly in the case of patches to a recent driver such as this one. To

[PATCH] iio: cros_ec: Remove unused variables

Fix gcc warnings about variable 'ec_device' being set but not used in these files: common/cros_ec_sensors/cros_ec_sensors.c:194:25 light/cros_ec_light_prox.c:184:25 Signed-off-by: Paolo Cretaro --- drivers/iio/common/cros_ec_sensors/cros_ec_sensors.c | 2 -- drivers