From: "Sa, Nuno" <Nuno.Sa@analog.com> To: Jonathan Cameron <Jonathan.Cameron@Huawei.com> Cc: "linux-iio@vger.kernel.org" <linux-iio@vger.kernel.org>, "devel@driverdev.osuosl.org" <devel@driverdev.osuosl.org>, Lars-Peter Clausen <lars@metafoo.de>, "Hennerich, Michael" <Michael.Hennerich@analog.com>, Jonathan Cameron <jic23@kernel.org>, Hartmut Knaack <knaack.h@gmx.de>, Peter Meerwald-Stadler <pmeerw@pmeerw.net>, "Bogdan, Dragos" <Dragos.Bogdan@analog.com>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, "Ardelean, Alexandru" <alexandru.Ardelean@analog.com> Subject: RE: [PATCH 10/10] iio: adis: Drop non Managed device functions Date: Tue, 15 Sep 2020 10:20:20 +0000 [thread overview] Message-ID: <CY4PR03MB2631DD532FE132207FCE3E9799200@CY4PR03MB2631.namprd03.prod.outlook.com> (raw) In-Reply-To: <20200915111015.00004707@Huawei.com> > -----Original Message----- > From: Jonathan Cameron <Jonathan.Cameron@Huawei.com> > Sent: Tuesday, September 15, 2020 12:10 PM > To: Sa, Nuno <Nuno.Sa@analog.com> > Cc: linux-iio@vger.kernel.org; devel@driverdev.osuosl.org; Lars-Peter > Clausen <lars@metafoo.de>; Hennerich, Michael > <Michael.Hennerich@analog.com>; Jonathan Cameron <jic23@kernel.org>; > Hartmut Knaack <knaack.h@gmx.de>; Peter Meerwald-Stadler > <pmeerw@pmeerw.net>; Bogdan, Dragos <Dragos.Bogdan@analog.com>; > Greg Kroah-Hartman <gregkh@linuxfoundation.org>; Ardelean, Alexandru > <alexandru.Ardelean@analog.com> > Subject: Re: [PATCH 10/10] iio: adis: Drop non Managed device functions > > [External] > > On Tue, 15 Sep 2020 11:33:45 +0200 > Nuno Sá <nuno.sa@analog.com> wrote: > > > Drop `adis_setup_buffer_and_trigger()`. All users were updated to use > > the devm version of this function. This avoids having almost the same > > code repeated. > > > > Signed-off-by: Nuno Sá <nuno.sa@analog.com> > > Good to see this cleanup, as long as we tidy up the few issues in the > earlier patches. > > Note I think I only commented on first instance of each thing to fix. > Please carry them through all the patches. > > Thanks, > > Jonathan So, I did thought about further cleaning the probe functions in order to get rid of the remove function but it felt like a different series to me. Anyways, since it's fine with you to include those changes here I will send a v2. - Nuno Sá > > --- > > drivers/iio/imu/adis_buffer.c | 64 +++------------------------------- > > drivers/iio/imu/adis_trigger.c | 60 ------------------------------- > > include/linux/iio/imu/adis.h | 27 -------------- > > 3 files changed, 4 insertions(+), 147 deletions(-) > > > > diff --git a/drivers/iio/imu/adis_buffer.c b/drivers/iio/imu/adis_buffer.c > > index 5b4225ee09b9..df6144285470 100644 > > --- a/drivers/iio/imu/adis_buffer.c > > +++ b/drivers/iio/imu/adis_buffer.c > > @@ -169,48 +169,6 @@ static void adis_buffer_cleanup(void *arg) > > kfree(adis->xfer); > > } > > > > -/** > > - * adis_setup_buffer_and_trigger() - Sets up buffer and trigger for the > adis device > > - * @adis: The adis device. > > - * @indio_dev: The IIO device. > > - * @trigger_handler: Optional trigger handler, may be NULL. > > - * > > - * Returns 0 on success, a negative error code otherwise. > > - * > > - * This function sets up the buffer and trigger for a adis devices. If > > - * 'trigger_handler' is NULL the default trigger handler will be used. The > > - * default trigger handler will simply read the registers assigned to the > > - * currently active channels. > > - * > > - * adis_cleanup_buffer_and_trigger() should be called to free the > resources > > - * allocated by this function. > > - */ > > -int adis_setup_buffer_and_trigger(struct adis *adis, struct iio_dev > *indio_dev, > > - irqreturn_t (*trigger_handler)(int, void *)) > > -{ > > - int ret; > > - > > - if (!trigger_handler) > > - trigger_handler = adis_trigger_handler; > > - > > - ret = iio_triggered_buffer_setup(indio_dev, > &iio_pollfunc_store_time, > > - trigger_handler, NULL); > > - if (ret) > > - return ret; > > - > > - if (adis->spi->irq) { > > - ret = adis_probe_trigger(adis, indio_dev); > > - if (ret) > > - goto error_buffer_cleanup; > > - } > > - return 0; > > - > > -error_buffer_cleanup: > > - iio_triggered_buffer_cleanup(indio_dev); > > - return ret; > > -} > > -EXPORT_SYMBOL_GPL(adis_setup_buffer_and_trigger); > > - > > /** > > * devm_adis_setup_buffer_and_trigger() - Sets up buffer and trigger for > > * the managed adis device > > @@ -220,7 +178,10 @@ > EXPORT_SYMBOL_GPL(adis_setup_buffer_and_trigger); > > * > > * Returns 0 on success, a negative error code otherwise. > > * > > - * This function perfoms exactly the same as > adis_setup_buffer_and_trigger() > > + * This function sets up the buffer and trigger for a adis devices. If > > + * 'trigger_handler' is NULL the default trigger handler will be used. The > > + * default trigger handler will simply read the registers assigned to the > > + * currently active channels. > > */ > > int > > devm_adis_setup_buffer_and_trigger(struct adis *adis, struct iio_dev > *indio_dev, > > @@ -248,20 +209,3 @@ devm_adis_setup_buffer_and_trigger(struct adis > *adis, struct iio_dev *indio_dev, > > } > > EXPORT_SYMBOL_GPL(devm_adis_setup_buffer_and_trigger); > > > > -/** > > - * adis_cleanup_buffer_and_trigger() - Free buffer and trigger resources > > - * @adis: The adis device. > > - * @indio_dev: The IIO device. > > - * > > - * Frees resources allocated by adis_setup_buffer_and_trigger() > > - */ > > -void adis_cleanup_buffer_and_trigger(struct adis *adis, > > - struct iio_dev *indio_dev) > > -{ > > - if (adis->spi->irq) > > - adis_remove_trigger(adis); > > - kfree(adis->buffer); > > - kfree(adis->xfer); > > - iio_triggered_buffer_cleanup(indio_dev); > > -} > > -EXPORT_SYMBOL_GPL(adis_cleanup_buffer_and_trigger); > > diff --git a/drivers/iio/imu/adis_trigger.c b/drivers/iio/imu/adis_trigger.c > > index 8afe71947c00..64e0ba51cb18 100644 > > --- a/drivers/iio/imu/adis_trigger.c > > +++ b/drivers/iio/imu/adis_trigger.c > > @@ -55,53 +55,6 @@ static int adis_validate_irq_flag(struct adis *adis) > > > > return 0; > > } > > -/** > > - * adis_probe_trigger() - Sets up trigger for a adis device > > - * @adis: The adis device > > - * @indio_dev: The IIO device > > - * > > - * Returns 0 on success or a negative error code > > - * > > - * adis_remove_trigger() should be used to free the trigger. > > - */ > > -int adis_probe_trigger(struct adis *adis, struct iio_dev *indio_dev) > > -{ > > - int ret; > > - > > - adis->trig = iio_trigger_alloc("%s-dev%d", indio_dev->name, > > - indio_dev->id); > > - if (adis->trig == NULL) > > - return -ENOMEM; > > - > > - adis_trigger_setup(adis); > > - > > - ret = adis_validate_irq_flag(adis); > > - if (ret) > > - return ret; > > - > > - ret = request_irq(adis->spi->irq, > > - &iio_trigger_generic_data_rdy_poll, > > - adis->irq_flag, > > - indio_dev->name, > > - adis->trig); > > - if (ret) > > - goto error_free_trig; > > - > > - ret = iio_trigger_register(adis->trig); > > - > > - indio_dev->trig = iio_trigger_get(adis->trig); > > - if (ret) > > - goto error_free_irq; > > - > > - return 0; > > - > > -error_free_irq: > > - free_irq(adis->spi->irq, adis->trig); > > -error_free_trig: > > - iio_trigger_free(adis->trig); > > - return ret; > > -} > > -EXPORT_SYMBOL_GPL(adis_probe_trigger); > > > > /** > > * devm_adis_probe_trigger() - Sets up trigger for a managed adis device > > @@ -137,16 +90,3 @@ int devm_adis_probe_trigger(struct adis *adis, > struct iio_dev *indio_dev) > > } > > EXPORT_SYMBOL_GPL(devm_adis_probe_trigger); > > > > -/** > > - * adis_remove_trigger() - Remove trigger for a adis devices > > - * @adis: The adis device > > - * > > - * Removes the trigger previously registered with adis_probe_trigger(). > > - */ > > -void adis_remove_trigger(struct adis *adis) > > -{ > > - iio_trigger_unregister(adis->trig); > > - free_irq(adis->spi->irq, adis->trig); > > - iio_trigger_free(adis->trig); > > -} > > -EXPORT_SYMBOL_GPL(adis_remove_trigger); > > diff --git a/include/linux/iio/imu/adis.h b/include/linux/iio/imu/adis.h > > index 2df67448f0d1..01ba691da2f3 100644 > > --- a/include/linux/iio/imu/adis.h > > +++ b/include/linux/iio/imu/adis.h > > @@ -517,14 +517,8 @@ struct adis_burst { > > int > > devm_adis_setup_buffer_and_trigger(struct adis *adis, struct iio_dev > *indio_dev, > > irq_handler_t trigger_handler); > > -int adis_setup_buffer_and_trigger(struct adis *adis, > > - struct iio_dev *indio_dev, irqreturn_t (*trigger_handler)(int, void > *)); > > -void adis_cleanup_buffer_and_trigger(struct adis *adis, > > - struct iio_dev *indio_dev); > > > > int devm_adis_probe_trigger(struct adis *adis, struct iio_dev *indio_dev); > > -int adis_probe_trigger(struct adis *adis, struct iio_dev *indio_dev); > > -void adis_remove_trigger(struct adis *adis); > > > > int adis_update_scan_mode(struct iio_dev *indio_dev, > > const unsigned long *scan_mask); > > @@ -538,33 +532,12 @@ devm_adis_setup_buffer_and_trigger(struct adis > *adis, struct iio_dev *indio_dev, > > return 0; > > } > > > > -static inline int adis_setup_buffer_and_trigger(struct adis *adis, > > - struct iio_dev *indio_dev, irqreturn_t (*trigger_handler)(int, void *)) > > -{ > > - return 0; > > -} > > - > > -static inline void adis_cleanup_buffer_and_trigger(struct adis *adis, > > - struct iio_dev *indio_dev) > > -{ > > -} > > - > > static inline int devm_adis_probe_trigger(struct adis *adis, > > struct iio_dev *indio_dev) > > { > > return 0; > > } > > > > -static inline int adis_probe_trigger(struct adis *adis, > > - struct iio_dev *indio_dev) > > -{ > > - return 0; > > -} > > - > > -static inline void adis_remove_trigger(struct adis *adis) > > -{ > > -} > > - > > #define adis_update_scan_mode NULL > > > > #endif /* CONFIG_IIO_BUFFER */ >
WARNING: multiple messages have this Message-ID (diff)
From: "Sa, Nuno" <Nuno.Sa@analog.com> To: Jonathan Cameron <Jonathan.Cameron@Huawei.com> Cc: "devel@driverdev.osuosl.org" <devel@driverdev.osuosl.org>, Lars-Peter Clausen <lars@metafoo.de>, "Hennerich, Michael" <Michael.Hennerich@analog.com>, "linux-iio@vger.kernel.org" <linux-iio@vger.kernel.org>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, "Bogdan, Dragos" <Dragos.Bogdan@analog.com>, Peter Meerwald-Stadler <pmeerw@pmeerw.net>, Hartmut Knaack <knaack.h@gmx.de>, "Ardelean, Alexandru" <alexandru.Ardelean@analog.com>, Jonathan Cameron <jic23@kernel.org> Subject: RE: [PATCH 10/10] iio: adis: Drop non Managed device functions Date: Tue, 15 Sep 2020 10:20:20 +0000 [thread overview] Message-ID: <CY4PR03MB2631DD532FE132207FCE3E9799200@CY4PR03MB2631.namprd03.prod.outlook.com> (raw) In-Reply-To: <20200915111015.00004707@Huawei.com> > -----Original Message----- > From: Jonathan Cameron <Jonathan.Cameron@Huawei.com> > Sent: Tuesday, September 15, 2020 12:10 PM > To: Sa, Nuno <Nuno.Sa@analog.com> > Cc: linux-iio@vger.kernel.org; devel@driverdev.osuosl.org; Lars-Peter > Clausen <lars@metafoo.de>; Hennerich, Michael > <Michael.Hennerich@analog.com>; Jonathan Cameron <jic23@kernel.org>; > Hartmut Knaack <knaack.h@gmx.de>; Peter Meerwald-Stadler > <pmeerw@pmeerw.net>; Bogdan, Dragos <Dragos.Bogdan@analog.com>; > Greg Kroah-Hartman <gregkh@linuxfoundation.org>; Ardelean, Alexandru > <alexandru.Ardelean@analog.com> > Subject: Re: [PATCH 10/10] iio: adis: Drop non Managed device functions > > [External] > > On Tue, 15 Sep 2020 11:33:45 +0200 > Nuno Sá <nuno.sa@analog.com> wrote: > > > Drop `adis_setup_buffer_and_trigger()`. All users were updated to use > > the devm version of this function. This avoids having almost the same > > code repeated. > > > > Signed-off-by: Nuno Sá <nuno.sa@analog.com> > > Good to see this cleanup, as long as we tidy up the few issues in the > earlier patches. > > Note I think I only commented on first instance of each thing to fix. > Please carry them through all the patches. > > Thanks, > > Jonathan So, I did thought about further cleaning the probe functions in order to get rid of the remove function but it felt like a different series to me. Anyways, since it's fine with you to include those changes here I will send a v2. - Nuno Sá > > --- > > drivers/iio/imu/adis_buffer.c | 64 +++------------------------------- > > drivers/iio/imu/adis_trigger.c | 60 ------------------------------- > > include/linux/iio/imu/adis.h | 27 -------------- > > 3 files changed, 4 insertions(+), 147 deletions(-) > > > > diff --git a/drivers/iio/imu/adis_buffer.c b/drivers/iio/imu/adis_buffer.c > > index 5b4225ee09b9..df6144285470 100644 > > --- a/drivers/iio/imu/adis_buffer.c > > +++ b/drivers/iio/imu/adis_buffer.c > > @@ -169,48 +169,6 @@ static void adis_buffer_cleanup(void *arg) > > kfree(adis->xfer); > > } > > > > -/** > > - * adis_setup_buffer_and_trigger() - Sets up buffer and trigger for the > adis device > > - * @adis: The adis device. > > - * @indio_dev: The IIO device. > > - * @trigger_handler: Optional trigger handler, may be NULL. > > - * > > - * Returns 0 on success, a negative error code otherwise. > > - * > > - * This function sets up the buffer and trigger for a adis devices. If > > - * 'trigger_handler' is NULL the default trigger handler will be used. The > > - * default trigger handler will simply read the registers assigned to the > > - * currently active channels. > > - * > > - * adis_cleanup_buffer_and_trigger() should be called to free the > resources > > - * allocated by this function. > > - */ > > -int adis_setup_buffer_and_trigger(struct adis *adis, struct iio_dev > *indio_dev, > > - irqreturn_t (*trigger_handler)(int, void *)) > > -{ > > - int ret; > > - > > - if (!trigger_handler) > > - trigger_handler = adis_trigger_handler; > > - > > - ret = iio_triggered_buffer_setup(indio_dev, > &iio_pollfunc_store_time, > > - trigger_handler, NULL); > > - if (ret) > > - return ret; > > - > > - if (adis->spi->irq) { > > - ret = adis_probe_trigger(adis, indio_dev); > > - if (ret) > > - goto error_buffer_cleanup; > > - } > > - return 0; > > - > > -error_buffer_cleanup: > > - iio_triggered_buffer_cleanup(indio_dev); > > - return ret; > > -} > > -EXPORT_SYMBOL_GPL(adis_setup_buffer_and_trigger); > > - > > /** > > * devm_adis_setup_buffer_and_trigger() - Sets up buffer and trigger for > > * the managed adis device > > @@ -220,7 +178,10 @@ > EXPORT_SYMBOL_GPL(adis_setup_buffer_and_trigger); > > * > > * Returns 0 on success, a negative error code otherwise. > > * > > - * This function perfoms exactly the same as > adis_setup_buffer_and_trigger() > > + * This function sets up the buffer and trigger for a adis devices. If > > + * 'trigger_handler' is NULL the default trigger handler will be used. The > > + * default trigger handler will simply read the registers assigned to the > > + * currently active channels. > > */ > > int > > devm_adis_setup_buffer_and_trigger(struct adis *adis, struct iio_dev > *indio_dev, > > @@ -248,20 +209,3 @@ devm_adis_setup_buffer_and_trigger(struct adis > *adis, struct iio_dev *indio_dev, > > } > > EXPORT_SYMBOL_GPL(devm_adis_setup_buffer_and_trigger); > > > > -/** > > - * adis_cleanup_buffer_and_trigger() - Free buffer and trigger resources > > - * @adis: The adis device. > > - * @indio_dev: The IIO device. > > - * > > - * Frees resources allocated by adis_setup_buffer_and_trigger() > > - */ > > -void adis_cleanup_buffer_and_trigger(struct adis *adis, > > - struct iio_dev *indio_dev) > > -{ > > - if (adis->spi->irq) > > - adis_remove_trigger(adis); > > - kfree(adis->buffer); > > - kfree(adis->xfer); > > - iio_triggered_buffer_cleanup(indio_dev); > > -} > > -EXPORT_SYMBOL_GPL(adis_cleanup_buffer_and_trigger); > > diff --git a/drivers/iio/imu/adis_trigger.c b/drivers/iio/imu/adis_trigger.c > > index 8afe71947c00..64e0ba51cb18 100644 > > --- a/drivers/iio/imu/adis_trigger.c > > +++ b/drivers/iio/imu/adis_trigger.c > > @@ -55,53 +55,6 @@ static int adis_validate_irq_flag(struct adis *adis) > > > > return 0; > > } > > -/** > > - * adis_probe_trigger() - Sets up trigger for a adis device > > - * @adis: The adis device > > - * @indio_dev: The IIO device > > - * > > - * Returns 0 on success or a negative error code > > - * > > - * adis_remove_trigger() should be used to free the trigger. > > - */ > > -int adis_probe_trigger(struct adis *adis, struct iio_dev *indio_dev) > > -{ > > - int ret; > > - > > - adis->trig = iio_trigger_alloc("%s-dev%d", indio_dev->name, > > - indio_dev->id); > > - if (adis->trig == NULL) > > - return -ENOMEM; > > - > > - adis_trigger_setup(adis); > > - > > - ret = adis_validate_irq_flag(adis); > > - if (ret) > > - return ret; > > - > > - ret = request_irq(adis->spi->irq, > > - &iio_trigger_generic_data_rdy_poll, > > - adis->irq_flag, > > - indio_dev->name, > > - adis->trig); > > - if (ret) > > - goto error_free_trig; > > - > > - ret = iio_trigger_register(adis->trig); > > - > > - indio_dev->trig = iio_trigger_get(adis->trig); > > - if (ret) > > - goto error_free_irq; > > - > > - return 0; > > - > > -error_free_irq: > > - free_irq(adis->spi->irq, adis->trig); > > -error_free_trig: > > - iio_trigger_free(adis->trig); > > - return ret; > > -} > > -EXPORT_SYMBOL_GPL(adis_probe_trigger); > > > > /** > > * devm_adis_probe_trigger() - Sets up trigger for a managed adis device > > @@ -137,16 +90,3 @@ int devm_adis_probe_trigger(struct adis *adis, > struct iio_dev *indio_dev) > > } > > EXPORT_SYMBOL_GPL(devm_adis_probe_trigger); > > > > -/** > > - * adis_remove_trigger() - Remove trigger for a adis devices > > - * @adis: The adis device > > - * > > - * Removes the trigger previously registered with adis_probe_trigger(). > > - */ > > -void adis_remove_trigger(struct adis *adis) > > -{ > > - iio_trigger_unregister(adis->trig); > > - free_irq(adis->spi->irq, adis->trig); > > - iio_trigger_free(adis->trig); > > -} > > -EXPORT_SYMBOL_GPL(adis_remove_trigger); > > diff --git a/include/linux/iio/imu/adis.h b/include/linux/iio/imu/adis.h > > index 2df67448f0d1..01ba691da2f3 100644 > > --- a/include/linux/iio/imu/adis.h > > +++ b/include/linux/iio/imu/adis.h > > @@ -517,14 +517,8 @@ struct adis_burst { > > int > > devm_adis_setup_buffer_and_trigger(struct adis *adis, struct iio_dev > *indio_dev, > > irq_handler_t trigger_handler); > > -int adis_setup_buffer_and_trigger(struct adis *adis, > > - struct iio_dev *indio_dev, irqreturn_t (*trigger_handler)(int, void > *)); > > -void adis_cleanup_buffer_and_trigger(struct adis *adis, > > - struct iio_dev *indio_dev); > > > > int devm_adis_probe_trigger(struct adis *adis, struct iio_dev *indio_dev); > > -int adis_probe_trigger(struct adis *adis, struct iio_dev *indio_dev); > > -void adis_remove_trigger(struct adis *adis); > > > > int adis_update_scan_mode(struct iio_dev *indio_dev, > > const unsigned long *scan_mask); > > @@ -538,33 +532,12 @@ devm_adis_setup_buffer_and_trigger(struct adis > *adis, struct iio_dev *indio_dev, > > return 0; > > } > > > > -static inline int adis_setup_buffer_and_trigger(struct adis *adis, > > - struct iio_dev *indio_dev, irqreturn_t (*trigger_handler)(int, void *)) > > -{ > > - return 0; > > -} > > - > > -static inline void adis_cleanup_buffer_and_trigger(struct adis *adis, > > - struct iio_dev *indio_dev) > > -{ > > -} > > - > > static inline int devm_adis_probe_trigger(struct adis *adis, > > struct iio_dev *indio_dev) > > { > > return 0; > > } > > > > -static inline int adis_probe_trigger(struct adis *adis, > > - struct iio_dev *indio_dev) > > -{ > > - return 0; > > -} > > - > > -static inline void adis_remove_trigger(struct adis *adis) > > -{ > > -} > > - > > #define adis_update_scan_mode NULL > > > > #endif /* CONFIG_IIO_BUFFER */ > _______________________________________________ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
next prev parent reply other threads:[~2020-09-15 10:20 UTC|newest] Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-09-15 9:33 [PATCH 00/10] Use ADIS devm functions for trigger/buffer setup Nuno Sá 2020-09-15 9:33 ` Nuno Sá 2020-09-15 9:33 ` [PATCH 01/10] iio: adis16201: Use Managed device functions Nuno Sá 2020-09-15 9:33 ` Nuno Sá 2020-09-15 10:05 ` Jonathan Cameron 2020-09-15 10:05 ` Jonathan Cameron 2020-09-15 9:33 ` [PATCH 02/10] iio: adis16209: " Nuno Sá 2020-09-15 9:33 ` Nuno Sá 2020-09-15 9:33 ` [PATCH 03/10] iio: adis16136: " Nuno Sá 2020-09-15 9:33 ` Nuno Sá 2020-09-15 10:06 ` Jonathan Cameron 2020-09-15 10:06 ` Jonathan Cameron 2020-09-15 9:33 ` [PATCH 04/10] iio: adis16260: " Nuno Sá 2020-09-15 9:33 ` Nuno Sá 2020-09-15 9:33 ` [PATCH 05/10] iio: adis16400: " Nuno Sá 2020-09-15 9:33 ` Nuno Sá 2020-09-15 9:33 ` [PATCH 06/10] iio: adis16460: " Nuno Sá 2020-09-15 9:33 ` Nuno Sá 2020-09-15 9:33 ` [PATCH 07/10] iio: adis16480: " Nuno Sá 2020-09-15 9:33 ` Nuno Sá 2020-09-15 10:07 ` Jonathan Cameron 2020-09-15 10:07 ` Jonathan Cameron 2020-09-15 9:33 ` [PATCH 08/10] staging: iio: adis16203: " Nuno Sá 2020-09-15 9:33 ` Nuno Sá 2020-09-15 9:33 ` [PATCH 09/10] staging: iio: adis16240: " Nuno Sá 2020-09-15 9:33 ` Nuno Sá 2020-09-15 9:33 ` [PATCH 10/10] iio: adis: Drop non " Nuno Sá 2020-09-15 9:33 ` Nuno Sá 2020-09-15 10:10 ` Jonathan Cameron 2020-09-15 10:10 ` Jonathan Cameron 2020-09-15 10:20 ` Sa, Nuno [this message] 2020-09-15 10:20 ` Sa, Nuno 2020-09-15 10:26 ` Jonathan Cameron 2020-09-15 10:26 ` Jonathan Cameron
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=CY4PR03MB2631DD532FE132207FCE3E9799200@CY4PR03MB2631.namprd03.prod.outlook.com \ --to=nuno.sa@analog.com \ --cc=Dragos.Bogdan@analog.com \ --cc=Jonathan.Cameron@Huawei.com \ --cc=Michael.Hennerich@analog.com \ --cc=alexandru.Ardelean@analog.com \ --cc=devel@driverdev.osuosl.org \ --cc=gregkh@linuxfoundation.org \ --cc=jic23@kernel.org \ --cc=knaack.h@gmx.de \ --cc=lars@metafoo.de \ --cc=linux-iio@vger.kernel.org \ --cc=pmeerw@pmeerw.net \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.