public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
From: t.figa@samsung.com (Tomasz Figa)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 4/6] pinctrl: samsung: Add support for SoC-specific suspend/resume callbacks
Date: Fri, 24 May 2013 11:20:06 +0200	[thread overview]
Message-ID: <22452279.DN7eJUzGA7@amdc1227> (raw)
In-Reply-To: <CACRpkdY-d3YYLJoWydakbT7P4wnBAK2XL7X0Fs0EcNSFakpUDw@mail.gmail.com>

Hi Linus,

On Friday 24 of May 2013 11:07:41 Linus Walleij wrote:
> On Fri, May 17, 2013 at 6:24 PM, Tomasz Figa <t.figa@samsung.com> wrote:
> > SoC-specific driver might require additional save and restore of
> > registers. This patch adds pair of SoC-specific callbacks per pinctrl
> > device to account for this.
> > 
> > Signed-off-by: Tomasz Figa <t.figa@samsung.com>
> > Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
> 
> Patch applied for fixes.

Thanks.

> Hm this is quite a lot of code for "fixes", can you confirm that
> the system is really unusable without all these patches?

Well, it is something that should have been already sent as a fix for 3.8, when 
this pin control driver was added, because since then suspend/resume has been 
broken on DT-enabled Exynos boards.

Without this, suspending the board and then trying to wake it up is rather 
unpredictable, leading usually to board reset, interrupt storm or at least 
some devices failing to resume.

Best regards,
-- 
Tomasz Figa
Linux Kernel Developer
Samsung R&D Institute Poland
Samsung Electronics

  reply	other threads:[~2013-05-24  9:20 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-17 16:24 [PATCH 0/6] Fix suspend/resume issues created by pinmux on exynos, part 2 Tomasz Figa
2013-05-17 16:24 ` [PATCH 1/6] pinctrl: exynos: Add support for set_irq_wake of wake-up EINTs Tomasz Figa
2013-05-17 19:17   ` Doug Anderson
2013-05-21 11:25   ` Linus Walleij
2013-05-17 16:24 ` [PATCH 2/6] ARM: EXYNOS: Fix EINT wake-up mask configuration when pinctrl is used Tomasz Figa
2013-05-17 19:22   ` Doug Anderson
2013-05-17 19:49     ` Tomasz Figa
2013-05-21 11:27   ` Linus Walleij
2013-05-17 16:24 ` [PATCH 3/6] ARM: SAMSUNG: pm: Adjust for pinctrl- and DT-enabled platforms Tomasz Figa
2013-05-17 19:24   ` Doug Anderson
2013-05-17 20:23     ` Tomasz Figa
2013-05-17 20:56       ` Doug Anderson
2013-05-17 21:07         ` Tomasz Figa
2013-05-21 11:29           ` Linus Walleij
2013-05-21 13:15             ` Tomasz Figa
2013-05-21 17:06               ` Tomasz Figa
2013-06-10 14:45   ` Tomasz Figa
2013-06-10 16:13     ` Linus Walleij
2013-06-11  7:45       ` Olof Johansson
2013-06-11  8:21         ` Olof Johansson
2013-06-12  0:15           ` Tomasz Figa
2013-06-12  0:20             ` Olof Johansson
2013-05-17 16:24 ` [PATCH 4/6] pinctrl: samsung: Add support for SoC-specific suspend/resume callbacks Tomasz Figa
2013-05-17 19:24   ` Doug Anderson
2013-05-17 20:51     ` Tomasz Figa
2013-05-24  9:07   ` Linus Walleij
2013-05-24  9:20     ` Tomasz Figa [this message]
2013-05-17 16:24 ` [PATCH 5/6] pinctrl: samsung: Allow per-bank SoC-specific private data Tomasz Figa
2013-05-17 19:24   ` Doug Anderson
2013-05-24  9:09   ` Linus Walleij
2013-05-17 16:24 ` [PATCH 6/6] pinctrl: exynos: Handle suspend/resume of GPIO EINT registers Tomasz Figa
2013-05-17 19:25   ` Doug Anderson
2013-05-17 20:34     ` Tomasz Figa
2013-05-17 21:18       ` Doug Anderson
2013-05-21 17:05         ` [PATCH v2 " Tomasz Figa
2013-05-22  4:46           ` Doug Anderson
2013-05-22 13:32             ` Tomasz Figa
2013-05-22 14:03             ` [PATCH v3 " Tomasz Figa
2013-05-22 15:57               ` Doug Anderson
2013-05-24  9:12               ` Linus Walleij
2013-05-24  9:23                 ` Tomasz Figa
2013-05-20  9:35 ` [PATCH 0/6] Fix suspend/resume issues created by pinmux on exynos, part 2 Tushar Behera

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=22452279.DN7eJUzGA7@amdc1227 \
    --to=t.figa@samsung.com \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox