From: NeilBrown <neilb@suse.de>
To: Mark Rutland <mark.rutland@arm.com>,
One Thousand Gnomes <gnomes@lxorguk.ukuu.org.uk>,
Peter Hurley <peter@hurleysoftware.com>,
Arnd Bergmann <arnd@arndb.de>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Sebastian Reichel <sre@kernel.org>,
Grant Likely <grant.likely@linaro.org>,
Jiri Slaby <jslaby@suse.cz>
Cc: GTA04 owners <gta04-owner@goldelico.com>,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: [PATCH 0/2] tty slave devices support - version 2
Date: Sat, 20 Dec 2014 11:09:20 +1100 [thread overview]
Message-ID: <20141219235827.13943.45713.stgit@notabene.brown> (raw)
Thanks for all the great feedback. I have incorporated a lot of it,
though not all ... partly because there was not yet unanimity on some
issues.
Big changes:
- children of a uart are no longer automatically managed.
The driver for the child device must register with the tty
after which it will be told when the tty is opened or closed.
The driver can then do whatever it likes, which may involve
powering the device on.
- I am now only providing a single drivers: serial-power-manager.
It can be used for devices which only want power management.
Currently it can provide this using a regulator and/or a
toggle GPIO. This one driver supports both of my devices.
non-changes:
- The interface functionality is still provided by the 'tty'
layer, not the 'serial' layer. I have no strong feelings on
this and doubt that I would have until some other user
appeared for this functionality. That would help show
if the current arrangement was helpful or inconvenient.
- We still treat any child node of a tty device which has a
'compatible' field gets allocated a device. If there is no
platform driver which supports that 'compatible' value, then
the device will be inactive. Specific action will only be
taken if there is a child node with a 'compatible' field for which
there is a supporting driver.
small changes:
- proper 'compatible' names are used
- switch to use 'gpiod' instead of 'gpio'
- no longer misuse the 'tty_' prefix.
Comments and suggestions always welcome,
Thanks,
NeilBrown
---
NeilBrown (2):
TTY: add support for "tty slave" devices.
misc: add a driver to power on/off UART attached devices.
.../devicetree/bindings/misc/wi2wi,w2cbw003.txt | 19 +
.../devicetree/bindings/misc/wi2wi,w2sg0004.txt | 37 +
.../devicetree/bindings/serial/of-serial.txt | 4
.../devicetree/bindings/vendor-prefixes.txt | 1
drivers/misc/Kconfig | 12
drivers/misc/Makefile | 1
drivers/misc/serial-power-manager.c | 494 ++++++++++++++++++++
drivers/tty/tty_io.c | 73 +++
include/linux/tty.h | 16 +
9 files changed, 654 insertions(+), 3 deletions(-)
create mode 100644 Documentation/devicetree/bindings/misc/wi2wi,w2cbw003.txt
create mode 100644 Documentation/devicetree/bindings/misc/wi2wi,w2sg0004.txt
create mode 100644 drivers/misc/serial-power-manager.c
--
Signature
next reply other threads:[~2014-12-20 0:09 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-12-20 0:09 NeilBrown [this message]
2014-12-20 0:09 ` [PATCH 1/2] TTY: add support for "tty slave" devices NeilBrown
[not found] ` <20141220000920.13943.22511.stgit-wvvUuzkyo1EYVZTmpyfIwg@public.gmane.org>
2014-12-21 10:20 ` Sebastian Reichel
[not found] ` <20141221102017.GA18161-SfvFxonMDyemK9LvCR3Hrw@public.gmane.org>
2014-12-23 5:16 ` NeilBrown
2014-12-20 0:09 ` [PATCH 2/2] misc: add a driver to power on/off UART attached devices NeilBrown
[not found] ` <20141220000920.13943.50943.stgit-wvvUuzkyo1EYVZTmpyfIwg@public.gmane.org>
2014-12-20 12:50 ` One Thousand Gnomes
2014-12-20 16:02 ` [Gta04-owner] " Christ van Willegen
[not found] ` <CA+Ot1Own-g=vjYhOygzVgNPspw6FgCHZqY8wQo31=ahf62E1gA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-12-23 3:17 ` NeilBrown
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=20141219235827.13943.45713.stgit@notabene.brown \
--to=neilb@suse.de \
--cc=arnd@arndb.de \
--cc=devicetree@vger.kernel.org \
--cc=gnomes@lxorguk.ukuu.org.uk \
--cc=grant.likely@linaro.org \
--cc=gregkh@linuxfoundation.org \
--cc=gta04-owner@goldelico.com \
--cc=jslaby@suse.cz \
--cc=linux-kernel@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=peter@hurleysoftware.com \
--cc=sre@kernel.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).