2023-11-03 15:02:10

by Ana-Maria Cusco

[permalink] [raw]
Subject: [PATCH 0/2] Add ADRF5740 driver

From: Ana-Maria Cusco <[email protected]>

This patch series adds support for ADRF5740 Attenuator.

The ADRF5740 is a silicon, 4-bit digital attenuator with 22 dB
attenuation control range in 2 dB steps.
It offers parallel control mode through four digitally controlled inputs.

Ana-Maria Cusco (2):
iio: amplifiers: adrf5740: Add Support ADRF5740 4-bit Attenuator
dt-bindings: iio: hmc425a: add entry for ADRF5740

.../bindings/iio/amplifiers/adi,hmc425a.yaml | 4 ++++
drivers/iio/amplifiers/hmc425a.c | 23 +++++++++++++++++++
2 files changed, 27 insertions(+)

--
2.34.1


2023-11-03 15:02:13

by Ana-Maria Cusco

[permalink] [raw]
Subject: [PATCH 1/2] iio: amplifiers: adrf5740: Add Support ADRF5740 4-bit Attenuator

From: Ana-Maria Cusco <[email protected]>

This adds support for the Analog Devices ADRF5740 2 dB LSB, 4-Bit,
Silicon Digital Attenuator, 10 MHz to 60 GHz

Signed-off-by: Ana-Maria Cusco <[email protected]>
---
drivers/iio/amplifiers/hmc425a.c | 23 +++++++++++++++++++++++
1 file changed, 23 insertions(+)

diff --git a/drivers/iio/amplifiers/hmc425a.c b/drivers/iio/amplifiers/hmc425a.c
index e87d35d50a95..ed4d72922696 100644
--- a/drivers/iio/amplifiers/hmc425a.c
+++ b/drivers/iio/amplifiers/hmc425a.c
@@ -5,6 +5,7 @@
* Copyright 2020 Analog Devices Inc.
*/

+#include <linux/bitops.h>
#include <linux/device.h>
#include <linux/err.h>
#include <linux/gpio/consumer.h>
@@ -22,6 +23,7 @@
enum hmc425a_type {
ID_HMC425A,
ID_HMC540S,
+ ID_ADRF5740
};

struct hmc425a_chip_info {
@@ -74,6 +76,10 @@ static int hmc425a_read_raw(struct iio_dev *indio_dev,
case ID_HMC540S:
gain = ~code * -1000;
break;
+ case ID_ADRF5740:
+ code = code & BIT(3) ? code & ~BIT(2) : code;
+ gain = code * -2000;
+ break;
}

*val = gain / 1000;
@@ -113,6 +119,10 @@ static int hmc425a_write_raw(struct iio_dev *indio_dev,
case ID_HMC540S:
code = ~((abs(gain) / 1000) & 0xF);
break;
+ case ID_ADRF5740:
+ code = (abs(gain) / 2000) & 0xF;
+ code = code & BIT(3) ? code | BIT(2) : code;
+ break;
}

mutex_lock(&st->lock);
@@ -165,6 +175,7 @@ static const struct iio_chan_spec hmc425a_channels[] = {
static const struct of_device_id hmc425a_of_match[] = {
{ .compatible = "adi,hmc425a", .data = (void *)ID_HMC425A },
{ .compatible = "adi,hmc540s", .data = (void *)ID_HMC540S },
+ { .compatible = "adi,adrf5740", .data = (void *)ID_ADRF5740 },
{},
};
MODULE_DEVICE_TABLE(of, hmc425a_of_match);
@@ -188,6 +199,15 @@ static struct hmc425a_chip_info hmc425a_chip_info_tbl[] = {
.gain_max = 0,
.default_gain = -0x10, /* set default gain -15.0db*/
},
+ [ID_ADRF5740] = {
+ .name = "adrf5740",
+ .channels = hmc425a_channels,
+ .num_channels = ARRAY_SIZE(hmc425a_channels),
+ .num_gpios = 4,
+ .gain_min = -22000,
+ .gain_max = 0,
+ .default_gain = 0xF, /* set default gain -22.0db*/
+ },
};

static int hmc425a_probe(struct platform_device *pdev)
@@ -229,6 +249,9 @@ static int hmc425a_probe(struct platform_device *pdev)
indio_dev->info = &hmc425a_info;
indio_dev->modes = INDIO_DIRECT_MODE;

+ /* Set default gain */
+ hmc425a_write(indio_dev, st->gain);
+
return devm_iio_device_register(&pdev->dev, indio_dev);
}

--
2.34.1

