On Wed, Apr 03, 2024 at 09:41:26AM +0800, Shawn Guo wrote: > On Tue, Apr 02, 2024 at 11:09:17AM -0400, Frank Li wrote: > > On Mon, Apr 01, 2024 at 08:04:56PM -0300, Fabio Estevam wrote: > > > On Mon, Apr 1, 2024 at 7:25 PM Frank Li <Frank.Li@xxxxxxx> wrote: > > > > > > > > lpcg's arg0 should use clock indices instead of index. > > > > > > > > pwm0_lpcg: clock-controller@5d400000 { > > > > ... // Col1 Col2 > > > > clocks = <&clk IMX_SC_R_PWM_0 IMX_SC_PM_CLK_PER>, // 0 0 > > > > <&clk IMX_SC_R_PWM_0 IMX_SC_PM_CLK_PER>, // 1 1 > > > > <&clk IMX_SC_R_PWM_0 IMX_SC_PM_CLK_PER>, // 2 4 > > > > <&lsio_bus_clk>, // 3 5 > > > > <&clk IMX_SC_R_PWM_0 IMX_SC_PM_CLK_PER>; // 4 6 > > > > clock-indices = <IMX_LPCG_CLK_0>, <IMX_LPCG_CLK_1>, > > > > <IMX_LPCG_CLK_4>, <IMX_LPCG_CLK_5>, > > > > <IMX_LPCG_CLK_6>; > > > > }; > > > > > > > > Col1: index, which exited dts try to get. > > > > > > I cannot understand this sentence, sorry. > > > > This base on downstream dts code. Downstream code use index in 'Col1' to > > get clock. > > So s/exited/existing you meant? Yes, sorry for typo. Frank > > Shawn >