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
WARNING: multiple messages have this Message-ID (diff)
From: johan@kernel.org (Johan Hovold) To: linux-arm-kernel@lists.infradead.org Subject: [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
next prev parent reply other threads:[~2018-01-12 14:46 UTC|newest] Thread overview: 100+ 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 ` 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 7:49 ` H. Nikolaus Schaller 2017-12-01 14:44 ` Andreas Färber 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 ` H. Nikolaus Schaller 2017-12-01 7:49 ` H. Nikolaus Schaller 2017-12-01 7:49 ` [PATCH v5 3/5] misc serdev: Add w2sg0004 (gps receiver) power control driver H. Nikolaus Schaller 2017-12-01 7:49 ` H. Nikolaus Schaller 2017-12-01 7:49 ` H. Nikolaus Schaller 2017-12-22 12:44 ` Johan Hovold 2017-12-22 12:44 ` Johan Hovold 2017-12-22 12:44 ` Johan Hovold 2017-12-22 14:40 ` H. Nikolaus Schaller 2017-12-22 14:40 ` H. Nikolaus Schaller 2017-12-22 14:40 ` H. Nikolaus Schaller 2018-01-09 11:55 ` [Letux-kernel] " H. Nikolaus Schaller 2018-01-09 11:55 ` H. Nikolaus Schaller 2018-01-09 11:55 ` H. Nikolaus Schaller 2018-01-12 15:39 ` Johan Hovold 2018-01-12 15:39 ` Johan Hovold 2018-01-12 15:39 ` Johan Hovold 2018-01-12 17:59 ` H. Nikolaus Schaller 2018-01-12 17:59 ` H. Nikolaus Schaller 2018-01-12 17:59 ` H. Nikolaus Schaller 2018-01-18 6:13 ` Johan Hovold 2018-01-18 6:13 ` Johan Hovold 2018-01-18 6:13 ` Johan Hovold 2018-01-18 13:43 ` H. Nikolaus Schaller 2018-01-18 13:43 ` H. Nikolaus Schaller 2018-01-18 13:43 ` H. Nikolaus Schaller 2018-03-07 15:53 ` H. Nikolaus Schaller 2018-03-07 15:53 ` H. Nikolaus Schaller 2018-03-07 15:53 ` H. Nikolaus Schaller 2018-03-19 14:05 ` Johan Hovold 2018-03-19 14:05 ` Johan Hovold 2018-03-19 14:05 ` Johan Hovold 2018-02-12 15:26 ` [Letux-kernel] " Pavel Machek 2018-02-12 15:26 ` Pavel Machek 2018-02-12 15:26 ` Pavel Machek 2018-02-27 7:04 ` Johan Hovold 2018-02-27 7:04 ` Johan Hovold 2018-02-27 7:04 ` Johan Hovold 2018-02-27 7:32 ` H. Nikolaus Schaller 2018-02-27 7:32 ` H. Nikolaus Schaller 2018-02-27 7:32 ` H. Nikolaus Schaller 2018-03-19 13:54 ` Johan Hovold 2018-03-19 13:54 ` Johan Hovold 2018-03-19 13:54 ` Johan Hovold 2018-03-20 14:49 ` H. Nikolaus Schaller 2018-03-20 14:49 ` H. Nikolaus Schaller 2018-03-20 14:49 ` H. Nikolaus Schaller 2018-03-26 13:59 ` Pavel Machek 2018-03-26 13:59 ` Pavel Machek 2018-02-27 18:38 ` Pavel Machek 2018-02-27 18:38 ` Pavel Machek 2018-02-27 18:38 ` Pavel Machek 2018-02-12 15:25 ` Pavel Machek 2018-02-12 15:25 ` Pavel Machek 2018-01-09 17:43 ` Andreas Kemnade 2018-01-09 17:43 ` Andreas Kemnade 2018-01-09 17:43 ` Andreas Kemnade 2018-01-12 14:46 ` Johan Hovold [this message] 2018-01-12 14:46 ` Johan Hovold 2018-01-12 18:40 ` Andreas Kemnade 2018-01-12 18:40 ` Andreas Kemnade 2018-01-12 18:40 ` Andreas Kemnade 2018-01-18 6:47 ` Johan Hovold 2018-01-18 6:47 ` Johan Hovold 2018-01-18 6:47 ` Johan Hovold 2018-03-08 6:16 ` Andreas Kemnade 2018-03-08 6:16 ` Andreas Kemnade 2018-03-19 14:02 ` Johan Hovold 2018-03-19 14:02 ` Johan Hovold 2017-12-01 7:49 ` [PATCH v5 4/5] DTS: gta04: add uart2 child node for w2sg00x4 H. Nikolaus Schaller 2017-12-01 7:49 ` H. Nikolaus Schaller 2017-12-21 14:53 ` Tony Lindgren 2017-12-21 14:53 ` Tony Lindgren 2017-12-21 14:53 ` Tony Lindgren 2017-12-01 7:49 ` [PATCH v5 5/5] misc serdev: w2sg0004: add debugging code and Kconfig H. Nikolaus Schaller 2017-12-01 7:49 ` H. Nikolaus Schaller 2017-12-01 7:49 ` H. Nikolaus Schaller 2017-12-22 12:51 ` Johan Hovold 2017-12-22 12:51 ` Johan Hovold 2017-12-22 12:51 ` Johan Hovold 2017-12-22 14:41 ` H. Nikolaus Schaller 2017-12-22 14:41 ` H. Nikolaus Schaller 2017-12-22 14:58 ` Greg Kroah-Hartman 2017-12-22 14:58 ` Greg Kroah-Hartman 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 8:52 ` H. Nikolaus Schaller 2017-12-18 8:52 ` H. Nikolaus Schaller 2017-12-18 14:48 ` Greg Kroah-Hartman 2017-12-18 14:48 ` Greg Kroah-Hartman 2017-12-18 14:52 ` H. Nikolaus Schaller 2017-12-18 14:52 ` H. Nikolaus Schaller 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: linkBe 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.