linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: thomas.petazzoni@free-electrons.com (Thomas Petazzoni)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCHv2] Input: omap4-keypad: Add pinctrl support
Date: Tue, 23 Oct 2012 13:11:56 +0200	[thread overview]
Message-ID: <20121023131156.13e2d0ae@skate> (raw)
In-Reply-To: <CACRpkdZhUwMTg0suqbZiukN7Se6fzdYP3-0Dspm9WF6qZniNVQ@mail.gmail.com>


On Tue, 23 Oct 2012 12:45:33 +0200, Linus Walleij wrote:

> Hm so I have had this idea of runtime PM core helping out
> with pins, so I could add something like
> 
> pm_pins_fetch()
> pm_pins_default()
> pm_pins_idle()
> pm_pins_sleep()
> 
> So if one is using the pin states defined in
> <linux/pinctrl/pinctrl-state.h> then the PM core can help out in
> keeping track of the pins and states, and the driver will just tell
> the PM core what to do and when.
> 
> Would this fit the bill for everyone's code consolidation needs?
> It would sure work for us...

That surely would work but is kind of non-obvious when reading a
driver's code: that's the problem with bus notifier, they do things a
bit "behind your back" without you noticing. Having the driver request
its own pinctrl state, and switch between states upon suspend/resume is
a lot more explicit, IMO.

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

  parent reply	other threads:[~2012-10-23 11:11 UTC|newest]

Thread overview: 69+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-10-22 13:13 [PATCHv2] Input: omap4-keypad: Add pinctrl support Sourav Poddar
2012-10-22 15:50 ` Dmitry Torokhov
2012-10-23  9:13   ` Linus Walleij
2012-10-23  9:35     ` Benoit Cousson
2012-10-23 10:04       ` Linus Walleij
2012-10-23 10:03         ` Felipe Balbi
2012-10-23 10:23           ` Thomas Petazzoni
2012-10-23 10:29             ` Linus Walleij
2012-10-23 10:29               ` Felipe Balbi
2012-10-23 10:45                 ` Linus Walleij
2012-10-23 10:42                   ` Felipe Balbi
2012-10-23 11:11                   ` Thomas Petazzoni [this message]
2012-10-23 17:02           ` Mitch Bradley
2012-10-23 17:20             ` Felipe Balbi
2012-10-23 17:51               ` Mitch Bradley
2012-10-23 17:51                 ` Felipe Balbi
2012-10-23  9:18   ` Benoit Cousson
2012-10-23 20:02     ` Dmitry Torokhov
2012-10-24  8:37       ` Felipe Balbi
2012-10-24 16:14         ` Dmitry Torokhov
2012-10-24 16:51           ` Linus Walleij
2012-10-24 17:28             ` Dmitry Torokhov
2012-10-24 18:58               ` Felipe Balbi
2012-10-25 20:59                 ` Mark Brown
2012-10-26  6:20                   ` Felipe Balbi
2012-10-26 16:03                     ` Mark Brown
2012-10-29 19:49                       ` Felipe Balbi
2012-10-30 11:24                         ` Mark Brown
2012-10-30 11:49                           ` Felipe Balbi
2012-10-30 14:07                             ` Mark Brown
2012-10-30 14:16                               ` Linus Walleij
2012-10-30 14:54                                 ` Mark Brown
2012-10-30 15:16                               ` Felipe Balbi
2012-10-30 15:58                                 ` Mark Brown
2012-10-30 17:25                                   ` Felipe Balbi
2012-10-30 18:20                                     ` Dmitry Torokhov
2012-10-30 18:48                                       ` Felipe Balbi
2012-10-30 18:37                                     ` Mark Brown
2012-10-30 21:51                                       ` Linus Walleij
2012-10-30 22:57                                         ` Rafael J. Wysocki
2012-11-02 18:26                                         ` Mark Brown
2012-10-30 14:11                             ` Linus Walleij
2012-10-28 20:12               ` Linus Walleij
2012-10-30 11:34                 ` Mark Brown
2012-10-30 14:02                   ` Linus Walleij
2012-10-30 14:37                     ` Mark Brown
2012-10-31 20:10                     ` Kevin Hilman
2012-11-01  8:54                       ` Linus Walleij
2012-11-01  8:56                         ` Fwd: " Linus Walleij
2012-11-01 11:42                           ` Kevin Hilman
2012-11-01 13:22                             ` Linus Walleij
2012-11-01 12:07                         ` Mark Brown
2012-11-01 14:01                           ` Linus Walleij
2012-11-01 14:19                             ` Mark Brown
2012-11-11 12:32                             ` Linus Walleij
2012-10-31 13:19                 ` Jean-Christophe PLAGNIOL-VILLARD
2012-10-24 16:52           ` Felipe Balbi
2012-10-24 17:13             ` Linus Walleij
2012-10-24 17:34             ` Dmitry Torokhov
2012-10-24 17:46           ` Benoit Cousson
2012-10-24 12:54       ` Linus Walleij
2012-10-24 16:18         ` Dmitry Torokhov
2012-10-24 16:57           ` Felipe Balbi
2012-10-24 17:18             ` Linus Walleij
2012-10-24 17:58             ` Dmitry Torokhov
2012-10-24 19:10               ` Felipe Balbi
2012-10-24 19:38                 ` Dmitry Torokhov
2012-10-24 19:51                   ` Felipe Balbi
2012-10-24 17:01           ` Linus Walleij

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=20121023131156.13e2d0ae@skate \
    --to=thomas.petazzoni@free-electrons.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;
as well as URLs for NNTP newsgroup(s).