RE: [PATCH] clk: imx7ulp: Correct system clock source option #7

From: Anson Huang
Date: Fri Oct 11 2019 - 21:05:31 EST


Hi, Fabio

> On Fri, Oct 11, 2019 at 5:39 AM Anson Huang <Anson.Huang@xxxxxxx>
> wrote:
> >
> > In the latest reference manual Rev.0,06/2019, the SCG1's system
>
> This should be SCS instead of SCG1.

The reference states SCG system clock, SCS stands for system clock source, so I think
it is actually meaning SCG1's system clock sourceð

Selects the SCG system clock.
0000b - Reserved
0001b - System OSC (SOSC_CLK)
0010b - Slow IRC (SIRC_CLK)
0011b - Fast IRC (FIRC_CLK)
0100b - Reserved
0101b - Auxiliary PLL (APLL_CLK)
0110b - System PLL (SPLL_CLK)
0111b - Reserved

>
> > clock source option #7 is no longer from upll, it is reserved, update
> > clock driver accordingly.
> >
> > Fixes: b1260067ac3d ("clk: imx: add imx7ulp clk driver")
> > Signed-off-by: Anson Huang <Anson.Huang@xxxxxxx>
>
> Reviewed-by: Fabio Estevam <festevam@xxxxxxxxx>

Thanks,
Anson