public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] mfd: tps65910: fix initialisation of interrupts
@ 2011-08-15 10:42 Johan Hovold
  2011-08-15 11:06 ` Graeme Gregory
  2011-08-22 14:04 ` Samuel Ortiz
  0 siblings, 2 replies; 3+ messages in thread
From: Johan Hovold @ 2011-08-15 10:42 UTC (permalink / raw)
  To: Samuel Ortiz
  Cc: Liam Girdwood, Jorge Eduardo Candelaria, linux-kernel,
	Johan Hovold, stable

Fix regression introduced by commit
a2974732ca7614aaf0baf9d6dd3ad893d50ce1c5 (TPS65911: Add new irq
definitions) which caused irq_num to be incorrectly set for tps65910.

Cc: stable@kernel.org
Signed-off-by: Johan Hovold <jhovold@gmail.com>
---
 drivers/mfd/tps65910-irq.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/drivers/mfd/tps65910-irq.c b/drivers/mfd/tps65910-irq.c
index 2bfad5c..a56be93 100644
--- a/drivers/mfd/tps65910-irq.c
+++ b/drivers/mfd/tps65910-irq.c
@@ -178,8 +178,10 @@ int tps65910_irq_init(struct tps65910 *tps65910, int irq,
 	switch (tps65910_chip_id(tps65910)) {
 	case TPS65910:
 		tps65910->irq_num = TPS65910_NUM_IRQ;
+		break;
 	case TPS65911:
 		tps65910->irq_num = TPS65911_NUM_IRQ;
+		break;
 	}
 
 	/* Register with genirq */
-- 
1.7.6


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

* Re: [PATCH] mfd: tps65910: fix initialisation of interrupts
  2011-08-15 10:42 [PATCH] mfd: tps65910: fix initialisation of interrupts Johan Hovold
@ 2011-08-15 11:06 ` Graeme Gregory
  2011-08-22 14:04 ` Samuel Ortiz
  1 sibling, 0 replies; 3+ messages in thread
From: Graeme Gregory @ 2011-08-15 11:06 UTC (permalink / raw)
  To: Johan Hovold
  Cc: Samuel Ortiz, Liam Girdwood, Jorge Eduardo Candelaria,
	linux-kernel, stable

On 15/08/2011 11:42, Johan Hovold wrote:
> Fix regression introduced by commit
> a2974732ca7614aaf0baf9d6dd3ad893d50ce1c5 (TPS65911: Add new irq
> definitions) which caused irq_num to be incorrectly set for tps65910.
>
> Cc: stable@kernel.org
> Signed-off-by: Johan Hovold <jhovold@gmail.com>
> ---
>  drivers/mfd/tps65910-irq.c |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/mfd/tps65910-irq.c b/drivers/mfd/tps65910-irq.c
> index 2bfad5c..a56be93 100644
> --- a/drivers/mfd/tps65910-irq.c
> +++ b/drivers/mfd/tps65910-irq.c
> @@ -178,8 +178,10 @@ int tps65910_irq_init(struct tps65910 *tps65910, int irq,
>  	switch (tps65910_chip_id(tps65910)) {
>  	case TPS65910:
>  		tps65910->irq_num = TPS65910_NUM_IRQ;
> +		break;
>  	case TPS65911:
>  		tps65910->irq_num = TPS65911_NUM_IRQ;
> +		break;
>  	}
>  
>  	/* Register with genirq */
Whole series

Acked-by: Graeme Gregory <gg@slimlogic.co.uk>


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

* Re: [PATCH] mfd: tps65910: fix initialisation of interrupts
  2011-08-15 10:42 [PATCH] mfd: tps65910: fix initialisation of interrupts Johan Hovold
  2011-08-15 11:06 ` Graeme Gregory
@ 2011-08-22 14:04 ` Samuel Ortiz
  1 sibling, 0 replies; 3+ messages in thread
From: Samuel Ortiz @ 2011-08-22 14:04 UTC (permalink / raw)
  To: Johan Hovold
  Cc: Liam Girdwood, Jorge Eduardo Candelaria, linux-kernel, stable

Hi Johan

On Mon, Aug 15, 2011 at 12:42:03PM +0200, Johan Hovold wrote:
> Fix regression introduced by commit
> a2974732ca7614aaf0baf9d6dd3ad893d50ce1c5 (TPS65911: Add new irq
> definitions) which caused irq_num to be incorrectly set for tps65910.
Patch applied to bot my for-next and for-linus branches, thanks.

Cheers,
Samuel.

-- 
Intel Open Source Technology Centre
http://oss.intel.com/

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

end of thread, other threads:[~2011-08-22 14:02 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-08-15 10:42 [PATCH] mfd: tps65910: fix initialisation of interrupts Johan Hovold
2011-08-15 11:06 ` Graeme Gregory
2011-08-22 14:04 ` Samuel Ortiz

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