Re: [PATCH] iio: acpi-als: Fix Kconfig and remove unneeded pointer casting

Jonathan Cameron
2015-05-10T19:28:38Z

On 09/05/15 20:30, Gabriele Mazzotta wrote:
> Signed-off-by: Gabriele Mazzotta <gabriele.mzt@gmail.com>
Applied as a fixup to the original patch which is now in the
togreg branch of iio.git - initially pushed out as testing for
the autobuilders to play with it.

Thanks,

Jonathan
> ---
> This patch fixes the issues of:
> [PATCH v9] iio: acpi: Add support for ACPI0008 Ambient Light Sensor
>
> Link: http://lkml.kernel.org/g/1430569857-31386-1-git-send-email-gabriele.mzt@gmail.com
>
> drivers/iio/light/Kconfig | 1 +
> drivers/iio/light/acpi-als.c | 3 +--
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/iio/light/Kconfig b/drivers/iio/light/Kconfig
> index 898b2b5..fecd743 100644
> --- a/drivers/iio/light/Kconfig
> +++ b/drivers/iio/light/Kconfig
> @@ -8,6 +8,7 @@ menu "Light sensors"
> config ACPI_ALS
> tristate "ACPI Ambient Light Sensor"
> depends on ACPI
> + select IIO_BUFFER
> select IIO_TRIGGERED_BUFFER
> select IIO_KFIFO_BUF
> help
> diff --git a/drivers/iio/light/acpi-als.c b/drivers/iio/light/acpi-als.c
> index 9839c9a..1dafa07 100644
> --- a/drivers/iio/light/acpi-als.c
> +++ b/drivers/iio/light/acpi-als.c
> @@ -138,8 +138,7 @@ static void acpi_als_notify(struct acpi_device *device, u32 event)
> goto out;
> }
>
> - iio_push_to_buffers_with_timestamp(indio_dev,
> - (u8 *)als->evt_buffer, time_ns);
> + iio_push_to_buffers_with_timestamp(indio_dev, als->evt_buffer, time_ns);
>
> out:
> mutex_unlock(&als->lock);
>