Re: [PATCH 3.10 02/20] iio: bmg160: IIO_BUFFER and IIO_TRIGGERED_BUFFER are required

[Luis Henriques]

On Sat, Sep 19, 2015 at 10:27:17AM -0700, Greg Kroah-Hartman wrote:
> 3.10-stable review patch.  If anyone has any objections, please let me know.
> 
> ------------------
> 
> From: Markus Pargmann <mpa@xxxxxxxxxxxxxx>
> 
> commit 06d2f6ca5a38abe92f1f3a132b331eee773868c3 upstream.
> 
> This patch adds selects for IIO_BUFFER and IIO_TRIGGERED_BUFFER. Without
> IIO_BUFFER, the driver does not compile.
> 
> Signed-off-by: Markus Pargmann <mpa@xxxxxxxxxxxxxx>
> Reviewed-by: Srinivas Pandruvada <srinivas.pandruvada@xxxxxxxxxxxxxxx>
> Signed-off-by: Jonathan Cameron <jic23@xxxxxxxxxx>
> Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
> 
> ---
>  drivers/iio/gyro/Kconfig |    3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> --- a/drivers/iio/gyro/Kconfig
> +++ b/drivers/iio/gyro/Kconfig
> @@ -73,7 +73,8 @@ config IIO_ST_GYRO_SPI_3AXIS
>  config ITG3200

I know it's a bit too late as this kernel has been released already,
but this patch doesn't look right to me.  The original patch is
changing the BMG160 driver, not the ITG3200.

The same applies to the 3.14 kernel -- it's changing the wrong
driver.

Cheers,
--
Luís

>  	tristate "InvenSense ITG3200 Digital 3-Axis Gyroscope I2C driver"
>  	depends on I2C
> -	select IIO_TRIGGERED_BUFFER if IIO_BUFFER
> +	select IIO_BUFFER
> +	select IIO_TRIGGERED_BUFFER
>  	help
>  	  Say yes here to add support for the InvenSense ITG3200 digital
>  	  3-axis gyroscope sensor.
> 
> 
> --
> To unsubscribe from this list: send the line "unsubscribe stable" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/