2023-11-03 15:03:04

by Ana-Maria Cusco

[permalink] [raw]
Subject: [PATCH 2/2] dt-bindings: iio: hmc425a: add entry for ADRF5740

From: Ana-Maria Cusco <[email protected]>

The ADRF5740 is a silicon, 4-bit digital attenuator with 22 dB
attenuation control range in 2 dB steps.

Signed-off-by: Ana-Maria Cusco <[email protected]>
---
.../devicetree/bindings/iio/amplifiers/adi,hmc425a.yaml | 4 ++++
1 file changed, 4 insertions(+)

diff --git a/Documentation/devicetree/bindings/iio/amplifiers/adi,hmc425a.yaml b/Documentation/devicetree/bindings/iio/amplifiers/adi,hmc425a.yaml
index 2ee6080deac7..42341fbf8d74 100644
--- a/Documentation/devicetree/bindings/iio/amplifiers/adi,hmc425a.yaml
+++ b/Documentation/devicetree/bindings/iio/amplifiers/adi,hmc425a.yaml
@@ -18,12 +18,16 @@ description: |
HMC540S 1 dB LSB Silicon MMIC 4-Bit Digital Positive Control Attenuator, 0.1 - 8 GHz
https://www.analog.com/media/en/technical-documentation/data-sheets/hmc540s.pdf

+ ADRF5750 2 dB LSB, 4-Bit, Silicon Digital Attenuator, 10 MHz to 60 GHz
+ https://www.analog.com/media/en/technical-documentation/data-sheets/adrf5740.pdf
+

properties:
compatible:
enum:
- adi,hmc425a
- adi,hmc540s
+ - adi,adrf5740

vcc-supply: true

--
2.34.1

2023-11-05 14:05:39

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH 2/2] dt-bindings: iio: hmc425a: add entry for ADRF5740

On 03/11/2023 16:01, Ana-Maria Cusco wrote:
> From: Ana-Maria Cusco <[email protected]>
>
> The ADRF5740 is a silicon, 4-bit digital attenuator with 22 dB
> attenuation control range in 2 dB steps.
>
> Signed-off-by: Ana-Maria Cusco <[email protected]>
> ---
> .../devicetree/bindings/iio/amplifiers/adi,hmc425a.yaml | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/iio/amplifiers/adi,hmc425a.yaml b/Documentation/devicetree/bindings/iio/amplifiers/adi,hmc425a.yaml
> index 2ee6080deac7..42341fbf8d74 100644
> --- a/Documentation/devicetree/bindings/iio/amplifiers/adi,hmc425a.yaml
> +++ b/Documentation/devicetree/bindings/iio/amplifiers/adi,hmc425a.yaml
> @@ -18,12 +18,16 @@ description: |
> HMC540S 1 dB LSB Silicon MMIC 4-Bit Digital Positive Control Attenuator, 0.1 - 8 GHz
> https://www.analog.com/media/en/technical-documentation/data-sheets/hmc540s.pdf
>
> + ADRF5750 2 dB LSB, 4-Bit, Silicon Digital Attenuator, 10 MHz to 60 GHz
> + https://www.analog.com/media/en/technical-documentation/data-sheets/adrf5740.pdf
> +
>
> properties:
> compatible:
> enum:
> - adi,hmc425a
> - adi,hmc540s
> + - adi,adrf5740

Please keep alphabetical order of entries.

Best regards,
Krzysztof

2023-11-06 11:10:23

by Jonathan Cameron

[permalink] [raw]
Subject: Re: [PATCH 0/2] Add ADRF5740 driver

On Fri, 3 Nov 2023 17:01:28 +0200
Ana-Maria Cusco <[email protected]> wrote:

> From: Ana-Maria Cusco <[email protected]>
Hi Ana-Maria,

If you do a v2, then make it clear in the title that this is adding support
to an additional driver rather than a whole new one.


>
> This patch series adds support for ADRF5740 Attenuator.
>
> The ADRF5740 is a silicon, 4-bit digital attenuator with 22 dB
> attenuation control range in 2 dB steps.
> It offers parallel control mode through four digitally controlled inputs.
>
> Ana-Maria Cusco (2):
> iio: amplifiers: adrf5740: Add Support ADRF5740 4-bit Attenuator
> dt-bindings: iio: hmc425a: add entry for ADRF5740
>
> .../bindings/iio/amplifiers/adi,hmc425a.yaml | 4 ++++
> drivers/iio/amplifiers/hmc425a.c | 23 +++++++++++++++++++
> 2 files changed, 27 insertions(+)
>