[PATCH 2/2] clk: tegra: Use definition for pll_u override bit

2015-12-21 Thread Jon Hunter
The definition, PLLU_BASE_OVERRIDE, for the pll_u OVERRIDE bit is defined but not used and when the OVERRIDE bit is cleared in tegra210_pll_init() the code directly uses the bit number. Therefore, use the definition, PLLU_BASE_OVERRIDE when clearing the OVERRIDE bit. Signed-off-by: Jon Hunter

[PATCH 1/2] clk: tegra: Fix warning caused by pll_u failing to lock

2015-12-21 Thread Jon Hunter
-by: Jon Hunter --- drivers/clk/tegra/clk-tegra210.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/clk/tegra/clk-tegra210.c b/drivers/clk/tegra/clk-tegra210.c index 849be30b52bf..791215747863 100644 --- a/drivers/clk/tegra/clk-tegra210.c +++ b/drivers/clk/tegra/clk-tegra210.c

[PATCH] clk: tegra: Fix clock sources for tegra210 EMC

2015-12-18 Thread Jon Hunter
a system hang on boot. This is because the kernel will disable the pll_mb when disabling unused clock as it appears to be unused when it is not. Also add the additional pll_p clock source for the EMC. Signed-off-by: Jon Hunter --- drivers/clk/tegra/clk-tegra210.c | 3 ++- 1 file changed, 2

Re: [PATCH 3/3] clk: tegra-super: Fix sparse warnings for functions not declared as static

2015-12-08 Thread Jon Hunter
On 07/12/15 17:51, Rhyland Klein wrote: > On 12/7/2015 10:51 AM, Rhyland Klein wrote: >> On 12/4/2015 12:04 PM, Jon Hunter wrote: >>> Sparse reports the following warnings for structures and functions that >>> should be declared static: >>> >>> d

[PATCH 2/3] clk: tegra210: Fix sparse warnings for functions not declared as static

2015-12-04 Thread Jon Hunter
t be static? drivers/clk/tegra/clk-tegra210.c:1215:15: warning: symbol 'tegra210_clk_adjust_vco_min' was not declared. Should it be static? Fix this by declaring the above as static. Fixes: b31eba5ff3f7 ("clk: tegra: Add support for Tegra210 clocks") Signed-off-by: Jon Hunt

[PATCH 1/3] clk: tegra: Fix sparse warning for pll_m

2015-12-04 Thread Jon Hunter
("clk: tegra: Add support for Tegra210 clocks") Signed-off-by: Jon Hunter --- drivers/clk/tegra/clk-tegra210.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/clk/tegra/clk-tegra210.c b/drivers/clk/tegra/clk-tegra210.c index 58514c44ea83..3bd831c2b9db 10

[PATCH 3/3] clk: tegra-super: Fix sparse warnings for functions not declared as static

2015-12-04 Thread Jon Hunter
gra: Add Super Gen5 Logic") Signed-off-by: Jon Hunter --- drivers/clk/tegra/clk-tegra-super-gen4.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/clk/tegra/clk-tegra-super-gen4.c b/drivers/clk/tegra/clk-tegra-super-gen4.c index 4559a20e3af6..474de0f0c26d

Re: [PATCH] clk: tegra: Fix bypassing of PLLs

2015-11-26 Thread Jon Hunter
On 25/11/15 17:48, Jon Hunter wrote: > > On 25/11/15 15:52, Tyler Baker wrote: >> On 25 November 2015 at 07:11, Thierry Reding >> wrote: >>> On Mon, Nov 23, 2015 at 03:18:59PM -0800, Tyler Baker wrote: >>>> Hi Jon, >>>> >>>

Re: [PATCH] clk: tegra: Fix bypassing of PLLs

2015-11-25 Thread Jon Hunter
On 25/11/15 15:52, Tyler Baker wrote: > On 25 November 2015 at 07:11, Thierry Reding wrote: >> On Mon, Nov 23, 2015 at 03:18:59PM -0800, Tyler Baker wrote: >>> Hi Jon, >>> >>> On 20 November 2015 at 07:11, Jon Hunter wrote: >>>> The _clk_disable

Re: [PATCH] clk: tegra: Fix bypassing of PLLs

2015-11-24 Thread Jon Hunter
Hi Tyler, On 23/11/15 23:18, Tyler Baker wrote: > Hi Jon, > > On 20 November 2015 at 07:11, Jon Hunter wrote: >> The _clk_disable_pll() function will attempt to place a PLL into bypass >> if the TEGRA_PLL_BYPASS is specified for the PLL and then disable the PLL >> by

Re: [PATCH] clk: tegra: Fix bypassing of PLLs

2015-11-24 Thread Jon Hunter
Hi Tyler, On 23/11/15 23:18, Tyler Baker wrote: > Hi Jon, > > On 20 November 2015 at 07:11, Jon Hunter wrote: >> The _clk_disable_pll() function will attempt to place a PLL into bypass >> if the TEGRA_PLL_BYPASS is specified for the PLL and then disable the PLL >> by

Re: [PATCH] clk: tegra: Fix bypassing of PLLs

2015-11-23 Thread Jon Hunter
On 20/11/15 17:15, Stephen Boyd wrote: > On 11/20, Jon Hunter wrote: >> The _clk_disable_pll() function will attempt to place a PLL into bypass >> if the TEGRA_PLL_BYPASS is specified for the PLL and then disable the PLL >> by clearing the enable bit. To place the PLL into b

[PATCH] clk: tegra: Fix bypassing of PLLs

2015-11-20 Thread Jon Hunter
clearing it. Signed-off-by: Jon Hunter --- drivers/clk/tegra/clk-pll.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/clk/tegra/clk-pll.c b/drivers/clk/tegra/clk-pll.c index d6d4ecb88e94..e5aa9c87df4c 100644 --- a/drivers/clk/tegra/clk-pll.c +++ b/drivers/clk/tegra/clk