Re: [PATCH v2 3/3] clk: aspeed: add AST2700 clk driver

From: Krzysztof Kozlowski
Date: Wed Aug 28 2024 - 03:30:25 EST


On 28/08/2024 08:27, Ryan Chen wrote:
> Add AST2700 clock controller driver. This driver also selects MFD_SYSCON,
> which provides access to system controller registers, and register the
> reset controller.
>
> Signed-off-by: Ryan Chen <ryan_chen@xxxxxxxxxxxxxx>


> + clks[SCU0_CLK_GATE_EMMCCLK] =
> + ast2700_clk_hw_register_gate(NULL, "emmcclk-gate", "emmcclk",
> + 0, clk_base + SCU0_CLK_STOP,
> + 27, 0, &ast2700_clk_lock);
> +
> + clks[SCU0_CLK_GATE_RVAS1CLK] =
> + ast2700_clk_hw_register_gate(NULL, "rvas2clk", NULL,
> + 0, clk_base + SCU0_CLK_STOP,
> + 28, 0, &ast2700_clk_lock);
> +
> + of_clk_add_hw_provider(soc0_node, of_clk_hw_onecell_get, clk_data);
> +
> + return 0;
> +};
> +
> +CLK_OF_DECLARE_DRIVER(ast2700_soc0, "aspeed,ast2700-scu0", ast2700_soc0_clk_init);
> +CLK_OF_DECLARE_DRIVER(ast2700_soc1, "aspeed,ast2700-scu1", ast2700_soc1_clk_init);

Nope, this is not documented.

You cannot add new compatibles without bindings.

Best regards,
Krzysztof