* [PATCH 1/2] staging:iio:ad7606: Consolidate PM ops
@ 2016-02-05 10:32 Lars-Peter Clausen
2016-02-05 10:32 ` [PATCH 2/2] staging:iio:ad7606: Consolidate channel specs Lars-Peter Clausen
2016-02-06 11:22 ` [PATCH 1/2] staging:iio:ad7606: Consolidate PM ops Jonathan Cameron
0 siblings, 2 replies; 6+ messages in thread
From: Lars-Peter Clausen @ 2016-02-05 10:32 UTC (permalink / raw)
To: Jonathan Cameron
Cc: Hartmut Knaack, Peter Meerwald, linux-iio, Lars-Peter Clausen
Both the SPI and platform device driver for the ad7606 use the same set of
PM ops. Consolidate them in the common part of the driver.
Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
---
drivers/staging/iio/adc/ad7606.h | 10 ++++++++--
drivers/staging/iio/adc/ad7606_core.c | 16 ++++++++++++++--
drivers/staging/iio/adc/ad7606_par.c | 32 +-------------------------------
drivers/staging/iio/adc/ad7606_spi.c | 32 +-------------------------------
4 files changed, 24 insertions(+), 66 deletions(-)
diff --git a/drivers/staging/iio/adc/ad7606.h b/drivers/staging/iio/adc/ad7606.h
index ec89d05..cca9469 100644
--- a/drivers/staging/iio/adc/ad7606.h
+++ b/drivers/staging/iio/adc/ad7606.h
@@ -85,8 +85,6 @@ struct ad7606_bus_ops {
int (*read_block)(struct device *, int, void *);
};
-void ad7606_suspend(struct iio_dev *indio_dev);
-void ad7606_resume(struct iio_dev *indio_dev);
struct iio_dev *ad7606_probe(struct device *dev, int irq,
void __iomem *base_address, unsigned id,
const struct ad7606_bus_ops *bops);
@@ -101,4 +99,12 @@ enum ad7606_supported_device_ids {
int ad7606_register_ring_funcs_and_init(struct iio_dev *indio_dev);
void ad7606_ring_cleanup(struct iio_dev *indio_dev);
+
+#ifdef CONFIG_PM_SLEEP
+extern const struct dev_pm_ops ad7606_pm_ops;
+#define AD7606_PM_OPS (&ad7606_pm_ops)
+#else
+#define AD7606_PM_OPS NULL
+#endif
+
#endif /* IIO_ADC_AD7606_H_ */
diff --git a/drivers/staging/iio/adc/ad7606_core.c b/drivers/staging/iio/adc/ad7606_core.c
index 5796ed2..d0b634d 100644
--- a/drivers/staging/iio/adc/ad7606_core.c
+++ b/drivers/staging/iio/adc/ad7606_core.c
@@ -576,8 +576,11 @@ int ad7606_remove(struct iio_dev *indio_dev, int irq)
return 0;
}
-void ad7606_suspend(struct iio_dev *indio_dev)
+#ifdef CONFIG_PM_SLEEP
+
+static int ad7606_suspend(struct device *dev)
{
+ struct iio_dev *indio_dev = dev_get_drvdata(dev);
struct ad7606_state *st = iio_priv(indio_dev);
if (gpio_is_valid(st->pdata->gpio_stby)) {
@@ -585,10 +588,13 @@ void ad7606_suspend(struct iio_dev *indio_dev)
gpio_set_value(st->pdata->gpio_range, 1);
gpio_set_value(st->pdata->gpio_stby, 0);
}
+
+ return 0;
}
-void ad7606_resume(struct iio_dev *indio_dev)
+static int ad7606_resume(struct device *dev)
{
+ struct iio_dev *indio_dev = dev_get_drvdata(dev);
struct ad7606_state *st = iio_priv(indio_dev);
if (gpio_is_valid(st->pdata->gpio_stby)) {
@@ -599,8 +605,14 @@ void ad7606_resume(struct iio_dev *indio_dev)
gpio_set_value(st->pdata->gpio_stby, 1);
ad7606_reset(st);
}
+
+ return 0;
}
+SIMPLE_DEV_PM_OPS(ad7606_pm_ops, ad7606_suspend, ad7606_resume);
+
+#endif
+
MODULE_AUTHOR("Michael Hennerich <hennerich@blackfin.uclinux.org>");
MODULE_DESCRIPTION("Analog Devices AD7606 ADC");
MODULE_LICENSE("GPL v2");
diff --git a/drivers/staging/iio/adc/ad7606_par.c b/drivers/staging/iio/adc/ad7606_par.c
index adc370e..84d2393 100644
--- a/drivers/staging/iio/adc/ad7606_par.c
+++ b/drivers/staging/iio/adc/ad7606_par.c
@@ -90,36 +90,6 @@ static int ad7606_par_remove(struct platform_device *pdev)
return 0;
}
-#ifdef CONFIG_PM
-static int ad7606_par_suspend(struct device *dev)
-{
- struct iio_dev *indio_dev = dev_get_drvdata(dev);
-
- ad7606_suspend(indio_dev);
-
- return 0;
-}
-
-static int ad7606_par_resume(struct device *dev)
-{
- struct iio_dev *indio_dev = dev_get_drvdata(dev);
-
- ad7606_resume(indio_dev);
-
- return 0;
-}
-
-static const struct dev_pm_ops ad7606_pm_ops = {
- .suspend = ad7606_par_suspend,
- .resume = ad7606_par_resume,
-};
-
-#define AD7606_PAR_PM_OPS (&ad7606_pm_ops)
-
-#else
-#define AD7606_PAR_PM_OPS NULL
-#endif /* CONFIG_PM */
-
static const struct platform_device_id ad7606_driver_ids[] = {
{
.name = "ad7606-8",
@@ -142,7 +112,7 @@ static struct platform_driver ad7606_driver = {
.id_table = ad7606_driver_ids,
.driver = {
.name = "ad7606",
- .pm = AD7606_PAR_PM_OPS,
+ .pm = AD7606_PM_OPS,
},
};
diff --git a/drivers/staging/iio/adc/ad7606_spi.c b/drivers/staging/iio/adc/ad7606_spi.c
index cbb3631..d873a51 100644
--- a/drivers/staging/iio/adc/ad7606_spi.c
+++ b/drivers/staging/iio/adc/ad7606_spi.c
@@ -62,36 +62,6 @@ static int ad7606_spi_remove(struct spi_device *spi)
return ad7606_remove(indio_dev, spi->irq);
}
-#ifdef CONFIG_PM
-static int ad7606_spi_suspend(struct device *dev)
-{
- struct iio_dev *indio_dev = dev_get_drvdata(dev);
-
- ad7606_suspend(indio_dev);
-
- return 0;
-}
-
-static int ad7606_spi_resume(struct device *dev)
-{
- struct iio_dev *indio_dev = dev_get_drvdata(dev);
-
- ad7606_resume(indio_dev);
-
- return 0;
-}
-
-static const struct dev_pm_ops ad7606_pm_ops = {
- .suspend = ad7606_spi_suspend,
- .resume = ad7606_spi_resume,
-};
-
-#define AD7606_SPI_PM_OPS (&ad7606_pm_ops)
-
-#else
-#define AD7606_SPI_PM_OPS NULL
-#endif
-
static const struct spi_device_id ad7606_id[] = {
{"ad7606-8", ID_AD7606_8},
{"ad7606-6", ID_AD7606_6},
@@ -103,7 +73,7 @@ MODULE_DEVICE_TABLE(spi, ad7606_id);
static struct spi_driver ad7606_driver = {
.driver = {
.name = "ad7606",
- .pm = AD7606_SPI_PM_OPS,
+ .pm = AD7606_PM_OPS,
},
.probe = ad7606_spi_probe,
.remove = ad7606_spi_remove,
--
2.1.4
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH 2/2] staging:iio:ad7606: Consolidate channel specs
2016-02-05 10:32 [PATCH 1/2] staging:iio:ad7606: Consolidate PM ops Lars-Peter Clausen
@ 2016-02-05 10:32 ` Lars-Peter Clausen
2016-02-06 11:24 ` Jonathan Cameron
2016-02-06 11:22 ` [PATCH 1/2] staging:iio:ad7606: Consolidate PM ops Jonathan Cameron
1 sibling, 1 reply; 6+ messages in thread
From: Lars-Peter Clausen @ 2016-02-05 10:32 UTC (permalink / raw)
To: Jonathan Cameron
Cc: Hartmut Knaack, Peter Meerwald, linux-iio, Lars-Peter Clausen
By slightly reordering the channels in the channel spec array we can reuse
the same array for all variant of the chip.
Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
---
drivers/staging/iio/adc/ad7606_core.c | 34 ++++++++--------------------------
1 file changed, 8 insertions(+), 26 deletions(-)
diff --git a/drivers/staging/iio/adc/ad7606_core.c b/drivers/staging/iio/adc/ad7606_core.c
index d0b634d..ce9b2c3 100644
--- a/drivers/staging/iio/adc/ad7606_core.c
+++ b/drivers/staging/iio/adc/ad7606_core.c
@@ -250,7 +250,8 @@ static const struct attribute_group ad7606_attribute_group_range = {
}, \
}
-static const struct iio_chan_spec ad7606_8_channels[] = {
+static const struct iio_chan_spec ad7606_channels[] = {
+ IIO_CHAN_SOFT_TIMESTAMP(8),
AD7606_CHANNEL(0),
AD7606_CHANNEL(1),
AD7606_CHANNEL(2),
@@ -259,25 +260,6 @@ static const struct iio_chan_spec ad7606_8_channels[] = {
AD7606_CHANNEL(5),
AD7606_CHANNEL(6),
AD7606_CHANNEL(7),
- IIO_CHAN_SOFT_TIMESTAMP(8),
-};
-
-static const struct iio_chan_spec ad7606_6_channels[] = {
- AD7606_CHANNEL(0),
- AD7606_CHANNEL(1),
- AD7606_CHANNEL(2),
- AD7606_CHANNEL(3),
- AD7606_CHANNEL(4),
- AD7606_CHANNEL(5),
- IIO_CHAN_SOFT_TIMESTAMP(6),
-};
-
-static const struct iio_chan_spec ad7606_4_channels[] = {
- AD7606_CHANNEL(0),
- AD7606_CHANNEL(1),
- AD7606_CHANNEL(2),
- AD7606_CHANNEL(3),
- IIO_CHAN_SOFT_TIMESTAMP(4),
};
static const struct ad7606_chip_info ad7606_chip_info_tbl[] = {
@@ -287,20 +269,20 @@ static const struct ad7606_chip_info ad7606_chip_info_tbl[] = {
[ID_AD7606_8] = {
.name = "ad7606",
.int_vref_mv = 2500,
- .channels = ad7606_8_channels,
- .num_channels = 8,
+ .channels = ad7606_channels,
+ .num_channels = 9,
},
[ID_AD7606_6] = {
.name = "ad7606-6",
.int_vref_mv = 2500,
- .channels = ad7606_6_channels,
- .num_channels = 6,
+ .channels = ad7606_channels,
+ .num_channels = 7,
},
[ID_AD7606_4] = {
.name = "ad7606-4",
.int_vref_mv = 2500,
- .channels = ad7606_4_channels,
- .num_channels = 4,
+ .channels = ad7606_channels,
+ .num_channels = 5,
},
};
--
2.1.4
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH 2/2] staging:iio:ad7606: Consolidate channel specs
2016-02-05 10:32 ` [PATCH 2/2] staging:iio:ad7606: Consolidate channel specs Lars-Peter Clausen
@ 2016-02-06 11:24 ` Jonathan Cameron
0 siblings, 0 replies; 6+ messages in thread
From: Jonathan Cameron @ 2016-02-06 11:24 UTC (permalink / raw)
To: Lars-Peter Clausen; +Cc: Hartmut Knaack, Peter Meerwald, linux-iio
On 05/02/16 10:32, Lars-Peter Clausen wrote:
> By slightly reordering the channels in the channel spec array we can reuse
> the same array for all variant of the chip.
>
> Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Fair enough I suppose, though I find it hard to care much about this sort
of consolidation as it trades slight readability disadvantage off against
reduced code size.
Applied to the togreg branch of iio.git.
Thanks,
Jonathan
> ---
> drivers/staging/iio/adc/ad7606_core.c | 34 ++++++++--------------------------
> 1 file changed, 8 insertions(+), 26 deletions(-)
>
> diff --git a/drivers/staging/iio/adc/ad7606_core.c b/drivers/staging/iio/adc/ad7606_core.c
> index d0b634d..ce9b2c3 100644
> --- a/drivers/staging/iio/adc/ad7606_core.c
> +++ b/drivers/staging/iio/adc/ad7606_core.c
> @@ -250,7 +250,8 @@ static const struct attribute_group ad7606_attribute_group_range = {
> }, \
> }
>
> -static const struct iio_chan_spec ad7606_8_channels[] = {
> +static const struct iio_chan_spec ad7606_channels[] = {
> + IIO_CHAN_SOFT_TIMESTAMP(8),
> AD7606_CHANNEL(0),
> AD7606_CHANNEL(1),
> AD7606_CHANNEL(2),
> @@ -259,25 +260,6 @@ static const struct iio_chan_spec ad7606_8_channels[] = {
> AD7606_CHANNEL(5),
> AD7606_CHANNEL(6),
> AD7606_CHANNEL(7),
> - IIO_CHAN_SOFT_TIMESTAMP(8),
> -};
> -
> -static const struct iio_chan_spec ad7606_6_channels[] = {
> - AD7606_CHANNEL(0),
> - AD7606_CHANNEL(1),
> - AD7606_CHANNEL(2),
> - AD7606_CHANNEL(3),
> - AD7606_CHANNEL(4),
> - AD7606_CHANNEL(5),
> - IIO_CHAN_SOFT_TIMESTAMP(6),
> -};
> -
> -static const struct iio_chan_spec ad7606_4_channels[] = {
> - AD7606_CHANNEL(0),
> - AD7606_CHANNEL(1),
> - AD7606_CHANNEL(2),
> - AD7606_CHANNEL(3),
> - IIO_CHAN_SOFT_TIMESTAMP(4),
> };
>
> static const struct ad7606_chip_info ad7606_chip_info_tbl[] = {
> @@ -287,20 +269,20 @@ static const struct ad7606_chip_info ad7606_chip_info_tbl[] = {
> [ID_AD7606_8] = {
> .name = "ad7606",
> .int_vref_mv = 2500,
> - .channels = ad7606_8_channels,
> - .num_channels = 8,
> + .channels = ad7606_channels,
> + .num_channels = 9,
> },
> [ID_AD7606_6] = {
> .name = "ad7606-6",
> .int_vref_mv = 2500,
> - .channels = ad7606_6_channels,
> - .num_channels = 6,
> + .channels = ad7606_channels,
> + .num_channels = 7,
> },
> [ID_AD7606_4] = {
> .name = "ad7606-4",
> .int_vref_mv = 2500,
> - .channels = ad7606_4_channels,
> - .num_channels = 4,
> + .channels = ad7606_channels,
> + .num_channels = 5,
> },
> };
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] staging:iio:ad7606: Consolidate PM ops
2016-02-05 10:32 [PATCH 1/2] staging:iio:ad7606: Consolidate PM ops Lars-Peter Clausen
2016-02-05 10:32 ` [PATCH 2/2] staging:iio:ad7606: Consolidate channel specs Lars-Peter Clausen
@ 2016-02-06 11:22 ` Jonathan Cameron
2016-02-06 11:28 ` Lars-Peter Clausen
1 sibling, 1 reply; 6+ messages in thread
From: Jonathan Cameron @ 2016-02-06 11:22 UTC (permalink / raw)
To: Lars-Peter Clausen; +Cc: Hartmut Knaack, Peter Meerwald, linux-iio
On 05/02/16 10:32, Lars-Peter Clausen wrote:
> Both the SPI and platform device driver for the ad7606 use the same set of
> PM ops. Consolidate them in the common part of the driver.
>
> Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Lars, this doesn't apply to the togreg branch.
Looks like you are locally missing the patch:
0046a46a8f93f5b99b499118779521ba3b017d69
staging/ad7606: Actually build the interface modules
Which was fixing some module building issues back in October last year.
Jonathan
> ---
> drivers/staging/iio/adc/ad7606.h | 10 ++++++++--
> drivers/staging/iio/adc/ad7606_core.c | 16 ++++++++++++++--
> drivers/staging/iio/adc/ad7606_par.c | 32 +-------------------------------
> drivers/staging/iio/adc/ad7606_spi.c | 32 +-------------------------------
> 4 files changed, 24 insertions(+), 66 deletions(-)
>
> diff --git a/drivers/staging/iio/adc/ad7606.h b/drivers/staging/iio/adc/ad7606.h
> index ec89d05..cca9469 100644
> --- a/drivers/staging/iio/adc/ad7606.h
> +++ b/drivers/staging/iio/adc/ad7606.h
> @@ -85,8 +85,6 @@ struct ad7606_bus_ops {
> int (*read_block)(struct device *, int, void *);
> };
>
> -void ad7606_suspend(struct iio_dev *indio_dev);
> -void ad7606_resume(struct iio_dev *indio_dev);
> struct iio_dev *ad7606_probe(struct device *dev, int irq,
> void __iomem *base_address, unsigned id,
> const struct ad7606_bus_ops *bops);
> @@ -101,4 +99,12 @@ enum ad7606_supported_device_ids {
>
> int ad7606_register_ring_funcs_and_init(struct iio_dev *indio_dev);
> void ad7606_ring_cleanup(struct iio_dev *indio_dev);
> +
> +#ifdef CONFIG_PM_SLEEP
> +extern const struct dev_pm_ops ad7606_pm_ops;
> +#define AD7606_PM_OPS (&ad7606_pm_ops)
> +#else
> +#define AD7606_PM_OPS NULL
> +#endif
> +
> #endif /* IIO_ADC_AD7606_H_ */
> diff --git a/drivers/staging/iio/adc/ad7606_core.c b/drivers/staging/iio/adc/ad7606_core.c
> index 5796ed2..d0b634d 100644
> --- a/drivers/staging/iio/adc/ad7606_core.c
> +++ b/drivers/staging/iio/adc/ad7606_core.c
> @@ -576,8 +576,11 @@ int ad7606_remove(struct iio_dev *indio_dev, int irq)
> return 0;
> }
>
> -void ad7606_suspend(struct iio_dev *indio_dev)
> +#ifdef CONFIG_PM_SLEEP
> +
> +static int ad7606_suspend(struct device *dev)
> {
> + struct iio_dev *indio_dev = dev_get_drvdata(dev);
> struct ad7606_state *st = iio_priv(indio_dev);
>
> if (gpio_is_valid(st->pdata->gpio_stby)) {
> @@ -585,10 +588,13 @@ void ad7606_suspend(struct iio_dev *indio_dev)
> gpio_set_value(st->pdata->gpio_range, 1);
> gpio_set_value(st->pdata->gpio_stby, 0);
> }
> +
> + return 0;
> }
>
> -void ad7606_resume(struct iio_dev *indio_dev)
> +static int ad7606_resume(struct device *dev)
> {
> + struct iio_dev *indio_dev = dev_get_drvdata(dev);
> struct ad7606_state *st = iio_priv(indio_dev);
>
> if (gpio_is_valid(st->pdata->gpio_stby)) {
> @@ -599,8 +605,14 @@ void ad7606_resume(struct iio_dev *indio_dev)
> gpio_set_value(st->pdata->gpio_stby, 1);
> ad7606_reset(st);
> }
> +
> + return 0;
> }
>
> +SIMPLE_DEV_PM_OPS(ad7606_pm_ops, ad7606_suspend, ad7606_resume);
> +
> +#endif
> +
> MODULE_AUTHOR("Michael Hennerich <hennerich@blackfin.uclinux.org>");
> MODULE_DESCRIPTION("Analog Devices AD7606 ADC");
> MODULE_LICENSE("GPL v2");
> diff --git a/drivers/staging/iio/adc/ad7606_par.c b/drivers/staging/iio/adc/ad7606_par.c
> index adc370e..84d2393 100644
> --- a/drivers/staging/iio/adc/ad7606_par.c
> +++ b/drivers/staging/iio/adc/ad7606_par.c
> @@ -90,36 +90,6 @@ static int ad7606_par_remove(struct platform_device *pdev)
> return 0;
> }
>
> -#ifdef CONFIG_PM
> -static int ad7606_par_suspend(struct device *dev)
> -{
> - struct iio_dev *indio_dev = dev_get_drvdata(dev);
> -
> - ad7606_suspend(indio_dev);
> -
> - return 0;
> -}
> -
> -static int ad7606_par_resume(struct device *dev)
> -{
> - struct iio_dev *indio_dev = dev_get_drvdata(dev);
> -
> - ad7606_resume(indio_dev);
> -
> - return 0;
> -}
> -
> -static const struct dev_pm_ops ad7606_pm_ops = {
> - .suspend = ad7606_par_suspend,
> - .resume = ad7606_par_resume,
> -};
> -
> -#define AD7606_PAR_PM_OPS (&ad7606_pm_ops)
> -
> -#else
> -#define AD7606_PAR_PM_OPS NULL
> -#endif /* CONFIG_PM */
> -
> static const struct platform_device_id ad7606_driver_ids[] = {
> {
> .name = "ad7606-8",
> @@ -142,7 +112,7 @@ static struct platform_driver ad7606_driver = {
> .id_table = ad7606_driver_ids,
> .driver = {
> .name = "ad7606",
> - .pm = AD7606_PAR_PM_OPS,
> + .pm = AD7606_PM_OPS,
> },
> };
>
> diff --git a/drivers/staging/iio/adc/ad7606_spi.c b/drivers/staging/iio/adc/ad7606_spi.c
> index cbb3631..d873a51 100644
> --- a/drivers/staging/iio/adc/ad7606_spi.c
> +++ b/drivers/staging/iio/adc/ad7606_spi.c
> @@ -62,36 +62,6 @@ static int ad7606_spi_remove(struct spi_device *spi)
> return ad7606_remove(indio_dev, spi->irq);
> }
>
> -#ifdef CONFIG_PM
> -static int ad7606_spi_suspend(struct device *dev)
> -{
> - struct iio_dev *indio_dev = dev_get_drvdata(dev);
> -
> - ad7606_suspend(indio_dev);
> -
> - return 0;
> -}
> -
> -static int ad7606_spi_resume(struct device *dev)
> -{
> - struct iio_dev *indio_dev = dev_get_drvdata(dev);
> -
> - ad7606_resume(indio_dev);
> -
> - return 0;
> -}
> -
> -static const struct dev_pm_ops ad7606_pm_ops = {
> - .suspend = ad7606_spi_suspend,
> - .resume = ad7606_spi_resume,
> -};
> -
> -#define AD7606_SPI_PM_OPS (&ad7606_pm_ops)
> -
> -#else
> -#define AD7606_SPI_PM_OPS NULL
> -#endif
> -
> static const struct spi_device_id ad7606_id[] = {
> {"ad7606-8", ID_AD7606_8},
> {"ad7606-6", ID_AD7606_6},
> @@ -103,7 +73,7 @@ MODULE_DEVICE_TABLE(spi, ad7606_id);
> static struct spi_driver ad7606_driver = {
> .driver = {
> .name = "ad7606",
> - .pm = AD7606_SPI_PM_OPS,
> + .pm = AD7606_PM_OPS,
> },
> .probe = ad7606_spi_probe,
> .remove = ad7606_spi_remove,
>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH 1/2] staging:iio:ad7606: Consolidate PM ops
2016-02-06 11:22 ` [PATCH 1/2] staging:iio:ad7606: Consolidate PM ops Jonathan Cameron
@ 2016-02-06 11:28 ` Lars-Peter Clausen
2016-02-06 17:54 ` Jonathan Cameron
0 siblings, 1 reply; 6+ messages in thread
From: Lars-Peter Clausen @ 2016-02-06 11:28 UTC (permalink / raw)
To: Jonathan Cameron; +Cc: Hartmut Knaack, Peter Meerwald, linux-iio
On 02/06/2016 12:22 PM, Jonathan Cameron wrote:
> On 05/02/16 10:32, Lars-Peter Clausen wrote:
>> Both the SPI and platform device driver for the ad7606 use the same set of
>> PM ops. Consolidate them in the common part of the driver.
>>
>> Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
> Lars, this doesn't apply to the togreg branch.
>
> Looks like you are locally missing the patch:
>
> 0046a46a8f93f5b99b499118779521ba3b017d69
> staging/ad7606: Actually build the interface modules
Which is not yet in the public togreg branch. I'll rebase and resend next
week when the public branch has been updated.
- Lars
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] staging:iio:ad7606: Consolidate PM ops
2016-02-06 11:28 ` Lars-Peter Clausen
@ 2016-02-06 17:54 ` Jonathan Cameron
0 siblings, 0 replies; 6+ messages in thread
From: Jonathan Cameron @ 2016-02-06 17:54 UTC (permalink / raw)
To: Lars-Peter Clausen; +Cc: Hartmut Knaack, Peter Meerwald, linux-iio
On 06/02/16 11:28, Lars-Peter Clausen wrote:
> On 02/06/2016 12:22 PM, Jonathan Cameron wrote:
>> On 05/02/16 10:32, Lars-Peter Clausen wrote:
>>> Both the SPI and platform device driver for the ad7606 use the same set of
>>> PM ops. Consolidate them in the common part of the driver.
>>>
>>> Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
>> Lars, this doesn't apply to the togreg branch.
>>
>> Looks like you are locally missing the patch:
>>
>> 0046a46a8f93f5b99b499118779521ba3b017d69
>> staging/ad7606: Actually build the interface modules
>
> Which is not yet in the public togreg branch. I'll rebase and resend next
> week when the public branch has been updated.
Odd... Pull request went out last weekend and hence even if I'd accidentally
kept it in testing until then it should have been in togreg last week.
Ah well, definitely is now as it's made it through to staging-next (and
Linux next). oops!
Jonathan
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-02-06 17:54 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-02-05 10:32 [PATCH 1/2] staging:iio:ad7606: Consolidate PM ops Lars-Peter Clausen
2016-02-05 10:32 ` [PATCH 2/2] staging:iio:ad7606: Consolidate channel specs Lars-Peter Clausen
2016-02-06 11:24 ` Jonathan Cameron
2016-02-06 11:22 ` [PATCH 1/2] staging:iio:ad7606: Consolidate PM ops Jonathan Cameron
2016-02-06 11:28 ` Lars-Peter Clausen
2016-02-06 17:54 ` Jonathan Cameron
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.