All of lore.kernel.org
 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 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.