mmc: sdhci: eliminate sdhci_of_host and sdhci_of_data
authorShawn Guo <shawn.guo@linaro.org>
Wed, 20 Jul 2011 21:13:36 +0000 (17:13 -0400)
committerChris Ball <cjb@laptop.org>
Wed, 20 Jul 2011 21:20:38 +0000 (17:20 -0400)
The patch migrates the use of sdhci_of_host and sdhci_of_data to
sdhci_pltfm_host and sdhci_pltfm_data, so that the former pair can
be eliminated.

Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
Reviewed-by: Grant Likely <grant.likely@secretlab.ca>
Reviewed-by: Wolfram Sang <w.sang@pengutronix.de>
Acked-by: Anton Vorontsov <cbouatmailru@gmail.com>
Signed-off-by: Chris Ball <cjb@laptop.org>
drivers/mmc/host/sdhci-of-core.c [deleted file]
drivers/mmc/host/sdhci-of-esdhc.c
drivers/mmc/host/sdhci-of-hlwd.c
drivers/mmc/host/sdhci-of.h
drivers/mmc/host/sdhci-pltfm.h

diff --git a/drivers/mmc/host/sdhci-of-core.c b/drivers/mmc/host/sdhci-of-core.c
deleted file mode 100644 (file)
index 60e4186..0000000
+++ /dev/null
@@ -1,253 +0,0 @@
-/*
- * OpenFirmware bindings for Secure Digital Host Controller Interface.
- *
- * Copyright (c) 2007 Freescale Semiconductor, Inc.
- * Copyright (c) 2009 MontaVista Software, Inc.
- *
- * Authors: Xiaobo Xie <X.Xie@freescale.com>
- *         Anton Vorontsov <avorontsov@ru.mvista.com>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or (at
- * your option) any later version.
- */
-
-#include <linux/err.h>
-#include <linux/module.h>
-#include <linux/init.h>
-#include <linux/io.h>
-#include <linux/interrupt.h>
-#include <linux/delay.h>
-#include <linux/of.h>
-#include <linux/of_platform.h>
-#include <linux/of_address.h>
-#include <linux/of_irq.h>
-#include <linux/mmc/host.h>
-#ifdef CONFIG_PPC
-#include <asm/machdep.h>
-#endif
-#include "sdhci-of.h"
-#include "sdhci.h"
-
-#ifdef CONFIG_MMC_SDHCI_BIG_ENDIAN_32BIT_BYTE_SWAPPER
-
-/*
- * These accessors are designed for big endian hosts doing I/O to
- * little endian controllers incorporating a 32-bit hardware byte swapper.
- */
-
-u32 sdhci_be32bs_readl(struct sdhci_host *host, int reg)
-{
-       return in_be32(host->ioaddr + reg);
-}
-
-u16 sdhci_be32bs_readw(struct sdhci_host *host, int reg)
-{
-       return in_be16(host->ioaddr + (reg ^ 0x2));
-}
-
-u8 sdhci_be32bs_readb(struct sdhci_host *host, int reg)
-{
-       return in_8(host->ioaddr + (reg ^ 0x3));
-}
-
-void sdhci_be32bs_writel(struct sdhci_host *host, u32 val, int reg)
-{
-       out_be32(host->ioaddr + reg, val);
-}
-
-void sdhci_be32bs_writew(struct sdhci_host *host, u16 val, int reg)
-{
-       struct sdhci_of_host *of_host = sdhci_priv(host);
-       int base = reg & ~0x3;
-       int shift = (reg & 0x2) * 8;
-
-       switch (reg) {
-       case SDHCI_TRANSFER_MODE:
-               /*
-                * Postpone this write, we must do it together with a
-                * command write that is down below.
-                */
-               of_host->xfer_mode_shadow = val;
-               return;
-       case SDHCI_COMMAND:
-               sdhci_be32bs_writel(host, val << 16 | of_host->xfer_mode_shadow,
-                                   SDHCI_TRANSFER_MODE);
-               return;
-       }
-       clrsetbits_be32(host->ioaddr + base, 0xffff << shift, val << shift);
-}
-
-void sdhci_be32bs_writeb(struct sdhci_host *host, u8 val, int reg)
-{
-       int base = reg & ~0x3;
-       int shift = (reg & 0x3) * 8;
-
-       clrsetbits_be32(host->ioaddr + base , 0xff << shift, val << shift);
-}
-#endif /* CONFIG_MMC_SDHCI_BIG_ENDIAN_32BIT_BYTE_SWAPPER */
-
-#ifdef CONFIG_PM
-
-static int sdhci_of_suspend(struct platform_device *ofdev, pm_message_t state)
-{
-       struct sdhci_host *host = dev_get_drvdata(&ofdev->dev);
-
-       return mmc_suspend_host(host->mmc);
-}
-
-static int sdhci_of_resume(struct platform_device *ofdev)
-{
-       struct sdhci_host *host = dev_get_drvdata(&ofdev->dev);
-
-       return mmc_resume_host(host->mmc);
-}
-
-#else
-
-#define sdhci_of_suspend NULL
-#define sdhci_of_resume NULL
-
-#endif
-
-static bool __devinit sdhci_of_wp_inverted(struct device_node *np)
-{
-       if (of_get_property(np, "sdhci,wp-inverted", NULL))
-               return true;
-
-       /* Old device trees don't have the wp-inverted property. */
-#ifdef CONFIG_PPC
-       return machine_is(mpc837x_rdb) || machine_is(mpc837x_mds);
-#else
-       return false;
-#endif
-}
-
-static const struct of_device_id sdhci_of_match[];
-static int __devinit sdhci_of_probe(struct platform_device *ofdev)
-{
-       const struct of_device_id *match;
-       struct device_node *np = ofdev->dev.of_node;
-       struct sdhci_of_data *sdhci_of_data;
-       struct sdhci_host *host;
-       struct sdhci_of_host *of_host;
-       const __be32 *clk;
-       int size;
-       int ret;
-
-       match = of_match_device(sdhci_of_match, &ofdev->dev);
-       if (!match)
-               return -EINVAL;
-       sdhci_of_data = match->data;
-
-       if (!of_device_is_available(np))
-               return -ENODEV;
-
-       host = sdhci_alloc_host(&ofdev->dev, sizeof(*of_host));
-       if (IS_ERR(host))
-               return -ENOMEM;
-
-       of_host = sdhci_priv(host);
-       dev_set_drvdata(&ofdev->dev, host);
-
-       host->ioaddr = of_iomap(np, 0);
-       if (!host->ioaddr) {
-               ret = -ENOMEM;
-               goto err_addr_map;
-       }
-
-       host->irq = irq_of_parse_and_map(np, 0);
-       if (!host->irq) {
-               ret = -EINVAL;
-               goto err_no_irq;
-       }
-
-       host->hw_name = dev_name(&ofdev->dev);
-       if (sdhci_of_data) {
-               host->quirks = sdhci_of_data->quirks;
-               host->ops = &sdhci_of_data->ops;
-       }
-
-       if (of_get_property(np, "sdhci,auto-cmd12", NULL))
-               host->quirks |= SDHCI_QUIRK_MULTIBLOCK_READ_ACMD12;
-
-
-       if (of_get_property(np, "sdhci,1-bit-only", NULL))
-               host->quirks |= SDHCI_QUIRK_FORCE_1_BIT_DATA;
-
-       if (sdhci_of_wp_inverted(np))
-               host->quirks |= SDHCI_QUIRK_INVERTED_WRITE_PROTECT;
-
-       clk = of_get_property(np, "clock-frequency", &size);
-       if (clk && size == sizeof(*clk) && *clk)
-               of_host->clock = be32_to_cpup(clk);
-
-       ret = sdhci_add_host(host);
-       if (ret)
-               goto err_add_host;
-
-       return 0;
-
-err_add_host:
-       irq_dispose_mapping(host->irq);
-err_no_irq:
-       iounmap(host->ioaddr);
-err_addr_map:
-       sdhci_free_host(host);
-       return ret;
-}
-
-static int __devexit sdhci_of_remove(struct platform_device *ofdev)
-{
-       struct sdhci_host *host = dev_get_drvdata(&ofdev->dev);
-
-       sdhci_remove_host(host, 0);
-       sdhci_free_host(host);
-       irq_dispose_mapping(host->irq);
-       iounmap(host->ioaddr);
-       return 0;
-}
-
-static const struct of_device_id sdhci_of_match[] = {
-#ifdef CONFIG_MMC_SDHCI_OF_ESDHC
-       { .compatible = "fsl,mpc8379-esdhc", .data = &sdhci_esdhc, },
-       { .compatible = "fsl,mpc8536-esdhc", .data = &sdhci_esdhc, },
-       { .compatible = "fsl,esdhc", .data = &sdhci_esdhc, },
-#endif
-#ifdef CONFIG_MMC_SDHCI_OF_HLWD
-       { .compatible = "nintendo,hollywood-sdhci", .data = &sdhci_hlwd, },
-#endif
-       { .compatible = "generic-sdhci", },
-       {},
-};
-MODULE_DEVICE_TABLE(of, sdhci_of_match);
-
-static struct platform_driver sdhci_of_driver = {
-       .driver = {
-               .name = "sdhci-of",
-               .owner = THIS_MODULE,
-               .of_match_table = sdhci_of_match,
-       },
-       .probe = sdhci_of_probe,
-       .remove = __devexit_p(sdhci_of_remove),
-       .suspend = sdhci_of_suspend,
-       .resume = sdhci_of_resume,
-};
-
-static int __init sdhci_of_init(void)
-{
-       return platform_driver_register(&sdhci_of_driver);
-}
-module_init(sdhci_of_init);
-
-static void __exit sdhci_of_exit(void)
-{
-       platform_driver_unregister(&sdhci_of_driver);
-}
-module_exit(sdhci_of_exit);
-
-MODULE_DESCRIPTION("Secure Digital Host Controller Interface OF driver");
-MODULE_AUTHOR("Xiaobo Xie <X.Xie@freescale.com>, "
-             "Anton Vorontsov <avorontsov@ru.mvista.com>");
-MODULE_LICENSE("GPL");
index ba40d6d..492bcd7 100644 (file)
@@ -60,32 +60,34 @@ static int esdhc_of_enable_dma(struct sdhci_host *host)
 
 static unsigned int esdhc_of_get_max_clock(struct sdhci_host *host)
 {
-       struct sdhci_of_host *of_host = sdhci_priv(host);
+       struct sdhci_pltfm_host *pltfm_host = sdhci_priv(host);
 
-       return of_host->clock;
+       return pltfm_host->clock;
 }
 
 static unsigned int esdhc_of_get_min_clock(struct sdhci_host *host)
 {
-       struct sdhci_of_host *of_host = sdhci_priv(host);
+       struct sdhci_pltfm_host *pltfm_host = sdhci_priv(host);
 
-       return of_host->clock / 256 / 16;
+       return pltfm_host->clock / 256 / 16;
 }
 
