[PATCH v3 2/2] clk: visconti: Add VIIF clocks

[yuji.ishikawa.x51 at mail.toshiba]

> -----Original Message-----
> From: Stephen Boyd <sboyd at kernel.org>
> Sent: Friday, November 21, 2025 11:57 AM
> To: Conor Dooley <conor+dt at kernel.org>; Krzysztof Kozlowski
> <krzk+dt at kernel.org>; Michael Turquette <mturquette at baylibre.com>;
> iwamatsu nobuhiro(岩松 信洋 □ＤＩＴＣ○ＣＰＴ)
> <nobuhiro.iwamatsu.x90 at mail.toshiba>; Philipp Zabel
> <p.zabel at pengutronix.de>; Rob Herring <robh at kernel.org>; ishikawa yuji(石
> 川 悠司 □ＡＩＤＣ○ＥＡ開) <yuji2.ishikawa at toshiba.co.jp>
> Cc: linux-clk at vger.kernel.org; linux-arm-kernel at lists.infradead.org;
> linux-kernel at vger.kernel.org; devicetree at vger.kernel.org; ishikawa yuji(石川
> 悠司 □ＡＩＤＣ○ＥＡ開) <yuji2.ishikawa at toshiba.co.jp>
> Subject: Re: [PATCH v3 2/2] clk: visconti: Add VIIF clocks
> 
> 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.

Thank you for pointing this out. I'll check the driver and consider how to clean up the usage.

Regards,
Yuji Ishikawa