All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paul Bolle <pebolle@tiscali.nl>
To: Lee Jones <lee.jones@linaro.org>
Cc: linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, kernel@stlinux.com,
	devicetree@vger.kernel.org, daniel.lezcano@linaro.org,
	tglx@linutronix.de, wim@iguana.be, a.zummo@towertech.it,
	linux-watchdog@vger.kernel.org, rtc-linux@googlegroups.com,
	linux@roeck-us.net
Subject: [rtc-linux] Re: [PATCH 03/12] clocksource: sti: Provide 'use timer as sched clock' capability
Date: Fri, 08 May 2015 11:52:10 +0200	[thread overview]
Message-ID: <1431078730.13593.14.camel@x220> (raw)
In-Reply-To: <20150508093603.GB29044@x1>

On Fri, 2015-05-08 at 10:36 +0100, Lee Jones wrote:
> On Fri, 08 May 2015, Paul Bolle wrote:
> > On Thu, 2015-05-07 at 14:38 +0100, Lee Jones wrote:
> > > --- a/drivers/clocksource/Kconfig
> > > +++ b/drivers/clocksource/Kconfig
> > 
> > > +config CLKSRC_ST_LPC_TIMER_SCHED_CLOCK
> > > +	bool
> > > +	depends on CLKSRC_ST_LPC
> > > +	default y
> > > +	help
> > > +	  Use Low Power controller timer clock source as sched_clock
> > 
> > This adds, as far as I can see, a Kconfig symbol that will always be in
> > lockstep with CLKSRC_ST_LPC. Ie, CLKSRC_ST_LPC and
> > CLKSRC_ST_LPC_TIMER_SCHED_CLOCK will always both be 'n' or both be 'y'.
> > 
> > Are there plans to break that lockstep in the future?
> 
> What's stopping CONFIG_CLKSRC_ST_LPC_TIMER_SCHED_CLOCK=n?

If CLKSRC_ST_LPC=y you mean? The lack of a prompt, I'd say. Ie, you
would need
	bool "A very short description"

to be able to set these independently.

But now I notice that I missed that there appears to be a problem with
CLKSRC_ST_LPC too. I'll reply to 2/12 shortly.

Thanks,


Paul Bolle

-- 
-- 
You received this message because you are subscribed to "rtc-linux".
Membership options at http://groups.google.com/group/rtc-linux .
Please read http://groups.google.com/group/rtc-linux/web/checklist
before submitting a driver.
--- 
You received this message because you are subscribed to the Google Groups "rtc-linux" group.
To unsubscribe from this group and stop receiving emails from it, send an email to rtc-linux+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

WARNING: multiple messages have this Message-ID (diff)
From: Paul Bolle <pebolle@tiscali.nl>
To: Lee Jones <lee.jones@linaro.org>
Cc: linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, kernel@stlinux.com,
	devicetree@vger.kernel.org, daniel.lezcano@linaro.org,
	tglx@linutronix.de, wim@iguana.be, a.zummo@towertech.it,
	linux-watchdog@vger.kernel.org, rtc-linux@googlegroups.com,
	linux@roeck-us.net
Subject: Re: [PATCH 03/12] clocksource: sti: Provide 'use timer as sched clock' capability
Date: Fri, 08 May 2015 11:52:10 +0200	[thread overview]
Message-ID: <1431078730.13593.14.camel@x220> (raw)
In-Reply-To: <20150508093603.GB29044@x1>

On Fri, 2015-05-08 at 10:36 +0100, Lee Jones wrote:
> On Fri, 08 May 2015, Paul Bolle wrote:
> > On Thu, 2015-05-07 at 14:38 +0100, Lee Jones wrote:
> > > --- a/drivers/clocksource/Kconfig
> > > +++ b/drivers/clocksource/Kconfig
> > 
> > > +config CLKSRC_ST_LPC_TIMER_SCHED_CLOCK
> > > +	bool
> > > +	depends on CLKSRC_ST_LPC
> > > +	default y
> > > +	help
> > > +	  Use Low Power controller timer clock source as sched_clock
> > 
> > This adds, as far as I can see, a Kconfig symbol that will always be in
> > lockstep with CLKSRC_ST_LPC. Ie, CLKSRC_ST_LPC and
> > CLKSRC_ST_LPC_TIMER_SCHED_CLOCK will always both be 'n' or both be 'y'.
> > 
> > Are there plans to break that lockstep in the future?
> 
> What's stopping CONFIG_CLKSRC_ST_LPC_TIMER_SCHED_CLOCK=n?

If CLKSRC_ST_LPC=y you mean? The lack of a prompt, I'd say. Ie, you
would need
	bool "A very short description"

to be able to set these independently.

But now I notice that I missed that there appears to be a problem with
CLKSRC_ST_LPC too. I'll reply to 2/12 shortly.

Thanks,


Paul Bolle


WARNING: multiple messages have this Message-ID (diff)
From: pebolle@tiscali.nl (Paul Bolle)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 03/12] clocksource: sti: Provide 'use timer as sched clock' capability
Date: Fri, 08 May 2015 11:52:10 +0200	[thread overview]
Message-ID: <1431078730.13593.14.camel@x220> (raw)
In-Reply-To: <20150508093603.GB29044@x1>

On Fri, 2015-05-08 at 10:36 +0100, Lee Jones wrote:
> On Fri, 08 May 2015, Paul Bolle wrote:
> > On Thu, 2015-05-07 at 14:38 +0100, Lee Jones wrote:
> > > --- a/drivers/clocksource/Kconfig
> > > +++ b/drivers/clocksource/Kconfig
> > 
> > > +config CLKSRC_ST_LPC_TIMER_SCHED_CLOCK
> > > +	bool
> > > +	depends on CLKSRC_ST_LPC
> > > +	default y
> > > +	help
> > > +	  Use Low Power controller timer clock source as sched_clock
> > 
> > This adds, as far as I can see, a Kconfig symbol that will always be in
> > lockstep with CLKSRC_ST_LPC. Ie, CLKSRC_ST_LPC and
> > CLKSRC_ST_LPC_TIMER_SCHED_CLOCK will always both be 'n' or both be 'y'.
> > 
> > Are there plans to break that lockstep in the future?
> 
> What's stopping CONFIG_CLKSRC_ST_LPC_TIMER_SCHED_CLOCK=n?

If CLKSRC_ST_LPC=y you mean? The lack of a prompt, I'd say. Ie, you
would need
	bool "A very short description"

to be able to set these independently.

But now I notice that I missed that there appears to be a problem with
CLKSRC_ST_LPC too. I'll reply to 2/12 shortly.

Thanks,


Paul Bolle

WARNING: multiple messages have this Message-ID (diff)
From: Paul Bolle <pebolle-IWqWACnzNjzz+pZb47iToQ@public.gmane.org>
To: Lee Jones <lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	kernel-F5mvAk5X5gdBDgjK7y7TUQ@public.gmane.org,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	daniel.lezcano-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org,
	tglx-hfZtesqFncYOwBW4kG4KsQ@public.gmane.org,
	wim-IQzOog9fTRqzQB+pC5nmwQ@public.gmane.org,
	a.zummo-BfzFCNDTiLLj+vYz1yj4TQ@public.gmane.org,
	linux-watchdog-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	rtc-linux-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org,
	linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org
Subject: Re: [PATCH 03/12] clocksource: sti: Provide 'use timer as sched clock' capability
Date: Fri, 08 May 2015 11:52:10 +0200	[thread overview]
Message-ID: <1431078730.13593.14.camel@x220> (raw)
In-Reply-To: <20150508093603.GB29044@x1>

On Fri, 2015-05-08 at 10:36 +0100, Lee Jones wrote:
> On Fri, 08 May 2015, Paul Bolle wrote:
> > On Thu, 2015-05-07 at 14:38 +0100, Lee Jones wrote:
> > > --- a/drivers/clocksource/Kconfig
> > > +++ b/drivers/clocksource/Kconfig
> > 
> > > +config CLKSRC_ST_LPC_TIMER_SCHED_CLOCK
> > > +	bool
> > > +	depends on CLKSRC_ST_LPC
> > > +	default y
> > > +	help
> > > +	  Use Low Power controller timer clock source as sched_clock
> > 
> > This adds, as far as I can see, a Kconfig symbol that will always be in
> > lockstep with CLKSRC_ST_LPC. Ie, CLKSRC_ST_LPC and
> > CLKSRC_ST_LPC_TIMER_SCHED_CLOCK will always both be 'n' or both be 'y'.
> > 
> > Are there plans to break that lockstep in the future?
> 
> What's stopping CONFIG_CLKSRC_ST_LPC_TIMER_SCHED_CLOCK=n?

If CLKSRC_ST_LPC=y you mean? The lack of a prompt, I'd say. Ie, you
would need
	bool "A very short description"

to be able to set these independently.

But now I notice that I missed that there appears to be a problem with
CLKSRC_ST_LPC too. I'll reply to 2/12 shortly.

Thanks,


Paul Bolle

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2015-05-08  9:52 UTC|newest]

Thread overview: 170+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-07 13:38 [rtc-linux] [PATCH 00/12] mfd: watchdog: rtc: Add LPC Clocksource driver Lee Jones
2015-05-07 13:38 ` Lee Jones
2015-05-07 13:38 ` Lee Jones
2015-05-07 13:38 ` Lee Jones
2015-05-07 13:38 ` [rtc-linux] [PATCH 01/12] mfd: dt-bindings: Provide human readable define for Clocksource mode Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38 ` [rtc-linux] [PATCH 02/12] clocksource: sti: Provide support for the ST LPC Clocksource IP Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 15:04   ` [rtc-linux] Re: [STLinux Kernel] " Peter Griffin
2015-05-07 15:04     ` Peter Griffin
2015-05-07 15:04     ` Peter Griffin
2015-05-07 15:04     ` Peter Griffin
2015-05-07 19:13     ` [rtc-linux] " Lee Jones
2015-05-07 19:13       ` Lee Jones
2015-05-07 19:13       ` Lee Jones
2015-05-07 19:13       ` Lee Jones
2015-05-07 19:13       ` Lee Jones
2015-05-08 10:09   ` [rtc-linux] " Paul Bolle
2015-05-08 10:09     ` Paul Bolle
2015-05-08 10:09     ` Paul Bolle
2015-05-08 10:09     ` Paul Bolle
2015-05-08 10:28     ` [rtc-linux] " Lee Jones
2015-05-08 10:28       ` Lee Jones
2015-05-08 10:28       ` Lee Jones
2015-05-08 10:28       ` Lee Jones
2015-05-08 11:30       ` [rtc-linux] " Paul Bolle
2015-05-08 11:30         ` Paul Bolle
2015-05-08 11:30         ` Paul Bolle
2015-05-11  8:42   ` [rtc-linux] " Daniel Lezcano
2015-05-11  8:42     ` Daniel Lezcano
2015-05-11  8:42     ` Daniel Lezcano
2015-05-11  8:42     ` Daniel Lezcano
2015-05-11  8:42     ` Daniel Lezcano
2015-05-11 10:53     ` [rtc-linux] " Lee Jones
2015-05-11 10:53       ` Lee Jones
2015-05-11 10:53       ` Lee Jones
2015-05-11 10:53       ` Lee Jones
2015-05-11 10:53       ` Lee Jones
2015-05-07 13:38 ` [rtc-linux] [PATCH 03/12] clocksource: sti: Provide 'use timer as sched clock' capability Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-08  8:44   ` [rtc-linux] " Paul Bolle
2015-05-08  8:44     ` Paul Bolle
2015-05-08  8:44     ` Paul Bolle
2015-05-08  8:44     ` Paul Bolle
2015-05-08  9:36     ` [rtc-linux] " Lee Jones
2015-05-08  9:36       ` Lee Jones
2015-05-08  9:36       ` Lee Jones
2015-05-08  9:36       ` Lee Jones
2015-05-08  9:36       ` Lee Jones
2015-05-08  9:52       ` Paul Bolle [this message]
2015-05-08  9:52         ` Paul Bolle
2015-05-08  9:52         ` Paul Bolle
2015-05-08  9:52         ` Paul Bolle
2015-05-08 10:29         ` [rtc-linux] " Lee Jones
2015-05-08 10:29           ` Lee Jones
2015-05-08 10:29           ` Lee Jones
2015-05-08 10:29           ` Lee Jones
2015-05-08 10:29           ` Lee Jones
2015-05-08 11:47           ` [rtc-linux] " Paul Bolle
2015-05-08 11:47             ` Paul Bolle
2015-05-08 11:47             ` Paul Bolle
2015-05-08 11:47             ` Paul Bolle
2015-05-11  8:47             ` [rtc-linux] " Daniel Lezcano
2015-05-11  8:47               ` Daniel Lezcano
2015-05-11  8:47               ` Daniel Lezcano
2015-05-11  8:47               ` Daniel Lezcano
2015-05-11  8:47               ` Daniel Lezcano
2015-05-11  9:22               ` [rtc-linux] " Paul Bolle
2015-05-11  9:22                 ` Paul Bolle
2015-05-11  9:22                 ` Paul Bolle
2015-05-07 13:38 ` [rtc-linux] [PATCH 04/12] clocksource: bindings: Provide bindings for ST's LPC Clocksource device Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38 ` [rtc-linux] [PATCH 05/12] watchdog: st_wdt: Update IP layout information to include Clocksource Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-08  0:04   ` [rtc-linux] " Guenter Roeck
2015-05-08  0:04     ` Guenter Roeck
2015-05-08  0:04     ` Guenter Roeck
2015-05-08  0:04     ` Guenter Roeck
2015-05-07 13:38 ` [rtc-linux] [PATCH 06/12] watchdog: bindings: Supply knowledge of a third supported device - clocksource Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 14:18   ` [rtc-linux] Re: [STLinux Kernel] " Peter Griffin
2015-05-07 14:18     ` Peter Griffin
2015-05-07 14:18     ` Peter Griffin
2015-05-07 14:18     ` Peter Griffin
2015-05-07 19:19     ` [rtc-linux] " Lee Jones
2015-05-07 19:19       ` Lee Jones
2015-05-07 19:19       ` Lee Jones
2015-05-07 19:19       ` Lee Jones
2015-05-07 19:19       ` Lee Jones
2015-05-07 19:55       ` [rtc-linux] " Peter Griffin
2015-05-07 19:55         ` Peter Griffin
2015-05-07 19:55         ` Peter Griffin
2015-05-07 19:55         ` Peter Griffin
2015-05-08  6:49         ` [rtc-linux] " Lee Jones
2015-05-08  6:49           ` Lee Jones
2015-05-08  6:49           ` Lee Jones
2015-05-08  6:49           ` Lee Jones
2015-05-08  6:49           ` Lee Jones
2015-05-08  7:16           ` [rtc-linux] " Peter Griffin
2015-05-08  7:16             ` Peter Griffin
2015-05-08  7:16             ` Peter Griffin
2015-05-08  7:16             ` Peter Griffin
2015-05-08  7:25             ` [rtc-linux] " Lee Jones
2015-05-08  7:25               ` Lee Jones
2015-05-08  7:25               ` Lee Jones
2015-05-08  7:25               ` Lee Jones
2015-05-08  7:25               ` Lee Jones
2015-05-07 13:38 ` [rtc-linux] [PATCH 07/12] rtc: st: Update IP layout information to include Clocksource Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-09 13:58   ` [rtc-linux] " Alexandre Belloni
2015-05-09 13:58     ` Alexandre Belloni
2015-05-11 10:54     ` Lee Jones
2015-05-11 10:54       ` Lee Jones
2015-05-11 10:54       ` Lee Jones
2015-05-11 10:54       ` Lee Jones
2015-05-11 11:03       ` Alexandre Belloni
2015-05-11 11:03         ` Alexandre Belloni
2015-05-11 11:03         ` Alexandre Belloni
2015-05-11 11:03         ` Alexandre Belloni
2015-05-11 11:52         ` Lee Jones
2015-05-11 11:52           ` Lee Jones
2015-05-11 11:52           ` Lee Jones
2015-05-11 11:52           ` Lee Jones
2015-05-11 11:52           ` Lee Jones
2015-05-07 13:38 ` [rtc-linux] [PATCH 08/12] rtc: bindings: Supply knowledge of a third supported device - clocksource Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 14:12   ` [rtc-linux] Re: [STLinux Kernel] " Peter Griffin
2015-05-07 14:12     ` Peter Griffin
2015-05-07 14:12     ` Peter Griffin
2015-05-07 19:21     ` [rtc-linux] " Lee Jones
2015-05-07 19:21       ` Lee Jones
2015-05-07 19:21       ` Lee Jones
2015-05-07 19:21       ` Lee Jones
2015-05-07 19:21       ` Lee Jones
2015-05-07 13:38 ` [rtc-linux] [PATCH 09/12] MAINTAINERS: Add the LPC Clocksource to STi maintained driver list Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38 ` [rtc-linux] [PATCH 10/12] ARM: STi: DT: STih407: Switch LPC mode from RTC to Clocksource Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38 ` [rtc-linux] [PATCH 11/12] ARM: STi: Enable LPC Clocksource Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38 ` [rtc-linux] [PATCH 12/12] ARM: STi: Update platform level menuconfig 'help' Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 14:15   ` [rtc-linux] Re: [STLinux Kernel] " Peter Griffin
2015-05-07 14:15     ` Peter Griffin
2015-05-07 14:15     ` Peter Griffin
2015-05-07 19:20     ` [rtc-linux] " Lee Jones
2015-05-07 19:20       ` Lee Jones
2015-05-07 19:20       ` Lee Jones
2015-05-07 19:20       ` Lee Jones

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=1431078730.13593.14.camel@x220 \
    --to=pebolle@tiscali.nl \
    --cc=a.zummo@towertech.it \
    --cc=daniel.lezcano@linaro.org \
    --cc=devicetree@vger.kernel.org \
    --cc=kernel@stlinux.com \
    --cc=lee.jones@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-watchdog@vger.kernel.org \
    --cc=linux@roeck-us.net \
    --cc=rtc-linux@googlegroups.com \
    --cc=tglx@linutronix.de \
    --cc=wim@iguana.be \
    /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.