-struct sdhci_of_data sdhci_esdhc = {
+static struct sdhci_ops sdhci_esdhc_ops = {
+       .read_l = sdhci_be32bs_readl,
+       .read_w = esdhc_readw,
+       .read_b = sdhci_be32bs_readb,
+       .write_l = sdhci_be32bs_writel,
+       .write_w = esdhc_writew,
+       .write_b = esdhc_writeb,
+       .set_clock = esdhc_set_clock,
+       .enable_dma = esdhc_of_enable_dma,
+       .get_max_clock = esdhc_of_get_max_clock,
+       .get_min_clock = esdhc_of_get_min_clock,
+};
+
+struct sdhci_pltfm_data sdhci_esdhc_pdata = {
        /* card detection could be handled via GPIO */
        .quirks = ESDHC_DEFAULT_QUIRKS | SDHCI_QUIRK_BROKEN_CARD_DETECTION
                | SDHCI_QUIRK_NO_CARD_NO_RESET,
-       .ops = {
-               .read_l = sdhci_be32bs_readl,
-               .read_w = esdhc_readw,
-               .read_b = sdhci_be32bs_readb,
-               .write_l = sdhci_be32bs_writel,
-               .write_w = esdhc_writew,
-               .write_b = esdhc_writeb,
-               .set_clock = esdhc_set_clock,
-               .enable_dma = esdhc_of_enable_dma,
-               .get_max_clock = esdhc_of_get_max_clock,
-               .get_min_clock = esdhc_of_get_min_clock,
-       },
+       .ops = &sdhci_esdhc_ops,
 };
index 68ddb75..380e896 100644 (file)
@@ -51,15 +51,17 @@ static void sdhci_hlwd_writeb(struct sdhci_host *host, u8 val, int reg)
        udelay(SDHCI_HLWD_WRITE_DELAY);
 }
 
