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


Groups > linux.kernel > #1691935 > unrolled thread

[PATCH 042/102] mtd: nand: sunxi: explicitly request exclusive reset control

Started byPhilipp Zabel <p.zabel@pengutronix.de>
First post2017-07-19 17:50 +0200
Last post2017-07-20 11:30 +0200
Articles 3 — 2 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 042/102] mtd: nand: sunxi: explicitly request exclusive reset control Philipp Zabel <p.zabel@pengutronix.de> - 2017-07-19 17:50 +0200
    Re: [PATCH 042/102] mtd: nand: sunxi: explicitly request exclusive  reset control Boris Brezillon <boris.brezillon@free-electrons.com> - 2017-07-20 09:10 +0200
      Re: [PATCH 042/102] mtd: nand: sunxi: explicitly request exclusive  reset control Philipp Zabel <p.zabel@pengutronix.de> - 2017-07-20 11:30 +0200

#1691935 — [PATCH 042/102] mtd: nand: sunxi: explicitly request exclusive reset control

FromPhilipp Zabel <p.zabel@pengutronix.de>
Date2017-07-19 17:50 +0200
Subject[PATCH 042/102] mtd: nand: sunxi: explicitly request exclusive reset control
Message-ID<u4ZoD-27S-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: Boris Brezillon <boris.brezillon@free-electrons.com>
Cc: Richard Weinberger <richard@nod.at>
Cc: David Woodhouse <dwmw2@infradead.org>
Cc: Brian Norris <computersforpeace@gmail.com>
Cc: Marek Vasut <marek.vasut@gmail.com>
Cc: Cyrille Pitchen <cyrille.pitchen@wedev4u.fr>
Cc: Maxime Ripard <maxime.ripard@free-electrons.com>
Cc: Chen-Yu Tsai <wens@csie.org>
Cc: linux-mtd@lists.infradead.org
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
---
 drivers/mtd/nand/sunxi_nand.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mtd/nand/sunxi_nand.c b/drivers/mtd/nand/sunxi_nand.c
index d0b6f8f9f297a..6eb97451f485c 100644
--- a/drivers/mtd/nand/sunxi_nand.c
+++ b/drivers/mtd/nand/sunxi_nand.c
@@ -2208,7 +2208,7 @@ static int sunxi_nfc_probe(struct platform_device *pdev)
 	if (ret)
 		goto out_ahb_clk_unprepare;
 
-	nfc->reset = devm_reset_control_get_optional(dev, "ahb");
+	nfc->reset = devm_reset_control_get_optional_exclusive(dev, "ahb");
 	if (IS_ERR(nfc->reset)) {
 		ret = PTR_ERR(nfc->reset);
 		goto out_mod_clk_unprepare;
-- 
2.11.0

[toc] | [next] | [standalone]


#1692480 — Re: [PATCH 042/102] mtd: nand: sunxi: explicitly request exclusive reset control

FromBoris Brezillon <boris.brezillon@free-electrons.com>
Date2017-07-20 09:10 +0200
SubjectRe: [PATCH 042/102] mtd: nand: sunxi: explicitly request exclusive reset control
Message-ID<u5dKW-3TR-9@gated-at.bofh.it>
In reply to#1691935
Hi Philipp,

On Wed, 19 Jul 2017 17:25:46 +0200
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.

If you don't mind, I'd like to take this patch in the nand tree, just in
case we have other modifications on sunxi_nand.c for this cycle.

Regards,

Boris

> 
> Cc: Boris Brezillon <boris.brezillon@free-electrons.com>
> Cc: Richard Weinberger <richard@nod.at>
> Cc: David Woodhouse <dwmw2@infradead.org>
> Cc: Brian Norris <computersforpeace@gmail.com>
> Cc: Marek Vasut <marek.vasut@gmail.com>
> Cc: Cyrille Pitchen <cyrille.pitchen@wedev4u.fr>
> Cc: Maxime Ripard <maxime.ripard@free-electrons.com>
> Cc: Chen-Yu Tsai <wens@csie.org>
> Cc: linux-mtd@lists.infradead.org
> Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
> ---
>  drivers/mtd/nand/sunxi_nand.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/mtd/nand/sunxi_nand.c b/drivers/mtd/nand/sunxi_nand.c
> index d0b6f8f9f297a..6eb97451f485c 100644
> --- a/drivers/mtd/nand/sunxi_nand.c
> +++ b/drivers/mtd/nand/sunxi_nand.c
> @@ -2208,7 +2208,7 @@ static int sunxi_nfc_probe(struct platform_device *pdev)
>  	if (ret)
>  		goto out_ahb_clk_unprepare;
>  
> -	nfc->reset = devm_reset_control_get_optional(dev, "ahb");
> +	nfc->reset = devm_reset_control_get_optional_exclusive(dev, "ahb");
>  	if (IS_ERR(nfc->reset)) {
>  		ret = PTR_ERR(nfc->reset);
>  		goto out_mod_clk_unprepare;

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


#1692691 — Re: [PATCH 042/102] mtd: nand: sunxi: explicitly request exclusive reset control

FromPhilipp Zabel <p.zabel@pengutronix.de>
Date2017-07-20 11:30 +0200
SubjectRe: [PATCH 042/102] mtd: nand: sunxi: explicitly request exclusive reset control
Message-ID<u5fWr-5m0-39@gated-at.bofh.it>
In reply to#1692480
On Thu, 2017-07-20 at 09:00 +0200, Boris Brezillon wrote:
> Hi Philipp,
> 
> On Wed, 19 Jul 2017 17:25:46 +0200
> 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.
> 
> If you don't mind, I'd like to take this patch in the nand tree, just in
> case we have other modifications on sunxi_nand.c for this cycle.

Absolutely, I'd like to only to apply the last two patches myself,
eventually.

regards
Philipp

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web