linux-omap.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Johan Hovold <johan@kernel.org>
To: Andreas Kemnade <andreas@kemnade.info>
Cc: "Johan Hovold" <johan@kernel.org>,
	"Mark Rutland" <mark.rutland@arm.com>,
	devicetree@vger.kernel.org,
	"Discussions about the Letux Kernel"
	<letux-kernel@openphoenux.org>,
	"Jonathan Cameron" <jic23@kernel.org>,
	"Arnd Bergmann" <arnd@arndb.de>,
	"Tony Lindgren" <tony@atomide.com>,
	"H. Nikolaus Schaller" <hns@goldelico.com>,
	kernel@pyra-handheld.com, "Russell King" <linux@armlinux.org.uk>,
	linux-kernel@vger.kernel.org,
	"Thierry Reding" <treding@nvidia.com>,
	"Rob Herring" <robh+dt@kernel.org>,
	"Kevin Hilman" <khilman@baylibre.com>,
	"Benoît Cousson" <bcousson@baylibre.com>,
	"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
	linux-omap@vger.kernel.org, "Andreas Färber" <afaerber@suse.de>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [Letux-kernel] [PATCH v5 3/5] misc serdev: Add w2sg0004 (gps receiver) power control driver
Date: Fri, 12 Jan 2018 15:46:47 +0100	[thread overview]
Message-ID: <20180112144647.GA5992@localhost> (raw)
In-Reply-To: <20180109184347.28ba0a6e@aktux>

On Tue, Jan 09, 2018 at 06:43:47PM +0100, Andreas Kemnade wrote:
> On Fri, 22 Dec 2017 13:44:27 +0100
> Johan Hovold <johan@kernel.org> wrote:
> 
> [...]
> > I'd suggest reiterating the problem you're trying to solve and
> > enumerating the previously discussed potential solutions in order to
> > find a proper abstraction level for this (before getting lost in
> > implementation details).
> > 
> The main point here is in short words: Having a device powered on or off
> when the uart it is attached to, is used or not used anymore,
> so the already available userspace applications do not need to be changed.

So we'd end up with something in-between a kernel driver and a
user-space solution. What about devices that need to be (partially)
powered also when the port isn't open? A pure user-space solution would
be able to handle all variants.

> I digged out a bit around:
> alternative aproaches were:
> adding hooks to the uart/tty layer:
> https://marc.info/?l=linux-kernel&m=143333222014616&w=2
> https://marc.info/?l=devicetree&m=143130955414580&w=2

Thanks for the pointers, I remember those threads...

> I do not find it right now in my archive:
> adding a virtual gpio for dtr to the omap_serial driver.
> The driver behind the virtual io would then handle pm. One reason it was
> rejected was that the devicetree should only contain real hardware and
> not virtual stuff.

Oh, yeah, I think something like that made it in briefly before getting
reverted again.

I'll respond to Nikolaus mail as well.

