All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paul Bolle <pebolle@tiscali.nl>
To: Daniel Lezcano <daniel.lezcano@linaro.org>
Cc: Lee Jones <lee.jones@linaro.org>,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, kernel@stlinux.com,
	devicetree@vger.kernel.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: Mon, 11 May 2015 11:22:50 +0200	[thread overview]
Message-ID: <1431336170.2398.73.camel@x220> (raw)
In-Reply-To: <55506CB0.2030909@linaro.org>

On Mon, 2015-05-11 at 10:47 +0200, Daniel Lezcano wrote:
> The general policy is to have non-prompted options in the clocksource's 
> Kconfig and let the platform's config to select the right components.

If you want to stick to that policy here there's no reason to add
CLKSRC_ST_LPC_TIMER_SCHED_CLOCK in this patch. It currently adds
nothing, as it will always be equal to whatever value CLKSRC_ST_LPC will
have, right?


Paul Bolle


WARNING: multiple messages have this Message-ID (diff)
From: Paul Bolle <pebolle@tiscali.nl>
To: Daniel Lezcano <daniel.lezcano@linaro.org>
Cc: Lee Jones <lee.jones@linaro.org>,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, kernel@stlinux.com,
	devicetree@vger.kernel.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: Mon, 11 May 2015 11:22:50 +0200	[thread overview]
Message-ID: <1431336170.2398.73.camel@x220> (raw)
In-Reply-To: <55506CB0.2030909@linaro.org>

On Mon, 2015-05-11 at 10:47 +0200, Daniel Lezcano wrote:
> The general policy is to have non-prompted options in the clocksource's 
> Kconfig and let the platform's config to select the right components.

If you want to stick to that policy here there's no reason to add
CLKSRC_ST_LPC_TIMER_SCHED_CLOCK in this patch. It currently adds
nothing, as it will always be equal to whatever value CLKSRC_ST_LPC will
have, right?


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: 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: Mon, 11 May 2015 11:22:50 +0200	[thread overview]
Message-ID: <1431336170.2398.73.camel@x220> (raw)
In-Reply-To: <55506CB0.2030909@linaro.org>

On Mon, 2015-05-11 at 10:47 +0200, Daniel Lezcano wrote:
> The general policy is to have non-prompted options in the clocksource's 
> Kconfig and let the platform's config to select the right components.

If you want to stick to that policy here there's no reason to add
CLKSRC_ST_LPC_TIMER_SCHED_CLOCK in this patch. It currently adds
nothing, as it will always be equal to whatever value CLKSRC_ST_LPC will
have, right?


Paul Bolle

  reply	other threads:[~2015-05-11  9:23 UTC|newest]

Thread overview: 170+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-07 13:38 [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 ` [rtc-linux] " Lee Jones
2015-05-07 13:38 ` [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   ` [rtc-linux] " Lee Jones
2015-05-07 13:38 ` [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   ` [rtc-linux] " Lee Jones
2015-05-07 15:04   ` [STLinux Kernel] " Peter Griffin
2015-05-07 15:04     ` Peter Griffin
2015-05-07 15:04     ` Peter Griffin
2015-05-07 15:04     ` [rtc-linux] " Peter Griffin
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-07 19:13       ` [rtc-linux] " Lee Jones
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:09     ` [rtc-linux] " Paul Bolle
2015-05-08 10:28     ` Lee Jones
2015-05-08 10:28       ` Lee Jones
2015-05-08 10:28       ` Lee Jones
2015-05-08 10:28       ` [rtc-linux] " Lee Jones
2015-05-08 11:30       ` Paul Bolle
2015-05-08 11:30         ` Paul Bolle
2015-05-08 11:30         ` [rtc-linux] " Paul Bolle
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  8:42     ` [rtc-linux] " Daniel Lezcano
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-11 10:53       ` [rtc-linux] " Lee Jones
2015-05-07 13:38 ` [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   ` [rtc-linux] " Lee Jones
2015-05-08  8:44   ` Paul Bolle
2015-05-08  8:44     ` Paul Bolle
2015-05-08  8:44     ` Paul Bolle
2015-05-08  8:44     ` [rtc-linux] " Paul Bolle
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:36       ` [rtc-linux] " Lee Jones
2015-05-08  9:52       ` Paul Bolle
2015-05-08  9:52         ` Paul Bolle
2015-05-08  9:52         ` Paul Bolle
2015-05-08  9:52         ` [rtc-linux] " Paul Bolle
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 10:29           ` [rtc-linux] " Lee Jones
2015-05-08 11:47           ` Paul Bolle
2015-05-08 11:47             ` Paul Bolle
2015-05-08 11:47             ` Paul Bolle
2015-05-08 11:47             ` [rtc-linux] " Paul Bolle
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  8:47               ` [rtc-linux] " Daniel Lezcano
2015-05-11  9:22               ` Paul Bolle [this message]
2015-05-11  9:22                 ` Paul Bolle
2015-05-11  9:22                 ` [rtc-linux] " Paul Bolle
2015-05-07 13:38 ` [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   ` [rtc-linux] " Lee Jones
2015-05-07 13:38 ` [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   ` [rtc-linux] " Lee Jones
2015-05-08  0:04   ` Guenter Roeck
2015-05-08  0:04     ` Guenter Roeck
2015-05-08  0:04     ` Guenter Roeck
2015-05-08  0:04     ` [rtc-linux] " Guenter Roeck
2015-05-07 13:38 ` [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   ` [rtc-linux] " Lee Jones
2015-05-07 14:18   ` [STLinux Kernel] " Peter Griffin
2015-05-07 14:18     ` Peter Griffin
2015-05-07 14:18     ` Peter Griffin
2015-05-07 14:18     ` [rtc-linux] " Peter Griffin
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:19       ` [rtc-linux] " Lee Jones
2015-05-07 19:55       ` Peter Griffin
2015-05-07 19:55         ` Peter Griffin
2015-05-07 19:55         ` Peter Griffin
2015-05-07 19:55         ` [rtc-linux] " Peter Griffin
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  6:49           ` [rtc-linux] " Lee Jones
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:16             ` [rtc-linux] " Peter Griffin
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-08  7:25               ` [rtc-linux] " Lee Jones
2015-05-07 13:38 ` [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   ` [rtc-linux] " Lee Jones
2015-05-09 13:58   ` 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 ` [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   ` [rtc-linux] " Lee Jones
2015-05-07 14:12   ` [STLinux Kernel] " Peter Griffin
2015-05-07 14:12     ` Peter Griffin
2015-05-07 14:12     ` [rtc-linux] " Peter Griffin
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 19:21       ` [rtc-linux] " Lee Jones
2015-05-07 13:38 ` [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   ` [rtc-linux] " Lee Jones
2015-05-07 13:38 ` [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   ` [rtc-linux] " Lee Jones
2015-05-07 13:38 ` [PATCH 11/12] ARM: STi: Enable LPC Clocksource Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38   ` [rtc-linux] " Lee Jones
2015-05-07 13:38 ` [PATCH 12/12] ARM: STi: Update platform level menuconfig 'help' Lee Jones
2015-05-07 13:38   ` Lee Jones
2015-05-07 13:38   ` [rtc-linux] " Lee Jones
2015-05-07 14:15   ` [STLinux Kernel] " Peter Griffin
2015-05-07 14:15     ` Peter Griffin
2015-05-07 14:15     ` [rtc-linux] " Peter Griffin
2015-05-07 19:20     ` Lee Jones
2015-05-07 19:20       ` Lee Jones
2015-05-07 19:20       ` Lee Jones
2015-05-07 19:20       ` [rtc-linux] " 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=1431336170.2398.73.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.