Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1691967 > unrolled thread

[PATCH 038/102] mmc: dw_mmc: explicitly request exclusive reset control

Started byPhilipp Zabel <p.zabel@pengutronix.de>
First post2017-07-19 18:00 +0200
Last post2017-07-27 16:50 +0200
Articles 3 — 3 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  [PATCH 038/102] mmc: dw_mmc: explicitly request exclusive reset control Philipp Zabel <p.zabel@pengutronix.de> - 2017-07-19 18:00 +0200
    Re: [PATCH 038/102] mmc: dw_mmc: explicitly request exclusive reset  control Shawn Lin <shawn.lin@rock-chips.com> - 2017-07-22 15:20 +0200
    Re: [PATCH 038/102] mmc: dw_mmc: explicitly request exclusive reset control Ulf Hansson <ulf.hansson@linaro.org> - 2017-07-27 16:50 +0200

#1691967 — [PATCH 038/102] mmc: dw_mmc: explicitly request exclusive reset control

FromPhilipp Zabel <p.zabel@pengutronix.de>
Date2017-07-19 18:00 +0200
Subject[PATCH 038/102] mmc: dw_mmc: explicitly request exclusive reset control
Message-ID<u4Zyk-2bX-55@gated-at.bofh.it>
Commit a53e35db70d1 ("reset: Ensure drivers are explicit when requesting
reset lines") started to transition the reset control request API calls
to explicitly state whether the driver needs exclusive or shared reset
control behavior. Convert all drivers requesting exclusive resets to the
explicit API call so the temporary transition helpers can be removed.

No functional changes.

Cc: Jaehoon Chung <jh80.chung@samsung.com>
Cc: Ulf Hansson <ulf.hansson@linaro.org>
Cc: linux-mmc@vger.kernel.org
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
---
 drivers/mmc/host/dw_mmc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mmc/host/dw_mmc.c b/drivers/mmc/host/dw_mmc.c
index a9dfb26972f21..7c1e2dbc76776 100644
--- a/drivers/mmc/host/dw_mmc.c
+++ b/drivers/mmc/host/dw_mmc.c
@@ -2950,7 +2950,7 @@ static struct dw_mci_board *dw_mci_parse_dt(struct dw_mci *host)
 		return ERR_PTR(-ENOMEM);
 
 	/* find reset controller when exist */
-	pdata->rstc = devm_reset_control_get_optional(dev, "reset");
+	pdata->rstc = devm_reset_control_get_optional_exclusive(dev, "reset");
 	if (IS_ERR(pdata->rstc)) {
 		if (PTR_ERR(pdata->rstc) == -EPROBE_DEFER)
 			return ERR_PTR(-EPROBE_DEFER);
-- 
2.11.0

[toc] | [next] | [standalone]


#1694158 — Re: [PATCH 038/102] mmc: dw_mmc: explicitly request exclusive reset control

FromShawn Lin <shawn.lin@rock-chips.com>
Date2017-07-22 15:20 +0200
SubjectRe: [PATCH 038/102] mmc: dw_mmc: explicitly request exclusive reset control
Message-ID<u62u6-2pk-13@gated-at.bofh.it>
In reply to#1691967
On 2017/7/19 23:25, Philipp Zabel wrote:
> Commit a53e35db70d1 ("reset: Ensure drivers are explicit when requesting
> reset lines") started to transition the reset control request API calls
> to explicitly state whether the driver needs exclusive or shared reset
> control behavior. Convert all drivers requesting exclusive resets to the
> explicit API call so the temporary transition helpers can be removed.
>
> No functional changes.
>
> Cc: Jaehoon Chung <jh80.chung@samsung.com>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>
> Cc: linux-mmc@vger.kernel.org
> Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>

Reviewed-by: Shawn Lin <shawn.lin@rock-chips.com>

> ---
>  drivers/mmc/host/dw_mmc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/mmc/host/dw_mmc.c b/drivers/mmc/host/dw_mmc.c
> index a9dfb26972f21..7c1e2dbc76776 100644
> --- a/drivers/mmc/host/dw_mmc.c
> +++ b/drivers/mmc/host/dw_mmc.c
> @@ -2950,7 +2950,7 @@ static struct dw_mci_board *dw_mci_parse_dt(struct dw_mci *host)
>  		return ERR_PTR(-ENOMEM);
>
>  	/* find reset controller when exist */
> -	pdata->rstc = devm_reset_control_get_optional(dev, "reset");
> +	pdata->rstc = devm_reset_control_get_optional_exclusive(dev, "reset");
>  	if (IS_ERR(pdata->rstc)) {
>  		if (PTR_ERR(pdata->rstc) == -EPROBE_DEFER)
>  			return ERR_PTR(-EPROBE_DEFER);
>


-- 
Best Regards
Shawn Lin

[toc] | [prev] | [next] | [standalone]


#1698061

FromUlf Hansson <ulf.hansson@linaro.org>
Date2017-07-27 16:50 +0200
Message-ID<u7SgX-7jy-41@gated-at.bofh.it>
In reply to#1691967
On 19 July 2017 at 17:25, Philipp Zabel <p.zabel@pengutronix.de> wrote:
> Commit a53e35db70d1 ("reset: Ensure drivers are explicit when requesting
> reset lines") started to transition the reset control request API calls
> to explicitly state whether the driver needs exclusive or shared reset
> control behavior. Convert all drivers requesting exclusive resets to the
> explicit API call so the temporary transition helpers can be removed.
>
> No functional changes.
>
> Cc: Jaehoon Chung <jh80.chung@samsung.com>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>
> Cc: linux-mmc@vger.kernel.org
> Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>

Thanks, applied for next!

Kind regards
Uffe

> ---
>  drivers/mmc/host/dw_mmc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/mmc/host/dw_mmc.c b/drivers/mmc/host/dw_mmc.c
> index a9dfb26972f21..7c1e2dbc76776 100644
> --- a/drivers/mmc/host/dw_mmc.c
> +++ b/drivers/mmc/host/dw_mmc.c
> @@ -2950,7 +2950,7 @@ static struct dw_mci_board *dw_mci_parse_dt(struct dw_mci *host)
>                 return ERR_PTR(-ENOMEM);
>
>         /* find reset controller when exist */
> -       pdata->rstc = devm_reset_control_get_optional(dev, "reset");
> +       pdata->rstc = devm_reset_control_get_optional_exclusive(dev, "reset");
>         if (IS_ERR(pdata->rstc)) {
>                 if (PTR_ERR(pdata->rstc) == -EPROBE_DEFER)
>                         return ERR_PTR(-EPROBE_DEFER);
> --
> 2.11.0
>

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web