Commit 689c0953 authored by Nuno Sa's avatar Nuno Sa Committed by Jonathan Cameron

iio: imu: adis16475: make use of irq_get_trigger_type()

There's no need to call both irq_get_irq_data() and
irqd_get_trigger_type() as we already have an helper for that. This
allows for code simplification.
Signed-off-by: default avatarNuno Sa <nuno.sa@analog.com>
Link: https://lore.kernel.org/r/20240117-adis-improv-v1-2-7f90e9fad200@analog.comSigned-off-by: default avatarJonathan Cameron <Jonathan.Cameron@huawei.com>
parent 680ce1d3
...@@ -1363,22 +1363,16 @@ static int adis16475_config_sync_mode(struct adis16475 *st) ...@@ -1363,22 +1363,16 @@ static int adis16475_config_sync_mode(struct adis16475 *st)
static int adis16475_config_irq_pin(struct adis16475 *st) static int adis16475_config_irq_pin(struct adis16475 *st)
{ {
int ret; int ret;
struct irq_data *desc;
u32 irq_type; u32 irq_type;
u16 val = 0; u16 val = 0;
u8 polarity; u8 polarity;
struct spi_device *spi = st->adis.spi; struct spi_device *spi = st->adis.spi;
desc = irq_get_irq_data(spi->irq);
if (!desc) {
dev_err(&spi->dev, "Could not find IRQ %d\n", spi->irq);
return -EINVAL;
}
/* /*
* It is possible to configure the data ready polarity. Furthermore, we * It is possible to configure the data ready polarity. Furthermore, we
* need to update the adis struct if we want data ready as active low. * need to update the adis struct if we want data ready as active low.
*/ */
irq_type = irqd_get_trigger_type(desc); irq_type = irq_get_trigger_type(spi->irq);
if (irq_type == IRQ_TYPE_EDGE_RISING) { if (irq_type == IRQ_TYPE_EDGE_RISING) {
polarity = 1; polarity = 1;
st->adis.irq_flag = IRQF_TRIGGER_RISING; st->adis.irq_flag = IRQF_TRIGGER_RISING;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment