[patch 2.6.29-rc3] omap_hsmmc: card detect irq bugfix

[patch 2.6.29-rc3] omap_hsmmc: card detect irq bugfix

[David Brownell]

From: David Brownell <dbrownell@users.sourceforge.net>

Work around lockdep issue when card detect IRQ handlers run in
thread context ... it forces IRQF_DISABLED, which prevents all
access to twl4030 card detect signals.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
---
 drivers/mmc/host/omap_hsmmc.c |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

--- a/drivers/mmc/host/omap_hsmmc.c
+++ b/drivers/mmc/host/omap_hsmmc.c
@@ -517,6 +517,9 @@ static void mmc_omap_detect(struct work_
 {
 	struct mmc_omap_host *host = container_of(work, struct mmc_omap_host,
 						mmc_carddetect_work);
+	struct omap_mmc_slot_data *slot = &mmc_slot(host);
+
+	host->carddetect = slot->card_detect(slot->card_detect_irq);
 
 	sysfs_notify(&host->mmc->class_dev.kobj, NULL, "cover_switch");
 	if (host->carddetect) {
@@ -538,7 +541,6 @@ static irqreturn_t omap_mmc_cd_handler(i
 {
 	struct mmc_omap_host *host = (struct mmc_omap_host *)dev_id;
 
-	host->carddetect = mmc_slot(host).card_detect(irq);
 	schedule_work(&host->mmc_carddetect_work);
 
 	return IRQ_HANDLED;

Re: [patch 2.6.29-rc3] omap_hsmmc: card detect irq bugfix

[Pierre Ossman]

On Wed, 4 Feb 2009 14:42:03 -0800
David Brownell <david-b@pacbell.net> wrote:

> From: David Brownell <dbrownell@users.sourceforge.net>
> 
> Work around lockdep issue when card detect IRQ handlers run in
> thread context ... it forces IRQF_DISABLED, which prevents all
> access to twl4030 card detect signals.
> 
> Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
> ---

An ack from eiter Tony or one of the listed authors of this driver
would be nice.

Rgds
-- 
     -- Pierre Ossman

  Linux kernel, MMC maintainer        http://www.kernel.org
  rdesktop, core developer          http://www.rdesktop.org

  WARNING: This correspondence is being monitored by the
  Swedish government. Make sure your server uses encryption
  for SMTP traffic and consider using PGP for end-to-end
  encryption.

Re: [patch 2.6.29-rc3] omap_hsmmc: card detect irq bugfix

[Tony Lindgren]

* Pierre Ossman <drzeus-mmc@drzeus.cx> [090217 21:25]:
> On Wed, 4 Feb 2009 14:42:03 -0800
> David Brownell <david-b@pacbell.net> wrote:
> 
> > From: David Brownell <dbrownell@users.sourceforge.net>
> > 
> > Work around lockdep issue when card detect IRQ handlers run in
> > thread context ... it forces IRQF_DISABLED, which prevents all
> > access to twl4030 card detect signals.
> > 
> > Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
> > ---
> 
> An ack from eiter Tony or one of the listed authors of this driver
> would be nice.

Here you are:

Acked-by: Tony Lindgren <tony@atomide.com>

Regards,

Tony

Re: [patch 2.6.29-rc3] omap_hsmmc: card detect irq bugfix

[Pierre Ossman]

On Tue, 17 Feb 2009 22:13:59 -0800
Tony Lindgren <tony@atomide.com> wrote:

> * Pierre Ossman <drzeus-mmc@drzeus.cx> [090217 21:25]:
> > On Wed, 4 Feb 2009 14:42:03 -0800
> > David Brownell <david-b@pacbell.net> wrote:
> > 
> > > From: David Brownell <dbrownell@users.sourceforge.net>
> > > 
> > > Work around lockdep issue when card detect IRQ handlers run in
> > > thread context ... it forces IRQF_DISABLED, which prevents all
> > > access to twl4030 card detect signals.
> > > 
> > > Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
> > > ---
> > 
> > An ack from eiter Tony or one of the listed authors of this driver
> > would be nice.
> 
> Here you are:
> 
> Acked-by: Tony Lindgren <tony@atomide.com>
> 

Queued. Could you have a look at the other omap_hsmmc patches as well?
And a patch for MAINTAINERS so that I can keep track of who's baby this
is. :)

Rgds
-- 
     -- Pierre Ossman

  Linux kernel, MMC maintainer        http://www.kernel.org
  rdesktop, core developer          http://www.rdesktop.org

  WARNING: This correspondence is being monitored by the
  Swedish government. Make sure your server uses encryption
  for SMTP traffic and consider using PGP for end-to-end
  encryption.

Re: [patch 2.6.29-rc3] omap_hsmmc: card detect irq bugfix

[Tony Lindgren]

* Pierre Ossman <drzeus-mmc@drzeus.cx> [090218 12:27]:
> On Tue, 17 Feb 2009 22:13:59 -0800
> Tony Lindgren <tony@atomide.com> wrote:
> 
> > * Pierre Ossman <drzeus-mmc@drzeus.cx> [090217 21:25]:
> > > On Wed, 4 Feb 2009 14:42:03 -0800
> > > David Brownell <david-b@pacbell.net> wrote:
> > > 
> > > > From: David Brownell <dbrownell@users.sourceforge.net>
> > > > 
> > > > Work around lockdep issue when card detect IRQ handlers run in
> > > > thread context ... it forces IRQF_DISABLED, which prevents all
> > > > access to twl4030 card detect signals.
> > > > 
> > > > Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
> > > > ---
> > > 
> > > An ack from eiter Tony or one of the listed authors of this driver
> > > would be nice.
> > 
> > Here you are:
> > 
> > Acked-by: Tony Lindgren <tony@atomide.com>
> > 
> 
> Queued. Could you have a look at the other omap_hsmmc patches as well?
> And a patch for MAINTAINERS so that I can keep track of who's baby this
> is. :)

Thanks. We've been meaning to add some omap maintainers for some time
now, so I'll do a patch that adds an entries to MAINTAINERS for most
omap drivers and will post it to all the associated mailing lists.

Regards,

Tony