On Fri 10 Nov 2023 at 18:50, Jerome Brunet <jbrunet@xxxxxxxxxxxx> wrote: > On Fri 10 Nov 2023 at 14:20, Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> wrote: > >> On 06/11/2023 09:55, Xianwei Zhao wrote: >>> The patchset adds support for the peripheral and PLL clock controller >>> found on the Amlogic C3 SoC family, such as C302X or C308L. >>> >>> Changes since V5 [3]: >>> - Fix some typo and modify formart for MARCO. Suggested by Jerome. >>> - Add pad clock for peripheral input clock in bindings. >>> - Add some description for explaining why ddr_dpll_pt_clk and cts_msr_clk are out of tree. >>> Changes since V4 [10]: >>> - Change some fw_name of clocks. Suggested by Jerome. >>> - Delete minItem of clocks. >>> - Add CLk_GET_RATE_NOCACHE flags for gp1_pll >>> - Fix some format. and fix width as 8 for mclk_pll_dco. >>> - exchange gate and divder for fclk_50m clock. >>> - add CLK_SET_RATE_PARENT for axi_a_divder & axi_b_divder. >>> - add CLK_IS_CRITICAL for axi_clk >>> - Optimized macro define for pwm clk. >>> - add cts_oscin_clk mux between 24M and 32k >>> - add some missing gate clock, such as ddr_pll. >> >> Where are all these versions? Please provide links. > > I have provided some guidance offline at the request of Amlogic. > > This should have been v4 and the cover-letter should have summarized the > change from v3 to this. Unfortunately it was sent as v6 :/ > >> >> Best regards, >> Krzysztof While labeling this v6 was a mistake, please continue from there: next to be v7. Don't reset to v4 or v5. If more versions are needed, I don't want to end up with 2 v6 on the list, that would be even more confusing. Thanks