linux-iio.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/3] axp20x minor fixes
@ 2014-11-11 19:30 Jacob Pan
  2014-11-11 19:30 ` [PATCH 1/3] iio: adc: Add module device table for autoloading Jacob Pan
                   ` (3 more replies)
  0 siblings, 4 replies; 10+ messages in thread
From: Jacob Pan @ 2014-11-11 19:30 UTC (permalink / raw)
  To: LKML, IIO, Jonathan Cameron, Lee Jones
  Cc: Carlo Caione, Samuel Ortiz, Ramakrishna Pallala, Todd Brandt,
	Peter Meerwald, Rafael Wysocki, Hartmut Knaack, Jacob Pan

A few fixes based on mfd/ib-mfd-iio-3.19.


Aaron Lu (1):
  iio: adc: Add module device table for autoloading

Jacob Pan (2):
  mfd/axp20x: avoid irq numbering collision
  iio/axp288_adc: remove THIS_MODULE owner

 drivers/iio/adc/axp288_adc.c |  9 ++++++++-
 drivers/mfd/axp20x.c         | 18 +++++++++---------
 2 files changed, 17 insertions(+), 10 deletions(-)

-- 
1.9.1

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

* [PATCH 1/3] iio: adc: Add module device table for autoloading
  2014-11-11 19:30 [PATCH 0/3] axp20x minor fixes Jacob Pan
@ 2014-11-11 19:30 ` Jacob Pan
  2014-11-15 16:00   ` Jonathan Cameron
  2014-11-18 15:26   ` Lee Jones
  2014-11-11 19:30 ` [PATCH 2/3] mfd/axp20x: avoid irq numbering collision Jacob Pan
                   ` (2 subsequent siblings)
  3 siblings, 2 replies; 10+ messages in thread
From: Jacob Pan @ 2014-11-11 19:30 UTC (permalink / raw)
  To: LKML, IIO, Jonathan Cameron, Lee Jones
  Cc: Carlo Caione, Samuel Ortiz, Ramakrishna Pallala, Todd Brandt,
	Peter Meerwald, Rafael Wysocki, Hartmut Knaack, Aaron Lu,
	Jacob Pan

From: Aaron Lu <aaron.lu@intel.com>

Add the module device id table so that the driver can be automatically
loaded once the platform device is created.

Signed-off-by: Aaron Lu <aaron.lu@intel.com>
Signed-off-by: Jacob Pan <jacob.jun.pan@linux.intel.com>
---
 drivers/iio/adc/axp288_adc.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/drivers/iio/adc/axp288_adc.c b/drivers/iio/adc/axp288_adc.c
index 4800286..4a6cf43 100644
--- a/drivers/iio/adc/axp288_adc.c
+++ b/drivers/iio/adc/axp288_adc.c
@@ -238,15 +238,23 @@ static int axp288_adc_remove(struct platform_device *pdev)
 	return 0;
 }
 
+static struct platform_device_id axp288_adc_id_table[] = {
+	{ .name = "axp288_adc" },
+	{},
+};
+
 static struct platform_driver axp288_adc_driver = {
 	.probe = axp288_adc_probe,
 	.remove = axp288_adc_remove,
+	.id_table = axp288_adc_id_table,
 	.driver = {
 		.name = "axp288_adc",
 		.owner = THIS_MODULE,
 	},
 };
 
+MODULE_DEVICE_TABLE(platform, axp288_adc_id_table);
+
 module_platform_driver(axp288_adc_driver);
 
 MODULE_AUTHOR("Jacob Pan <jacob.jun.pan@linux.intel.com>");
-- 
1.9.1

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

* [PATCH 2/3] mfd/axp20x: avoid irq numbering collision
  2014-11-11 19:30 [PATCH 0/3] axp20x minor fixes Jacob Pan
  2014-11-11 19:30 ` [PATCH 1/3] iio: adc: Add module device table for autoloading Jacob Pan
