Commit 0f6f3235 authored by Linus Walleij's avatar Linus Walleij Committed by Ulf Hansson

mmc: jz4740: Use GPIO descriptor for power

The power GPIO line is passed with inversion flags and all from
the platform data. Switch to using an optional GPIO descriptor and
use this to switch the power.

Augment the only boardfile to pass in the proper "power" descriptor
in the GPIO descriptor machine table instead.

As the GPIO handling is now much simpler, we can cut down on some
overhead code.

Cc: Paul Cercueil <paul@crapouillou.net>
Cc: linux-mips@linux-mips.org
Signed-off-by: default avatarLinus Walleij <linus.walleij@linaro.org>
Acked-by: default avatarPaul Burton <paul.burton@mips.com>
Signed-off-by: default avatarUlf Hansson <ulf.hansson@linaro.org>
parent 0c901c05
...@@ -3,10 +3,8 @@ ...@@ -3,10 +3,8 @@
#define __LINUX_MMC_JZ4740_MMC #define __LINUX_MMC_JZ4740_MMC
struct jz4740_mmc_platform_data { struct jz4740_mmc_platform_data {
int gpio_power;
unsigned card_detect_active_low:1; unsigned card_detect_active_low:1;
unsigned read_only_active_low:1; unsigned read_only_active_low:1;
unsigned power_active_low:1;
unsigned data_1bit:1; unsigned data_1bit:1;
}; };
......
...@@ -43,8 +43,6 @@ ...@@ -43,8 +43,6 @@
#include "clock.h" #include "clock.h"
/* GPIOs */ /* GPIOs */
#define QI_LB60_GPIO_SD_VCC_EN_N JZ_GPIO_PORTD(2)
#define QI_LB60_GPIO_KEYOUT(x) (JZ_GPIO_PORTC(10) + (x)) #define QI_LB60_GPIO_KEYOUT(x) (JZ_GPIO_PORTC(10) + (x))
#define QI_LB60_GPIO_KEYIN(x) (JZ_GPIO_PORTD(18) + (x)) #define QI_LB60_GPIO_KEYIN(x) (JZ_GPIO_PORTD(18) + (x))
#define QI_LB60_GPIO_KEYIN8 JZ_GPIO_PORTD(26) #define QI_LB60_GPIO_KEYIN8 JZ_GPIO_PORTD(26)
...@@ -385,14 +383,14 @@ static struct platform_device qi_lb60_gpio_keys = { ...@@ -385,14 +383,14 @@ static struct platform_device qi_lb60_gpio_keys = {
}; };
static struct jz4740_mmc_platform_data qi_lb60_mmc_pdata = { static struct jz4740_mmc_platform_data qi_lb60_mmc_pdata = {
.gpio_power = QI_LB60_GPIO_SD_VCC_EN_N, /* Intentionally left blank */
.power_active_low = 1,
}; };
static struct gpiod_lookup_table qi_lb60_mmc_gpio_table = { static struct gpiod_lookup_table qi_lb60_mmc_gpio_table = {
.dev_id = "jz4740-mmc.0", .dev_id = "jz4740-mmc.0",
.table = { .table = {
GPIO_LOOKUP("GPIOD", 0, "cd", GPIO_ACTIVE_HIGH), GPIO_LOOKUP("GPIOD", 0, "cd", GPIO_ACTIVE_HIGH),
GPIO_LOOKUP("GPIOD", 2, "power", GPIO_ACTIVE_LOW),
{ }, { },
}, },
}; };
......
...@@ -21,7 +21,7 @@ ...@@ -21,7 +21,7 @@
#include <linux/dmaengine.h> #include <linux/dmaengine.h>
#include <linux/dma-mapping.h> #include <linux/dma-mapping.h>
#include <linux/err.h> #include <linux/err.h>
#include <linux/gpio.h> #include <linux/gpio/consumer.h>
#include <linux/interrupt.h> #include <linux/interrupt.h>
#include <linux/io.h> #include <linux/io.h>
#include <linux/irq.h> #include <linux/irq.h>
...@@ -136,6 +136,7 @@ struct jz4740_mmc_host { ...@@ -136,6 +136,7 @@ struct jz4740_mmc_host {
struct platform_device *pdev; struct platform_device *pdev;
struct jz4740_mmc_platform_data *pdata; struct jz4740_mmc_platform_data *pdata;
struct clk *clk; struct clk *clk;
struct gpio_desc *power;
enum jz4740_mmc_version version; enum jz4740_mmc_version version;
...@@ -903,18 +904,16 @@ static void jz4740_mmc_set_ios(struct mmc_host *mmc, struct mmc_ios *ios) ...@@ -903,18 +904,16 @@ static void jz4740_mmc_set_ios(struct mmc_host *mmc, struct mmc_ios *ios)
switch (ios->power_mode) { switch (ios->power_mode) {
case MMC_POWER_UP: case MMC_POWER_UP:
jz4740_mmc_reset(host); jz4740_mmc_reset(host);
if (host->pdata && gpio_is_valid(host->pdata->gpio_power)) if (host->power)
gpio_set_value(host->pdata->gpio_power, gpiod_set_value(host->power, 1);
!host->pdata->power_active_low);
host->cmdat |= JZ_MMC_CMDAT_INIT; host->cmdat |= JZ_MMC_CMDAT_INIT;
clk_prepare_enable(host->clk); clk_prepare_enable(host->clk);
break; break;
case MMC_POWER_ON: case MMC_POWER_ON:
break; break;
default: default:
if (host->pdata && gpio_is_valid(host->pdata->gpio_power)) if (host->power)
gpio_set_value(host->pdata->gpio_power, gpiod_set_value(host->power, 0);
host->pdata->power_active_low);
clk_disable_unprepare(host->clk); clk_disable_unprepare(host->clk);
break; break;
} }
...@@ -947,30 +946,9 @@ static const struct mmc_host_ops jz4740_mmc_ops = { ...@@ -947,30 +946,9 @@ static const struct mmc_host_ops jz4740_mmc_ops = {
.enable_sdio_irq = jz4740_mmc_enable_sdio_irq, .enable_sdio_irq = jz4740_mmc_enable_sdio_irq,
}; };
static int jz4740_mmc_request_gpio(struct device *dev, int gpio, static int jz4740_mmc_request_gpios(struct jz4740_mmc_host *host,
const char *name, bool output, int value) struct mmc_host *mmc,
{ struct platform_device *pdev)
int ret;
if (!gpio_is_valid(gpio))
return 0;
ret = gpio_request(gpio, name);
if (ret) {
dev_err(dev, "Failed to request %s gpio: %d\n", name, ret);
return ret;
}
if (output)
gpio_direction_output(gpio, value);
else
gpio_direction_input(gpio);
return 0;
}
static int jz4740_mmc_request_gpios(struct mmc_host *mmc,
struct platform_device *pdev)
{ {
struct jz4740_mmc_platform_data *pdata = dev_get_platdata(&pdev->dev); struct jz4740_mmc_platform_data *pdata = dev_get_platdata(&pdev->dev);
int ret = 0; int ret = 0;
...@@ -995,19 +973,12 @@ static int jz4740_mmc_request_gpios(struct mmc_host *mmc, ...@@ -995,19 +973,12 @@ static int jz4740_mmc_request_gpios(struct mmc_host *mmc,
if (ret == -EPROBE_DEFER) if (ret == -EPROBE_DEFER)
return ret; return ret;
return jz4740_mmc_request_gpio(&pdev->dev, pdata->gpio_power, host->power = devm_gpiod_get_optional(&pdev->dev, "power",
"MMC read only", true, pdata->power_active_low); GPIOD_OUT_HIGH);
} if (IS_ERR(host->power))
return PTR_ERR(host->power);
static void jz4740_mmc_free_gpios(struct platform_device *pdev)
{
struct jz4740_mmc_platform_data *pdata = dev_get_platdata(&pdev->dev);
if (!pdata)
return;
if (gpio_is_valid(pdata->gpio_power)) return 0;
gpio_free(pdata->gpio_power);
} }
static const struct of_device_id jz4740_mmc_of_match[] = { static const struct of_device_id jz4740_mmc_of_match[] = {
...@@ -1053,7 +1024,7 @@ static int jz4740_mmc_probe(struct platform_device* pdev) ...@@ -1053,7 +1024,7 @@ static int jz4740_mmc_probe(struct platform_device* pdev)
mmc->caps |= MMC_CAP_SDIO_IRQ; mmc->caps |= MMC_CAP_SDIO_IRQ;
if (!(pdata && pdata->data_1bit)) if (!(pdata && pdata->data_1bit))
mmc->caps |= MMC_CAP_4_BIT_DATA; mmc->caps |= MMC_CAP_4_BIT_DATA;
ret = jz4740_mmc_request_gpios(mmc, pdev); ret = jz4740_mmc_request_gpios(host, mmc, pdev);
if (ret) if (ret)
goto err_free_host; goto err_free_host;
} }
...@@ -1104,7 +1075,7 @@ static int jz4740_mmc_probe(struct platform_device* pdev) ...@@ -1104,7 +1075,7 @@ static int jz4740_mmc_probe(struct platform_device* pdev)
dev_name(&pdev->dev), host); dev_name(&pdev->dev), host);
if (ret) { if (ret) {
dev_err(&pdev->dev, "Failed to request irq: %d\n", ret); dev_err(&pdev->dev, "Failed to request irq: %d\n", ret);
goto err_free_gpios; goto err_free_host;
} }
jz4740_mmc_clock_disable(host); jz4740_mmc_clock_disable(host);
...@@ -1135,8 +1106,6 @@ static int jz4740_mmc_probe(struct platform_device* pdev) ...@@ -1135,8 +1106,6 @@ static int jz4740_mmc_probe(struct platform_device* pdev)
jz4740_mmc_release_dma_channels(host); jz4740_mmc_release_dma_channels(host);
err_free_irq: err_free_irq:
free_irq(host->irq, host); free_irq(host->irq, host);
err_free_gpios:
jz4740_mmc_free_gpios(pdev);
err_free_host: err_free_host:
mmc_free_host(mmc); mmc_free_host(mmc);
...@@ -1155,8 +1124,6 @@ static int jz4740_mmc_remove(struct platform_device *pdev) ...@@ -1155,8 +1124,6 @@ static int jz4740_mmc_remove(struct platform_device *pdev)
free_irq(host->irq, host); free_irq(host->irq, host);
jz4740_mmc_free_gpios(pdev);
if (host->use_dma) if (host->use_dma)
jz4740_mmc_release_dma_channels(host); jz4740_mmc_release_dma_channels(host);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment