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, On 20 November 2015 at 07:11, Jon Hunter wrote: > The _clk_disabl

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_pll() function will attempt to place a PLL into by

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

2015-11-25 Thread Tyler Baker
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_pll() function will attempt to place a PLL into bypass >> > if the TEGRA_PLL_BYPASS is specifie

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

2015-11-25 Thread Thierry Reding
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_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

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 clearing the enable bit. To

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

2015-11-24 Thread Thierry Reding
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_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

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 clearing the enable bit. To

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

2015-11-23 Thread Tyler Baker
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 clearing the enable bit. To place the PLL into bypass, the bypass bit > needs to be se

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 bypass, the bypass bit >

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

2015-11-20 Thread Stephen Boyd
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 bypass, the bypass bit > needs to be set and not cleared. Fix this by

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

2015-11-20 Thread Rhyland Klein
On 11/20/2015 10:11 AM, 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 bypass, the bypass bit > needs to be set and not cleare

[PATCH] clk: tegra: Fix bypassing of PLLs

2015-11-20 Thread Jon Hunter
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 bypass, the bypass bit needs to be set and not cleared. Fix this by setting the bypass bit and not clea