@ 2014-11-11 19:30 ` Jacob Pan
  2014-11-18 15:24   ` Lee Jones
  2014-11-11 19:30 ` [PATCH 3/3] iio/axp288_adc: remove THIS_MODULE owner Jacob Pan
  2014-11-18 15:49 ` [GIT PULL] Immutable branch between MFD and IIO (second tag) Lee Jones
  3 siblings, 1 reply; 10+ messages in thread
From: Jacob Pan @ 2014-11-11 19:30 UTC (permalink / raw)
  To: LKML, IIO, Jonathan Cameron, Lee Jones
  Cc: Carlo Caione, Samuel Ortiz, Ramakrishna Pallala, Todd Brandt,
	Peter Meerwald, Rafael Wysocki, Hartmut Knaack, Jacob Pan

IRQ numbers in axp20x devices are defined with high-order bit first
in each IRQ enable/status registers. On Intel platforms it is more
common to number IRQs with least significant bit first. Therefore,
sharing IRQ# between the two is very difficult. Since AXP288 is a
customized PMIC for Intel platform and the amount of shared IRQs are
very small, we use separate IRQ numbering. This also fixes collision
and a duplicate in WBTO interrupt.

e.g. For the 16 interrupts controlled in IRQ enabled registers 1 & 2,
on axp20x for ARM, the PMIC local IRQ numbers and register bits are
mapped as:
IRQ#:  0  1  2  3  4  5  6  7      8  9 10 11 12 13 14 15
---------------------------------------------------------
ARM:   7  6  5  4  3  2  1  0      7  6  5  4  3  2  1  0
Intel: 0  1  2  3  4  5  6  7      0  1  2  3  4  5  6  7

Signed-off-by: Todd Brandt <todd.e.brandt@linux.intel.com>
Signed-off-by: Jacob Pan <jacob.jun.pan@linux.intel.com>
---
 drivers/mfd/axp20x.c | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/drivers/mfd/axp20x.c b/drivers/mfd/axp20x.c
