Re: [PATCH 2/4] clk: Ingenic: Add support for TCU of X1000.

Zhou Yanjie
2020-03-04T10:42:03Z

Hi,

On 2020年02月19日 16:29, 周琰杰 (Zhou Yanjie) wrote:
> X1000 has a different TCU containing OST, since X1000, OST has been
> independent of TCU. This patch is prepare for later OST driver.
>
> Signed-off-by: 周琰杰 (Zhou Yanjie) <zhouyanjie@wanyeetech.com>

Any review/ack from clock framework subsystem?

Thanks and best regards!

> ---
> drivers/clk/ingenic/tcu.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/drivers/clk/ingenic/tcu.c b/drivers/clk/ingenic/tcu.c
> index ad7daa4..8799907 100644
> --- a/drivers/clk/ingenic/tcu.c
> +++ b/drivers/clk/ingenic/tcu.c
> @@ -317,10 +317,17 @@ static const struct ingenic_soc_info jz4770_soc_info = {
> .has_tcu_clk = false,
> };
>
> +static const struct ingenic_soc_info x1000_soc_info = {
> + .num_channels = 8,
> + .has_ost = false, /* X1000 has OST, but it not belong TCU */
> + .has_tcu_clk = false,
> +};
> +
> static const struct of_device_id ingenic_tcu_of_match[] __initconst = {
> { .compatible = "ingenic,jz4740-tcu", .data = &jz4740_soc_info, },
> { .compatible = "ingenic,jz4725b-tcu", .data = &jz4725b_soc_info, },
> { .compatible = "ingenic,jz4770-tcu", .data = &jz4770_soc_info, },
> + { .compatible = "ingenic,x1000-tcu", .data = &x1000_soc_info, },
> { /* sentinel */ }
> };
>
> @@ -471,3 +478,4 @@ static void __init ingenic_tcu_init(struct device_node *np)
> CLK_OF_DECLARE_DRIVER(jz4740_cgu, "ingenic,jz4740-tcu", ingenic_tcu_init);
> CLK_OF_DECLARE_DRIVER(jz4725b_cgu, "ingenic,jz4725b-tcu", ingenic_tcu_init);
> CLK_OF_DECLARE_DRIVER(jz4770_cgu, "ingenic,jz4770-tcu", ingenic_tcu_init);
> +CLK_OF_DECLARE_DRIVER(x1000_cgu, "ingenic,x1000-tcu", ingenic_tcu_init);