On 22/01/2024 02:32, Peng Fan (OSS) wrote: > + > +/* The index should match i.MX95 SCMI Firmware */ > +#define IMX95_CLK_32K 1 > +#define IMX95_CLK_24M 2 > +#define IMX95_CLK_FRO 3 > +#define IMX95_CLK_SYSPLL1_VCO 4 > +#define IMX95_CLK_SYSPLL1_PFD0_UNGATED 5 > +#define IMX95_CLK_SYSPLL1_PFD0 6 > +#define IMX95_CLK_SYSPLL1_PFD0_DIV2 7 > +#define IMX95_CLK_SYSPLL1_PFD1_UNGATED 8 > +#define IMX95_CLK_SYSPLL1_PFD1 9 > +#define IMX95_CLK_SYSPLL1_PFD1_DIV2 10 > +#define IMX95_CLK_SYSPLL1_PFD2_UNGATED 11 > +#define IMX95_CLK_SYSPLL1_PFD2 12 > +#define IMX95_CLK_SYSPLL1_PFD2_DIV2 13 > +#define IMX95_CLK_AUDIOPLL1_VCO 14 > +#define IMX95_CLK_AUDIOPLL1 15 > +#define IMX95_CLK_AUDIOPLL2_VCO 16 > +#define IMX95_CLK_AUDIOPLL2 17 > +#define IMX95_CLK_VIDEOPLL1_VCO 18 > +#define IMX95_CLK_VIDEOPLL1 19 > +#define IMX95_CLK_RESERVED20 20 > +#define IMX95_CLK_RESERVED21 21 > +#define IMX95_CLK_RESERVED22 22 > +#define IMX95_CLK_RESERVED23 23 > +#define IMX95_CLK_ARMPLL_VCO 24 > +#define IMX95_CLK_ARMPLL_PFD0_UNGATED 25 > +#define IMX95_CLK_ARMPLL_PFD0 26 > +#define IMX95_CLK_ARMPLL_PFD1_UNGATED 27 > +#define IMX95_CLK_ARMPLL_PFD1 28 > +#define IMX95_CLK_ARMPLL_PFD2_UNGATED 29 > +#define IMX95_CLK_ARMPLL_PFD2 30 > +#define IMX95_CLK_ARMPLL_PFD3_UNGATED 31 > +#define IMX95_CLK_ARMPLL_PFD3 32 > +#define IMX95_CLK_DRAMPLL_VCO 33 > +#define IMX95_CLK_DRAMPLL 34 > +#define IMX95_CLK_HSIOPLL_VCO 35 > +#define IMX95_CLK_HSIOPLL 36 > +#define IMX95_CLK_LDBPLL_VCO 37 > +#define IMX95_CLK_LDBPLL 38 > +#define IMX95_CLK_EXT1 39 > +#define IMX95_CLK_EXT2 40 > + > +#define IMX95_CCM_NUM_CLK_SRC 41 All my comments from previous version or for other patchsets you sent recently apply. Best regards, Krzysztof