[PATCH v13 08/10] clk: nuvoton: Add clock driver for ma35d1 clock controller

Stephen Boyd sboyd at kernel.org
Mon Jun 12 19:16:03 PDT 2023


Quoting Jacky Huang (2023-06-12 18:37:31)
> On 2023/6/13 上午 09:02, Stephen Boyd wrote:
> > Quoting Jacky Huang (2023-06-04 21:07:47)
> >> diff --git a/drivers/clk/nuvoton/clk-ma35d1-divider.c b/drivers/clk/nuvoton/clk-ma35d1-divider.c
> >> +                                           int num_pdata)
> >> +{
> >> +       return clk_hw_register_mux_parent_data(dev, name, pdata, num_pdata,
> >> +                                              CLK_SET_RATE_NO_REPARENT, reg, shift,
> >> +                                              width, 0, &ma35d1_lock);
> >> +}
> >> +
> >> +static struct clk_hw *ma35d1_clk_mux(struct device *dev, const char *name,
> >> +                                    void __iomem *reg, u8 shift, u8 width,
> >> +                                    const char *const *parents, int num_parents)
> > Please don't use string arrays for parent descriptions. Everything
> > should use clk_parent_data or direct clk_hw pointers.
> 
> I will use clk_parent_data instead of strings.
> 

When you use clk_parent_data, just don't set anything for the "dummy"
clks besides a -1 .index to indicate there isn't a parent for that
entry.



More information about the linux-arm-kernel mailing list