* [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 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
* [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
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).