All of lore.kernel.org
 help / color / mirror / Atom feed
From: swarren@nvidia.com (Stephen Warren)
To: linux-arm-kernel@lists.infradead.org
Subject: An extremely simplified pinctrl bindings proposal
Date: Mon, 06 Feb 2012 21:28:25 -0800	[thread overview]
Message-ID: <4F30B679.4060106@nvidia.com> (raw)
In-Reply-To: <20120206190315.GU1426@atomide.com>

On 02/06/2012 11:03 AM, Tony Lindgren wrote:
> * Linus Walleij <linus.walleij@linaro.org> [120206 08:58]:
>> On Mon, Feb 6, 2012 at 6:53 AM, Stephen Warren <swarren@nvidia.com> wrote:
>>
>> I will certainly finalize the pinctrl subsystem as-is, adding the
>> pin configurations states as the last major piece. If for nothing
>> else it provides some understanding of the problem space.
>>
>> I think we should keep both for the time being and consider the
>> alternative approach when patches appear. So if/when someone
>> creates a new subsystem like this, drivers can move over to it on a
>> per-driver basis. If there are zero drivers left in pinctrl it can be
>> deleted.
> 
> Yes it seems that we can easily do both. So far the only
> change needed for pinctrl drivers containing no data is that
> we should make the string names optional and structure debugfs
> around the physical register addresses instead. I'm basically
> just setting the mux register physcal address as the pin name
> for now to work around this.

I was thinking that since there was just a plain list of register
writes, there wouldn't be any concept of pins, groups, functions, etc.
at all. As such, it wouldn't really fit into pinctrl as-is; it'd need to
be either something separate, or pinctrl to change substantially more
than just allowing unnamed pins, wouldn't it?

-- 
nvpublic

WARNING: multiple messages have this Message-ID (diff)
From: Stephen Warren <swarren@nvidia.com>
To: Tony Lindgren <tony@atomide.com>
Cc: Linus Walleij <linus.walleij@linaro.org>,
	Dong Aisheng <dongas86@gmail.com>,
	Shawn Guo <shawn.guo@linaro.org>,
	Dong Aisheng-B29396 <B29396@freescale.com>,
	"Sascha Hauer (s.hauer@pengutronix.de)" <s.hauer@pengutronix.de>,
	"rob.herring@calxeda.com" <rob.herring@calxeda.com>,
	"kernel@pengutronix.de" <kernel@pengutronix.de>,
	"cjb@laptop.org" <cjb@laptop.org>,
	"Simon Glass (sjg@chromium.org)" <sjg@chromium.org>,
	Thomas Abraham <thomas.abraham@linaro.org>,
	"Grant Likely (grant.likely@secretlab.ca)"
	<grant.likely@secretlab.ca>,
	"devicetree-discuss@lists.ozlabs.org"
	<devicetree-discuss@lists.ozlabs.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: An extremely simplified pinctrl bindings proposal
Date: Mon, 06 Feb 2012 21:28:25 -0800	[thread overview]
Message-ID: <4F30B679.4060106@nvidia.com> (raw)
In-Reply-To: <20120206190315.GU1426@atomide.com>

On 02/06/2012 11:03 AM, Tony Lindgren wrote:
> * Linus Walleij <linus.walleij@linaro.org> [120206 08:58]:
>> On Mon, Feb 6, 2012 at 6:53 AM, Stephen Warren <swarren@nvidia.com> wrote:
>>
>> I will certainly finalize the pinctrl subsystem as-is, adding the
>> pin configurations states as the last major piece. If for nothing
>> else it provides some understanding of the problem space.
>>
>> I think we should keep both for the time being and consider the
>> alternative approach when patches appear. So if/when someone
>> creates a new subsystem like this, drivers can move over to it on a
>> per-driver basis. If there are zero drivers left in pinctrl it can be
>> deleted.
> 
> Yes it seems that we can easily do both. So far the only
> change needed for pinctrl drivers containing no data is that
> we should make the string names optional and structure debugfs
> around the physical register addresses instead. I'm basically
> just setting the mux register physcal address as the pin name
> for now to work around this.

I was thinking that since there was just a plain list of register
writes, there wouldn't be any concept of pins, groups, functions, etc.
at all. As such, it wouldn't really fit into pinctrl as-is; it'd need to
be either something separate, or pinctrl to change substantially more
than just allowing unnamed pins, wouldn't it?

-- 
nvpublic

  parent reply	other threads:[~2012-02-07  5:28 UTC|newest]

Thread overview: 56+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-02-05  5:31 An extremely simplified pinctrl bindings proposal Stephen Warren
2012-02-05  5:31 ` Stephen Warren
2012-02-05  5:31 ` Stephen Warren
2012-02-05  6:07 ` Richard Zhao
2012-02-05  6:07   ` Richard Zhao
2012-02-05  6:07   ` Richard Zhao
2012-02-06  3:07 ` Thomas Abraham
2012-02-06  3:07   ` Thomas Abraham
2012-02-06  5:44   ` Stephen Warren
2012-02-06  5:44     ` Stephen Warren
2012-02-06  4:20 ` Linus Walleij
2012-02-06  4:20   ` Linus Walleij
2012-02-06  5:53   ` Stephen Warren
2012-02-06  5:53     ` Stephen Warren
2012-02-06  5:53     ` Stephen Warren
2012-02-06 17:29     ` Linus Walleij
2012-02-06 17:29       ` Linus Walleij
2012-02-06 19:03       ` Tony Lindgren
2012-02-06 19:03         ` Tony Lindgren
2012-02-06 19:03         ` Tony Lindgren
2012-02-06 19:56         ` Linus Walleij
2012-02-06 19:56           ` Linus Walleij
2012-02-06 19:56           ` Linus Walleij
2012-02-06 21:04           ` Tony Lindgren
2012-02-06 21:04             ` Tony Lindgren
2012-02-06 21:04             ` Tony Lindgren
2012-02-06 23:15             ` Linus Walleij
2012-02-06 23:15               ` Linus Walleij
2012-02-06 23:57               ` Tony Lindgren
2012-02-06 23:57                 ` Tony Lindgren
2012-02-06 23:57                 ` Tony Lindgren
2012-02-07  1:07                 ` Linus Walleij
2012-02-07  1:07                   ` Linus Walleij
2012-02-07  1:07                   ` Linus Walleij
2012-02-07  5:28         ` Stephen Warren [this message]
2012-02-07  5:28           ` Stephen Warren
2012-02-06 19:41     ` Mark Brown
2012-02-06 19:41       ` Mark Brown
2012-02-06 19:41       ` Mark Brown
2012-02-06 18:57 ` Tony Lindgren
2012-02-06 18:57   ` Tony Lindgren
2012-02-06 18:57   ` Tony Lindgren
2012-02-06 19:05 ` Mitch Bradley
2012-02-06 19:05   ` Mitch Bradley
2012-02-06 19:05   ` Mitch Bradley
2012-02-06 19:26   ` Linus Walleij
2012-02-06 19:26     ` Linus Walleij
2012-02-06 21:24     ` Mitch Bradley
2012-02-06 21:24       ` Mitch Bradley
2012-02-06 21:24       ` Mitch Bradley
2012-02-07  5:33   ` Stephen Warren
2012-02-07  5:33     ` Stephen Warren
2012-02-07  5:33     ` Stephen Warren
2012-02-07  7:07     ` Mitch Bradley
2012-02-07  7:07       ` Mitch Bradley
2012-02-07  7:07       ` Mitch Bradley

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=4F30B679.4060106@nvidia.com \
    --to=swarren@nvidia.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 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.