* [PATCH 1/2] ARM: tegra: remove unused functions in mach/clk.h
@ 2012-11-27 6:59 Sivaram Nair
2012-11-27 6:59 ` [PATCH 2/2] ARM: tegra: moving stuff away from mach/clk.h Sivaram Nair
2012-11-29 6:20 ` [PATCH 1/2] ARM: tegra: remove unused functions in mach/clk.h Prashant Gaikwad
0 siblings, 2 replies; 6+ messages in thread
From: Sivaram Nair @ 2012-11-27 6:59 UTC (permalink / raw)
To: linux-arm-kernel
This is a cleaning up patch that removes couple of functions in
mach/clk.h that are no longer used and does not have any body.
Signed-off-by: Sivaram Nair <sivaramn@nvidia.com>
---
arch/arm/mach-tegra/include/mach/clk.h | 5 -----
1 file changed, 5 deletions(-)
diff --git a/arch/arm/mach-tegra/include/mach/clk.h b/arch/arm/mach-tegra/include/mach/clk.h
index 95f3a54..64d2657 100644
--- a/arch/arm/mach-tegra/include/mach/clk.h
+++ b/arch/arm/mach-tegra/include/mach/clk.h
@@ -34,11 +34,6 @@ enum tegra_clk_ex_param {
void tegra_periph_reset_deassert(struct clk *c);
void tegra_periph_reset_assert(struct clk *c);
-#ifndef CONFIG_COMMON_CLK
-unsigned long clk_get_rate_all_locked(struct clk *c);
-#endif
-
-void tegra2_sdmmc_tap_delay(struct clk *c, int delay);
int tegra_clk_cfg_ex(struct clk *c, enum tegra_clk_ex_param p, u32 setting);
#endif
--
1.7.9.5
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 2/2] ARM: tegra: moving stuff away from mach/clk.h
2012-11-27 6:59 [PATCH 1/2] ARM: tegra: remove unused functions in mach/clk.h Sivaram Nair
@ 2012-11-27 6:59 ` Sivaram Nair
2012-11-29 6:12 ` Prashant Gaikwad
2012-11-29 6:20 ` [PATCH 1/2] ARM: tegra: remove unused functions in mach/clk.h Prashant Gaikwad
1 sibling, 1 reply; 6+ messages in thread
From: Sivaram Nair @ 2012-11-27 6:59 UTC (permalink / raw)
To: linux-arm-kernel
This patch moves some stuff away from mach/clk.h to other mach-tegra
files. This is part of the efforts to get rid of mach/clk.h which in
turn will help to enable single zImage.
Signed-off-by: Sivaram Nair <sivaramn@nvidia.com>
---
arch/arm/mach-tegra/clock.c | 1 +
arch/arm/mach-tegra/clock.h | 12 ++++++++++--
arch/arm/mach-tegra/include/mach/clk.h | 11 -----------
3 files changed, 11 insertions(+), 13 deletions(-)
diff --git a/arch/arm/mach-tegra/clock.c b/arch/arm/mach-tegra/clock.c
index 867bf8b..4440626 100644
--- a/arch/arm/mach-tegra/clock.c
+++ b/arch/arm/mach-tegra/clock.c
@@ -26,6 +26,7 @@
#include <linux/sched.h>
#include <linux/seq_file.h>
#include <linux/slab.h>
+#include <mach/clk.h>
#include "board.h"
#include "clock.h"
diff --git a/arch/arm/mach-tegra/clock.h b/arch/arm/mach-tegra/clock.h
index 2aa37f5..e6685d4 100644
--- a/arch/arm/mach-tegra/clock.h
+++ b/arch/arm/mach-tegra/clock.h
@@ -25,8 +25,6 @@
#include <linux/clkdev.h>
#include <linux/list.h>
-#include <mach/clk.h>
-
#define DIV_BUS (1 << 0)
#define DIV_U71 (1 << 1)
#define DIV_U71_FIXED (1 << 2)
@@ -70,6 +68,15 @@ struct clk_pll_freq_table {
u8 cpcon;
};
+enum tegra_clk_ex_param {
+ TEGRA_CLK_VI_INP_SEL,
+ TEGRA_CLK_DTV_INVERT,
+ TEGRA_CLK_NAND_PAD_DIV2_ENB,
+ TEGRA_CLK_PLLD_CSI_OUT_ENB,
+ TEGRA_CLK_PLLD_DSI_OUT_ENB,
+ TEGRA_CLK_PLLD_MIPI_MUX_SEL,
+};
+
enum clk_state {
UNINITIALIZED = 0,
ON,
@@ -149,5 +156,6 @@ void tegra2_init_clocks(void);
void tegra30_init_clocks(void);
struct clk *tegra_get_clock_by_name(const char *name);
void tegra_clk_init_from_table(struct tegra_clk_init_table *table);
+int tegra_clk_cfg_ex(struct clk *c, enum tegra_clk_ex_param p, u32 setting);
#endif
diff --git a/arch/arm/mach-tegra/include/mach/clk.h b/arch/arm/mach-tegra/include/mach/clk.h
index 64d2657..6338652 100644
--- a/arch/arm/mach-tegra/include/mach/clk.h
+++ b/arch/arm/mach-tegra/include/mach/clk.h
@@ -22,18 +22,7 @@
struct clk;
-enum tegra_clk_ex_param {
- TEGRA_CLK_VI_INP_SEL,
- TEGRA_CLK_DTV_INVERT,
- TEGRA_CLK_NAND_PAD_DIV2_ENB,
- TEGRA_CLK_PLLD_CSI_OUT_ENB,
- TEGRA_CLK_PLLD_DSI_OUT_ENB,
- TEGRA_CLK_PLLD_MIPI_MUX_SEL,
-};
-
void tegra_periph_reset_deassert(struct clk *c);
void tegra_periph_reset_assert(struct clk *c);
-int tegra_clk_cfg_ex(struct clk *c, enum tegra_clk_ex_param p, u32 setting);
-
#endif
--
1.7.9.5
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 2/2] ARM: tegra: moving stuff away from mach/clk.h
2012-11-27 6:59 ` [PATCH 2/2] ARM: tegra: moving stuff away from mach/clk.h Sivaram Nair
@ 2012-11-29 6:12 ` Prashant Gaikwad
2012-11-29 18:25 ` Stephen Warren
0 siblings, 1 reply; 6+ messages in thread
From: Prashant Gaikwad @ 2012-11-29 6:12 UTC (permalink / raw)
To: linux-arm-kernel
On Tuesday 27 November 2012 12:29 PM, Sivaram Nair wrote:
> This patch moves some stuff away from mach/clk.h to other mach-tegra
> files. This is part of the efforts to get rid of mach/clk.h which in
> turn will help to enable single zImage.
>
> Signed-off-by: Sivaram Nair <sivaramn@nvidia.com>
> ---
> arch/arm/mach-tegra/clock.c | 1 +
> arch/arm/mach-tegra/clock.h | 12 ++++++++++--
> arch/arm/mach-tegra/include/mach/clk.h | 11 -----------
> 3 files changed, 11 insertions(+), 13 deletions(-)
This patch will not add any real value since I have removed clk_cfg_ex
functionality in clock code rework.
It's just that I will have to rebase my changes on this.
Reviewed-by: Prashant Gaikwad <pgaikwad@nvidia.com>
> diff --git a/arch/arm/mach-tegra/clock.c b/arch/arm/mach-tegra/clock.c
> index 867bf8b..4440626 100644
> --- a/arch/arm/mach-tegra/clock.c
> +++ b/arch/arm/mach-tegra/clock.c
> @@ -26,6 +26,7 @@
> #include <linux/sched.h>
> #include <linux/seq_file.h>
> #include <linux/slab.h>
> +#include <mach/clk.h>
>
> #include "board.h"
> #include "clock.h"
> diff --git a/arch/arm/mach-tegra/clock.h b/arch/arm/mach-tegra/clock.h
> index 2aa37f5..e6685d4 100644
> --- a/arch/arm/mach-tegra/clock.h
> +++ b/arch/arm/mach-tegra/clock.h
> @@ -25,8 +25,6 @@
> #include <linux/clkdev.h>
> #include <linux/list.h>
>
> -#include <mach/clk.h>
> -
> #define DIV_BUS (1 << 0)
> #define DIV_U71 (1 << 1)
> #define DIV_U71_FIXED (1 << 2)
> @@ -70,6 +68,15 @@ struct clk_pll_freq_table {
> u8 cpcon;
> };
>
> +enum tegra_clk_ex_param {
> + TEGRA_CLK_VI_INP_SEL,
> + TEGRA_CLK_DTV_INVERT,
> + TEGRA_CLK_NAND_PAD_DIV2_ENB,
> + TEGRA_CLK_PLLD_CSI_OUT_ENB,
> + TEGRA_CLK_PLLD_DSI_OUT_ENB,
> + TEGRA_CLK_PLLD_MIPI_MUX_SEL,
> +};
> +
> enum clk_state {
> UNINITIALIZED = 0,
> ON,
> @@ -149,5 +156,6 @@ void tegra2_init_clocks(void);
> void tegra30_init_clocks(void);
> struct clk *tegra_get_clock_by_name(const char *name);
> void tegra_clk_init_from_table(struct tegra_clk_init_table *table);
> +int tegra_clk_cfg_ex(struct clk *c, enum tegra_clk_ex_param p, u32 setting);
>
> #endif
> diff --git a/arch/arm/mach-tegra/include/mach/clk.h b/arch/arm/mach-tegra/include/mach/clk.h
> index 64d2657..6338652 100644
> --- a/arch/arm/mach-tegra/include/mach/clk.h
> +++ b/arch/arm/mach-tegra/include/mach/clk.h
> @@ -22,18 +22,7 @@
>
> struct clk;
>
> -enum tegra_clk_ex_param {
> - TEGRA_CLK_VI_INP_SEL,
> - TEGRA_CLK_DTV_INVERT,
> - TEGRA_CLK_NAND_PAD_DIV2_ENB,
> - TEGRA_CLK_PLLD_CSI_OUT_ENB,
> - TEGRA_CLK_PLLD_DSI_OUT_ENB,
> - TEGRA_CLK_PLLD_MIPI_MUX_SEL,
> -};
> -
> void tegra_periph_reset_deassert(struct clk *c);
> void tegra_periph_reset_assert(struct clk *c);
>
> -int tegra_clk_cfg_ex(struct clk *c, enum tegra_clk_ex_param p, u32 setting);
> -
> #endif
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/2] ARM: tegra: remove unused functions in mach/clk.h
2012-11-27 6:59 [PATCH 1/2] ARM: tegra: remove unused functions in mach/clk.h Sivaram Nair
2012-11-27 6:59 ` [PATCH 2/2] ARM: tegra: moving stuff away from mach/clk.h Sivaram Nair
@ 2012-11-29 6:20 ` Prashant Gaikwad
1 sibling, 0 replies; 6+ messages in thread
From: Prashant Gaikwad @ 2012-11-29 6:20 UTC (permalink / raw)
To: linux-arm-kernel
On Tuesday 27 November 2012 12:29 PM, Sivaram Nair wrote:
> This is a cleaning up patch that removes couple of functions in
> mach/clk.h that are no longer used and does not have any body.
>
> Signed-off-by: Sivaram Nair <sivaramn@nvidia.com>
> ---
> arch/arm/mach-tegra/include/mach/clk.h | 5 -----
> 1 file changed, 5 deletions(-)
>
> diff --git a/arch/arm/mach-tegra/include/mach/clk.h b/arch/arm/mach-tegra/include/mach/clk.h
> index 95f3a54..64d2657 100644
> --- a/arch/arm/mach-tegra/include/mach/clk.h
> +++ b/arch/arm/mach-tegra/include/mach/clk.h
> @@ -34,11 +34,6 @@ enum tegra_clk_ex_param {
> void tegra_periph_reset_deassert(struct clk *c);
> void tegra_periph_reset_assert(struct clk *c);
>
> -#ifndef CONFIG_COMMON_CLK
> -unsigned long clk_get_rate_all_locked(struct clk *c);
> -#endif
> -
> -void tegra2_sdmmc_tap_delay(struct clk *c, int delay);
> int tegra_clk_cfg_ex(struct clk *c, enum tegra_clk_ex_param p, u32 setting);
>
> #endif
Reviewed-by: Prashant Gaikwad <pgaikwad@nvidia.com>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 2/2] ARM: tegra: moving stuff away from mach/clk.h
2012-11-29 6:12 ` Prashant Gaikwad
@ 2012-11-29 18:25 ` Stephen Warren
2012-11-30 7:09 ` Prashant Gaikwad
0 siblings, 1 reply; 6+ messages in thread
From: Stephen Warren @ 2012-11-29 18:25 UTC (permalink / raw)
To: linux-arm-kernel
On 11/28/2012 11:12 PM, Prashant Gaikwad wrote:
> On Tuesday 27 November 2012 12:29 PM, Sivaram Nair wrote:
>> This patch moves some stuff away from mach/clk.h to other mach-tegra
>> files. This is part of the efforts to get rid of mach/clk.h which in
>> turn will help to enable single zImage.
>>
>> Signed-off-by: Sivaram Nair <sivaramn@nvidia.com>
>> ---
>> arch/arm/mach-tegra/clock.c | 1 +
>> arch/arm/mach-tegra/clock.h | 12 ++++++++++--
>> arch/arm/mach-tegra/include/mach/clk.h | 11 -----------
>> 3 files changed, 11 insertions(+), 13 deletions(-)
>
> This patch will not add any real value since I have removed clk_cfg_ex
> functionality in clock code rework.
> It's just that I will have to rebase my changes on this.
>
> Reviewed-by: Prashant Gaikwad <pgaikwad@nvidia.com>
In that case, I may as well not apply patch 2/2 at all, and I'll just
take your clock code cleanup instead. I assume that will be posted for
inclusion into 3.9?
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 2/2] ARM: tegra: moving stuff away from mach/clk.h
2012-11-29 18:25 ` Stephen Warren
@ 2012-11-30 7:09 ` Prashant Gaikwad
0 siblings, 0 replies; 6+ messages in thread
From: Prashant Gaikwad @ 2012-11-30 7:09 UTC (permalink / raw)
To: linux-arm-kernel
On Thursday 29 November 2012 11:55 PM, Stephen Warren wrote:
> On 11/28/2012 11:12 PM, Prashant Gaikwad wrote:
>> On Tuesday 27 November 2012 12:29 PM, Sivaram Nair wrote:
>>> This patch moves some stuff away from mach/clk.h to other mach-tegra
>>> files. This is part of the efforts to get rid of mach/clk.h which in
>>> turn will help to enable single zImage.
>>>
>>> Signed-off-by: Sivaram Nair <sivaramn@nvidia.com>
>>> ---
>>> arch/arm/mach-tegra/clock.c | 1 +
>>> arch/arm/mach-tegra/clock.h | 12 ++++++++++--
>>> arch/arm/mach-tegra/include/mach/clk.h | 11 -----------
>>> 3 files changed, 11 insertions(+), 13 deletions(-)
>> This patch will not add any real value since I have removed clk_cfg_ex
>> functionality in clock code rework.
>> It's just that I will have to rebase my changes on this.
>>
>> Reviewed-by: Prashant Gaikwad <pgaikwad@nvidia.com>
> In that case, I may as well not apply patch 2/2 at all, and I'll just
> take your clock code cleanup instead. I assume that will be posted for
> inclusion into 3.9?
Yes, I am targeting for 3.9.
> --
> To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-11-30 7:09 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-11-27 6:59 [PATCH 1/2] ARM: tegra: remove unused functions in mach/clk.h Sivaram Nair
2012-11-27 6:59 ` [PATCH 2/2] ARM: tegra: moving stuff away from mach/clk.h Sivaram Nair
2012-11-29 6:12 ` Prashant Gaikwad
2012-11-29 18:25 ` Stephen Warren
2012-11-30 7:09 ` Prashant Gaikwad
2012-11-29 6:20 ` [PATCH 1/2] ARM: tegra: remove unused functions in mach/clk.h Prashant Gaikwad
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).