[prev in list] [next in list] [prev in thread] [next in thread]
List: linux-serial
Subject: Re: [PATCH V2 1/8] interconnect: Add devm_of_icc_get() as exported API for users
From: Bjorn Andersson <bjorn.andersson () linaro ! org>
Date: 2020-03-27 23:02:21
Message-ID: 20200327230221.GK5063 () builder
[Download RAW message or body]
On Fri 13 Mar 06:12 PDT 2020, Akash Asthana wrote:
> Users can use devm version of of_icc_get() to benefit from automatic
> resource release.
>
> Signed-off-by: Akash Asthana <akashast@codeaurora.org>
Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Regards,
Bjorn
> ---
> drivers/interconnect/core.c | 25 +++++++++++++++++++++++++
> include/linux/interconnect.h | 7 +++++++
> 2 files changed, 32 insertions(+)
>
> diff --git a/drivers/interconnect/core.c b/drivers/interconnect/core.c
> index 2c6515e..f5699ed 100644
> --- a/drivers/interconnect/core.c
> +++ b/drivers/interconnect/core.c
> @@ -350,6 +350,31 @@ static struct icc_node *of_icc_get_from_provider(struct \
> of_phandle_args *spec) return node;
> }
>
> +static void devm_icc_release(struct device *dev, void *res)
> +{
> + icc_put(*(struct icc_path **)res);
> +}
> +
> +struct icc_path *devm_of_icc_get(struct device *dev, const char *name)
> +{
> + struct icc_path **ptr, *path;
> +
> + ptr = devres_alloc(devm_icc_release, sizeof(**ptr), GFP_KERNEL);
> + if (!ptr)
> + return ERR_PTR(-ENOMEM);
> +
> + path = of_icc_get(dev, name);
> + if (!IS_ERR(path)) {
> + *ptr = path;
> + devres_add(dev, ptr);
> + } else {
> + devres_free(ptr);
> + }
> +
> + return path;
> +}
> +EXPORT_SYMBOL_GPL(devm_of_icc_get);
> +
> /**
> * of_icc_get() - get a path handle from a DT node based on name
> * @dev: device pointer for the consumer device
> diff --git a/include/linux/interconnect.h b/include/linux/interconnect.h
> index d70a914..7706924 100644
> --- a/include/linux/interconnect.h
> +++ b/include/linux/interconnect.h
> @@ -28,6 +28,7 @@ struct device;
> struct icc_path *icc_get(struct device *dev, const int src_id,
> const int dst_id);
> struct icc_path *of_icc_get(struct device *dev, const char *name);
> +struct icc_path *devm_of_icc_get(struct device *dev, const char *name);
> void icc_put(struct icc_path *path);
> int icc_set_bw(struct icc_path *path, u32 avg_bw, u32 peak_bw);
> void icc_set_tag(struct icc_path *path, u32 tag);
> @@ -46,6 +47,12 @@ static inline struct icc_path *of_icc_get(struct device *dev,
> return NULL;
> }
>
> +static inline struct icc_path *devm_of_icc_get(struct device *dev,
> + const char *name)
> +{
> + return NULL;
> +}
> +
> static inline void icc_put(struct icc_path *path)
> {
> }
> --
> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,\na Linux \
> Foundation Collaborative Project
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic