From: Drew Fustini <drew@beagleboard.org>
To: "Enrico Weigelt, metux IT consult" <lkml@metux.net>
Cc: Linus Walleij <linus.walleij@linaro.org>,
linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org,
Tony Lindgren <tony@atomide.com>,
Andy Shevchenko <andy.shevchenko@gmail.com>,
Alexandre Belloni <alexandre.belloni@bootlin.com>,
Geert Uytterhoeven <geert@linux-m68k.org>,
Pantelis Antoniou <pantelis.antoniou@konsulko.com>,
Jason Kridner <jkridner@beagleboard.org>,
Robert Nelson <robertcnelson@beagleboard.org>,
Joe Perches <joe@perches.com>,
Dan Carpenter <dan.carpenter@oracle.com>,
Jonathan Corbet <corbet@lwn.net>,
linux-doc@vger.kernel.org,
Geert Uytterhoeven <geert+renesas@glider.be>
Subject: Re: [PATCH v9 2/4] pinctrl: pinmux: Add pinmux-select debugfs file
Date: Sat, 13 Mar 2021 00:42:39 -0800 [thread overview]
Message-ID: <20210313084239.GB1172642@x1> (raw)
In-Reply-To: <349b09f8-fe99-d0d4-dd11-c288bf66cb4d@metux.net>
On Fri, Mar 12, 2021 at 02:57:54PM +0100, Enrico Weigelt, metux IT consult wrote:
> On 02.03.21 06:30, Drew Fustini wrote:
>
> Hi folks,
>
> > Add "pinmux-select" to debugfs which will activate a pin function for a
> > given pin group:
> >
> > echo "<group-name function-name>" > pinmux-select
> >
> > The write operation pinmux_select() handles this by checking that the
> > names map to valid selectors and then calling ops->set_mux().
>
> I've already been playing with similar idea, but for external muxes.
> For example, some boards have multiple SIM slots that can be switched
> via some gpio pin.
>
> Not sure whether traditional pinmux would be a good match for that.
>
>
> --mtx
>
Thank you for your comment. I am interested in understanding other
use cases.
Linus can hopefully correct me, but I think this is covered by section
"Drivers needing both pin control and GPIOs" in pinctl.rst [1]. The
driver could be both a GPIO consumer and pinctrl consumer and then be
able to call pinctrl_select_state() and devm_gpio_request().
Thank you,
Drew
[1] https://www.kernel.org/doc/html/latest/driver-api/pinctl.html
next prev parent reply other threads:[~2021-03-13 8:43 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-02 5:30 [PATCH v9 0/4] pinctrl: pinmux: Add pinmux-select debugfs file Drew Fustini
2021-03-02 5:30 ` [PATCH v9 1/4] pinctrl: use to octal permissions for debugfs files Drew Fustini
2021-03-02 5:30 ` [PATCH v9 2/4] pinctrl: pinmux: Add pinmux-select debugfs file Drew Fustini
2021-03-12 13:57 ` Enrico Weigelt, metux IT consult
2021-03-13 8:42 ` Drew Fustini [this message]
2021-03-13 9:28 ` Alexandre Belloni
2021-03-13 18:47 ` Geert Uytterhoeven
2021-03-18 19:40 ` Enrico Weigelt, metux IT consult
2021-03-15 16:24 ` Linus Walleij
2021-03-02 5:30 ` [PATCH v9 3/4] Documentation: rename pinctl to pin-control Drew Fustini
2021-03-02 5:30 ` [PATCH v9 4/4] docs/pinctrl: document debugfs files Drew Fustini
2021-03-10 13:43 ` [PATCH v9 0/4] pinctrl: pinmux: Add pinmux-select debugfs file 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=20210313084239.GB1172642@x1 \
--to=drew@beagleboard.org \
--cc=alexandre.belloni@bootlin.com \
--cc=andy.shevchenko@gmail.com \
--cc=corbet@lwn.net \
--cc=dan.carpenter@oracle.com \
--cc=geert+renesas@glider.be \
--cc=geert@linux-m68k.org \
--cc=jkridner@beagleboard.org \
--cc=joe@perches.com \
--cc=linus.walleij@linaro.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lkml@metux.net \
--cc=pantelis.antoniou@konsulko.com \
--cc=robertcnelson@beagleboard.org \
--cc=tony@atomide.com \
/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