Quoting Dong Aisheng (2018-03-20 20:19:49)
> @@ -50,6 +52,38 @@ static int __must_check of_clk_bulk_get(struct device_node *np, int num_clks,
> return ret;
> }
> EXPORT_SYMBOL(of_clk_bulk_get);
> +
> +static int __must_check of_clk_bulk_get_all(struct device_node *np,
> + struct clk_bulk_data **clks)
> +{
> + struct clk_bulk_data *clk_bulk;
> + int num_clks;
> + int ret;
> +
> + num_clks = of_clk_get_parent_count(np);
> + if (!num_clks)
> + return 0;
> +
> + clk_bulk = kcalloc(num_clks, sizeof(*clk_bulk), GFP_KERNEL);
Can be kmalloc_array? of_clk_bulk_get() already clears things out
appropriately.
> + if (!clk_bulk)
> + return -ENOMEM;
> +
> + ret = of_clk_bulk_get(np, num_clks, clk_bulk);
> + if (ret) {
> + kfree(clk_bulk);
> + return ret;
> + }
> +
> + *clks = clk_bulk;
> +
> + return num_clks;
> +}
> +#else
> +static int __must_check of_clk_bulk_get_all(struct device_node *np,
> + struct clk_bulk_data **clks)
> +{
> + return -ENOENT;
> +}
> #endif
This else can probably be dropped too.
>
> void clk_bulk_put(int num_clks, struct clk_bulk_data *clks)
> @@ -90,6 +124,29 @@ int __must_check clk_bulk_get(struct device *dev, int num_clks,
> }
> EXPORT_SYMBOL(clk_bulk_get);
>
> +void clk_bulk_put_all(int num_clks, struct clk_bulk_data *clks)
> +{
> + if (IS_ERR_OR_NULL(clks))
> + return;
> +
> + clk_bulk_put(num_clks, clks);
> +
> + kfree(clks);
> +}
> +EXPORT_SYMBOL(clk_bulk_put_all);
> +
> +int __must_check clk_bulk_get_all(struct device *dev,
> + struct clk_bulk_data **clks)
> +{
> + struct device_node *np = dev_of_node(dev);
> +
> + if (!np)
> + return 0;
> +
> + return of_clk_bulk_get_all(np, clks);
> +}
> +EXPORT_SYMBOL(clk_bulk_get_all);
Looks better!
--
To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