-struct sdhci_of_data sdhci_hlwd = {
+static struct sdhci_ops sdhci_hlwd_ops = {
+       .read_l = sdhci_be32bs_readl,
+       .read_w = sdhci_be32bs_readw,
+       .read_b = sdhci_be32bs_readb,
+       .write_l = sdhci_hlwd_writel,
+       .write_w = sdhci_hlwd_writew,
+       .write_b = sdhci_hlwd_writeb,
+};
+
+struct sdhci_pltfm_data sdhci_hlwd_pdata = {
        .quirks = SDHCI_QUIRK_32BIT_DMA_ADDR |
                  SDHCI_QUIRK_32BIT_DMA_SIZE,
-       .ops = {
-               .read_l = sdhci_be32bs_readl,
-               .read_w = sdhci_be32bs_readw,
-               .read_b = sdhci_be32bs_readb,
-               .write_l = sdhci_hlwd_writel,
-               .write_w = sdhci_hlwd_writew,
-               .write_b = sdhci_hlwd_writeb,
-       },
+       .ops = &sdhci_hlwd_ops,
 };
index ad09ad9..5bdb5e7 100644 (file)
 
 #include <linux/types.h>
 #include "sdhci.h"
-
-struct sdhci_of_data {
-       unsigned int quirks;
-       struct sdhci_ops ops;
-};
-
-struct sdhci_of_host {
-       unsigned int clock;
-       u16 xfer_mode_shadow;
-};
+#include "sdhci-pltfm.h"
 
 extern u32 sdhci_be32bs_readl(struct sdhci_host *host, int reg);
 extern u16 sdhci_be32bs_readw(struct sdhci_host *host, int reg);
@@ -36,7 +27,7 @@ extern void sdhci_be32bs_writel(struct sdhci_host *host, u32 val, int reg);
 extern void sdhci_be32bs_writew(struct sdhci_host *host, u16 val, int reg);
 extern void sdhci_be32bs_writeb(struct sdhci_host *host, u8 val, int reg);
 
-extern struct sdhci_of_data sdhci_esdhc;
-extern struct sdhci_of_data sdhci_hlwd;
+extern struct sdhci_pltfm_data sdhci_esdhc_pdata;
+extern struct sdhci_pltfm_data sdhci_hlwd_pdata;
 
 #endif /* __SDHCI_OF_H */
index ff4b7eb..3cac450 100644 (file)
 struct sdhci_pltfm_host {
        struct clk *clk;
        void *priv; /* to handle quirks across io-accessor calls */
+
+       /* migrate from sdhci_of_host */
+       unsigned int clock;
+       u16 xfer_mode_shadow;
 };
 
 extern struct sdhci_host *sdhci_pltfm_init(struct platform_device *pdev,