All of lore.kernel.org
 help / color / mirror / Atom feed
From: mad_soft@inbox.ru (Dmitry Artamonow)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] pxa/hx4700: Fix basic suspend/resume
Date: Fri, 13 May 2011 14:32:55 +0400	[thread overview]
Message-ID: <20110513103255.GB18716@rainbow> (raw)
In-Reply-To: <E1QKFPN-0003xV-Sf@outmx07.plus.net>

On 19:49 Wed 11 May     , Paul Parsons wrote:
> 
> Driver-specific bugs still remain. In particular the mmc driver does not resume properly after returning from sleep mode, thus preventing sleep mode from being entered a second time.

Sorry, no comments about your patch, just wanted to say about mmc
driver. Some time ago when I tried to look at suspend on hx4700,
this patch helped me to survive suspend-wakeup cycle with root on MMC:

diff --git a/drivers/mfd/asic3.c b/drivers/mfd/asic3.c
index 4896a3c..4c686d9 100644
--- a/drivers/mfd/asic3.c
+++ b/drivers/mfd/asic3.c
@@ -895,6 +895,8 @@ static struct mfd_cell asic3_cell_mmc = {
 	.name          = "tmio-mmc",
 	.enable        = asic3_mmc_enable,
 	.disable       = asic3_mmc_disable,
+	.suspend	= asic3_mmc_disable,
+	.resume		= asic3_mmc_enable,
 	.driver_data   = &asic3_mmc_data,
 	.num_resources = ARRAY_SIZE(asic3_mmc_resources),
 	.resources     = asic3_mmc_resources,

That was around 2.6.37. Not sure if it still helps and I have no possibility
to check - tmio_mmc on ASIC3 seems to be totally borken in 2.6.39-rcX - it
doesn't even detect card on my hx4700 :( And that's pity, as I mainly use
SD for rootfs.

-- 
Best regards,
Dmitry "MAD" Artamonow

  reply	other threads:[~2011-05-13 10:32 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-05-11 19:49 [PATCH] pxa/hx4700: Fix basic suspend/resume Paul Parsons
2011-05-13 10:32 ` Dmitry Artamonow [this message]
2011-05-13 19:18   ` Paul Parsons
2011-05-14 20:18     ` Dmitry Artamonow
2011-05-14 22:32       ` Paul Parsons
2011-05-15 13:14         ` Dmitry Artamonow
2011-05-15 13:27           ` Paul Parsons
2012-02-26 10:54 ` Philipp Zabel
2012-02-27  2:12   ` Haojian Zhuang
2012-02-27  2:32     ` Paul Parsons
2012-02-27  3:15       ` Haojian Zhuang
2012-02-27  9:04   ` Russell King - ARM Linux
2012-02-27  9:54     ` Haojian Zhuang
2012-02-27 14:31     ` Paul Parsons

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20110513103255.GB18716@rainbow \
    --to=mad_soft@inbox.ru \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.