All of lore.kernel.org
 help / color / mirror / Atom feed
From: Johan Hovold <johan@kernel.org>
To: "H. Nikolaus Schaller" <hns@goldelico.com>
Cc: "Johan Hovold" <johan@kernel.org>,
	"Mark Rutland" <mark.rutland@arm.com>,
	DTML <devicetree@vger.kernel.org>,
	linux-omap <linux-omap@vger.kernel.org>,
	"Jonathan Cameron" <jic23@kernel.org>,
	"Arnd Bergmann" <arnd@arndb.de>,
	"Tony Lindgren" <tony@atomide.com>,
	"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
	"Russell King" <linux@armlinux.org.uk>,
	"Linux Kernel Mailing List" <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>,
	kernel@pyra-handheld.com,
	"Andreas Kemnade" <andreas@kemnade.info>,
	"Discussions about the Letux Kernel"
	<letux-kernel@openphoenux.org>,
	"Andreas Färber" <afaerber@suse.de>
Subject: Re: [PATCH v5 3/5] misc serdev: Add w2sg0004 (gps receiver) power control driver
Date: Mon, 19 Mar 2018 15:05:55 +0100	[thread overview]
Message-ID: <20180319140555.GN18359@localhost> (raw)
In-Reply-To: <461DBB7C-5FF7-4723-9518-C9A5E7E5610D@goldelico.com>

On Wed, Mar 07, 2018 at 04:53:12PM +0100, H. Nikolaus Schaller wrote:

> If I look for example at the camera module drivers provided by
> drivers/media/i2c, most of them could be easily power-controlled from
> user-space by i2c-tools and 1-2 gpios through /sys/class/gpio and
> a big set of scripts.
> 
> Still they have a place in the kernel and cameras are powered on
> if the device is opened and powered down if it is closed.
> 
> So I am still trying to understand the rationale and logic (if one exists)
> behind having them in kernel but rejecting our driver which does the
> same for a different class of devices.

For media we have a framework in place; for gps we do not (yet).

Johan

WARNING: multiple messages have this Message-ID (diff)
From: johan@kernel.org (Johan Hovold)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v5 3/5] misc serdev: Add w2sg0004 (gps receiver) power control driver
Date: Mon, 19 Mar 2018 15:05:55 +0100	[thread overview]
Message-ID: <20180319140555.GN18359@localhost> (raw)
In-Reply-To: <461DBB7C-5FF7-4723-9518-C9A5E7E5610D@goldelico.com>

On Wed, Mar 07, 2018 at 04:53:12PM +0100, H. Nikolaus Schaller wrote:

> If I look for example at the camera module drivers provided by
> drivers/media/i2c, most of them could be easily power-controlled from
> user-space by i2c-tools and 1-2 gpios through /sys/class/gpio and
> a big set of scripts.
> 
> Still they have a place in the kernel and cameras are powered on
> if the device is opened and powered down if it is closed.
> 
> So I am still trying to understand the rationale and logic (if one exists)
> behind having them in kernel but rejecting our driver which does the
> same for a different class of devices.

For media we have a framework in place; for gps we do not (yet).

Johan

WARNING: multiple messages have this Message-ID (diff)
From: Johan Hovold <johan@kernel.org>
To: "H. Nikolaus Schaller" <hns@goldelico.com>
Cc: "Johan Hovold" <johan@kernel.org>,
	"Mark Rutland" <mark.rutland@arm.com>,
	DTML <devicetree@vger.kernel.org>,
	linux-omap <linux-omap@vger.kernel.org>,
	"Jonathan Cameron" <jic23@kernel.org>,
	"Arnd Bergmann" <arnd@arndb.de>,
	"Tony Lindgren" <tony@atomide.com>,
	"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
	"Russell King" <linux@armlinux.org.uk>,
	"Linux Kernel Mailing List" <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>,
	kernel@pyra-handheld.com,
	"Andreas Kemnade" <andreas@kemnade.info>,
	"Discussions about the Letux Kernel"
	<letux-kernel@openphoenux.org>,
	"Andreas Färber" <afaerber@suse.de>,
	"Linux ARM" <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v5 3/5] misc serdev: Add w2sg0004 (gps receiver) power control driver
Date: Mon, 19 Mar 2018 15:05:55 +0100	[thread overview]
Message-ID: <20180319140555.GN18359@localhost> (raw)
In-Reply-To: <461DBB7C-5FF7-4723-9518-C9A5E7E5610D@goldelico.com>

On Wed, Mar 07, 2018 at 04:53:12PM +0100, H. Nikolaus Schaller wrote:

> If I look for example at the camera module drivers provided by
> drivers/media/i2c, most of them could be easily power-controlled from
> user-space by i2c-tools and 1-2 gpios through /sys/class/gpio and
> a big set of scripts.
> 
> Still they have a place in the kernel and cameras are powered on
> if the device is opened and powered down if it is closed.
> 
> So I am still trying to understand the rationale and logic (if one exists)
> behind having them in kernel but rejecting our driver which does the
> same for a different class of devices.

For media we have a framework in place; for gps we do not (yet).

Johan

  reply	other threads:[~2018-03-19 14:05 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 4/5] DTS: gta04: add uart2 child node for w2sg00x4 H. Nikolaus Schaller
2017-12-01  7:49   ` H. Nikolaus Schaller
     [not found]   ` <1b1c7a8319eaf61a46eebd7a366ea745b7a54634.1512114577.git.hns-xXXSsgcRVICgSpxsJD1C4w@public.gmane.org>
2017-12-21 14:53     ` Tony Lindgren
2017-12-21 14:53       ` Tony Lindgren
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
2017-12-01  7:49     ` H. Nikolaus Schaller
2017-12-01  7:49     ` H. Nikolaus Schaller
     [not found]     ` <5494ad34b39a6c62601e3747440268dfb3be7d5a.1512114576.git.hns-xXXSsgcRVICgSpxsJD1C4w@public.gmane.org>
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
     [not found]             ` <90F9A8E4-035A-4A9E-8AAB-757491D63E69-xXXSsgcRVICgSpxsJD1C4w@public.gmane.org>
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 [this message]
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
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
     [not found]               ` <20180112194022.7da1e726-cLv4Z9ELZ06ZuzBka8ofvg@public.gmane.org>
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 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
     [not found]     ` <5816bfb9e7cab68591c133e20696d6188ebe70de.1512114577.git.hns-xXXSsgcRVICgSpxsJD1C4w@public.gmane.org>
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
     [not found]           ` <A2720858-5172-4627-8133-C677E4B4DC72-xXXSsgcRVICgSpxsJD1C4w@public.gmane.org>
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
     [not found]     ` <20171218144816.GA31130-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
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=20180319140555.GN18359@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-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 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.