From: Nicolas Ferre <nicolas.ferre@atmel.com>
To: Boris BREZILLON <boris.brezillon@free-electrons.com>,
Jean-Christophe Plagniol-Villard <plagnioj@jcrosoft.com>,
Alexandre Belloni <alexandre.belloni@free-electrons.com>,
Andrew Victor <linux@maxim.org.za>,
Alessandro Zummo <a.zummo@towertech.it>,
rtc-linux@googlegroups.com, Samuel Ortiz <sameo@linux.intel.com>,
Lee Jones <lee.jones@linaro.org>,
Johan Hovold <jhovold@gmail.com>
Cc: Mark Rutland <mark.rutland@arm.com>,
devicetree@vger.kernel.org, Pawel Moll <pawel.moll@arm.com>,
Ian Campbell <ijc+devicetree@hellion.org.uk>,
linux-kernel@vger.kernel.org, Rob Herring <robh+dt@kernel.org>,
Kumar Gala <galak@codeaurora.org>,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v6 6/9] ARM: at91: add clk_lookup entry for RTT devices
Date: Tue, 23 Sep 2014 10:23:16 +0200 [thread overview]
Message-ID: <54212DF4.1000301@atmel.com> (raw)
In-Reply-To: <1411400563-570-7-git-send-email-boris.brezillon@free-electrons.com>
On 22/09/2014 17:42, Boris BREZILLON :
> First export the clk32k clk.
> Then add clk_lookup entries for RTT devices so that rtc-at91sam9 driver
> can retrieve and manipulate the slow clk.
>
> Signed-off-by: Boris BREZILLON <boris.brezillon@free-electrons.com>
> Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
Ok until we remove the !CCF for the sam9's:
Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>
> ---
> arch/arm/mach-at91/at91sam9260.c | 2 ++
> arch/arm/mach-at91/at91sam9261.c | 2 ++
> arch/arm/mach-at91/at91sam9263.c | 4 ++++
> arch/arm/mach-at91/at91sam9g45.c | 2 ++
> arch/arm/mach-at91/at91sam9rl.c | 2 ++
> arch/arm/mach-at91/clock.c | 2 +-
> arch/arm/mach-at91/clock.h | 1 +
> 7 files changed, 14 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-at91/at91sam9260.c b/arch/arm/mach-at91/at91sam9260.c
> index 3477ba9..a853d4c 100644
> --- a/arch/arm/mach-at91/at91sam9260.c
> +++ b/arch/arm/mach-at91/at91sam9260.c
> @@ -217,6 +217,7 @@ static struct clk_lookup periph_clocks_lookups[] = {
> CLKDEV_CON_DEV_ID("pclk", "fffbc000.ssc", &ssc_clk),
> CLKDEV_CON_DEV_ID(NULL, "i2c-at91sam9260.0", &twi_clk),
> CLKDEV_CON_DEV_ID(NULL, "i2c-at91sam9g20.0", &twi_clk),
> + CLKDEV_CON_DEV_ID(NULL, "rtc-at91sam9.0", &clk32k),
> /* more usart lookup table for DT entries */
> CLKDEV_CON_DEV_ID("usart", "fffff200.serial", &mck),
> CLKDEV_CON_DEV_ID("usart", "fffb0000.serial", &usart0_clk),
> @@ -237,6 +238,7 @@ static struct clk_lookup periph_clocks_lookups[] = {
> CLKDEV_CON_DEV_ID("mci_clk", "fffa8000.mmc", &mmc_clk),
> CLKDEV_CON_DEV_ID("spi_clk", "fffc8000.spi", &spi0_clk),
> CLKDEV_CON_DEV_ID("spi_clk", "fffcc000.spi", &spi1_clk),
> + CLKDEV_CON_DEV_ID(NULL, "fffffd20.rtc", &clk32k),
> /* fake hclk clock */
> CLKDEV_CON_DEV_ID("hclk", "at91_ohci", &ohci_clk),
> CLKDEV_CON_ID("pioA", &pioA_clk),
> diff --git a/arch/arm/mach-at91/at91sam9261.c b/arch/arm/mach-at91/at91sam9261.c
> index fb164a5..3f5a299 100644
> --- a/arch/arm/mach-at91/at91sam9261.c
> +++ b/arch/arm/mach-at91/at91sam9261.c
> @@ -192,6 +192,7 @@ static struct clk_lookup periph_clocks_lookups[] = {
> CLKDEV_CON_ID("pioA", &pioA_clk),
> CLKDEV_CON_ID("pioB", &pioB_clk),
> CLKDEV_CON_ID("pioC", &pioC_clk),
> + CLKDEV_CON_DEV_ID(NULL, "rtc-at91sam9.0", &clk32k),
> /* more lookup table for DT entries */
> CLKDEV_CON_DEV_ID("usart", "fffff200.serial", &mck),
> CLKDEV_CON_DEV_ID("usart", "fffb0000.serial", &usart0_clk),
> @@ -209,6 +210,7 @@ static struct clk_lookup periph_clocks_lookups[] = {
> CLKDEV_CON_DEV_ID(NULL, "fffff400.gpio", &pioA_clk),
> CLKDEV_CON_DEV_ID(NULL, "fffff600.gpio", &pioB_clk),
> CLKDEV_CON_DEV_ID(NULL, "fffff800.gpio", &pioC_clk),
> + CLKDEV_CON_DEV_ID(NULL, "fffffd20.rtc", &clk32k),
> };
>
> static struct clk_lookup usart_clocks_lookups[] = {
> diff --git a/arch/arm/mach-at91/at91sam9263.c b/arch/arm/mach-at91/at91sam9263.c
> index 810fa5f..bef5b96 100644
> --- a/arch/arm/mach-at91/at91sam9263.c
> +++ b/arch/arm/mach-at91/at91sam9263.c
> @@ -201,6 +201,8 @@ static struct clk_lookup periph_clocks_lookups[] = {
> CLKDEV_CON_DEV_ID("t0_clk", "atmel_tcb.0", &tcb_clk),
> CLKDEV_CON_DEV_ID(NULL, "i2c-at91sam9260.0", &twi_clk),
> CLKDEV_CON_DEV_ID(NULL, "at91sam9rl-pwm", &pwm_clk),
> + CLKDEV_CON_DEV_ID(NULL, "rtc-at91sam9.0", &clk32k),
> + CLKDEV_CON_DEV_ID(NULL, "rtc-at91sam9.1", &clk32k),
> /* fake hclk clock */
> CLKDEV_CON_DEV_ID("hclk", "at91_ohci", &ohci_clk),
> CLKDEV_CON_ID("pioA", &pioA_clk),
> @@ -227,6 +229,8 @@ static struct clk_lookup periph_clocks_lookups[] = {
> CLKDEV_CON_DEV_ID(NULL, "fffff800.gpio", &pioCDE_clk),
> CLKDEV_CON_DEV_ID(NULL, "fffffa00.gpio", &pioCDE_clk),
> CLKDEV_CON_DEV_ID(NULL, "fffb8000.pwm", &pwm_clk),
> + CLKDEV_CON_DEV_ID(NULL, "fffffd20.rtc", &clk32k),
> + CLKDEV_CON_DEV_ID(NULL, "fffffd50.rtc", &clk32k),
> };
>
> static struct clk_lookup usart_clocks_lookups[] = {
> diff --git a/arch/arm/mach-at91/at91sam9g45.c b/arch/arm/mach-at91/at91sam9g45.c
> index 9d45496..69fdfcc 100644
> --- a/arch/arm/mach-at91/at91sam9g45.c
> +++ b/arch/arm/mach-at91/at91sam9g45.c
> @@ -253,6 +253,7 @@ static struct clk_lookup periph_clocks_lookups[] = {
> CLKDEV_CON_DEV_ID(NULL, "atmel_tdes", &aestdessha_clk),
> CLKDEV_CON_DEV_ID(NULL, "atmel_aes", &aestdessha_clk),
> CLKDEV_CON_DEV_ID(NULL, "at91sam9rl-pwm", &pwm_clk),
> + CLKDEV_CON_DEV_ID(NULL, "rtc-at91sam9.0", &clk32k),
> /* more usart lookup table for DT entries */
> CLKDEV_CON_DEV_ID("usart", "ffffee00.serial", &mck),
> CLKDEV_CON_DEV_ID("usart", "fff8c000.serial", &usart0_clk),
> @@ -279,6 +280,7 @@ static struct clk_lookup periph_clocks_lookups[] = {
> CLKDEV_CON_DEV_ID(NULL, "fffff600.gpio", &pioC_clk),
> CLKDEV_CON_DEV_ID(NULL, "fffff800.gpio", &pioDE_clk),
> CLKDEV_CON_DEV_ID(NULL, "fffffa00.gpio", &pioDE_clk),
> + CLKDEV_CON_DEV_ID(NULL, "fffffd20.rtc", &clk32k),
>
> CLKDEV_CON_ID("pioA", &pioA_clk),
> CLKDEV_CON_ID("pioB", &pioB_clk),
> diff --git a/arch/arm/mach-at91/at91sam9rl.c b/arch/arm/mach-at91/at91sam9rl.c
> index 878d501..a82dc61 100644
> --- a/arch/arm/mach-at91/at91sam9rl.c
> +++ b/arch/arm/mach-at91/at91sam9rl.c
> @@ -205,6 +205,7 @@ static struct clk_lookup periph_clocks_lookups[] = {
> CLKDEV_CON_ID("pioB", &pioB_clk),
> CLKDEV_CON_ID("pioC", &pioC_clk),
> CLKDEV_CON_ID("pioD", &pioD_clk),
> + CLKDEV_CON_DEV_ID(NULL, "rtc-at91sam9.0", &clk32k),
> /* more lookup table for DT entries */
> CLKDEV_CON_DEV_ID("usart", "fffff200.serial", &mck),
> CLKDEV_CON_DEV_ID("usart", "fffb0000.serial", &usart0_clk),
> @@ -223,6 +224,7 @@ static struct clk_lookup periph_clocks_lookups[] = {
> CLKDEV_CON_DEV_ID(NULL, "fffff600.gpio", &pioB_clk),
> CLKDEV_CON_DEV_ID(NULL, "fffff800.gpio", &pioC_clk),
> CLKDEV_CON_DEV_ID(NULL, "fffffa00.gpio", &pioD_clk),
> + CLKDEV_CON_DEV_ID(NULL, "fffffd20.rtc", &clk32k),
> CLKDEV_CON_ID("adc_clk", &tsc_clk),
> };
>
> diff --git a/arch/arm/mach-at91/clock.c b/arch/arm/mach-at91/clock.c
> index 034529d..f5cfed75 100644
> --- a/arch/arm/mach-at91/clock.c
> +++ b/arch/arm/mach-at91/clock.c
> @@ -115,7 +115,7 @@ static u32 at91_pllb_usb_init;
> * 48 MHz (unless no USB function clocks are needed). The main clock and
> * both PLLs are turned off to run in "slow clock mode" (system suspend).
> */
> -static struct clk clk32k = {
> +struct clk clk32k = {
> .name = "clk32k",
> .rate_hz = AT91_SLOW_CLOCK,
> .users = 1, /* always on */
> diff --git a/arch/arm/mach-at91/clock.h b/arch/arm/mach-at91/clock.h
> index a98a39b..6eb825a 100644
> --- a/arch/arm/mach-at91/clock.h
> +++ b/arch/arm/mach-at91/clock.h
> @@ -34,6 +34,7 @@ struct clk {
> extern int __init clk_register(struct clk *clk);
> extern struct clk mck;
> extern struct clk utmi_clk;
> +extern struct clk clk32k;
>
> #define CLKDEV_CON_ID(_id, _clk) \
> { \
>
--
Nicolas Ferre
next prev parent reply other threads:[~2014-09-23 8:23 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-22 15:42 [PATCH v6 0/9] rtc: at91sam9: add DT support Boris BREZILLON
[not found] ` <1411400563-570-1-git-send-email-boris.brezillon-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2014-09-22 15:42 ` [PATCH v6 1/9] rtc: at91sam9: remove references to mach specific headers Boris BREZILLON
[not found] ` <1411400563-570-2-git-send-email-boris.brezillon-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2014-09-23 8:13 ` Nicolas Ferre
2014-09-22 15:42 ` [PATCH v6 2/9] rtc: at91sam9: use standard readl/writel functions instead of raw versions Boris BREZILLON
2014-09-23 8:14 ` Nicolas Ferre
2014-09-22 15:42 ` [PATCH v6 3/9] rtc: at91sam9: replace devm_ioremap by devm_ioremap_resource Boris BREZILLON
[not found] ` <1411400563-570-4-git-send-email-boris.brezillon-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2014-09-23 8:15 ` Nicolas Ferre
2014-09-22 15:42 ` [PATCH v6 4/9] rtc: at91sam9: add DT support Boris BREZILLON
2014-09-23 8:16 ` Nicolas Ferre
2014-09-22 15:42 ` [PATCH v6 5/9] rtc: at91sam9: make use of syscon/regmap to access GPBR registers Boris BREZILLON
2014-09-23 8:31 ` Nicolas Ferre
2014-09-22 15:42 ` [PATCH v6 6/9] ARM: at91: add clk_lookup entry for RTT devices Boris BREZILLON
2014-09-23 8:23 ` Nicolas Ferre [this message]
2014-09-22 15:42 ` [PATCH v6 7/9] rtc: at91sam9: use clk API instead of relying on AT91_SLOW_CLOCK Boris BREZILLON
[not found] ` <1411400563-570-8-git-send-email-boris.brezillon-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2014-09-23 8:21 ` Nicolas Ferre
2014-09-22 15:42 ` [PATCH v6 8/9] rtc: at91sam9: add DT bindings documentation Boris BREZILLON
2014-09-23 7:57 ` Johan Hovold
2014-09-23 8:24 ` Nicolas Ferre
2014-09-22 15:42 ` [PATCH v6 9/9] mfd: syscon: add Atmel GPBR DT bindings documention Boris BREZILLON
2014-09-23 8:23 ` Nicolas Ferre
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=54212DF4.1000301@atmel.com \
--to=nicolas.ferre@atmel.com \
--cc=a.zummo@towertech.it \
--cc=alexandre.belloni@free-electrons.com \
--cc=boris.brezillon@free-electrons.com \
--cc=devicetree@vger.kernel.org \
--cc=galak@codeaurora.org \
--cc=ijc+devicetree@hellion.org.uk \
--cc=jhovold@gmail.com \
--cc=lee.jones@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@maxim.org.za \
--cc=mark.rutland@arm.com \
--cc=pawel.moll@arm.com \
--cc=plagnioj@jcrosoft.com \
--cc=robh+dt@kernel.org \
--cc=rtc-linux@googlegroups.com \
--cc=sameo@linux.intel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).