Johan

  reply	other threads:[~2018-01-12 14:46 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-01  7:49 [PATCH v5 0/5] misc serdev: new serdev based driver for Wi2Wi w2sg00x4 GPS module H. Nikolaus Schaller
2017-12-01  7:49 ` [PATCH v5 1/5] dt-bindings: define vendor prefix for Wi2Wi, Inc H. Nikolaus Schaller
2017-12-01 14:44   ` Andreas Färber
2017-12-01  7:49 ` [PATCH v5 2/5] dt-bindings: gps: add w2sg00x4 bindings documentation (GPS module with UART)) H. Nikolaus Schaller
2017-12-01  7:49 ` [PATCH v5 4/5] DTS: gta04: add uart2 child node for w2sg00x4 H. Nikolaus Schaller
     [not found]   ` <1b1c7a8319eaf61a46eebd7a366ea745b7a54634.1512114577.git.hns-xXXSsgcRVICgSpxsJD1C4w@public.gmane.org>
2017-12-21 14:53     ` Tony Lindgren
     [not found] ` <cover.1512114576.git.hns-xXXSsgcRVICgSpxsJD1C4w@public.gmane.org>
2017-12-01  7:49   ` [PATCH v5 3/5] misc serdev: Add w2sg0004 (gps receiver) power control driver H. Nikolaus Schaller
     [not found]     ` <5494ad34b39a6c62601e3747440268dfb3be7d5a.1512114576.git.hns-xXXSsgcRVICgSpxsJD1C4w@public.gmane.org>
2017-12-22 12:44       ` Johan Hovold
2017-12-22 14:40         ` H. Nikolaus Schaller
2018-01-09 11:55           ` [Letux-kernel] " H. Nikolaus Schaller
     [not found]             ` <90F9A8E4-035A-4A9E-8AAB-757491D63E69-xXXSsgcRVICgSpxsJD1C4w@public.gmane.org>
2018-01-12 15:39               ` Johan Hovold
2018-01-12 17:59                 ` H. Nikolaus Schaller
2018-01-18  6:13                   ` Johan Hovold
2018-01-18 13:43                     ` H. Nikolaus Schaller
2018-03-07 15:53                       ` H. Nikolaus Schaller
2018-03-19 14:05                         ` Johan Hovold
2018-02-12 15:26                 ` [Letux-kernel] " Pavel Machek
2018-02-27  7:04                   ` Johan Hovold
2018-02-27  7:32                     ` H. Nikolaus Schaller
2018-03-19 13:54                       ` Johan Hovold
2018-03-20 14:49                         ` H. Nikolaus Schaller
2018-03-26 13:59                           ` Pavel Machek
2018-02-27 18:38                     ` Pavel Machek
2018-02-12 15:25             ` Pavel Machek
2018-01-09 17:43         ` Andreas Kemnade
2018-01-12 14:46           ` Johan Hovold [this message]
2018-01-12 18:40             ` Andreas Kemnade
     [not found]               ` <20180112194022.7da1e726-cLv4Z9ELZ06ZuzBka8ofvg@public.gmane.org>
2018-01-18  6:47                 ` Johan Hovold
2018-03-08  6:16                   ` Andreas Kemnade
2018-03-19 14:02                     ` Johan Hovold
2017-12-01  7:49   ` [PATCH v5 5/5] misc serdev: w2sg0004: add debugging code and Kconfig H. Nikolaus Schaller
     [not found]     ` <5816bfb9e7cab68591c133e20696d6188ebe70de.1512114577.git.hns-xXXSsgcRVICgSpxsJD1C4w@public.gmane.org>
2017-12-22 12:51       ` Johan Hovold
2017-12-22 14:41         ` H. Nikolaus Schaller
     [not found]           ` <A2720858-5172-4627-8133-C677E4B4DC72-xXXSsgcRVICgSpxsJD1C4w@public.gmane.org>
2017-12-22 14:58             ` Greg Kroah-Hartman
2017-12-18  8:52 ` [PATCH v5 0/5] misc serdev: new serdev based driver for Wi2Wi w2sg00x4 GPS module H. Nikolaus Schaller
2017-12-18 14:48   ` Greg Kroah-Hartman
     [not found]     ` <20171218144816.GA31130-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
2017-12-18 14:52       ` H. Nikolaus Schaller

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=20180112144647.GA5992@localhost \
    --to=johan@kernel.org \
    --cc=afaerber@suse.de \
    --cc=andreas@kemnade.info \
    --cc=arnd@arndb.de \
    --cc=bcousson@baylibre.com \
    --cc=devicetree@vger.kernel.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=hns@goldelico.com \
    --cc=jic23@kernel.org \
    --cc=kernel@pyra-handheld.com \
    --cc=khilman@baylibre.com \
    --cc=letux-kernel@openphoenux.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=linux@armlinux.org.uk \
    --cc=mark.rutland@arm.com \
    --cc=robh+dt@kernel.org \
    --cc=tony@atomide.com \
    --cc=treding@nvidia.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;
as well as URLs for NNTP newsgroup(s).