[PATCH 2/2] clk: ast2600: Keep PLL power on
Joel Stanley
joel at jms.id.au
Wed Feb 22 00:07:53 CET 2023
On Tue, 21 Feb 2023 at 13:01, Dylan Hung <dylan_hung at aspeedtech.com> wrote:
>
> According to the PLL vendor, we should keep the PLL power on, so we
> shouldn't toggle the power-down bit during PLL initialization.
>
> Signed-off-by: Dylan Hung <dylan_hung at aspeedtech.com>
Reviewed-by: Joel Stanley <joel at jms.id.au>
> ---
> drivers/clk/aspeed/clk_ast2600.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/clk/aspeed/clk_ast2600.c b/drivers/clk/aspeed/clk_ast2600.c
> index 0df1dc3718d3..e5ada5b6d49c 100644
> --- a/drivers/clk/aspeed/clk_ast2600.c
> +++ b/drivers/clk/aspeed/clk_ast2600.c
> @@ -538,7 +538,7 @@ static uint32_t ast2600_configure_pll(struct ast2600_scu *scu,
> }
>
> p_cfg->reg.b.bypass = 0;
> - p_cfg->reg.b.off = 1;
> + p_cfg->reg.b.off = 0;
> p_cfg->reg.b.reset = 1;
>
> reg = readl(addr);
> @@ -549,7 +549,6 @@ static uint32_t ast2600_configure_pll(struct ast2600_scu *scu,
> /* write extend parameter */
> writel(p_cfg->ext_reg, addr_ext);
> udelay(100);
> - p_cfg->reg.b.off = 0;
> p_cfg->reg.b.reset = 0;
> reg &= ~GENMASK(25, 0);
> reg |= p_cfg->reg.w;
> --
> 2.25.1
>
More information about the U-Boot
mailing list