From: Jonathan Cameron <Jonathan.Cameron@Huawei.com>
To: Jonathan Cameron <jic23@kernel.org>
Cc: <linux-iio@vger.kernel.org>, <devicetree@vger.kernel.org>,
Rob Herring <robh+dt@kernel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Lars-Peter Clausen <lars@metafoo.de>,
Michael Hennerich <Michael.Hennerich@analog.com>,
Barry Song <baohua@kernel.org>
Subject: Re: [PATCH 11/12] staging: iio: accel: adis16203: Move out of staging
Date: Tue, 24 Jan 2023 09:44:50 +0000 [thread overview]
Message-ID: <20230124094450.0000272b@Huawei.com> (raw)
In-Reply-To: <20230123211758.563383-12-jic23@kernel.org>
On Mon, 23 Jan 2023 21:17:57 +0000
Jonathan Cameron <jic23@kernel.org> wrote:
> From: Jonathan Cameron <Jonathan.Cameron@huawei.com>
>
> The main blocker of this driver moving previously was non standard
> ABI for the 180 degree offset channel. Now support for that channel
> has been dropped, this simple driver can move out of staging.
>
> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
I forgot my own rules of posting graduations with move detection turned off.
I'm going to be doing a v2 anyway to fix the issues Rob's bot and
Krysztof pointed out in the binding so will hopefully remember to
turn move detection off for that.
> ---
> drivers/iio/accel/Kconfig | 12 ++++++++++++
> drivers/iio/accel/Makefile | 1 +
> drivers/{staging => }/iio/accel/adis16203.c | 0
> drivers/staging/iio/accel/Kconfig | 12 ------------
> drivers/staging/iio/accel/Makefile | 1 -
> 5 files changed, 13 insertions(+), 13 deletions(-)
>
> diff --git a/drivers/iio/accel/Kconfig b/drivers/iio/accel/Kconfig
> index b6b45d359f28..88d4b18bd5e8 100644
> --- a/drivers/iio/accel/Kconfig
> +++ b/drivers/iio/accel/Kconfig
> @@ -18,6 +18,18 @@ config ADIS16201
> To compile this driver as a module, say M here: the module will
> be called adis16201.
>
> +config ADIS16203
> + tristate "Analog Devices ADIS16203 Programmable 360 Degrees Inclinometer"
> + depends on SPI
> + select IIO_ADIS_LIB
> + select IIO_ADIS_LIB_BUFFER if IIO_BUFFER
> + help
> + Say Y here to build support for Analog Devices adis16203 Programmable
> + 360 Degrees Inclinometer.
> +
> + To compile this driver as a module, say M here: the module will be
> + called adis16203.
> +
> config ADIS16209
> tristate "Analog Devices ADIS16209 Dual-Axis Digital Inclinometer and Accelerometer"
> depends on SPI
> diff --git a/drivers/iio/accel/Makefile b/drivers/iio/accel/Makefile
> index 311ead9c3ef1..b6f0cee52659 100644
> --- a/drivers/iio/accel/Makefile
> +++ b/drivers/iio/accel/Makefile
> @@ -5,6 +5,7 @@
>
> # When adding new entries keep the list in alphabetical order
> obj-$(CONFIG_ADIS16201) += adis16201.o
> +obj-$(CONFIG_ADIS16203) += adis16203.o
> obj-$(CONFIG_ADIS16209) += adis16209.o
> obj-$(CONFIG_ADXL313) += adxl313_core.o
> obj-$(CONFIG_ADXL313_I2C) += adxl313_i2c.o
> diff --git a/drivers/staging/iio/accel/adis16203.c b/drivers/iio/accel/adis16203.c
> similarity index 100%
> rename from drivers/staging/iio/accel/adis16203.c
> rename to drivers/iio/accel/adis16203.c
> diff --git a/drivers/staging/iio/accel/Kconfig b/drivers/staging/iio/accel/Kconfig
> index 3318997a7009..3769af8719f3 100644
> --- a/drivers/staging/iio/accel/Kconfig
> +++ b/drivers/staging/iio/accel/Kconfig
> @@ -4,18 +4,6 @@
> #
> menu "Accelerometers"
>
> -config ADIS16203
> - tristate "Analog Devices ADIS16203 Programmable 360 Degrees Inclinometer"
> - depends on SPI
> - select IIO_ADIS_LIB
> - select IIO_ADIS_LIB_BUFFER if IIO_BUFFER
> - help
> - Say Y here to build support for Analog Devices adis16203 Programmable
> - 360 Degrees Inclinometer.
> -
> - To compile this driver as a module, say M here: the module will be
> - called adis16203.
> -
> config ADIS16240
> tristate "Analog Devices ADIS16240 Programmable Impact Sensor and Recorder"
> depends on SPI
> diff --git a/drivers/staging/iio/accel/Makefile b/drivers/staging/iio/accel/Makefile
> index 094cc9be35bd..b0beec471814 100644
> --- a/drivers/staging/iio/accel/Makefile
> +++ b/drivers/staging/iio/accel/Makefile
> @@ -3,5 +3,4 @@
> # Makefile for industrial I/O accelerometer drivers
> #
>
> -obj-$(CONFIG_ADIS16203) += adis16203.o
> obj-$(CONFIG_ADIS16240) += adis16240.o
next prev parent reply other threads:[~2023-01-24 9:44 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-01-23 21:17 [PATCH 00/12] staging: iio: accel: adis16203 Inclinometer cleanup and graduation Jonathan Cameron
2023-01-23 21:17 ` [PATCH 01/12] staging: iio: accel: adis16203: More conventional header ordering Jonathan Cameron
2023-01-23 21:17 ` [PATCH 02/12] staging: iio: accel: adis16203: Drop excessive register comments Jonathan Cameron
2023-01-23 21:17 ` [PATCH 03/12] staging: iio: accel: adis16203: Rename incli channel registers Jonathan Cameron
2023-01-23 21:17 ` [PATCH 04/12] staging: iio: accel: adis16203: Drop the 180 degree offset channel Jonathan Cameron
2023-01-23 21:17 ` [PATCH 05/12] staging: iio: accel: adis16203: Simplify handling of calibbias Jonathan Cameron
2023-01-23 21:17 ` [PATCH 06/12] staging: iio: accel: adis16203: Drop driver name define Jonathan Cameron
2023-01-23 21:17 ` [PATCH 07/12] staging: iio: accel: adis16203: Make units explicit in name of delay define Jonathan Cameron
2023-01-23 21:17 ` [PATCH 08/12] staging: iio: accel: adis16203: Trivial whitespace cleanup Jonathan Cameron
2023-01-23 21:17 ` [PATCH 09/12] staging: iio: accel: adis16203: Drop setting drvdata as nothing gets it Jonathan Cameron
2023-01-23 21:17 ` [PATCH 10/12] staging: iio: accel: adis16203: Trivial style cleanup Jonathan Cameron
2023-01-23 21:17 ` [PATCH 11/12] staging: iio: accel: adis16203: Move out of staging Jonathan Cameron
2023-01-24 9:44 ` Jonathan Cameron [this message]
2023-01-23 21:17 ` [PATCH 12/12] dt-bindings: iio: accel: Add ADIS16203 Inclinometer Jonathan Cameron
2023-01-24 1:22 ` Rob Herring
2023-01-24 8:39 ` Krzysztof Kozlowski
2023-01-24 9:43 ` Jonathan Cameron
2023-02-01 9:29 ` Linus Walleij
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230124094450.0000272b@Huawei.com \
--to=jonathan.cameron@huawei.com \
--cc=Michael.Hennerich@analog.com \
--cc=baohua@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=jic23@kernel.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=lars@metafoo.de \
--cc=linux-iio@vger.kernel.org \
--cc=robh+dt@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.