From: Michael Turquette <mturquette@baylibre.com>
To: Antoine Tenart <antoine.tenart@free-electrons.com>,
sebastian.hesselbarth@gmail.com, sboyd@codeaurora.org
Cc: "Antoine Tenart" <antoine.tenart@free-electrons.com>,
zmxu@marvell.com, jszhang@marvell.com,
linux-arm-kernel@lists.infradead.org, linux-clk@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 1/6] clk: berlin: move MAX_CLKS out of drivers/clk/berlin
Date: Tue, 25 Aug 2015 13:25:51 -0700 [thread overview]
Message-ID: <20150825202551.31346.93685@quantum> (raw)
In-Reply-To: <1439898368-11543-2-git-send-email-antoine.tenart@free-electrons.com>
Quoting Antoine Tenart (2015-08-18 04:46:03)
> Move the MAX_CLKS definition used in the Berlin clock driver outside the
> driver.
>
> Signed-off-by: Antoine Tenart <antoine.tenart@free-electrons.com>
Acked-by: Michael Turquette <mturquette@baylibre.com>
> ---
> drivers/clk/berlin/bg2.c | 1 -
> drivers/clk/berlin/bg2q.c | 1 -
> include/dt-bindings/clock/berlin2.h | 2 ++
> include/dt-bindings/clock/berlin2q.h | 2 ++
> 4 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/clk/berlin/bg2.c b/drivers/clk/berlin/bg2.c
> index 73153fc45ee9..469a2c57e1b4 100644
> --- a/drivers/clk/berlin/bg2.c
> +++ b/drivers/clk/berlin/bg2.c
> @@ -91,7 +91,6 @@
> *
> */
>
> -#define MAX_CLKS 41
> static struct clk *clks[MAX_CLKS];
> static struct clk_onecell_data clk_data;
> static DEFINE_SPINLOCK(lock);
> diff --git a/drivers/clk/berlin/bg2q.c b/drivers/clk/berlin/bg2q.c
> index 221f40c2b850..a27958efab92 100644
> --- a/drivers/clk/berlin/bg2q.c
> +++ b/drivers/clk/berlin/bg2q.c
> @@ -45,7 +45,6 @@
> #define REG_SDIO0XIN_CLKCTL 0x0158
> #define REG_SDIO1XIN_CLKCTL 0x015c
>
> -#define MAX_CLKS 27
> static struct clk *clks[MAX_CLKS];
> static struct clk_onecell_data clk_data;
> static DEFINE_SPINLOCK(lock);
> diff --git a/include/dt-bindings/clock/berlin2.h b/include/dt-bindings/clock/berlin2.h
> index 0c30800175df..97832b840beb 100644
> --- a/include/dt-bindings/clock/berlin2.h
> +++ b/include/dt-bindings/clock/berlin2.h
> @@ -43,3 +43,5 @@
> #define CLKID_VIDEO1 38
> #define CLKID_VIDEO2 39
> #define CLKID_TWD 40
> +
> +#define MAX_CLKS 41
> diff --git a/include/dt-bindings/clock/berlin2q.h b/include/dt-bindings/clock/berlin2q.h
> index 287fc3b4afb2..c79af928ae92 100644
> --- a/include/dt-bindings/clock/berlin2q.h
> +++ b/include/dt-bindings/clock/berlin2q.h
> @@ -29,3 +29,5 @@
> #define CLKID_SMEMC 24
> #define CLKID_PCIE 25
> #define CLKID_TWD 26
> +
> +#define MAX_CLKS 27
> --
> 2.5.0
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
WARNING: multiple messages have this Message-ID (diff)
From: mturquette@baylibre.com (Michael Turquette)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 1/6] clk: berlin: move MAX_CLKS out of drivers/clk/berlin
Date: Tue, 25 Aug 2015 13:25:51 -0700 [thread overview]
Message-ID: <20150825202551.31346.93685@quantum> (raw)
In-Reply-To: <1439898368-11543-2-git-send-email-antoine.tenart@free-electrons.com>
Quoting Antoine Tenart (2015-08-18 04:46:03)
> Move the MAX_CLKS definition used in the Berlin clock driver outside the
> driver.
>
> Signed-off-by: Antoine Tenart <antoine.tenart@free-electrons.com>
Acked-by: Michael Turquette <mturquette@baylibre.com>
> ---
> drivers/clk/berlin/bg2.c | 1 -
> drivers/clk/berlin/bg2q.c | 1 -
> include/dt-bindings/clock/berlin2.h | 2 ++
> include/dt-bindings/clock/berlin2q.h | 2 ++
> 4 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/clk/berlin/bg2.c b/drivers/clk/berlin/bg2.c
> index 73153fc45ee9..469a2c57e1b4 100644
> --- a/drivers/clk/berlin/bg2.c
> +++ b/drivers/clk/berlin/bg2.c
> @@ -91,7 +91,6 @@
> *
> */
>
> -#define MAX_CLKS 41
> static struct clk *clks[MAX_CLKS];
> static struct clk_onecell_data clk_data;
> static DEFINE_SPINLOCK(lock);
> diff --git a/drivers/clk/berlin/bg2q.c b/drivers/clk/berlin/bg2q.c
> index 221f40c2b850..a27958efab92 100644
> --- a/drivers/clk/berlin/bg2q.c
> +++ b/drivers/clk/berlin/bg2q.c
> @@ -45,7 +45,6 @@
> #define REG_SDIO0XIN_CLKCTL 0x0158
> #define REG_SDIO1XIN_CLKCTL 0x015c
>
> -#define MAX_CLKS 27
> static struct clk *clks[MAX_CLKS];
> static struct clk_onecell_data clk_data;
> static DEFINE_SPINLOCK(lock);
> diff --git a/include/dt-bindings/clock/berlin2.h b/include/dt-bindings/clock/berlin2.h
> index 0c30800175df..97832b840beb 100644
> --- a/include/dt-bindings/clock/berlin2.h
> +++ b/include/dt-bindings/clock/berlin2.h
> @@ -43,3 +43,5 @@
> #define CLKID_VIDEO1 38
> #define CLKID_VIDEO2 39
> #define CLKID_TWD 40
> +
> +#define MAX_CLKS 41
> diff --git a/include/dt-bindings/clock/berlin2q.h b/include/dt-bindings/clock/berlin2q.h
> index 287fc3b4afb2..c79af928ae92 100644
> --- a/include/dt-bindings/clock/berlin2q.h
> +++ b/include/dt-bindings/clock/berlin2q.h
> @@ -29,3 +29,5 @@
> #define CLKID_SMEMC 24
> #define CLKID_PCIE 25
> #define CLKID_TWD 26
> +
> +#define MAX_CLKS 27
> --
> 2.5.0
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
next prev parent reply other threads:[~2015-08-25 20:25 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-08-18 11:46 [PATCH v2 0/6] ARM: berlin: add cpufreq support Antoine Tenart
2015-08-18 11:46 ` Antoine Tenart
2015-08-18 11:46 ` [PATCH v2 1/6] clk: berlin: move MAX_CLKS out of drivers/clk/berlin Antoine Tenart
2015-08-18 11:46 ` Antoine Tenart
2015-08-25 20:25 ` Michael Turquette [this message]
2015-08-25 20:25 ` Michael Turquette
2015-08-18 11:46 ` [PATCH v2 2/6] clk: berlin: add cpuclk Antoine Tenart
2015-08-18 11:46 ` Antoine Tenart
2015-08-18 11:46 ` [PATCH v2 3/6] ARM: berlin: dts: add the cpufreq-dt bindings on the BG2Q Antoine Tenart
2015-08-18 11:46 ` Antoine Tenart
2015-08-18 11:46 ` Antoine Tenart
2015-08-18 11:46 ` [PATCH v2 4/6] ARM: berlin: dts: add the cpufreq-dt bindings on the BG2 Antoine Tenart
2015-08-18 11:46 ` Antoine Tenart
2015-08-18 11:46 ` [PATCH v2 5/6] ARM: berlin: dts: add the cpufreq-dt bindings on the BG2CD Antoine Tenart
2015-08-18 11:46 ` Antoine Tenart
2015-08-18 11:46 ` [PATCH v2 6/6] ARM: berlin: register cpufreq-dt Antoine Tenart
2015-08-18 11:46 ` Antoine Tenart
2015-08-18 12:56 ` [PATCH v2 0/6] ARM: berlin: add cpufreq support Viresh Kumar
2015-08-18 12:56 ` Viresh Kumar
2015-08-18 14:39 ` Antoine Tenart
2015-08-18 14:39 ` Antoine Tenart
2015-08-18 14:59 ` Viresh Kumar
2015-08-18 14:59 ` Viresh Kumar
2015-08-19 8:17 ` Antoine Tenart
2015-08-19 8:17 ` Antoine Tenart
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=20150825202551.31346.93685@quantum \
--to=mturquette@baylibre.com \
--cc=antoine.tenart@free-electrons.com \
--cc=jszhang@marvell.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=sboyd@codeaurora.org \
--cc=sebastian.hesselbarth@gmail.com \
--cc=zmxu@marvell.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 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.