Re: OMAP serial runtime PM and autosuspend (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding))

On Thu, May 24, 2018 at 06:32:37AM -0700, Tony Lindgren wrote: > * Johan Hovold [180524 09:20]: > > On Mon, May 21, 2018 at 08:48:32AM -0700, Tony Lindgren wrote: > > > Well if you have some better mechanism in mind let's try it out. Short of > > > sprinkling

Re: OMAP serial runtime PM and autosuspend (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding))

On Thu, May 24, 2018 at 06:32:37AM -0700, Tony Lindgren wrote: > * Johan Hovold [180524 09:20]: > > On Mon, May 21, 2018 at 08:48:32AM -0700, Tony Lindgren wrote: > > > Well if you have some better mechanism in mind let's try it out. Short of > > > sprinkling pm_runtime_force_suspend/resume

Re: OMAP serial runtime PM and autosuspend (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding))

* Johan Hovold [180524 09:20]: > On Mon, May 21, 2018 at 08:48:32AM -0700, Tony Lindgren wrote: > > > > Yes the bug for closed ports needs to be fixed for sure. > > I did some forensic on this and it seems this problem has "always" been > there. Specifically, closed ports have

Re: OMAP serial runtime PM and autosuspend (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding))

* Johan Hovold [180524 09:20]: > On Mon, May 21, 2018 at 08:48:32AM -0700, Tony Lindgren wrote: > > > > Yes the bug for closed ports needs to be fixed for sure. > > I did some forensic on this and it seems this problem has "always" been > there. Specifically, closed ports have never been

Re: OMAP serial runtime PM and autosuspend (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding))

On Mon, May 21, 2018 at 08:48:32AM -0700, Tony Lindgren wrote: > * Johan Hovold [180521 13:50]: > > On Thu, May 17, 2018 at 10:10:38AM -0700, Tony Lindgren wrote: > > > * Johan Hovold [180517 10:12]: > > > Well in that case we should just stick with -1 value

Re: OMAP serial runtime PM and autosuspend (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding))

On Mon, May 21, 2018 at 08:48:32AM -0700, Tony Lindgren wrote: > * Johan Hovold [180521 13:50]: > > On Thu, May 17, 2018 at 10:10:38AM -0700, Tony Lindgren wrote: > > > * Johan Hovold [180517 10:12]: > > > Well in that case we should just stick with -1 value for the > > > autosuspend. And just

Re: OMAP serial runtime PM and autosuspend (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding))

* Johan Hovold [180521 13:50]: > On Thu, May 17, 2018 at 10:10:38AM -0700, Tony Lindgren wrote: > > * Johan Hovold [180517 10:12]: > > > No, defaulting to "on" (i.e. calling pm_runtime_forbid()) wouldn't work > > > either as that would also prevent the device

Re: OMAP serial runtime PM and autosuspend (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding))

* Johan Hovold [180521 13:50]: > On Thu, May 17, 2018 at 10:10:38AM -0700, Tony Lindgren wrote: > > * Johan Hovold [180517 10:12]: > > > No, defaulting to "on" (i.e. calling pm_runtime_forbid()) wouldn't work > > > either as that would also prevent the device from runtime suspending > > > just

Re: OMAP serial runtime PM and autosuspend (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding))

On Thu, May 17, 2018 at 10:10:38AM -0700, Tony Lindgren wrote: > * Johan Hovold [180517 10:12]: > > [ Sorry about the late reply. ] > > > > On Wed, May 09, 2018 at 06:57:06AM -0700, Tony Lindgren wrote: > > > * Johan Hovold [180509 13:12]: > > > > > > It

Re: OMAP serial runtime PM and autosuspend (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding))

On Thu, May 17, 2018 at 10:10:38AM -0700, Tony Lindgren wrote: > * Johan Hovold [180517 10:12]: > > [ Sorry about the late reply. ] > > > > On Wed, May 09, 2018 at 06:57:06AM -0700, Tony Lindgren wrote: > > > * Johan Hovold [180509 13:12]: > > > > > > It seems we really should not be using the

Re: OMAP serial runtime PM and autosuspend (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding))

* Johan Hovold [180517 10:12]: > [ Sorry about the late reply. ] > > On Wed, May 09, 2018 at 06:57:06AM -0700, Tony Lindgren wrote: > > * Johan Hovold [180509 13:12]: > > > > It seems we really should not be using the negative autosuspend to > > > configure

Re: OMAP serial runtime PM and autosuspend (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding))

* Johan Hovold [180517 10:12]: > [ Sorry about the late reply. ] > > On Wed, May 09, 2018 at 06:57:06AM -0700, Tony Lindgren wrote: > > * Johan Hovold [180509 13:12]: > > > > It seems we really should not be using the negative autosuspend to > > > configure the RPM behaviour the way these

Re: Serdev runtime PM (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding)

On Wed, May 09, 2018 at 07:05:50AM -0700, Tony Lindgren wrote: > * Johan Hovold [180509 09:20]: > > On Tue, May 08, 2018 at 05:56:08PM +0200, Sebastian Reichel wrote: > > > I think using open/close for runtime pm is good enough for GPS, > > > since it regularly sends data and

Re: Serdev runtime PM (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding)

On Wed, May 09, 2018 at 07:05:50AM -0700, Tony Lindgren wrote: > * Johan Hovold [180509 09:20]: > > On Tue, May 08, 2018 at 05:56:08PM +0200, Sebastian Reichel wrote: > > > I think using open/close for runtime pm is good enough for GPS, > > > since it regularly sends data and draws lots of power

Re: OMAP serial runtime PM and autosuspend (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding))

[ Sorry about the late reply. ] On Wed, May 09, 2018 at 06:57:06AM -0700, Tony Lindgren wrote: > * Johan Hovold [180509 13:12]: > > It seems we really should not be using the negative autosuspend to > > configure the RPM behaviour the way these drivers do. Perhaps a new > >

Re: OMAP serial runtime PM and autosuspend (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding))

[ Sorry about the late reply. ] On Wed, May 09, 2018 at 06:57:06AM -0700, Tony Lindgren wrote: > * Johan Hovold [180509 13:12]: > > It seems we really should not be using the negative autosuspend to > > configure the RPM behaviour the way these drivers do. Perhaps a new > > mechanism is needed.

Re: Serdev runtime PM (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding)

* Johan Hovold [180509 09:20]: > On Tue, May 08, 2018 at 05:56:08PM +0200, Sebastian Reichel wrote: > > I think using open/close for runtime pm is good enough for GPS, > > since it regularly sends data and draws lots of power anyways. > > But devices, that have an out-of-band

Re: Serdev runtime PM (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding)

* Johan Hovold [180509 09:20]: > On Tue, May 08, 2018 at 05:56:08PM +0200, Sebastian Reichel wrote: > > I think using open/close for runtime pm is good enough for GPS, > > since it regularly sends data and draws lots of power anyways. > > But devices, that have an out-of-band wakeup signal can do

Re: OMAP serial runtime PM and autosuspend (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding))

* Johan Hovold [180509 13:12]: > While this seems to fix the idling of closed ports here too, I'm not > sure we can move use_autosuspend to startup() like this. > > First, this flag should be set before registering the tty so that udev > can be used to update the attributes. >

Re: OMAP serial runtime PM and autosuspend (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding))

* Johan Hovold [180509 13:12]: > While this seems to fix the idling of closed ports here too, I'm not > sure we can move use_autosuspend to startup() like this. > > First, this flag should be set before registering the tty so that udev > can be used to update the attributes. > > Second, this

OMAP serial runtime PM and autosuspend (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding))

[ Updating subject and adding linux-serial, linux-pm and linux-omap. ] On Tue, May 08, 2018 at 09:49:04AM -0700, Tony Lindgren wrote: > * Tony Lindgren [180508 08:54]: > > * Tony Lindgren [180508 08:47]: > > > * Tony Lindgren [180508

OMAP serial runtime PM and autosuspend (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding))

[ Updating subject and adding linux-serial, linux-pm and linux-omap. ] On Tue, May 08, 2018 at 09:49:04AM -0700, Tony Lindgren wrote: > * Tony Lindgren [180508 08:54]: > > * Tony Lindgren [180508 08:47]: > > > * Tony Lindgren [180508 08:22]: > > > > * Johan Hovold [180508 07:00]: > > > > >

Re: Serdev runtime PM (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding)

On Wed, May 09, 2018 at 11:18:31AM +0200, Johan Hovold wrote: > I've cooked up a patch which I'll be sending as a reply to this mail. Forgot to add the in-reply-to header of course. For the interested, the patch can be found here:

Re: Serdev runtime PM (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding)

On Wed, May 09, 2018 at 11:18:31AM +0200, Johan Hovold wrote: > I've cooked up a patch which I'll be sending as a reply to this mail. Forgot to add the in-reply-to header of course. For the interested, the patch can be found here:

Serdev runtime PM (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding)

[ Changing the subject line as this is thread is no longer about DT bindings. Also adding linux-serial and linux-pm while keeping some context. ] On Tue, May 08, 2018 at 05:56:08PM +0200, Sebastian Reichel wrote: > Hi, > > On Mon, May 07, 2018 at 06:34:39PM +0200, Johan Hovold wrote: > > On

Serdev runtime PM (was: Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding)

[ Changing the subject line as this is thread is no longer about DT bindings. Also adding linux-serial and linux-pm while keeping some context. ] On Tue, May 08, 2018 at 05:56:08PM +0200, Sebastian Reichel wrote: > Hi, > > On Mon, May 07, 2018 at 06:34:39PM +0200, Johan Hovold wrote: > > On

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

* Tony Lindgren [180508 08:54]: > * Tony Lindgren [180508 08:47]: > > * Tony Lindgren [180508 08:22]: > > > * Johan Hovold [180508 07:00]: > > > > With the negative autosuspend set in both omap drivers probe functions, > >

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

* Tony Lindgren [180508 08:54]: > * Tony Lindgren [180508 08:47]: > > * Tony Lindgren [180508 08:22]: > > > * Johan Hovold [180508 07:00]: > > > > With the negative autosuspend set in both omap drivers probe functions, > > > > this is the expected behaviour. Which I think we must fix. > > > >

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

Hi, On Mon, May 07, 2018 at 06:34:39PM +0200, Johan Hovold wrote: > On Mon, May 07, 2018 at 08:45:15AM -0700, Tony Lindgren wrote: > > * Johan Hovold [180507 03:03]: > > > On Fri, May 04, 2018 at 01:42:13PM +0200, Sebastian Reichel wrote: > > > > > > > Having said all of this,

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

Hi, On Mon, May 07, 2018 at 06:34:39PM +0200, Johan Hovold wrote: > On Mon, May 07, 2018 at 08:45:15AM -0700, Tony Lindgren wrote: > > * Johan Hovold [180507 03:03]: > > > On Fri, May 04, 2018 at 01:42:13PM +0200, Sebastian Reichel wrote: > > > > > > > Having said all of this, serdev does not

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

* Tony Lindgren [180508 08:47]: > * Tony Lindgren [180508 08:22]: > > * Johan Hovold [180508 07:00]: > > > With the negative autosuspend set in both omap drivers probe functions, > > > this is the expected behaviour. Which I think we must

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

* Tony Lindgren [180508 08:47]: > * Tony Lindgren [180508 08:22]: > > * Johan Hovold [180508 07:00]: > > > With the negative autosuspend set in both omap drivers probe functions, > > > this is the expected behaviour. Which I think we must fix. > > > > Yes indeed. I've been using my script for

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

* Tony Lindgren [180508 08:22]: > * Johan Hovold [180508 07:00]: > > With the negative autosuspend set in both omap drivers probe functions, > > this is the expected behaviour. Which I think we must fix. > > Yes indeed. I've been using my script for years now

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

* Tony Lindgren [180508 08:22]: > * Johan Hovold [180508 07:00]: > > With the negative autosuspend set in both omap drivers probe functions, > > this is the expected behaviour. Which I think we must fix. > > Yes indeed. I've been using my script for years now and have > completely missed the

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

* Johan Hovold [180508 07:00]: > On Mon, May 07, 2018 at 10:50:32AM -0700, Tony Lindgren wrote: > > I don't know, they are there for the ports that don't have any > > serdev device. But if there is a serdev child node, the sysfs > > disappear for the 8250 port like it's

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

* Johan Hovold [180508 07:00]: > On Mon, May 07, 2018 at 10:50:32AM -0700, Tony Lindgren wrote: > > I don't know, they are there for the ports that don't have any > > serdev device. But if there is a serdev child node, the sysfs > > disappear for the 8250 port like it's /dev/ttyS* entry. That is

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

On Mon, May 07, 2018 at 10:50:32AM -0700, Tony Lindgren wrote: > * Johan Hovold [180507 16:36]: > > On Mon, May 07, 2018 at 08:45:15AM -0700, Tony Lindgren wrote: > > > Hi, > > > > > > * Johan Hovold [180507 03:03]: > > > > On Fri, May 04, 2018 at 01:42:13PM

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

On Mon, May 07, 2018 at 10:50:32AM -0700, Tony Lindgren wrote: > * Johan Hovold [180507 16:36]: > > On Mon, May 07, 2018 at 08:45:15AM -0700, Tony Lindgren wrote: > > > Hi, > > > > > > * Johan Hovold [180507 03:03]: > > > > On Fri, May 04, 2018 at 01:42:13PM +0200, Sebastian Reichel wrote: > >

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

* Johan Hovold [180507 16:36]: > On Mon, May 07, 2018 at 08:45:15AM -0700, Tony Lindgren wrote: > > Hi, > > > > * Johan Hovold [180507 03:03]: > > > On Fri, May 04, 2018 at 01:42:13PM +0200, Sebastian Reichel wrote: > > > > > > > Having said all of this,

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

* Johan Hovold [180507 16:36]: > On Mon, May 07, 2018 at 08:45:15AM -0700, Tony Lindgren wrote: > > Hi, > > > > * Johan Hovold [180507 03:03]: > > > On Fri, May 04, 2018 at 01:42:13PM +0200, Sebastian Reichel wrote: > > > > > > > Having said all of this, serdev does not yet support runtime PM

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

On Mon, May 07, 2018 at 08:45:15AM -0700, Tony Lindgren wrote: > Hi, > > * Johan Hovold [180507 03:03]: > > On Fri, May 04, 2018 at 01:42:13PM +0200, Sebastian Reichel wrote: > > > > > Having said all of this, serdev does not yet support runtime PM (at > > > all). Tony is

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

On Mon, May 07, 2018 at 08:45:15AM -0700, Tony Lindgren wrote: > Hi, > > * Johan Hovold [180507 03:03]: > > On Fri, May 04, 2018 at 01:42:13PM +0200, Sebastian Reichel wrote: > > > > > Having said all of this, serdev does not yet support runtime PM (at > > > all). Tony is currently looking into

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

Hi, * Johan Hovold [180507 03:03]: > On Fri, May 04, 2018 at 01:42:13PM +0200, Sebastian Reichel wrote: > > > Having said all of this, serdev does not yet support runtime PM (at > > all). Tony is currently looking into it. Fortunately serdev allows > > us to enable runtime PM

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

Hi, * Johan Hovold [180507 03:03]: > On Fri, May 04, 2018 at 01:42:13PM +0200, Sebastian Reichel wrote: > > > Having said all of this, serdev does not yet support runtime PM (at > > all). Tony is currently looking into it. Fortunately serdev allows > > us to enable runtime PM by default (once

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

On Fri, May 04, 2018 at 02:04:15PM +0200, H. Nikolaus Schaller wrote: > It should be possible to cover this by a timer that is started > in this case. If there is serdev data received after assuming the module > is turned off, the driver has detected the wrong case - and can safely > close the

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

On Fri, May 04, 2018 at 02:04:15PM +0200, H. Nikolaus Schaller wrote: > It should be possible to cover this by a timer that is started > in this case. If there is serdev data received after assuming the module > is turned off, the driver has detected the wrong case - and can safely > close the

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

On Fri, May 04, 2018 at 01:42:13PM +0200, Sebastian Reichel wrote: > Having said all of this, serdev does not yet support runtime PM (at > all). Tony is currently looking into it. Fortunately serdev allows > us to enable runtime PM by default (once implemented), since we know > the remote side

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

On Fri, May 04, 2018 at 01:42:13PM +0200, Sebastian Reichel wrote: > Having said all of this, serdev does not yet support runtime PM (at > all). Tony is currently looking into it. Fortunately serdev allows > us to enable runtime PM by default (once implemented), since we know > the remote side

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

On Thu, May 03, 2018 at 11:35:21AM +0200, H. Nikolaus Schaller wrote: > I have realized that the w2sg0004 is an exception (although a Sirf chip) > that it does not provide a WAKEUP signal. And another significant > difference is that we have to keep the serdev UART enabled even if there > is no

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

On Thu, May 03, 2018 at 11:35:21AM +0200, H. Nikolaus Schaller wrote: > I have realized that the w2sg0004 is an exception (although a Sirf chip) > that it does not provide a WAKEUP signal. And another significant > difference is that we have to keep the serdev UART enabled even if there > is no

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

On Wed, May 02, 2018 at 08:16:11AM -0500, Rob Herring wrote: > On Wed, May 2, 2018 at 3:16 AM, Johan Hovold wrote: > > On Tue, May 01, 2018 at 09:05:42AM -0500, Rob Herring wrote: > >> On Thu, Apr 26, 2018 at 4:10 AM, Johan Hovold wrote: > >> > On Wed, Apr 25,

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

On Wed, May 02, 2018 at 08:16:11AM -0500, Rob Herring wrote: > On Wed, May 2, 2018 at 3:16 AM, Johan Hovold wrote: > > On Tue, May 01, 2018 at 09:05:42AM -0500, Rob Herring wrote: > >> On Thu, Apr 26, 2018 at 4:10 AM, Johan Hovold wrote: > >> > On Wed, Apr 25, 2018 at 01:16:58PM -0500, Rob

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

* Sebastian Reichel [180504 13:40]: > Hi, > > On Fri, May 04, 2018 at 02:04:15PM +0200, H. Nikolaus Schaller wrote: > > > Am 04.05.2018 um 13:42 schrieb Sebastian Reichel : > > >> I think it does not need much more (if at all) than a gpio controller on > > >>

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

* Sebastian Reichel [180504 13:40]: > Hi, > > On Fri, May 04, 2018 at 02:04:15PM +0200, H. Nikolaus Schaller wrote: > > > Am 04.05.2018 um 13:42 schrieb Sebastian Reichel : > > >> I think it does not need much more (if at all) than a gpio controller on > > >> the OMAP3 chip (I think the clocks

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

Hi, On Fri, May 04, 2018 at 02:04:15PM +0200, H. Nikolaus Schaller wrote: > > Am 04.05.2018 um 13:42 schrieb Sebastian Reichel : > >> I think it does not need much more (if at all) than a gpio controller on > >> the OMAP3 chip (I think the clocks are active anyways for use by the

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

Hi, On Fri, May 04, 2018 at 02:04:15PM +0200, H. Nikolaus Schaller wrote: > > Am 04.05.2018 um 13:42 schrieb Sebastian Reichel : > >> I think it does not need much more (if at all) than a gpio controller on > >> the OMAP3 chip (I think the clocks are active anyways for use by the other > >>

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

Hi Sebastian, > Am 04.05.2018 um 13:42 schrieb Sebastian Reichel : > > [+cc Tony] > > Hi, > >> I think it does not need much more (if at all) than a gpio controller on >> the OMAP3 chip (I think the clocks are active anyways for use by the other >> UARTs). >> >> We had

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

Hi Sebastian, > Am 04.05.2018 um 13:42 schrieb Sebastian Reichel : > > [+cc Tony] > > Hi, > >> I think it does not need much more (if at all) than a gpio controller on >> the OMAP3 chip (I think the clocks are active anyways for use by the other >> UARTs). >> >> We had proposed years ago to

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

[+cc Tony] Hi, On Fri, May 04, 2018 at 07:16:00AM +0200, H. Nikolaus Schaller wrote: > > Am 03.05.2018 um 20:50 schrieb Andreas Kemnade : > > On Thu, 3 May 2018 11:35:21 +0200 > > H. Nikolaus Schaller wrote: > > > >> I have realized that the w2sg0004

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

[+cc Tony] Hi, On Fri, May 04, 2018 at 07:16:00AM +0200, H. Nikolaus Schaller wrote: > > Am 03.05.2018 um 20:50 schrieb Andreas Kemnade : > > On Thu, 3 May 2018 11:35:21 +0200 > > H. Nikolaus Schaller wrote: > > > >> I have realized that the w2sg0004 is an exception (although a Sirf chip) > >>

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

Hi Andreas, > Am 03.05.2018 um 20:50 schrieb Andreas Kemnade : > > On Thu, 3 May 2018 11:35:21 +0200 > H. Nikolaus Schaller wrote: > > >> I have realized that the w2sg0004 is an exception (although a Sirf chip) >> that it does not provide a WAKEUP

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

Hi Andreas, > Am 03.05.2018 um 20:50 schrieb Andreas Kemnade : > > On Thu, 3 May 2018 11:35:21 +0200 > H. Nikolaus Schaller wrote: > > >> I have realized that the w2sg0004 is an exception (although a Sirf chip) >> that it does not provide a WAKEUP signal. And another significant >> difference

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

On Thu, 3 May 2018 11:35:21 +0200 H. Nikolaus Schaller wrote: > I have realized that the w2sg0004 is an exception (although a Sirf chip) > that it does not provide a WAKEUP signal. And another significant > difference is that we have to keep the serdev UART enabled even if

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

On Thu, 3 May 2018 11:35:21 +0200 H. Nikolaus Schaller wrote: > I have realized that the w2sg0004 is an exception (although a Sirf chip) > that it does not provide a WAKEUP signal. And another significant > difference is that we have to keep the serdev UART enabled even if there > is no

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

On Wed, May 2, 2018 at 3:16 AM, Johan Hovold wrote: > On Tue, May 01, 2018 at 09:05:42AM -0500, Rob Herring wrote: >> On Thu, Apr 26, 2018 at 4:10 AM, Johan Hovold wrote: >> > On Wed, Apr 25, 2018 at 01:16:58PM -0500, Rob Herring wrote: >> >> On Tue, Apr 24,

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

On Wed, May 2, 2018 at 3:16 AM, Johan Hovold wrote: > On Tue, May 01, 2018 at 09:05:42AM -0500, Rob Herring wrote: >> On Thu, Apr 26, 2018 at 4:10 AM, Johan Hovold wrote: >> > On Wed, Apr 25, 2018 at 01:16:58PM -0500, Rob Herring wrote: >> >> On Tue, Apr 24, 2018 at 11:34 AM, Johan Hovold

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

On Tue, May 01, 2018 at 09:05:42AM -0500, Rob Herring wrote: > On Thu, Apr 26, 2018 at 4:10 AM, Johan Hovold wrote: > > On Wed, Apr 25, 2018 at 01:16:58PM -0500, Rob Herring wrote: > >> On Tue, Apr 24, 2018 at 11:34 AM, Johan Hovold wrote: > >> > Add binding

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

On Tue, May 01, 2018 at 09:05:42AM -0500, Rob Herring wrote: > On Thu, Apr 26, 2018 at 4:10 AM, Johan Hovold wrote: > > On Wed, Apr 25, 2018 at 01:16:58PM -0500, Rob Herring wrote: > >> On Tue, Apr 24, 2018 at 11:34 AM, Johan Hovold wrote: > >> > Add binding for u-blox GNSS receivers. > >> > >

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

On Thu, Apr 26, 2018 at 4:10 AM, Johan Hovold wrote: > On Wed, Apr 25, 2018 at 01:16:58PM -0500, Rob Herring wrote: >> On Tue, Apr 24, 2018 at 11:34 AM, Johan Hovold wrote: >> > Add binding for u-blox GNSS receivers. >> > >> > Note that the u-blox product

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

On Thu, Apr 26, 2018 at 4:10 AM, Johan Hovold wrote: > On Wed, Apr 25, 2018 at 01:16:58PM -0500, Rob Herring wrote: >> On Tue, Apr 24, 2018 at 11:34 AM, Johan Hovold wrote: >> > Add binding for u-blox GNSS receivers. >> > >> > Note that the u-blox product names encodes form factor (e.g. "neo"),

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

On Wed, Apr 25, 2018 at 01:16:58PM -0500, Rob Herring wrote: > On Tue, Apr 24, 2018 at 11:34 AM, Johan Hovold wrote: > > Add binding for u-blox GNSS receivers. > > > > Note that the u-blox product names encodes form factor (e.g. "neo"), > > chipset (e.g. "8") and variant (e.g.

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

On Wed, Apr 25, 2018 at 01:16:58PM -0500, Rob Herring wrote: > On Tue, Apr 24, 2018 at 11:34 AM, Johan Hovold wrote: > > Add binding for u-blox GNSS receivers. > > > > Note that the u-blox product names encodes form factor (e.g. "neo"), > > chipset (e.g. "8") and variant (e.g. "q"), but that only

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

On Tue, Apr 24, 2018 at 11:34 AM, Johan Hovold wrote: > Add binding for u-blox GNSS receivers. > > Note that the u-blox product names encodes form factor (e.g. "neo"), > chipset (e.g. "8") and variant (e.g. "q"), but that only formfactor and > chipset is used for the compatible

Re: [PATCH 4/7] dt-bindings: gnss: add u-blox binding

On Tue, Apr 24, 2018 at 11:34 AM, Johan Hovold wrote: > Add binding for u-blox GNSS receivers. > > Note that the u-blox product names encodes form factor (e.g. "neo"), > chipset (e.g. "8") and variant (e.g. "q"), but that only formfactor and > chipset is used for the compatible strings (for now).

[PATCH 4/7] dt-bindings: gnss: add u-blox binding

Add binding for u-blox GNSS receivers. Note that the u-blox product names encodes form factor (e.g. "neo"), chipset (e.g. "8") and variant (e.g. "q"), but that only formfactor and chipset is used for the compatible strings (for now). Signed-off-by: Johan Hovold ---

[PATCH 4/7] dt-bindings: gnss: add u-blox binding

Add binding for u-blox GNSS receivers. Note that the u-blox product names encodes form factor (e.g. "neo"), chipset (e.g. "8") and variant (e.g. "q"), but that only formfactor and chipset is used for the compatible strings (for now). Signed-off-by: Johan Hovold ---