On 1/26/26 7:51 AM, Rodrigo Alencar via B4 Relay wrote:
> From: Rodrigo Alencar <rodrigo.alencar@analog.com>
>
> Some parts may consume enable GPIO to enable serial mode
> (HMC1119's and HMC792A P/S pin) or powerup the device
> (e.g. ADA4961's PWUP pin).
>
> Co-developed-by: Michael Hennerich <michael.hennerich@analog.com>
> Signed-off-by: Michael Hennerich <michael.hennerich@analog.com>
> Signed-off-by: Rodrigo Alencar <rodrigo.alencar@analog.com>
> ---
> drivers/iio/amplifiers/ad8366.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/drivers/iio/amplifiers/ad8366.c b/drivers/iio/amplifiers/ad8366.c
> index 134188db2e15..9ad6a628376c 100644
> --- a/drivers/iio/amplifiers/ad8366.c
> +++ b/drivers/iio/amplifiers/ad8366.c
> @@ -41,6 +41,7 @@ struct ad8366_state {
> struct spi_device *spi;
> struct mutex lock; /* protect sensor state */
> struct gpio_desc *reset_gpio;
> + struct gpio_desc *enable_gpio;
This isn't needed since it is never used outside of the probe function.
It can be a local variable.
> unsigned char ch[2];
> const struct ad8366_info *info;
> /*
> @@ -230,6 +231,11 @@ static int ad8366_probe(struct spi_device *spi)
> return dev_err_probe(dev, PTR_ERR(st->reset_gpio),
> "Failed to get reset GPIO\n");
>
> + st->enable_gpio = devm_gpiod_get_optional(dev, "enable", GPIOD_OUT_HIGH);
> + if (IS_ERR(st->enable_gpio))
> + return dev_err_probe(dev, PTR_ERR(st->enable_gpio),
> + "Failed to get enable GPIO\n");
> +
> indio_dev->name = spi_get_device_id(spi)->name;
> indio_dev->info = &ad8366_info;
> indio_dev->modes = INDIO_DIRECT_MODE;
>