index daf3c8d..0a8fcd9 100644
--- a/drivers/mfd/axp20x.c
+++ b/drivers/mfd/axp20x.c
@@ -183,21 +183,21 @@ static const struct regmap_irq axp20x_regmap_irqs[] = {
 
 /* some IRQs are compatible with axp20x models */
 static const struct regmap_irq axp288_regmap_irqs[] = {
-	INIT_REGMAP_IRQ(AXP20X, VBUS_REMOVAL,           0, 2),
-	INIT_REGMAP_IRQ(AXP20X, VBUS_PLUGIN,            0, 3),
-	INIT_REGMAP_IRQ(AXP20X, VBUS_OVER_V,            0, 4),
+	INIT_REGMAP_IRQ(AXP288, VBUS_FALL,              0, 2),
+	INIT_REGMAP_IRQ(AXP288, VBUS_RISE,              0, 3),
+	INIT_REGMAP_IRQ(AXP288, OV,                     0, 4),
 
-	INIT_REGMAP_IRQ(AXP20X, CHARG_DONE,             1, 2),
-	INIT_REGMAP_IRQ(AXP20X, CHARG,                  1, 3),
+	INIT_REGMAP_IRQ(AXP288, DONE,                   1, 2),
+	INIT_REGMAP_IRQ(AXP288, CHARGING,               1, 3),
 	INIT_REGMAP_IRQ(AXP288, SAFE_QUIT,              1, 4),
 	INIT_REGMAP_IRQ(AXP288, SAFE_ENTER,             1, 5),
-	INIT_REGMAP_IRQ(AXP20X, BATT_REMOVAL,           1, 6),
-	INIT_REGMAP_IRQ(AXP20X, BATT_PLUGIN,            1, 7),
+	INIT_REGMAP_IRQ(AXP288, ABSENT,                 1, 6),
+	INIT_REGMAP_IRQ(AXP288, APPEND,                 1, 7),
 
 	INIT_REGMAP_IRQ(AXP288, QWBTU,                  2, 0),
 	INIT_REGMAP_IRQ(AXP288, WBTU,                   2, 1),
 	INIT_REGMAP_IRQ(AXP288, QWBTO,                  2, 2),
-	INIT_REGMAP_IRQ(AXP288, WBTU,                   2, 3),
+	INIT_REGMAP_IRQ(AXP288, WBTO,                   2, 3),
 	INIT_REGMAP_IRQ(AXP288, QCBTU,                  2, 4),
 	INIT_REGMAP_IRQ(AXP288, CBTU,                   2, 5),
 	INIT_REGMAP_IRQ(AXP288, QCBTO,                  2, 6),
@@ -215,7 +215,7 @@ static const struct regmap_irq axp288_regmap_irqs[] = {
 	INIT_REGMAP_IRQ(AXP288, POKS,                   4, 4),
 	INIT_REGMAP_IRQ(AXP288, POKN,                   4, 5),
 	INIT_REGMAP_IRQ(AXP288, POKP,                   4, 6),
-	INIT_REGMAP_IRQ(AXP20X, TIMER,                  4, 7),
+	INIT_REGMAP_IRQ(AXP288, TIMER,                  4, 7),
 
 	INIT_REGMAP_IRQ(AXP288, MV_CHNG,                5, 0),
 	INIT_REGMAP_IRQ(AXP288, BC_USB_CHNG,            5, 1),
-- 
1.9.1

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

* [PATCH 3/3] iio/axp288_adc: remove THIS_MODULE owner
  2014-11-11 19:30 [PATCH 0/3] axp20x minor fixes Jacob Pan
  2014-11-11 19:30 ` [PATCH 1/3] iio: adc: Add module device table for autoloading Jacob Pan
  2014-11-11 19:30 ` [PATCH 2/3] mfd/axp20x: avoid irq numbering collision Jacob Pan
@ 2014-11-11 19:30 ` Jacob Pan
  2014-11-15 16:00   ` Jonathan Cameron
  2014-11-18 15:25   ` Lee Jones
  2014-11-18 15:49 ` [GIT PULL] Immutable branch between MFD and IIO (second tag) Lee Jones
  3 siblings, 2 replies; 10+ messages in thread
From: Jacob Pan @ 2014-11-11 19:30 UTC (permalink / raw)
  To: LKML, IIO, Jonathan Cameron, Lee Jones
  Cc: Carlo Caione, Samuel Ortiz, Ramakrishna Pallala, Todd Brandt,
	Peter Meerwald, Rafael Wysocki, Hartmut Knaack, Jacob Pan

This is no longer needed in that platform driver_register will do it.

Signed-off-by: Jacob Pan <jacob.jun.pan@linux.intel.com>
---
 drivers/iio/adc/axp288_adc.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/iio/adc/axp288_adc.c b/drivers/iio/adc/axp288_adc.c
index 4a6cf43..08bcfb0 100644
--- a/drivers/iio/adc/axp288_adc.c
+++ b/drivers/iio/adc/axp288_adc.c
@@ -249,7 +249,6 @@ static struct platform_driver axp288_adc_driver = {
 	.id_table = axp288_adc_id_table,
 	.driver = {
 		.name = "axp288_adc",
-		.owner = THIS_MODULE,
 	},
 };
 
-- 
1.9.1

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

* Re: [PATCH 1/3] iio: adc: Add module device table for autoloading
  2014-11-11 19:30 ` [PATCH 1/3] iio: adc: Add module device table for autoloading Jacob Pan
@ 2014-11-15 16:00   ` Jonathan Cameron
  2014-11-18 15:26   ` Lee Jones
  1 sibling, 0 replies; 10+ messages in thread
From: Jonathan Cameron @ 2014-11-15 16:00 UTC (permalink / raw)
  To: Jacob Pan, LKML, IIO, Lee Jones
  Cc: Carlo Caione, Samuel Ortiz, Ramakrishna Pallala, Todd Brandt,
	Peter Meerwald, Rafael Wysocki, Hartmut Knaack, Aaron Lu

On 11/11/14 19:30, Jacob Pan wrote:
> From: Aaron Lu <aaron.lu@intel.com>
> 
> Add the module device id table so that the driver can be automatically
> loaded once the platform device is created.
> 
> Signed-off-by: Aaron Lu <aaron.lu@intel.com>
> Signed-off-by: Jacob Pan <jacob.jun.pan@linux.intel.com>
Acked-by: Jonathan Cameron <jic23@kernel.org>
> ---
>  drivers/iio/adc/axp288_adc.c | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/drivers/iio/adc/axp288_adc.c b/drivers/iio/adc/axp288_adc.c
> index 4800286..4a6cf43 100644
> --- a/drivers/iio/adc/axp288_adc.c
> +++ b/drivers/iio/adc/axp288_adc.c
> @@ -238,15 +238,23 @@ static int axp288_adc_remove(struct platform_device *pdev)
>  	return 0;
>  }
>  
> +static struct platform_device_id axp288_adc_id_table[] = {
> +	{ .name = "axp288_adc" },
> +	{},
> +};
> +
>  static struct platform_driver axp288_adc_driver = {
>  	.probe = axp288_adc_probe,
>  	.remove = axp288_adc_remove,
> +	.id_table = axp288_adc_id_table,
>  	.driver = {
>  		.name = "axp288_adc",
>  		.owner = THIS_MODULE,
>  	},
>  };
>  
> +MODULE_DEVICE_TABLE(platform, axp288_adc_id_table);
> +
>  module_platform_driver(axp288_adc_driver);
>  
>  MODULE_AUTHOR("Jacob Pan <jacob.jun.pan@linux.intel.com>");
> 


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

* Re: [PATCH 3/3] iio/axp288_adc: remove THIS_MODULE owner
  2014-11-11 19:30 ` [PATCH 3/3] iio/axp288_adc: remove THIS_MODULE owner Jacob Pan
@ 2014-11-15 16:00   ` Jonathan Cameron
  2014-11-18 15:25   ` Lee Jones
  1 sibling, 0 replies; 10+ messages in thread
From: Jonathan Cameron @ 2014-11-15 16:00 UTC (permalink / raw)
  To: Jacob Pan, LKML, IIO, Lee Jones
  Cc: Carlo Caione, Samuel Ortiz, Ramakrishna Pallala, Todd Brandt,
	Peter Meerwald, Rafael Wysocki, Hartmut Knaack

On 11/11/14 19:30, Jacob Pan wrote:
> This is no longer needed in that platform driver_register will do it.
> 
> Signed-off-by: Jacob Pan <jacob.jun.pan@linux.intel.com>
Acked-by: Jonathan Cameron <jic23@kernel.org>
> ---
>  drivers/iio/adc/axp288_adc.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/iio/adc/axp288_adc.c b/drivers/iio/adc/axp288_adc.c
> index 4a6cf43..08bcfb0 100644
> --- a/drivers/iio/adc/axp288_adc.c
> +++ b/drivers/iio/adc/axp288_adc.c
> @@ -249,7 +249,6 @@ static struct platform_driver axp288_adc_driver = {
>  	.id_table = axp288_adc_id_table,
>  	.driver = {
>  		.name = "axp288_adc",
> -		.owner = THIS_MODULE,
>  	},
>  };
>  
> 


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

* Re: [PATCH 2/3] mfd/axp20x: avoid irq numbering collision
  2014-11-11 19:30 ` [PATCH 2/3] mfd/axp20x: avoid irq numbering collision Jacob Pan
@ 2014-11-18 15:24   ` Lee Jones
  0 siblings, 0 replies; 10+ messages in thread
From: Lee Jones @ 2014-11-18 15:24 UTC (permalink / raw)
  To: Jacob Pan
  Cc: LKML, IIO, Jonathan Cameron, Carlo Caione, Samuel Ortiz,
	Ramakrishna Pallala, Todd Brandt, Peter Meerwald, Rafael Wysocki,
	Hartmut Knaack

On Tue, 11 Nov 2014, Jacob Pan wrote:

> IRQ numbers in axp20x devices are defined with high-order bit first
> in each IRQ enable/status registers. On Intel platforms it is more
> common to number IRQs with least significant bit first. Therefore,
> sharing IRQ# between the two is very difficult. Since AXP288 is a
> customized PMIC for Intel platform and the amount of shared IRQs are
> very small, we use separate IRQ numbering. This also fixes collision
> and a duplicate in WBTO interrupt.
> 
> e.g. For the 16 interrupts controlled in IRQ enabled registers 1 & 2,
> on axp20x for ARM, the PMIC local IRQ numbers and register bits are
> mapped as:
> IRQ#:  0  1  2  3  4  5  6  7      8  9 10 11 12 13 14 15
> ---------------------------------------------------------
> ARM:   7  6  5  4  3  2  1  0      7  6  5  4  3  2  1  0
> Intel: 0  1  2  3  4  5  6  7      0  1  2  3  4  5  6  7
> 
> Signed-off-by: Todd Brandt <todd.e.brandt@linux.intel.com>
> Signed-off-by: Jacob Pan <jacob.jun.pan@linux.intel.com>
> ---
>  drivers/mfd/axp20x.c | 18 +++++++++---------
>  1 file changed, 9 insertions(+), 9 deletions(-)

Applied, thanks.

> diff --git a/drivers/mfd/axp20x.c b/drivers/mfd/axp20x.c
> index daf3c8d..0a8fcd9 100644
> --- a/drivers/mfd/axp20x.c
> +++ b/drivers/mfd/axp20x.c
> @@ -183,21 +183,21 @@ static const struct regmap_irq axp20x_regmap_irqs[] = {
>  
>  /* some IRQs are compatible with axp20x models */
>  static const struct regmap_irq axp288_regmap_irqs[] = {
> -	INIT_REGMAP_IRQ(AXP20X, VBUS_REMOVAL,           0, 2),
> -	INIT_REGMAP_IRQ(AXP20X, VBUS_PLUGIN,            0, 3),
> -	INIT_REGMAP_IRQ(AXP20X, VBUS_OVER_V,            0, 4),
> +	INIT_REGMAP_IRQ(AXP288, VBUS_FALL,              0, 2),
> +	INIT_REGMAP_IRQ(AXP288, VBUS_RISE,              0, 3),
> +	INIT_REGMAP_IRQ(AXP288, OV,                     0, 4),
>  
> -	INIT_REGMAP_IRQ(AXP20X, CHARG_DONE,             1, 2),
> -	INIT_REGMAP_IRQ(AXP20X, CHARG,                  1, 3),
> +	INIT_REGMAP_IRQ(AXP288, DONE,                   1, 2),
> +	INIT_REGMAP_IRQ(AXP288, CHARGING,               1, 3),
>  	INIT_REGMAP_IRQ(AXP288, SAFE_QUIT,              1, 4),
>  	INIT_REGMAP_IRQ(AXP288, SAFE_ENTER,             1, 5),
> -	INIT_REGMAP_IRQ(AXP20X, BATT_REMOVAL,           1, 6),
> -	INIT_REGMAP_IRQ(AXP20X, BATT_PLUGIN,            1, 7),
> +	INIT_REGMAP_IRQ(AXP288, ABSENT,                 1, 6),
> +	INIT_REGMAP_IRQ(AXP288, APPEND,                 1, 7),
>  
>  	INIT_REGMAP_IRQ(AXP288, QWBTU,                  2, 0),
>  	INIT_REGMAP_IRQ(AXP288, WBTU,                   2, 1),
>  	INIT_REGMAP_IRQ(AXP288, QWBTO,                  2, 2),
> -	INIT_REGMAP_IRQ(AXP288, WBTU,                   2, 3),
> +	INIT_REGMAP_IRQ(AXP288, WBTO,                   2, 3),
>  	INIT_REGMAP_IRQ(AXP288, QCBTU,                  2, 4),
>  	INIT_REGMAP_IRQ(AXP288, CBTU,                   2, 5),
>  	INIT_REGMAP_IRQ(AXP288, QCBTO,                  2, 6),
> @@ -215,7 +215,7 @@ static const struct regmap_irq axp288_regmap_irqs[] = {
>  	INIT_REGMAP_IRQ(AXP288, POKS,                   4, 4),
>  	INIT_REGMAP_IRQ(AXP288, POKN,                   4, 5),
>  	INIT_REGMAP_IRQ(AXP288, POKP,                   4, 6),
> -	INIT_REGMAP_IRQ(AXP20X, TIMER,                  4, 7),
> +	INIT_REGMAP_IRQ(AXP288, TIMER,                  4, 7),
>  
>  	INIT_REGMAP_IRQ(AXP288, MV_CHNG,                5, 0),
>  	INIT_REGMAP_IRQ(AXP288, BC_USB_CHNG,            5, 1),

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

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

* Re: [PATCH 3/3] iio/axp288_adc: remove THIS_MODULE owner
  2014-11-11 19:30 ` [PATCH 3/3] iio/axp288_adc: remove THIS_MODULE owner Jacob Pan
  2014-11-15 16:00   ` Jonathan Cameron
@ 2014-11-18 15:25   ` Lee Jones
  1 sibling, 0 replies; 10+ messages in thread
From: Lee Jones @ 2014-11-18 15:25 UTC (permalink / raw)
  To: Jacob Pan
  Cc: LKML, IIO, Jonathan Cameron, Carlo Caione, Samuel Ortiz,
	Ramakrishna Pallala, Todd Brandt, Peter Meerwald, Rafael Wysocki,
	Hartmut Knaack

On Tue, 11 Nov 2014, Jacob Pan wrote:

> This is no longer needed in that platform driver_register will do it.
> 
> Signed-off-by: Jacob Pan <jacob.jun.pan@linux.intel.com>
> ---
>  drivers/iio/adc/axp288_adc.c | 1 -
>  1 file changed, 1 deletion(-)

Applied thanks.

> diff --git a/drivers/iio/adc/axp288_adc.c b/drivers/iio/adc/axp288_adc.c
> index 4a6cf43..08bcfb0 100644
> --- a/drivers/iio/adc/axp288_adc.c
> +++ b/drivers/iio/adc/axp288_adc.c
> @@ -249,7 +249,6 @@ static struct platform_driver axp288_adc_driver = {
>  	.id_table = axp288_adc_id_table,
>  	.driver = {
>  		.name = "axp288_adc",
> -		.owner = THIS_MODULE,
>  	},
>  };
>  

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

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

* Re: [PATCH 1/3] iio: adc: Add module device table for autoloading
  2014-11-11 19:30 ` [PATCH 1/3] iio: adc: Add module device table for autoloading Jacob Pan
  2014-11-15 16:00   ` Jonathan Cameron
@ 2014-11-18 15:26   ` Lee Jones
  1 sibling, 0 replies; 10+ messages in thread
From: Lee Jones @ 2014-11-18 15:26 UTC (permalink / raw)
  To: Jacob Pan
  Cc: LKML, IIO, Jonathan Cameron, Carlo Caione, Samuel Ortiz,
	Ramakrishna Pallala, Todd Brandt, Peter Meerwald, Rafael Wysocki,
	Hartmut Knaack, Aaron Lu

On Tue, 11 Nov 2014, Jacob Pan wrote:

> From: Aaron Lu <aaron.lu@intel.com>
> 
> Add the module device id table so that the driver can be automatically
> loaded once the platform device is created.
> 
> Signed-off-by: Aaron Lu <aaron.lu@intel.com>
> Signed-off-by: Jacob Pan <jacob.jun.pan@linux.intel.com>
> ---
>  drivers/iio/adc/axp288_adc.c | 8 ++++++++
>  1 file changed, 8 insertions(+)

Applied, thanks.

> diff --git a/drivers/iio/adc/axp288_adc.c b/drivers/iio/adc/axp288_adc.c
> index 4800286..4a6cf43 100644
> --- a/drivers/iio/adc/axp288_adc.c
> +++ b/drivers/iio/adc/axp288_adc.c
> @@ -238,15 +238,23 @@ static int axp288_adc_remove(struct platform_device *pdev)
>  	return 0;
>  }
>  
> +static struct platform_device_id axp288_adc_id_table[] = {
> +	{ .name = "axp288_adc" },
> +	{},
> +};
> +
>  static struct platform_driver axp288_adc_driver = {
>  	.probe = axp288_adc_probe,
>  	.remove = axp288_adc_remove,
> +	.id_table = axp288_adc_id_table,
>  	.driver = {
>  		.name = "axp288_adc",
>  		.owner = THIS_MODULE,
>  	},
>  };
>  
> +MODULE_DEVICE_TABLE(platform, axp288_adc_id_table);
> +
>  module_platform_driver(axp288_adc_driver);
>  
>  MODULE_AUTHOR("Jacob Pan <jacob.jun.pan@linux.intel.com>");

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

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

* [GIT PULL] Immutable branch between MFD and IIO (second tag)
  2014-11-11 19:30 [PATCH 0/3] axp20x minor fixes Jacob Pan
                   ` (2 preceding siblings ...)
  2014-11-11 19:30 ` [PATCH 3/3] iio/axp288_adc: remove THIS_MODULE owner Jacob Pan
@ 2014-11-18 15:49 ` Lee Jones
  3 siblings, 0 replies; 10+ messages in thread
From: Lee Jones @ 2014-11-18 15:49 UTC (permalink / raw)
  To: Jacob Pan
  Cc: LKML, IIO, Jonathan Cameron, Carlo Caione, Samuel Ortiz,
	Ramakrishna Pallala, Todd Brandt, Peter Meerwald, Rafael Wysocki,
	Hartmut Knaack

The following changes since commit bfe01a5ba2490f299e1d2d5508cbbbadd897bbe9:

  Linux 3.17 (2014-10-05 12:23:04 -0700)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/lee/mfd.git tags/ib-mfd-iio-v3.19-1

for you to fetch changes up to 8019f6962708985782b65bd97be88046a55e1e4d:

  iio/axp288_adc: remove THIS_MODULE owner (2014-11-18 15:34:22 +0000)

----------------------------------------------------------------
Immutable branch between MFD and IIO due for v3.19.

This is the second tag, as more patches have been applied.

----------------------------------------------------------------
Aaron Lu (1):
      iio: adc: Add module device table for autoloading

Jacob Pan (4):
      mfd: axp20x: Extend axp20x to support axp288 pmic
      iio: adc: Add support for axp288 adc
      mfd/axp20x: avoid irq numbering collision
      iio/axp288_adc: remove THIS_MODULE owner

 drivers/iio/adc/Kconfig      |   8 +
 drivers/iio/adc/Makefile     |   1 +
 drivers/iio/adc/axp288_adc.c | 261 +++++++++++++++++++++++++++++++
 drivers/mfd/Kconfig          |   3 +-
 drivers/mfd/axp20x.c         | 361 ++++++++++++++++++++++++++++++++++++-------
 include/linux/mfd/axp20x.h   |  59 +++++++
 6 files changed, 637 insertions(+), 56 deletions(-)
 create mode 100644 drivers/iio/adc/axp288_adc.c

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

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

end of thread, other threads:[~2014-11-18 15:49 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-11-11 19:30 [PATCH 0/3] axp20x minor fixes Jacob Pan
2014-11-11 19:30 ` [PATCH 1/3] iio: adc: Add module device table for autoloading Jacob Pan
2014-11-15 16:00   ` Jonathan Cameron
2014-11-18 15:26   ` Lee Jones
2014-11-11 19:30 ` [PATCH 2/3] mfd/axp20x: avoid irq numbering collision Jacob Pan
2014-11-18 15:24   ` Lee Jones
2014-11-11 19:30 ` [PATCH 3/3] iio/axp288_adc: remove THIS_MODULE owner Jacob Pan
2014-11-15 16:00   ` Jonathan Cameron
2014-11-18 15:25   ` Lee Jones
2014-11-18 15:49 ` [GIT PULL] Immutable branch between MFD and IIO (second tag) Lee Jones

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).