From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=melexis.com header.i=@melexis.com header.b="S88a+TLT" Received: from mail-oi1-x22c.google.com (mail-oi1-x22c.google.com [IPv6:2607:f8b0:4864:20::22c]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D984DBC for ; Thu, 23 Nov 2023 02:20:27 -0800 (PST) Received: by mail-oi1-x22c.google.com with SMTP id 5614622812f47-3b2f2b9a176so491868b6e.0 for ; Thu, 23 Nov 2023 02:20:27 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=melexis.com; s=google; t=1700734827; x=1701339627; darn=vger.kernel.org; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=zgcr8M5KTV6qx0UUZCDKtyAM/hgXXxP53Wb8ozrdskQ=; b=S88a+TLTEAN3+uVnj+ORn8bossWXwevsFZPIgqMiHjVA6Xi3vQGY1qIoFHeu7B/b/7 okYiYedwRnV5BThEQ4vDkZ4uCcaKf0yzCZ/x5CCNUi7unMJBBbCBIpanBq9s0WeHYswz d9jlWTggp+cwFEMsle7Up3mAL1qEs5qfmPyJzg6eFV1kGF8DTeYDgv/lOXFCjmnHxNKO kHl11F0LQb6wdMbOmipXzCy/BfihSu1oqawTzbNZGFEu1JwrQwP8dDNb8VedM/gAS1Ua CFqnzXedVfZxKQAvSBBl5JgZcl+1RIO3uXi4n5qeouidHpPyk06zOmUklD3IAlBfTSE8 xq7w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1700734827; x=1701339627; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=zgcr8M5KTV6qx0UUZCDKtyAM/hgXXxP53Wb8ozrdskQ=; b=TkYIhpMXth4leC0rdI+O2QW12LuwHsySWvN6CPD1ZtnC5rl5xkY8BEluPgaZ+kOKBH 6da3Z1YFMdAI+sQqIIQ+23ZbdHXyKWAhTpLRx5DtM3+hL2fHC0iSao9/WeRIKy9n0NCi IuKFPDUmH19yReCYT/D+rI9qDtS4dtzpuZ3UkhkTz2vwvBkp4jFNIUjouT5xueUz2hbq ssOjJtER1BJ3FvxCWhrxTJvKe5tPef9OQ4tZl81MncwKzdmnEiPSo9EC7XA4Dh2xm/l+ VapNOUy3/LWHTKFz19iCyV4+456sykTtqEn7JfvSPjIfUv6ckcx+YbXHVDul4ILyaoyv j2SQ== X-Gm-Message-State: AOJu0YzLD/ckFevdv/YULThsHgDz+K6rp7lwv3w/HMPiGPAg4EzofzDe yhRVf23YTOgxLWj078awqlq32Haz9XjSvz0GyC+0YA== X-Google-Smtp-Source: AGHT+IFg6ttglwn9lhnew4BaGIc37d/TxP7u9vtdx9eRIYqHxFriSRQeo+Ts6Ui0MVdpa6gCuwNi+ofHFA0fFj1FhfE= X-Received: by 2002:a05:6870:7903:b0:1f0:36b6:ef25 with SMTP id hg3-20020a056870790300b001f036b6ef25mr7026131oab.23.1700734827146; Thu, 23 Nov 2023 02:20:27 -0800 (PST) Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 References: <20231123094357.222371-1-kimseer.paller@analog.com> <20231123094357.222371-2-kimseer.paller@analog.com> In-Reply-To: <20231123094357.222371-2-kimseer.paller@analog.com> From: Crt Mori Date: Thu, 23 Nov 2023 11:19:51 +0100 Message-ID: Subject: Re: [PATCH v4 2/2] iio: frequency: admfm2000: New driver To: Kim Seer Paller Cc: Jonathan Cameron , Lars-Peter Clausen , Michael Hennerich , Rob Herring , Krzysztof Kozlowski , Conor Dooley , linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset="UTF-8" Hi, Just minor remark inline. Best regards, Crt On Thu, 23 Nov 2023 at 10:44, Kim Seer Paller wrote: > > Dual microwave down converter module with input RF and LO frequency > ranges from 0.5 to 32 GHz and an output IF frequency range from 0.1 to > 8 GHz. It consists of a LNA, mixer, IF filter, DSA, and IF amplifier > for each down conversion path. > > Signed-off-by: Kim Seer Paller > --- > V1 -> V4: No changes. > > MAINTAINERS | 1 + > drivers/iio/frequency/Kconfig | 10 + > drivers/iio/frequency/Makefile | 1 + > drivers/iio/frequency/admfm2000.c | 309 ++++++++++++++++++++++++++++++ > 4 files changed, 321 insertions(+) > create mode 100644 drivers/iio/frequency/admfm2000.c > > diff --git a/MAINTAINERS b/MAINTAINERS > index f1692ec68..d8630e490 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -1253,6 +1253,7 @@ L: linux-iio@vger.kernel.org > S: Supported > W: https://ez.analog.com/linux-software-drivers > F: Documentation/devicetree/bindings/iio/frequency/adi,admfm2000.yaml > +F: drivers/iio/frequency/admfm2000.c > > ANALOG DEVICES INC ADMV1013 DRIVER > M: Antoniu Miclaus > diff --git a/drivers/iio/frequency/Kconfig b/drivers/iio/frequency/Kconfig > index 9e85dfa58..c455be7d4 100644 > --- a/drivers/iio/frequency/Kconfig > +++ b/drivers/iio/frequency/Kconfig > @@ -60,6 +60,16 @@ config ADF4377 > To compile this driver as a module, choose M here: the > module will be called adf4377. > > +config ADMFM2000 > + tristate "Analog Devices ADMFM2000 Dual Microwave Down Converter" > + depends on GPIOLIB > + help > + Say yes here to build support for Analog Devices ADMFM2000 Dual > + Microwave Down Converter. > + > + To compile this driver as a module, choose M here: the > + module will be called admfm2000. > + > config ADMV1013 > tristate "Analog Devices ADMV1013 Microwave Upconverter" > depends on SPI && COMMON_CLK > diff --git a/drivers/iio/frequency/Makefile b/drivers/iio/frequency/Makefile > index b616c29b4..70d0e0b70 100644 > --- a/drivers/iio/frequency/Makefile > +++ b/drivers/iio/frequency/Makefile > @@ -8,6 +8,7 @@ obj-$(CONFIG_AD9523) += ad9523.o > obj-$(CONFIG_ADF4350) += adf4350.o > obj-$(CONFIG_ADF4371) += adf4371.o > obj-$(CONFIG_ADF4377) += adf4377.o > +obj-$(CONFIG_ADMFM2000) += admfm2000.o > obj-$(CONFIG_ADMV1013) += admv1013.o > obj-$(CONFIG_ADMV1014) += admv1014.o > obj-$(CONFIG_ADMV4420) += admv4420.o > diff --git a/drivers/iio/frequency/admfm2000.c b/drivers/iio/frequency/admfm2000.c > new file mode 100644 > index 000000000..e0b5edce7 > --- /dev/null > +++ b/drivers/iio/frequency/admfm2000.c > @@ -0,0 +1,309 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* > + * ADMFM2000 Dual Microwave Down Converter > + * > + * Copyright 2023 Analog Devices Inc. > + */ > + > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > + > +#define ADMFM2000_MIXER_MODE 0 > +#define ADMFM2000_DIRECT_IF_MODE 1 > +#define ADMF20000_DSA_GPIOS 5 > +#define ADMF20000_MODE_GPIOS 2 > +#define ADMF20000_MAX_GAIN 0 > +#define ADMF20000_MIN_GAIN -31000 > +#define ADMF20000_DEFAULT_GAIN -0x20 > + > +struct admfm2000_state { > + struct mutex lock; /* protect sensor state */ > + struct gpio_descs *sw_ch[2]; > + struct gpio_descs *dsa_gpios[2]; > + u32 gain[2]; > +}; > + > +static int admfm2000_mode(struct iio_dev *indio_dev, u32 reg, u32 mode) > +{ > + struct admfm2000_state *st = iio_priv(indio_dev); > + DECLARE_BITMAP(values, 2); > + > + switch (mode) { > + case ADMFM2000_MIXER_MODE: > + values[0] = (reg == 0) ? 1 : 2; > + gpiod_set_array_value_cansleep(st->sw_ch[reg]->ndescs, > + st->sw_ch[reg]->desc, > + NULL, values); > + break; > + case ADMFM2000_DIRECT_IF_MODE: > + values[0] = (reg == 0) ? 2 : 1; > + gpiod_set_array_value_cansleep(st->sw_ch[reg]->ndescs, > + st->sw_ch[reg]->desc, > + NULL, values); > + break; > + default: > + return -EINVAL; > + } > + > + return 0; > +} > + > +static int admfm2000_attenuation(struct iio_dev *indio_dev, u32 chan, > + u32 value) > +{ > + struct admfm2000_state *st = iio_priv(indio_dev); > + DECLARE_BITMAP(values, BITS_PER_TYPE(value)); > + > + values[0] = value; > + > + gpiod_set_array_value_cansleep(st->dsa_gpios[chan]->ndescs, > + st->dsa_gpios[chan]->desc, > + NULL, values); > + return 0; > +} > + > +static int admfm2000_read_raw(struct iio_dev *indio_dev, > + struct iio_chan_spec const *chan, int *val, > + int *val2, long mask) > +{ > + struct admfm2000_state *st = iio_priv(indio_dev); > + int gain; > + > + switch (mask) { > + case IIO_CHAN_INFO_HARDWAREGAIN: > + mutex_lock(&st->lock); > + gain = ~(st->gain[chan->channel]) * -1000; > + *val = gain / 1000; > + *val2 = (gain % 1000) * 1000; > + mutex_unlock(&st->lock); > + > + return IIO_VAL_INT_PLUS_MICRO_DB; > + default: > + return -EINVAL; > + } > +} > + > +static int admfm2000_write_raw(struct iio_dev *indio_dev, > + struct iio_chan_spec const *chan, int val, > + int val2, long mask) > +{ > + struct admfm2000_state *st = iio_priv(indio_dev); > + int gain, ret; > + > + if (val < 0) > + gain = (val * 1000) - (val2 / 1000); > + else > + gain = (val * 1000) + (val2 / 1000); > + > + if (gain > ADMF20000_MAX_GAIN || gain < ADMF20000_MIN_GAIN) > + return -EINVAL; > + > + switch (mask) { > + case IIO_CHAN_INFO_HARDWAREGAIN: > + mutex_lock(&st->lock); > + st->gain[chan->channel] = ~((abs(gain) / 1000) & 0x1F); > + > + ret = admfm2000_attenuation(indio_dev, chan->channel, > + st->gain[chan->channel]); > + > + mutex_unlock(&st->lock); > + if (ret) > + return ret; > + break; > + default: > + return -EINVAL; > + } > + > + return 0; > +} > + > +static int admfm2000_write_raw_get_fmt(struct iio_dev *indio_dev, > + struct iio_chan_spec const *chan, > + long mask) > +{ > + switch (mask) { > + case IIO_CHAN_INFO_HARDWAREGAIN: > + return IIO_VAL_INT_PLUS_MICRO_DB; > + default: > + return -EINVAL; > + } > +} > + > +static const struct iio_info admfm2000_info = { > + .read_raw = &admfm2000_read_raw, > + .write_raw = &admfm2000_write_raw, > + .write_raw_get_fmt = &admfm2000_write_raw_get_fmt, > +}; > + > +#define ADMFM2000_CHAN(_channel) { \ > + .type = IIO_VOLTAGE, \ > + .output = 1, \ > + .indexed = 1, \ > + .channel = _channel, \ > + .info_mask_separate = BIT(IIO_CHAN_INFO_HARDWAREGAIN), \ > +} > + > +static const struct iio_chan_spec admfm2000_channels[] = { > + ADMFM2000_CHAN(0), > + ADMFM2000_CHAN(1), > +}; > + > +static int admfm2000_channel_config(struct admfm2000_state *st, > + struct iio_dev *indio_dev) > +{ > + struct platform_device *pdev = to_platform_device(indio_dev->dev.parent); > + struct device *dev = &pdev->dev; > + struct fwnode_handle *child; > + u32 reg, mode; > + int ret; > + > + device_for_each_child_node(dev, child) { > + ret = fwnode_property_read_u32(child, "reg", ®); > + if (ret) { > + fwnode_handle_put(child); > + return dev_err_probe(dev, ret, > + "Failed to get reg property\n"); > + } > + > + if (reg >= indio_dev->num_channels) { > + fwnode_handle_put(child); > + return dev_err_probe(dev, -EINVAL, "reg bigger than: %d\n", > + indio_dev->num_channels); > + } > + > + ret = fwnode_property_read_u32(child, "adi,mode", &mode); > + if (ret) { > + fwnode_handle_put(child); > + return dev_err_probe(dev, ret, > + "Failed to get mode property\n"); > + } > + > + if (mode >= 2) { > + fwnode_handle_put(child); > + return dev_err_probe(dev, -EINVAL, "mode bigger than: 1\n"); > + } > + > + ret = admfm2000_mode(indio_dev, reg, mode); > + if (ret) { > + fwnode_handle_put(child); > + return ret; > + } > + } > + > + return 0; > +} > + > +static int admfm2000_setup(struct admfm2000_state *st, > + struct iio_dev *indio_dev) > +{ > + struct platform_device *pdev = to_platform_device(indio_dev->dev.parent); > + struct device *dev = &pdev->dev; > + > + st->sw_ch[0] = devm_gpiod_get_array(dev, "switch1", GPIOD_OUT_LOW); > + if (IS_ERR(st->sw_ch[0])) > + return dev_err_probe(dev, PTR_ERR(st->sw_ch[0]), > + "Failed to get gpios\n"); > + > + if (st->sw_ch[0]->ndescs != ADMF20000_MODE_GPIOS) { > + dev_err_probe(dev, -ENODEV, "%d GPIOs needed to operate\n", > + ADMF20000_MODE_GPIOS); > + return -ENODEV; > + } > + > + st->sw_ch[1] = devm_gpiod_get_array(dev, "switch2", GPIOD_OUT_LOW); > + if (IS_ERR(st->sw_ch[1])) > + return dev_err_probe(dev, PTR_ERR(st->sw_ch[1]), > + "Failed to get gpios\n"); > + > + if (st->sw_ch[1]->ndescs != ADMF20000_MODE_GPIOS) { > + dev_err_probe(dev, -ENODEV, "%d GPIOs needed to operate\n", > + ADMF20000_MODE_GPIOS); > + return -ENODEV; > + } > + > + st->dsa_gpios[0] = devm_gpiod_get_array(dev, "attenuation1", > + GPIOD_OUT_LOW); > + if (IS_ERR(st->dsa_gpios[0])) > + return dev_err_probe(dev, PTR_ERR(st->dsa_gpios[0]), > + "Failed to get gpios\n"); > + > + if (st->dsa_gpios[0]->ndescs != ADMF20000_DSA_GPIOS) { > + dev_err_probe(dev, -ENODEV, "%d GPIOs needed to operate\n", > + ADMF20000_DSA_GPIOS); > + return -ENODEV; > + } > + > + st->dsa_gpios[1] = devm_gpiod_get_array(dev, "attenuation2", > + GPIOD_OUT_LOW); > + if (IS_ERR(st->dsa_gpios[1])) > + return dev_err_probe(dev, PTR_ERR(st->dsa_gpios[1]), > + "Failed to get gpios\n"); > + > + if (st->dsa_gpios[1]->ndescs != ADMF20000_DSA_GPIOS) { > + dev_err_probe(dev, -ENODEV, "%d GPIOs needed to operate\n", > + ADMF20000_DSA_GPIOS); no return -ENODEV here? > + } > + > + return 0; > +} > + > +static int admfm2000_probe(struct platform_device *pdev) > +{ > + struct device *dev = &pdev->dev; > + struct iio_dev *indio_dev; > + struct admfm2000_state *st; > + int ret; Order these in reverse christmass tree like you did above. > + > + indio_dev = devm_iio_device_alloc(dev, sizeof(*st)); > + if (!indio_dev) > + return -ENOMEM; > + > + st = iio_priv(indio_dev); > + > + indio_dev->name = "admfm2000"; > + indio_dev->num_channels = ARRAY_SIZE(admfm2000_channels); > + indio_dev->channels = admfm2000_channels; > + indio_dev->info = &admfm2000_info; > + indio_dev->modes = INDIO_DIRECT_MODE; > + > + st->gain[0] = ADMF20000_DEFAULT_GAIN; > + st->gain[1] = ADMF20000_DEFAULT_GAIN; > + > + mutex_init(&st->lock); > + > + ret = admfm2000_setup(st, indio_dev); > + if (ret) > + return ret; > + > + ret = admfm2000_channel_config(st, indio_dev); > + if (ret) > + return ret; > + > + return devm_iio_device_register(dev, indio_dev); > +} > + > +static const struct of_device_id admfm2000_of_match[] = { > + { .compatible = "adi,admfm2000" }, > + { } > +}; > +MODULE_DEVICE_TABLE(of, admfm2000_of_match); > + > +static struct platform_driver admfm2000_driver = { > + .driver = { > + .name = "admfm2000", > + .of_match_table = admfm2000_of_match, > + }, > + .probe = admfm2000_probe, > +}; > +module_platform_driver(admfm2000_driver); > + > +MODULE_AUTHOR("Kim Seer Paller "); > +MODULE_DESCRIPTION("ADMFM2000 Dual Microwave Down Converter"); > +MODULE_LICENSE("GPL"); > -- > 2.34.1 > >