[PATCH] mmc: pwrseq_simple: Fix regression with optional GPIOs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Commit ce037275861e ("mmc: pwrseq_simple: use GPIO descriptors array API")
changed the handling MMC power sequence so GPIOs no longer are optional.

This broke SDIO WLAN at least for omap5 that can't yet use the reset GPIOs
with pwrseq_simple as a wait is needed after enabling the SDIO device.

Let's fix the problem by allocating the GPIO values array during init
depending on the optional GPIOs found.

Note that depending on the board specific configuration, some of the GPIOs
can be permanently set up with pulls, so we want to keep pwrseq_simple
GPIOs optional.

Cc: Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx>
Signed-off-by: Tony Lindgren <tony@xxxxxxxxxxx>
---
 drivers/mmc/core/pwrseq_simple.c | 20 ++++++++++++++++----
 1 file changed, 16 insertions(+), 4 deletions(-)

--- a/drivers/mmc/core/pwrseq_simple.c
+++ b/drivers/mmc/core/pwrseq_simple.c
@@ -24,6 +24,7 @@ struct mmc_pwrseq_simple {
 	bool clk_enabled;
 	struct clk *ext_clk;
 	struct gpio_descs *reset_gpios;
+	int *values;
 };
 
 static void mmc_pwrseq_simple_set_gpios_value(struct mmc_pwrseq_simple *pwrseq,
@@ -31,13 +32,15 @@ static void mmc_pwrseq_simple_set_gpios_value(struct mmc_pwrseq_simple *pwrseq,
 {
 	int i;
 	struct gpio_descs *reset_gpios = pwrseq->reset_gpios;
-	int values[reset_gpios->ndescs];
+
+	if (!reset_gpios || !reset_gpios->ndescs)
+		return;
 
 	for (i = 0; i < reset_gpios->ndescs; i++)
-		values[i] = value;
+		pwrseq->values[i] = value;
 
 	gpiod_set_array_value_cansleep(reset_gpios->ndescs, reset_gpios->desc,
-				       values);
+				       pwrseq->values);
 }
 
 static void mmc_pwrseq_simple_pre_power_on(struct mmc_host *host)
@@ -84,6 +87,7 @@ static void mmc_pwrseq_simple_free(struct mmc_host *host)
 	if (!IS_ERR(pwrseq->ext_clk))
 		clk_put(pwrseq->ext_clk);
 
+	kfree(pwrseq->values);
 	kfree(pwrseq);
 }
 
@@ -111,12 +115,20 @@ struct mmc_pwrseq *mmc_pwrseq_simple_alloc(struct mmc_host *host,
 		goto free;
 	}
 
-	pwrseq->reset_gpios = gpiod_get_array(dev, "reset", GPIOD_OUT_HIGH);
+	pwrseq->reset_gpios = gpiod_get_array_optional(dev, "reset",
+						       GPIOD_OUT_HIGH);
 	if (IS_ERR(pwrseq->reset_gpios)) {
 		ret = PTR_ERR(pwrseq->reset_gpios);
 		goto clk_put;
 	}
 
+	if (pwrseq->reset_gpios && pwrseq->reset_gpios->ndescs) {
+		pwrseq->values = kzalloc(pwrseq->reset_gpios->ndescs *
+					 sizeof(int), GFP_KERNEL);
+		if (!pwrseq->values)
+			goto clk_put;
+	}
+
 	pwrseq->pwrseq.ops = &mmc_pwrseq_simple_ops;
 
 	return &pwrseq->pwrseq;
-- 
2.6.2

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux