On Monday, May 27, 2019 2:05:35 PM CEST Kefeng Wang wrote: > Use of_clk_get_parent_count() instead of open coding. > > Reviewed-by: Geert Uytterhoeven <[email protected]> > Signed-off-by: Kefeng Wang <[email protected]> > --- > drivers/base/power/clock_ops.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/base/power/clock_ops.c b/drivers/base/power/clock_ops.c > index 59d19dd64928..3e84e3085d43 100644 > --- a/drivers/base/power/clock_ops.c > +++ b/drivers/base/power/clock_ops.c > @@ -12,6 +12,7 @@ > #include <linux/pm_clock.h> > #include <linux/clk.h> > #include <linux/clkdev.h> > +#include <linux/of_clk.h> > #include <linux/slab.h> > #include <linux/err.h> > #include <linux/pm_domain.h> > @@ -195,8 +196,7 @@ int of_pm_clk_add_clks(struct device *dev) > if (!dev || !dev->of_node) > return -EINVAL; > > - count = of_count_phandle_with_args(dev->of_node, "clocks", > - "#clock-cells"); > + count = of_clk_get_parent_count(dev->of_node); > if (count <= 0) > return -ENODEV; > >
Applied, thanks!

