Linux IIO development
 help / color / mirror / Atom feed
* [PATCH 0/3] IIO break info_mask into a pair of masks - Additional drivers 2 (missed ones)
@ 2013-03-04 21:16 Jonathan Cameron
  2013-03-04 21:16 ` [PATCH 1/3] staging:iio:adc:spear move to info_mask_(shared_by_type/separate) Jonathan Cameron
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Jonathan Cameron @ 2013-03-04 21:16 UTC (permalink / raw)
  To: linux-iio; +Cc: Jonathan Cameron

As Lars-Peter noticed, I missed these 3 drivers for some reason in the earlier
two series of patches.  Oops.  Have now deployed grep more carefully/thoroughly.

These will shortly appear at an appropriate spot in the info_mask_split branch
of iio.git.

This short series applies the equivalent changes to three drivers missed
from the series in

[PATCH 00/67 V2] IIO break info_mask into a pair of masks
was under way.

If there are no issues, I'll slot these in just before patch 67 of that
series.

As a quick summary for anyone just looking at these, I am proposing to
split info_mask into info_mask_separate and info_mask_shared_by_type
so as to simplify the handling, shorten the code and make more space
for new elements.  With the benefit of hindsight info_mask should
have been done like this in the first place.

Jonathan


Jonathan Cameron (3):
  staging:iio:adc:spear move to info_mask_(shared_by_type/separate)
  staging:iio:adc:mxs move to info_mask_(shared_by_type/separate)
  staging:iio:adc:lpc32xx move to info_mask_(shared_by_type/separate)

 drivers/staging/iio/adc/lpc32xx_adc.c | 2 +-
 drivers/staging/iio/adc/mxs-lradc.c   | 2 +-
 drivers/staging/iio/adc/spear_adc.c   | 4 ++--
 3 files changed, 4 insertions(+), 4 deletions(-)

-- 
1.8.1.4


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 1/3] staging:iio:adc:spear move to info_mask_(shared_by_type/separate)
  2013-03-04 21:16 [PATCH 0/3] IIO break info_mask into a pair of masks - Additional drivers 2 (missed ones) Jonathan Cameron
@ 2013-03-04 21:16 ` Jonathan Cameron
  2013-03-05  7:28   ` Stefan Roese
  2013-03-04 21:16 ` [PATCH 2/3] staging:iio:adc:mxs " Jonathan Cameron
  2013-03-04 21:16 ` [PATCH 3/3] staging:iio:adc:lpc32xx " Jonathan Cameron
  2 siblings, 1 reply; 6+ messages in thread
From: Jonathan Cameron @ 2013-03-04 21:16 UTC (permalink / raw)
  To: linux-iio; +Cc: Jonathan Cameron, Stefan Roese

The original info_mask is going away in favour of the broken out versions.

Signed-off-by: Jonathan Cameron <jic23@kernel.org>
cc: Stefan Roese <sr@denx.de>
---
 drivers/staging/iio/adc/spear_adc.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/iio/adc/spear_adc.c b/drivers/staging/iio/adc/spear_adc.c
index 13052ce..f45da42 100644
--- a/drivers/staging/iio/adc/spear_adc.c
+++ b/drivers/staging/iio/adc/spear_adc.c
@@ -180,8 +180,8 @@ static int spear_read_raw(struct iio_dev *indio_dev,
 #define SPEAR_ADC_CHAN(idx) {				\
 	.type = IIO_VOLTAGE,				\
 	.indexed = 1,					\
-	.info_mask = IIO_CHAN_INFO_RAW_SEPARATE_BIT |	\
-	IIO_CHAN_INFO_SCALE_SHARED_BIT,			\
+	.info_mask_separate = BIT(IIO_CHAN_INFO_RAW),	\
+	.info_mask_shared_by_type = BIT(IIO_CHAN_INFO_SCALE),	\
 	.channel = idx,					\
 	.scan_type = {					\
 		.sign = 'u',				\
-- 
1.8.1.4


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH 2/3] staging:iio:adc:mxs move to info_mask_(shared_by_type/separate)
  2013-03-04 21:16 [PATCH 0/3] IIO break info_mask into a pair of masks - Additional drivers 2 (missed ones) Jonathan Cameron
  2013-03-04 21:16 ` [PATCH 1/3] staging:iio:adc:spear move to info_mask_(shared_by_type/separate) Jonathan Cameron
@ 2013-03-04 21:16 ` Jonathan Cameron
  2013-03-04 21:23   ` Marek Vasut
  2013-03-04 21:16 ` [PATCH 3/3] staging:iio:adc:lpc32xx " Jonathan Cameron
  2 siblings, 1 reply; 6+ messages in thread
From: Jonathan Cameron @ 2013-03-04 21:16 UTC (permalink / raw)
  To: linux-iio; +Cc: Jonathan Cameron, Marek Vasut

The original info_mask is going away in favour of the broken out versions.

Signed-off-by: Jonathan Cameron <jic23@kernel.org>
cc: Marek Vasut <marex@denx.de>
---
 drivers/staging/iio/adc/mxs-lradc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/iio/adc/mxs-lradc.c b/drivers/staging/iio/adc/mxs-lradc.c
index f5fb0df..4ef3070 100644
--- a/drivers/staging/iio/adc/mxs-lradc.c
+++ b/drivers/staging/iio/adc/mxs-lradc.c
@@ -821,7 +821,7 @@ static const struct iio_buffer_setup_ops mxs_lradc_buffer_ops = {
 	.type = (chan_type),					\
 	.indexed = 1,						\
 	.scan_index = (idx),					\
-	.info_mask = IIO_CHAN_INFO_RAW_SEPARATE_BIT,		\
+	.info_mask_separate = BIT(IIO_CHAN_INFO_RAW),		\
 	.channel = (idx),					\
 	.scan_type = {						\
 		.sign = 'u',					\
-- 
1.8.1.4

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH 3/3] staging:iio:adc:lpc32xx move to info_mask_(shared_by_type/separate)
  2013-03-04 21:16 [PATCH 0/3] IIO break info_mask into a pair of masks - Additional drivers 2 (missed ones) Jonathan Cameron
  2013-03-04 21:16 ` [PATCH 1/3] staging:iio:adc:spear move to info_mask_(shared_by_type/separate) Jonathan Cameron
  2013-03-04 21:16 ` [PATCH 2/3] staging:iio:adc:mxs " Jonathan Cameron
@ 2013-03-04 21:16 ` Jonathan Cameron
  2 siblings, 0 replies; 6+ messages in thread
From: Jonathan Cameron @ 2013-03-04 21:16 UTC (permalink / raw)
  To: linux-iio; +Cc: Jonathan Cameron, Roland Stigge

The original info_mask is going away in favour of the broken out versions.

Signed-off-by: Jonathan Cameron <jic23@kernel.org>
cc: Roland Stigge <stigge@antcom.de>
---
 drivers/staging/iio/adc/lpc32xx_adc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/iio/adc/lpc32xx_adc.c b/drivers/staging/iio/adc/lpc32xx_adc.c
index 0bf2a6c..2f2f7fd 100644
--- a/drivers/staging/iio/adc/lpc32xx_adc.c
+++ b/drivers/staging/iio/adc/lpc32xx_adc.c
@@ -103,7 +103,7 @@ static const struct iio_info lpc32xx_adc_iio_info = {
 	.type = IIO_VOLTAGE,				\
 	.indexed = 1,					\
 	.channel = _index,				\
-	.info_mask = IIO_CHAN_INFO_RAW_SEPARATE_BIT,	\
+	.info_mask_separate = BIT(IIO_CHAN_INFO_RAW),	\
 	.address = AD_IN * _index,			\
 	.scan_index = _index,				\
 }
-- 
1.8.1.4


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH 2/3] staging:iio:adc:mxs move to info_mask_(shared_by_type/separate)
  2013-03-04 21:16 ` [PATCH 2/3] staging:iio:adc:mxs " Jonathan Cameron
@ 2013-03-04 21:23   ` Marek Vasut
  0 siblings, 0 replies; 6+ messages in thread
From: Marek Vasut @ 2013-03-04 21:23 UTC (permalink / raw)
  To: Jonathan Cameron; +Cc: linux-iio

Dear Jonathan Cameron,

> The original info_mask is going away in favour of the broken out versions.
> 
> Signed-off-by: Jonathan Cameron <jic23@kernel.org>
> cc: Marek Vasut <marex@denx.de>
> ---
>  drivers/staging/iio/adc/mxs-lradc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/staging/iio/adc/mxs-lradc.c
> b/drivers/staging/iio/adc/mxs-lradc.c index f5fb0df..4ef3070 100644
> --- a/drivers/staging/iio/adc/mxs-lradc.c
> +++ b/drivers/staging/iio/adc/mxs-lradc.c
> @@ -821,7 +821,7 @@ static const struct iio_buffer_setup_ops
> mxs_lradc_buffer_ops = { .type = (chan_type),					
\
>  	.indexed = 1,						\
>  	.scan_index = (idx),					\
> -	.info_mask = IIO_CHAN_INFO_RAW_SEPARATE_BIT,		\
> +	.info_mask_separate = BIT(IIO_CHAN_INFO_RAW),		\
>  	.channel = (idx),					\
>  	.scan_type = {						\
>  		.sign = 'u',					\

Reviewed-by: Marek Vasut <marex@denx.de>

Best regards,
Marek Vasut

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/3] staging:iio:adc:spear move to info_mask_(shared_by_type/separate)
  2013-03-04 21:16 ` [PATCH 1/3] staging:iio:adc:spear move to info_mask_(shared_by_type/separate) Jonathan Cameron
@ 2013-03-05  7:28   ` Stefan Roese
  0 siblings, 0 replies; 6+ messages in thread
From: Stefan Roese @ 2013-03-05  7:28 UTC (permalink / raw)
  To: Jonathan Cameron; +Cc: linux-iio

On 04.03.2013 22:16, Jonathan Cameron wrote:
> The original info_mask is going away in favour of the broken out versions.
> 
> Signed-off-by: Jonathan Cameron <jic23@kernel.org>
> cc: Stefan Roese <sr@denx.de>

Acked-by: Stefan Roese <sr@denx.de>

Thanks,
Stefan


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2013-03-05  7:28 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-03-04 21:16 [PATCH 0/3] IIO break info_mask into a pair of masks - Additional drivers 2 (missed ones) Jonathan Cameron
2013-03-04 21:16 ` [PATCH 1/3] staging:iio:adc:spear move to info_mask_(shared_by_type/separate) Jonathan Cameron
2013-03-05  7:28   ` Stefan Roese
2013-03-04 21:16 ` [PATCH 2/3] staging:iio:adc:mxs " Jonathan Cameron
2013-03-04 21:23   ` Marek Vasut
2013-03-04 21:16 ` [PATCH 3/3] staging:iio:adc:lpc32xx " Jonathan Cameron

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox