From: Neil Armstrong <narmstrong@baylibre.com>
To: Daniel Lezcano <daniel.lezcano@linaro.org>, tglx@linutronix.de
Cc: linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org,
"Russell King" <linux@armlinux.org.uk>,
"Michal Simek" <monstr@monstr.eu>,
"John Crispin" <john@phrozen.org>,
"Ralf Baechle" <ralf@linux-mips.org>,
"Ley Foon Tan" <lftan@altera.com>,
"Vineet Gupta" <vgupta@synopsys.com>,
"Mark Rutland" <mark.rutland@arm.com>,
"Marc Zyngier" <marc.zyngier@arm.com>,
"Patrice Chotard" <patrice.chotard@st.com>,
"Maxime Coquelin" <mcoquelin.stm32@gmail.com>,
"Alexandre Torgue" <alexandre.torgue@st.com>,
"Florian Fainelli" <f.fainelli@gmail.com>,
"Ray Jui" <rjui@broadcom.com>,
"Scott Branden" <sbranden@broadcom.com>,
"maintainer:BROADCOM BCM281XX/BCM11XXX/BCM216XX ARM ARCHITE..."
<bcm-kernel-feedback-list@broadcom.com>,
"Stephen Warren" <swarren@wwwdotorg.org>,
"Lee Jones" <lee@kernel.org>, "Eric Anholt" <eric@anholt.net>,
"Sören Brinkmann" <soren.brinkmann@xilinx.com>,
"Linus Walleij" <linus.walleij@linaro.org>,
"Alexander Shiyan" <shc_work@mail.ru>,
"Kukjin Kim" <kgene@kernel.org>,
"Krzysztof Kozlowski" <krzk@kernel.org>,
"Javier Martinez Canillas" <javier@osg.samsung.com>,
"Yoshinori Sato" <ysato@users.sourceforge.jp>,
"Carlo Caione" <carlo@caione.org>,
"Kevin Hilman" <khilman@baylibre.com>,
"Liviu Dudau" <liviu.dudau@arm.com>,
"Sudeep Holla" <sudeep.holla@arm.com>,
"Lorenzo Pieralisi" <lorenzo.pieralisi@arm.com>,
"Matthias Brugger" <matthias.bgg@gmail.com>,
"Heiko Stuebner" <heiko@sntech.de>,
"Maxime Ripard" <maxime.ripard@free-electrons.com>,
"Chen-Yu Tsai" <wens@csie.org>,
"Marc Gonzalez" <marc_gonzalez@sigmadesigns.com>,
"Thierry Reding" <thierry.reding@gmail.com>,
"Alexandre Courbot" <gnurou@gmail.com>,
"Uwe Kleine-König" <kernel@pengutronix.de>,
"Joachim Eastwood" <manabian@gmail.com>,
"Vladimir Zapolskiy" <vz@mleia.com>,
"Sylvain Lemieux" <slemieux.tyco@gmail.com>,
"Barry Song" <baohua@kernel.org>,
"Baruch Siach" <baruch@tkos.co.il>,
"Santosh Shilimkar" <ssantosh@kernel.org>,
"Tony Prisk" <linux@prisktech.co.nz>,
"John Stultz" <john.stultz@linaro.org>,
"Stephen Boyd" <sboyd@codeaurora.org>,
"Anna-Maria Gleixner" <anna-maria@linutronix.de>,
"Richard Cochran" <rcochran@linutronix.de>,
"Ingo Molnar" <mingo@kernel.org>,
"Noam Camus" <noamca@mellanox.com>,
"Arnd Bergmann" <arnd@arndb.de>,
"open list:RALINK MIPS ARCHITECTURE" <linux-mips@linux-mips.org>,
"moderated list:NIOS2 ARCHITECTURE"
<nios2-dev@lists.rocketboards.org>,
"open list:SYNOPSYS ARC ARCHITECTURE"
<linux-snps-arc@lists.infradead.org>,
"open list:ARM/STI ARCHITECTURE" <kernel@stlinux.com>,
"moderated list:BROADCOM BCM2835 ARM ARCHITECTURE"
<linux-rpi-kernel@lists.infradead.org>,
"moderated list:ARM/SAMSUNG EXYNOS ARM ARCHITECTURES"
<linux-samsung-soc@vger.kernel.org>,
"moderated list:H8/300 ARCHITECTURE"
<uclinux-h8-devel@lists.sourceforge.jp>,
"open list:ARM/Amlogic Meson SoC support"
<linux-amlogic@lists.infradead.org>,
"moderated list:ARM/Mediatek SoC support"
<linux-mediatek@lists.infradead.org>,
"open list:ARM/Rockchip SoC support"
<linux-rockchip@lists.infradead.org>,
"open list:TEGRA ARCHITECTURE SUPPORT"
<linux-tegra@vger.kernel.org>,
"moderated list:ARM/OXNAS platform support"
<linux-oxnas@lists.tuxfamily.org>
Subject: Re: [PATCH 2/7] clocksource: Rename CLOCKSOURCE_OF_DECLARE
Date: Mon, 29 May 2017 10:00:46 +0200 [thread overview]
Message-ID: <4ccb7cac-1663-792a-7912-d9ee803c7e2b@baylibre.com> (raw)
In-Reply-To: <1495879129-28109-2-git-send-email-daniel.lezcano@linaro.org>
On 05/27/2017 11:58 AM, Daniel Lezcano wrote:
> The CLOCKSOUCE_OF_DECLARE macro is used widely for the timers to declare the
> clocksource at early stage. However, this macro is also used to initialize
> the clockevent if any, or the clockevent only.
>
> It was originally suggested to declare another macro to initialize a
> clockevent, so in order to separate the two entities even they belong to the
> same IP. This was not accepted because of the impact on the DT where splitting
> a clocksource/clockevent definition does not make sense as it is a Linux
> concept not a hardware description.
>
> On the other side, the clocksource has not interrupt declared while the
> clockevent has, so it is easy from the driver to know if the description is
> for a clockevent or a clocksource, IOW it could be implemented at the driver
> level.
>
> So instead of dealing with a named clocksource macro, let's use a more generic
> one: TIMER_OF_DECLARE.
>
> The patch has not functional changes.
>
> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> ---
> arch/arm/kernel/smp_twd.c | 6 +++---
> arch/microblaze/kernel/timer.c | 2 +-
> arch/mips/ralink/cevt-rt3352.c | 2 +-
> arch/nios2/kernel/time.c | 2 +-
> drivers/clocksource/arc_timer.c | 6 +++---
> drivers/clocksource/arm_arch_timer.c | 6 +++---
> drivers/clocksource/arm_global_timer.c | 2 +-
> drivers/clocksource/armv7m_systick.c | 2 +-
> drivers/clocksource/asm9260_timer.c | 2 +-
> drivers/clocksource/bcm2835_timer.c | 2 +-
> drivers/clocksource/bcm_kona_timer.c | 4 ++--
> drivers/clocksource/cadence_ttc_timer.c | 2 +-
> drivers/clocksource/clksrc-dbx500-prcmu.c | 2 +-
> drivers/clocksource/clksrc_st_lpc.c | 2 +-
> drivers/clocksource/clps711x-timer.c | 2 +-
> drivers/clocksource/dw_apb_timer_of.c | 8 ++++----
> drivers/clocksource/exynos_mct.c | 4 ++--
> drivers/clocksource/fsl_ftm_timer.c | 2 +-
> drivers/clocksource/h8300_timer16.c | 2 +-
> drivers/clocksource/h8300_timer8.c | 2 +-
> drivers/clocksource/h8300_tpu.c | 2 +-
> drivers/clocksource/jcore-pit.c | 2 +-
> drivers/clocksource/meson6_timer.c | 2 +-
> drivers/clocksource/mips-gic-timer.c | 2 +-
> drivers/clocksource/mps2-timer.c | 2 +-
> drivers/clocksource/mtk_timer.c | 2 +-
> drivers/clocksource/mxs_timer.c | 2 +-
> drivers/clocksource/nomadik-mtu.c | 2 +-
> drivers/clocksource/pxa_timer.c | 2 +-
> drivers/clocksource/qcom-timer.c | 4 ++--
> drivers/clocksource/renesas-ostm.c | 2 +-
> drivers/clocksource/rockchip_timer.c | 4 ++--
> drivers/clocksource/samsung_pwm_timer.c | 8 ++++----
> drivers/clocksource/sun4i_timer.c | 2 +-
> drivers/clocksource/tango_xtal.c | 2 +-
> drivers/clocksource/tegra20_timer.c | 4 ++--
> drivers/clocksource/time-armada-370-xp.c | 6 +++---
> drivers/clocksource/time-efm32.c | 4 ++--
> drivers/clocksource/time-lpc32xx.c | 2 +-
> drivers/clocksource/time-orion.c | 2 +-
> drivers/clocksource/time-pistachio.c | 2 +-
> drivers/clocksource/timer-atlas7.c | 2 +-
> drivers/clocksource/timer-atmel-pit.c | 2 +-
> drivers/clocksource/timer-atmel-st.c | 2 +-
> drivers/clocksource/timer-digicolor.c | 2 +-
> drivers/clocksource/timer-fttmr010.c | 10 +++++-----
> drivers/clocksource/timer-imx-gpt.c | 24 ++++++++++++------------
> drivers/clocksource/timer-integrator-ap.c | 2 +-
> drivers/clocksource/timer-keystone.c | 2 +-
> drivers/clocksource/timer-nps.c | 6 +++---
> drivers/clocksource/timer-oxnas-rps.c | 4 ++--
> drivers/clocksource/timer-prima2.c | 2 +-
> drivers/clocksource/timer-sp804.c | 4 ++--
> drivers/clocksource/timer-stm32.c | 2 +-
> drivers/clocksource/timer-sun5i.c | 4 ++--
> drivers/clocksource/timer-ti-32k.c | 2 +-
> drivers/clocksource/timer-u300.c | 2 +-
> drivers/clocksource/versatile.c | 4 ++--
> drivers/clocksource/vf_pit_timer.c | 2 +-
> drivers/clocksource/vt8500_timer.c | 2 +-
> drivers/clocksource/zevio-timer.c | 2 +-
> include/linux/clocksource.h | 2 +-
> 62 files changed, 103 insertions(+), 103 deletions(-)
>
[...]
> diff --git a/drivers/clocksource/timer-oxnas-rps.c b/drivers/clocksource/timer-oxnas-rps.c
> index d630bf4..eed6fef 100644
> --- a/drivers/clocksource/timer-oxnas-rps.c
> +++ b/drivers/clocksource/timer-oxnas-rps.c
> @@ -293,7 +293,7 @@ static int __init oxnas_rps_timer_init(struct device_node *np)
> return ret;
> }
>
> -CLOCKSOURCE_OF_DECLARE(ox810se_rps,
> +TIMER_OF_DECLARE(ox810se_rps,
> "oxsemi,ox810se-rps-timer", oxnas_rps_timer_init);
> -CLOCKSOURCE_OF_DECLARE(ox820_rps,
> +TIMER_OF_DECLARE(ox820_rps,
> "oxsemi,ox820se-rps-timer", oxnas_rps_timer_init);
[...]
For the timer-oxnas-rps driver,
Acked-by: Neil Armstrong <narmstrong@baylibre.com>
Thanks for the rework,
Neil
next prev parent reply other threads:[~2017-05-29 8:01 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1495879129-28109-1-git-send-email-daniel.lezcano@linaro.org>
2017-05-27 9:58 ` [PATCH 2/7] clocksource: Rename CLOCKSOURCE_OF_DECLARE Daniel Lezcano
2017-05-27 9:58 ` Daniel Lezcano
2017-05-28 13:48 ` Linus Walleij
2017-05-28 14:20 ` Daniel Lezcano
2017-05-29 7:39 ` Heiko Stübner
2017-05-29 8:00 ` Neil Armstrong [this message]
2017-05-29 8:41 ` Arnd Bergmann
2017-05-29 8:48 ` Daniel Lezcano
2017-05-29 9:57 ` Arnd Bergmann
2017-05-29 10:55 ` Daniel Lezcano
2017-05-29 11:21 ` Arnd Bergmann
2017-05-29 13:25 ` Matthias Brugger
2017-05-30 16:36 ` Vineet Gupta
2017-05-30 16:36 ` Vineet Gupta
2017-05-31 7:11 ` Patrice CHOTARD
2017-05-31 8:19 ` Krzysztof Kozlowski
2017-05-27 9:58 ` [PATCH 3/7] clocksource: Rename clocksource_probe Daniel Lezcano
2017-05-27 9:58 ` Daniel Lezcano
2017-05-28 13:49 ` Linus Walleij
2017-05-29 4:34 ` Viresh Kumar
2017-05-29 7:37 ` Heiko Stübner
2017-05-31 10:29 ` James Hartley
2017-05-31 10:29 ` James Hartley
2017-05-27 9:58 ` [PATCH 6/7] clocksource: Rename CLKSRC_OF to TIMER_OF Daniel Lezcano
2017-05-27 9:58 ` Daniel Lezcano
2017-05-28 3:00 ` Shawn Guo
2017-05-28 3:00 ` Shawn Guo
2017-05-28 9:18 ` Daniel Lezcano
2017-05-28 9:18 ` Daniel Lezcano
2017-05-28 13:52 ` Linus Walleij
2017-05-28 13:52 ` Linus Walleij
2017-05-31 8:24 ` Krzysztof Kozlowski
2017-05-31 8:24 ` Krzysztof Kozlowski
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=4ccb7cac-1663-792a-7912-d9ee803c7e2b@baylibre.com \
--to=narmstrong@baylibre.com \
--cc=alexandre.torgue@st.com \
--cc=anna-maria@linutronix.de \
--cc=arnd@arndb.de \
--cc=baohua@kernel.org \
--cc=baruch@tkos.co.il \
--cc=bcm-kernel-feedback-list@broadcom.com \
--cc=carlo@caione.org \
--cc=daniel.lezcano@linaro.org \
--cc=eric@anholt.net \
--cc=f.fainelli@gmail.com \
--cc=gnurou@gmail.com \
--cc=heiko@sntech.de \
--cc=javier@osg.samsung.com \
--cc=john.stultz@linaro.org \
--cc=john@phrozen.org \
--cc=kernel@pengutronix.de \
--cc=kernel@stlinux.com \
--cc=kgene@kernel.org \
--cc=khilman@baylibre.com \
--cc=krzk@kernel.org \
--cc=lee@kernel.org \
--cc=lftan@altera.com \
--cc=linus.walleij@linaro.org \
--cc=linux-amlogic@lists.infradead.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=linux-mips@linux-mips.org \
--cc=linux-oxnas@lists.tuxfamily.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=linux-rpi-kernel@lists.infradead.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=linux-snps-arc@lists.infradead.org \
--cc=linux-tegra@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=linux@prisktech.co.nz \
--cc=liviu.dudau@arm.com \
--cc=lorenzo.pieralisi@arm.com \
--cc=manabian@gmail.com \
--cc=marc.zyngier@arm.com \
--cc=marc_gonzalez@sigmadesigns.com \
--cc=mark.rutland@arm.com \
--cc=matthias.bgg@gmail.com \
--cc=maxime.ripard@free-electrons.com \
--cc=mcoquelin.stm32@gmail.com \
--cc=mingo@kernel.org \
--cc=monstr@monstr.eu \
--cc=nios2-dev@lists.rocketboards.org \
--cc=noamca@mellanox.com \
--cc=patrice.chotard@st.com \
--cc=ralf@linux-mips.org \
--cc=rcochran@linutronix.de \
--cc=rjui@broadcom.com \
--cc=sboyd@codeaurora.org \
--cc=sbranden@broadcom.com \
--cc=shc_work@mail.ru \
--cc=slemieux.tyco@gmail.com \
--cc=soren.brinkmann@xilinx.com \
--cc=ssantosh@kernel.org \
--cc=sudeep.holla@arm.com \
--cc=swarren@wwwdotorg.org \
--cc=tglx@linutronix.de \
--cc=thierry.reding@gmail.com \
--cc=uclinux-h8-devel@lists.sourceforge.jp \
--cc=vgupta@synopsys.com \
--cc=vz@mleia.com \
--cc=wens@csie.org \
--cc=ysato@users.sourceforge.jp \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox