lkml.org 
[lkml]   [2019]   [Mar]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH 2/5] reset: Add acquired flag to of_reset_control_array_get()
From
Date
On Thu, 2019-02-21 at 16:25 +0100, Thierry Reding wrote:
> From: Thierry Reding <treding@nvidia.com>
>
> In order to be able to request an array of reset controls in acquired or
> released mode, add the acquired flag to of_reset_control_array_get() and
> pass the flag to subsequent calls of __of_reset_control_get().
>
> Signed-off-by: Thierry Reding <treding@nvidia.com>
> ---
> drivers/reset/core.c | 9 ++++++---
> drivers/usb/dwc3/dwc3-of-simple.c | 3 ++-
> include/linux/reset.h | 14 ++++++++------
> 3 files changed, 16 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/reset/core.c b/drivers/reset/core.c
> index 1e8a42b16f23..f94da91c22af 100644
> --- a/drivers/reset/core.c
> +++ b/drivers/reset/core.c
> @@ -830,12 +830,15 @@ static int of_reset_control_get_count(struct device_node *node)
> * @np: device node for the device that requests the reset controls array
> * @shared: whether reset controls are shared or not
> * @optional: whether it is optional to get the reset controls
> + * @acquired: only one reset control may be acquired for a given controller
> + * and ID
> *
> * Returns pointer to allocated reset_control_array on success or
> * error on failure
> */
> struct reset_control *
> -of_reset_control_array_get(struct device_node *np, bool shared, bool optional)
> +of_reset_control_array_get(struct device_node *np, bool shared, bool optional,
> + bool acquired)
> {
> struct reset_control_array *resets;
> struct reset_control *rstc;
> @@ -851,7 +854,7 @@ of_reset_control_array_get(struct device_node *np, bool shared, bool optional)
>
> for (i = 0; i < num; i++) {
> rstc = __of_reset_control_get(np, NULL, i, shared, optional,
> - true);
> + acquired);
> if (IS_ERR(rstc))
> goto err_rst;
> resets->rstc[i] = rstc;
> @@ -898,7 +901,7 @@ devm_reset_control_array_get(struct device *dev, bool shared, bool optional)
> if (!devres)
> return ERR_PTR(-ENOMEM);
>
> - rstc = of_reset_control_array_get(dev->of_node, shared, optional);
> + rstc = of_reset_control_array_get(dev->of_node, shared, optional, true);
> if (IS_ERR(rstc)) {
> devres_free(devres);
> return rstc;
> diff --git a/drivers/usb/dwc3/dwc3-of-simple.c b/drivers/usb/dwc3/dwc3-of-simple.c
> index 4c2771c5e727..67ce2037472d 100644
> --- a/drivers/usb/dwc3/dwc3-of-simple.c
> +++ b/drivers/usb/dwc3/dwc3-of-simple.c
> @@ -107,7 +107,8 @@ static int dwc3_of_simple_probe(struct platform_device *pdev)
> simple->pulse_resets = true;
> }
>
> - simple->resets = of_reset_control_array_get(np, shared_resets, true);
> + simple->resets = of_reset_control_array_get(np, shared_resets, true,
> + true);

Felipe, could I get your acked-by to merge this through the reset tree?

dwc3-of-simple is a bit of a special case because it calls
of_reset_control_array_get directly instead of through the
of_reset_control_array_get_shared/exclusive wrappers.

regards
Philipp

\
 
 \ /
  Last update: 2019-03-19 17:38    [W:0.483 / U:0.164 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site