lkml.org 
[lkml]   [2018]   [Mar]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH V2 2/4] clk: add new APIs to operate on all available clocks
Date
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!

\
 
 \ /
  Last update: 2018-03-23 17:57    [W:0.382 / U:0.008 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site