linux-kernel - Re: [PATCH 3/3] lis3: SPI transport layer

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAMuHMdX-aX20QXMzKm5kxgCp+Bg+bYa-6+hNONy=rZ0MX4F2vA@mail.gmail.com>
Date:	Wed, 10 Aug 2016 12:31:08 +0200
From:	Geert Uytterhoeven <geert@...ux-m68k.org>
To:	Daniel Mack <daniel@...aq.de>
Cc:	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Éric Piel <eric.piel@...mplin-utc.net>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Arnd Bergmann <arnd@...db.de>,
	Greg KH <gregkh@...uxfoundation.org>,
	Guenter Roeck <linux@...ck-us.net>,
	linux-spi <linux-spi@...r.kernel.org>
Subject: Re: [PATCH 3/3] lis3: SPI transport layer

Hi Daniel,

On Mon, Mar 23, 2009 at 12:51 AM, Daniel Mack <daniel@...aq.de> wrote:
> Make use of the new abstraction layer and add a new transport layer for
> spi. Works fine on a PXA based board.
>
> Signed-off-by: Daniel Mack <daniel@...aq.de>
> Acked-by: Pavel Machek <pavel@....cz>
> Cc: Éric Piel <eric.piel@...mplin-utc.net>
> Cc: Andrew Morton <akpm@...ux-foundation.org>
> ---
>  drivers/hwmon/Kconfig         |   16 ++++++
>  drivers/hwmon/Makefile        |    1 +
>  drivers/hwmon/lis3lv02d_spi.c |  114 +++++++++++++++++++++++++++++++++++++++++
>  3 files changed, 131 insertions(+), 0 deletions(-)
>  create mode 100644 drivers/hwmon/lis3lv02d_spi.c
>
> diff --git a/drivers/hwmon/Kconfig b/drivers/hwmon/Kconfig
> index b84bf06..9d78ab4 100644
> --- a/drivers/hwmon/Kconfig
> +++ b/drivers/hwmon/Kconfig

In the mean time, this has moved to drivers/misc/lis3lv02d/Kconfig

> @@ -895,6 +895,22 @@ config SENSORS_LIS3LV02D
>           Say Y here if you have an applicable laptop and want to experience
>           the awesome power of lis3lv02d.
>
> +config SENSORS_LIS3_SPI
> +       tristate "STMicroeletronics LIS3LV02Dx three-axis digital accelerometer (SPI)"
> +       depends on !ACPI && SPI_MASTER && INPUT

Is there any specific reason this option depends on !ACPI?
This hurts multi-platform kernels, supporting both OF and ACPI.

> +       default n
> +       help
> +         This driver provides support for the LIS3LV02Dx accelerometer connected
> +         via SPI. The accelerometer data is readable via
> +         /sys/devices/platform/lis3lv02d.
> +
> +         This driver also provides an absolute input class device, allowing
> +         the laptop to act as a pinball machine-esque joystick.
> +
> +         This driver can also be built as modules.  If so, the core module
> +         will be called lis3lv02d and a specific module for the SPI transport
> +         is called lis3lv02d_spi.
> +

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@...ux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds