Re: [PATCH 4/7] drivers: clk: meson: Add Amlogic T7 sys pll support

[Jerome Brunet <jbrunet@baylibre.com>]

On mer. 18 févr. 2026 at 12:00, Ronald Claveau <linux-kernel-dev@aliel.fr> wrote:

> Add SYS PLL for the clock controller of the Amlogic T7 SoC family.

Like the FDIVs, I think this is provided by SCMI.

Check include/dt-bindings/clock/amlogic,t7-scmi.h

>
> Signed-off-by: Ronald Claveau <linux-kernel-dev@aliel.fr>
> ---
>  drivers/clk/meson/t7-peripherals.c | 134 ++++++++++++++++++++++++++++-
>  1 file changed, 131 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/clk/meson/t7-peripherals.c b/drivers/clk/meson/t7-peripherals.c
> index 214db7850d86..de206473f3a7 100644
> --- a/drivers/clk/meson/t7-peripherals.c
> +++ b/drivers/clk/meson/t7-peripherals.c
> @@ -176,6 +176,127 @@ static struct clk_regmap t7_rtc = {
>  	},
>  };
>  
> +static u32 t7_sys_parents_val_table[] = { 0, 1, 2, 3, 4, 5, 7 };
> +static const struct clk_parent_data t7_sys_parents[] = {
> +	{ .fw_name = "xtal", },
> +	{ .fw_name = "fdiv2", },
> +	{ .fw_name = "fdiv3", },
> +	{ .fw_name = "fdiv4", },
> +	{ .fw_name = "fdiv5", },
> +	{ .fw_name = "axi_clk_frcpu", },
> +	{ .hw = &t7_rtc.hw },
> +};
> +
> +static struct clk_regmap t7_sys_a_sel = {
> +	.data = &(struct clk_regmap_mux_data){
> +		.offset = SYS_CLK_CTRL0,
> +		.mask = 0x7,
> +		.shift = 10,
> +		.table = t7_sys_parents_val_table,
> +	},
> +	.hw.init = &(struct clk_init_data){
> +		.name = "sys_a_sel",
> +		.ops = &clk_regmap_mux_ops,
> +		.parent_data = t7_sys_parents,
> +		.num_parents = ARRAY_SIZE(t7_sys_parents),
> +	},
> +};
> +
> +static struct clk_regmap t7_sys_a_div = {
> +	.data = &(struct clk_regmap_div_data){
> +	.offset = SYS_CLK_CTRL0,
> +		.shift = 0,
> +		.width = 10,
> +	},
> +	.hw.init = &(struct clk_init_data){
> +		.name = "sys_a_div",
> +		.ops = &clk_regmap_divider_ops,
> +		.parent_hws = (const struct clk_hw *[]) {
> +			&t7_sys_a_sel.hw
> +		},
> +		.num_parents = 1,
> +		.flags = CLK_SET_RATE_PARENT,
> +	},
> +};
> +
> +static struct clk_regmap t7_sys_a = {
> +	.data = &(struct clk_regmap_gate_data){
> +		.offset = SYS_CLK_CTRL0,
> +		.bit_idx = 13,
> +	},
> +	.hw.init = &(struct clk_init_data){
> +		.name = "sys_a",
> +		.ops = &clk_regmap_gate_ops,
> +		.parent_hws = (const struct clk_hw *[]) {
> +			&t7_sys_a_div.hw
> +		},
> +		.num_parents = 1,
> +	},
> +};
> +
> +static struct clk_regmap t7_sys_b_sel = {
> +	.data = &(struct clk_regmap_mux_data){
> +		.offset = SYS_CLK_CTRL0,
> +		.mask = 0x7,
> +		.shift = 26,
> +		.table = t7_sys_parents_val_table,
> +	},
> +	.hw.init = &(struct clk_init_data){
> +		.name = "sys_b_sel",
> +		.ops = &clk_regmap_mux_ops,
> +		.parent_data = t7_sys_parents,
> +		.num_parents = ARRAY_SIZE(t7_sys_parents),
> +	},
> +};
> +
> +static struct clk_regmap t7_sys_b_div = {
> +	.data = &(struct clk_regmap_div_data){
> +		.offset = SYS_CLK_CTRL0,
> +		.shift = 16,
> +		.width = 10,
> +	},
> +	.hw.init = &(struct clk_init_data){
> +		.name = "sys_b_div",
> +		.ops = &clk_regmap_divider_ops,
> +		.parent_hws = (const struct clk_hw *[]) {
> +			&t7_sys_b_sel.hw
> +		},
> +		.num_parents = 1,
> +	},
> +};
> +
> +static struct clk_regmap t7_sys_b = {
> +	.data = &(struct clk_regmap_gate_data){
> +		.offset = SYS_CLK_CTRL0,
> +		.bit_idx = 29,
> +	},
> +	.hw.init = &(struct clk_init_data){
> +		.name = "sys_b",
> +		.ops = &clk_regmap_gate_ops,
> +		.parent_hws = (const struct clk_hw *[]) {
> +			&t7_sys_b_div.hw
> +		},
> +		.num_parents = 1,
> +	},
> +};
> +
> +static struct clk_regmap t7_sys = {
> +	.data = &(struct clk_regmap_mux_data){
> +		.offset = SYS_CLK_CTRL0,
> +		.mask = 0x1,
> +		.shift = 15,
> +	},
> +	.hw.init = &(struct clk_init_data){
> +		.name = "sys_clk",
> +		.ops = &clk_regmap_mux_ops,
> +		.parent_hws = (const struct clk_hw *[]) {
> +			&t7_sys_a.hw,
> +			&t7_sys_b.hw,
> +		},
> +		.num_parents = 2,
> +	},
> +};
> +
>  static struct clk_regmap t7_ceca_dualdiv_in = {
>  	.data = &(struct clk_regmap_gate_data){
>  		.offset = CECA_CTRL0,
> @@ -824,7 +945,7 @@ static T7_COMP_GATE(sd_emmc_c, NAND_CLK_CTRL, 7, 0);
>  
>  static const struct clk_parent_data t7_spicc_parents[] = {
>  	{ .fw_name = "xtal", },
> -	{ .fw_name = "sys", },
> +	{ .hw = &t7_sys.hw },
>  	{ .fw_name = "fdiv4", },
>  	{ .fw_name = "fdiv3", },
>  	{ .fw_name = "fdiv2", },
> @@ -859,7 +980,7 @@ static T7_COMP_GATE(spicc5, SPICC_CLK_CTRL2, 22, 0);
>  
>  static const struct clk_parent_data t7_saradc_parents[] = {
>  	{ .fw_name = "xtal" },
> -	{ .fw_name = "sys" },
> +	{ .hw = &t7_sys.hw },
>  };
>  
>  static T7_COMP_SEL(saradc, SAR_CLK_CTRL0, 9, 0x1, t7_saradc_parents);
> @@ -929,7 +1050,7 @@ static T7_COMP_SEL(pwm_ao_h, PWM_CLK_AO_GH_CTRL, 25, 0x3, t7_pwm_parents);
>  static T7_COMP_DIV(pwm_ao_h, PWM_CLK_AO_GH_CTRL, 16, 8);
>  static T7_COMP_GATE(pwm_ao_h, PWM_CLK_AO_GH_CTRL, 24, 0);
>  
> -static const struct clk_parent_data t7_sys_pclk_parents = { .fw_name = "sys" };
> +static const struct clk_parent_data t7_sys_pclk_parents = { .hw = &t7_sys.hw };
>  
>  #define T7_SYS_PCLK(_name, _reg, _bit, _flags) \
>  	MESON_PCLK(t7_##_name, _reg, _bit, &t7_sys_pclk_parents, _flags)
> @@ -1161,6 +1282,13 @@ static struct clk_hw *t7_peripherals_hw_clks[] = {
>  	[CLKID_PWM_AO_H_SEL]		= &t7_pwm_ao_h_sel.hw,
>  	[CLKID_PWM_AO_H_DIV]		= &t7_pwm_ao_h_div.hw,
>  	[CLKID_PWM_AO_H]		= &t7_pwm_ao_h.hw,
> +	[CLKID_SYS_A_SEL]		= &t7_sys_a_sel.hw,
> +	[CLKID_SYS_A_DIV]		= &t7_sys_a_div.hw,
> +	[CLKID_SYS_A]			= &t7_sys_a.hw,
> +	[CLKID_SYS_B_SEL]		= &t7_sys_b_sel.hw,
> +	[CLKID_SYS_B_DIV]		= &t7_sys_b_div.hw,
> +	[CLKID_SYS_B]			= &t7_sys_b.hw,
> +	[CLKID_SYS]				= &t7_sys.hw,
>  	[CLKID_SYS_DDR]			= &t7_sys_ddr.hw,
>  	[CLKID_SYS_DOS]			= &t7_sys_dos.hw,
>  	[CLKID_SYS_MIPI_DSI_A]		= &t7_sys_mipi_dsi_a.hw,

-- 
Jerome

_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic