[PATCH v3 2/2] clk: visconti: Add VIIF clocks
Stephen Boyd
sboyd at kernel.org
Thu Nov 20 18:56:54 PST 2025
Quoting Yuji Ishikawa (2025-11-13 23:05:12)
> Add the control sequence of register bits to handle the clocks and the
> resets of Video Input Interface.
>
> Signed-off-by: Yuji Ishikawa <yuji2.ishikawa at toshiba.co.jp>
> ---
Applied to clk-next but see below.
> diff --git a/drivers/clk/visconti/clkc-tmpv770x.c b/drivers/clk/visconti/clkc-tmpv770x.c
> index 94a736da6..1e2e8d643 100644
> --- a/drivers/clk/visconti/clkc-tmpv770x.c
> +++ b/drivers/clk/visconti/clkc-tmpv770x.c
> @@ -18,8 +18,8 @@
> #include "reset.h"
>
> /* Must be equal to the last clock/reset ID increased by one */
> -#define CLKS_NR (TMPV770X_CLK_BUSLCK + 1)
> -#define RESETS_NR (TMPV770X_RESET_SBUSCLK + 1)
> +#define CLKS_NR (TMPV770X_CLK_VIIFBS1_PROC + 1)
> +#define RESETS_NR (TMPV770X_RESET_VIIFBS1_L1ISP + 1)
>
> static DEFINE_SPINLOCK(tmpv770x_clk_lock);
> static DEFINE_SPINLOCK(tmpv770x_rst_lock);
> @@ -32,6 +32,10 @@ static const struct clk_parent_data pietherplls_parent_data[] = {
> { .fw_name = "pietherpll", .name = "pietherpll", },
> };
>
> +static const struct clk_parent_data pidnnplls_parent_data[] = {
> + { .fw_name = "pidnnpll", .name = "pidnnpll", },
> +};
This is wrong. There's no "pidnnpll" in the DT binding for
toshiba,tmpv770x-pismu 'clocks' property. Please send a follow up patch
that cleans up the usage of clk_parent_data in this driver.
More information about the linux-arm-kernel
mailing list