From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.6 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_2 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id A3985C43603 for ; Sun, 15 Dec 2019 12:16:19 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 6D9ED2465B for ; Sun, 15 Dec 2019 12:16:19 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1576412179; bh=xKHO/rHEjhI6JaY7eTb9EV6hYpFyOyCRWCVIpWTMZeI=; h=Date:From:To:Cc:Subject:In-Reply-To:References:List-ID:From; b=gRmlcmf2RtaqkbY+6QAZW4dS/GrEnJuMWh6FZsUi1+C2J4frzA6bXydigCAF9U+xY i/19zZoH7EVTlecRoLc3MDjcwWqBKjBB4M45zFc4jhSWAF5AzaqrMQ7s9XUNr+gyXF 4a1kNQToJIi+4HpqW97SKQI6EpJFvMKhzPVdnIC8= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726099AbfLOMQT (ORCPT ); Sun, 15 Dec 2019 07:16:19 -0500 Received: from mail.kernel.org ([198.145.29.99]:51136 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726135AbfLOMQS (ORCPT ); Sun, 15 Dec 2019 07:16:18 -0500 Received: from archlinux (cpc149474-cmbg20-2-0-cust94.5-4.cable.virginm.net [82.4.196.95]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 8906220866; Sun, 15 Dec 2019 12:16:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1576412177; bh=xKHO/rHEjhI6JaY7eTb9EV6hYpFyOyCRWCVIpWTMZeI=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=myNSw2szTdbbLOj6YU5+o4o84Jj07NNSs3EOGWwxsliU4iVbAHu04Fo1IOGOuEtKy QHLCDLqYYMWlLR1NMozpfsXolKnluhHnGFDt/gQipivfuitF3QJvDChTUuVI7fHZeE NuXgmwDE1DOKFqOiK5YfU3pMUBfHFrNdBk5PmtjM= Date: Sun, 15 Dec 2019 12:16:12 +0000 From: Jonathan Cameron To: Uwe =?UTF-8?B?S2xlaW5lLUvDtm5pZw==?= Cc: Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald-Stadler , Rob Herring , Mark Rutland , Michael Hennerich , Stefan Popa , Alexandru Ardelean , linux-iio@vger.kernel.org, kernel@pengutronix.de Subject: Re: [PATCH v4 3/3] iio: adc: new driver to support Linear technology's ltc2496 Message-ID: <20191215121612.5e84c8a8@archlinux> In-Reply-To: <20191209203248.21555-4-u.kleine-koenig@pengutronix.de> References: <20191209203248.21555-1-u.kleine-koenig@pengutronix.de> <20191209203248.21555-4-u.kleine-koenig@pengutronix.de> X-Mailer: Claws Mail 3.17.4 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Sender: linux-iio-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-iio@vger.kernel.org On Mon, 9 Dec 2019 21:32:48 +0100 Uwe Kleine-K=C3=B6nig wrote: > This chip is similar to the LTC2497 ADC, it just uses SPI instead of I2C > and so has a slightly different protocol. Only the actual hardware > access is different. The spi protocol is different enough to not be able > to map the differences via a regmap. >=20 > Also generalize the entry in MAINTAINER to cover the newly introduced > file. >=20 > Signed-off-by: Uwe Kleine-K=C3=B6nig Hmm. I'm a little surprised this doesn't run into the issues we used to have with single files being linked twice and the interactions between built in drivers and modules. However I've just tried every combination on x86_64 and it seems to be fine. Perhaps something got made more flexible in the build system recently that I missed, or there is a subtle detail that is different about this particular case. With that in mind, applied to the togreg branch of iio.git and pushed out as testing to let the autobuilders play with it. Thanks, Jonathan > --- > MAINTAINERS | 2 +- > drivers/iio/adc/Kconfig | 10 ++++ > drivers/iio/adc/Makefile | 1 + > drivers/iio/adc/ltc2496.c | 108 ++++++++++++++++++++++++++++++++++++++ > 4 files changed, 120 insertions(+), 1 deletion(-) > create mode 100644 drivers/iio/adc/ltc2496.c >=20 > diff --git a/MAINTAINERS b/MAINTAINERS > index eb19fad370d7..8b1211038617 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -1029,7 +1029,7 @@ S: Supported > F: Documentation/ABI/testing/sysfs-bus-iio-frequency-ad9523 > F: Documentation/ABI/testing/sysfs-bus-iio-frequency-adf4350 > F: drivers/iio/*/ad* > -F: drivers/iio/adc/ltc2497* > +F: drivers/iio/adc/ltc249* > X: drivers/iio/*/adjd* > F: drivers/staging/iio/*/ad* > =20 > diff --git a/drivers/iio/adc/Kconfig b/drivers/iio/adc/Kconfig > index f0af3a42f53c..deb86f6039b3 100644 > --- a/drivers/iio/adc/Kconfig > +++ b/drivers/iio/adc/Kconfig > @@ -492,6 +492,16 @@ config LTC2485 > To compile this driver as a module, choose M here: the module will be > called ltc2485. > =20 > +config LTC2496 > + tristate "Linear Technology LTC2496 ADC driver" > + depends on SPI > + help > + Say yes here to build support for Linear Technology LTC2496 > + 16-Bit 8-/16-Channel Delta Sigma ADC. > + > + To compile this driver as a module, choose M here: the module will be > + called ltc2496. > + > config LTC2497 > tristate "Linear Technology LTC2497 ADC driver" > depends on I2C > diff --git a/drivers/iio/adc/Makefile b/drivers/iio/adc/Makefile > index ee0c8dcfb501..c3dc2a12766a 100644 > --- a/drivers/iio/adc/Makefile > +++ b/drivers/iio/adc/Makefile > @@ -47,6 +47,7 @@ obj-$(CONFIG_LPC18XX_ADC) +=3D lpc18xx_adc.o > obj-$(CONFIG_LPC32XX_ADC) +=3D lpc32xx_adc.o > obj-$(CONFIG_LTC2471) +=3D ltc2471.o > obj-$(CONFIG_LTC2485) +=3D ltc2485.o > +obj-$(CONFIG_LTC2496) +=3D ltc2496.o ltc2497-core.o > obj-$(CONFIG_LTC2497) +=3D ltc2497.o ltc2497-core.o > obj-$(CONFIG_MAX1027) +=3D max1027.o > obj-$(CONFIG_MAX11100) +=3D max11100.o > diff --git a/drivers/iio/adc/ltc2496.c b/drivers/iio/adc/ltc2496.c > new file mode 100644 > index 000000000000..88a30156a849 > --- /dev/null > +++ b/drivers/iio/adc/ltc2496.c > @@ -0,0 +1,108 @@ > +// SPDX-License-Identifier: GPL-2.0-only > +/* > + * ltc2496.c - Driver for Analog Devices/Linear Technology LTC2496 ADC > + * > + * Based on ltc2497.c which has > + * Copyright (C) 2017 Analog Devices Inc. > + * > + * Licensed under the GPL-2. > + * > + * Datasheet: https://www.analog.com/media/en/technical-documentation/da= ta-sheets/2496fc.pdf > + */ > + > +#include > +#include > +#include > +#include > +#include > + > +#include "ltc2497.h" > + > +struct ltc2496_driverdata { > + /* this must be the first member */ > + struct ltc2497core_driverdata common_ddata; > + struct spi_device *spi; > + > + /* > + * DMA (thus cache coherency maintenance) requires the > + * transfer buffers to live in their own cache lines. > + */ > + unsigned char rxbuf[3] ____cacheline_aligned; > + unsigned char txbuf[3]; > +}; > + > +static int ltc2496_result_and_measure(struct ltc2497core_driverdata *dda= ta, > + u8 address, int *val) > +{ > + struct ltc2496_driverdata *st =3D > + container_of(ddata, struct ltc2496_driverdata, common_ddata); > + struct spi_transfer t =3D { > + .tx_buf =3D st->txbuf, > + .rx_buf =3D st->rxbuf, > + .len =3D sizeof(st->txbuf), > + }; > + int ret; > + > + st->txbuf[0] =3D LTC2497_ENABLE | address; > + > + ret =3D spi_sync_transfer(st->spi, &t, 1); > + if (ret < 0) { > + dev_err(&st->spi->dev, "spi_sync_transfer failed: %pe\n", > + ERR_PTR(ret)); > + return ret; > + } > + > + if (val) > + *val =3D ((st->rxbuf[0] & 0x3f) << 12 | > + st->rxbuf[1] << 4 | st->rxbuf[2] >> 4) - > + (1 << 17); > + > + return 0; > +} > + > +static int ltc2496_probe(struct spi_device *spi) > +{ > + struct iio_dev *indio_dev; > + struct ltc2496_driverdata *st; > + struct device *dev =3D &spi->dev; > + > + indio_dev =3D devm_iio_device_alloc(dev, sizeof(*st)); > + if (!indio_dev) > + return -ENOMEM; > + > + st =3D iio_priv(indio_dev); > + spi_set_drvdata(spi, indio_dev); > + st->spi =3D spi; > + st->common_ddata.result_and_measure =3D ltc2496_result_and_measure; > + > + return ltc2497core_probe(dev, indio_dev); > +} > + > +static int ltc2496_remove(struct spi_device *spi) > +{ > + struct iio_dev *indio_dev =3D spi_get_drvdata(spi); > + > + ltc2497core_remove(indio_dev); > + > + return 0; > +} > + > +static const struct of_device_id ltc2496_of_match[] =3D { > + { .compatible =3D "lltc,ltc2496", }, > + {}, > +}; > +MODULE_DEVICE_TABLE(of, ltc2496_of_match); > + > +static struct spi_driver ltc2496_driver =3D { > + .driver =3D { > + .name =3D "ltc2496", > + .of_match_table =3D of_match_ptr(ltc2496_of_match), > + }, > + .probe =3D ltc2496_probe, > + .remove =3D ltc2496_remove, > +}; > +module_spi_driver(ltc2496_driver); > + > +MODULE_AUTHOR("Uwe Kleine-K=C3=B6nig "); > +MODULE_DESCRIPTION("Linear Technology LTC2496 ADC driver"); > +MODULE_LICENSE("GPL v2");