From mboxrd@z Thu Jan 1 00:00:00 1970 From: Damien Le Moal Date: Fri, 9 Apr 2021 02:45:59 +0000 Subject: [PATCH v3 02/11] clk: k210: Fix PLLs not being enabled References: <20210409021313.433558-1-seanga2@gmail.com> <20210409021313.433558-3-seanga2@gmail.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 2021/04/09 11:13, Sean Anderson wrote: > After starting or setting the rate of a PLL, the enable bit must be set. > > This fixes a bug where the AI ram would not be accessible, because it > requires PLL1 to be running. > > Signed-off-by: Sean Anderson > --- > > (no changes since v1) > > drivers/clk/kendryte/pll.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/clk/kendryte/pll.c b/drivers/clk/kendryte/pll.c > index ab6d75d585..f198920113 100644 > --- a/drivers/clk/kendryte/pll.c > +++ b/drivers/clk/kendryte/pll.c > @@ -531,6 +531,7 @@ static int k210_pll_enable(struct clk *clk) > k210_pll_waitfor_lock(pll); > > reg &= ~K210_PLL_BYPASS; > + reg |= K210_PLL_EN; > writel(reg, pll->reg); > > return 0; > @@ -550,6 +551,7 @@ static int k210_pll_disable(struct clk *clk) > writel(reg, pll->reg); > > reg &= ~K210_PLL_PWRD; > + reg &= ~K210_PLL_EN; > writel(reg, pll->reg); > return 0; > } > Looks good. That matches what the linux driver is doing. Reviewed-by: Damien Le Moal -- Damien Le Moal Western Digital Research