* [PATCHv5 0/4]socfpga: Enable SD/MMC support
@ 2013-12-09 13:57 ` dinguyen at altera.com
0 siblings, 0 replies; 14+ messages in thread
From: dinguyen @ 2013-12-09 13:57 UTC (permalink / raw)
To: dinh.linux, arnd, mturquette, cjb, jh80.chung, tgih.jun
Cc: linux-mmc, linux-arm-kernel, Dinh Nguyen
From: Dinh Nguyen <dinguyen@altera.com>
Hi,
This is v5 of the patch series to enable SD/MMC on the SOCFPGA platform.
V5 differences from V4:
* This patch series is now dependent on patch series:
[mmc: dw_mmc: Make the use of the hold reg generic]
* Instead of re-using the platform specific "rockchip,rk2928-dw-mshc" binding,
just use the generic "snps,dw-mshc" binding.
* No longer need to CC DTS Bindings maintainers as there aren't any new bindings.
Thanks,
Dinh Nguyen (4):
arm: dts: Add support for SD/MMC on SOCFPGA
clk: socfpga: Add a hook for SD/MMC driver to control CIU clock
settings
mmc: dw_mmc-socfpga: Remove the SOCFPGA specific platform for dw_mmc
ARM: socfpga_defconfig: enable SD/MMC support
arch/arm/boot/dts/socfpga.dtsi | 11 +++
arch/arm/boot/dts/socfpga_arria5.dtsi | 12 +++
arch/arm/boot/dts/socfpga_cyclone5.dtsi | 12 +++
arch/arm/boot/dts/socfpga_vt.dts | 12 +++
arch/arm/configs/socfpga_defconfig | 2 +
drivers/clk/socfpga/clk.c | 25 ++++++
drivers/mmc/host/Kconfig | 8 --
drivers/mmc/host/dw_mmc-socfpga.c | 138 -------------------------------
8 files changed, 74 insertions(+), 146 deletions(-)
delete mode 100644 drivers/mmc/host/dw_mmc-socfpga.c
--
1.7.9.5
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCHv5 0/4]socfpga: Enable SD/MMC support
@ 2013-12-09 13:57 ` dinguyen at altera.com
0 siblings, 0 replies; 14+ messages in thread
From: dinguyen at altera.com @ 2013-12-09 13:57 UTC (permalink / raw)
To: linux-arm-kernel
From: Dinh Nguyen <dinguyen@altera.com>
Hi,
This is v5 of the patch series to enable SD/MMC on the SOCFPGA platform.
V5 differences from V4:
* This patch series is now dependent on patch series:
[mmc: dw_mmc: Make the use of the hold reg generic]
* Instead of re-using the platform specific "rockchip,rk2928-dw-mshc" binding,
just use the generic "snps,dw-mshc" binding.
* No longer need to CC DTS Bindings maintainers as there aren't any new bindings.
Thanks,
Dinh Nguyen (4):
arm: dts: Add support for SD/MMC on SOCFPGA
clk: socfpga: Add a hook for SD/MMC driver to control CIU clock
settings
mmc: dw_mmc-socfpga: Remove the SOCFPGA specific platform for dw_mmc
ARM: socfpga_defconfig: enable SD/MMC support
arch/arm/boot/dts/socfpga.dtsi | 11 +++
arch/arm/boot/dts/socfpga_arria5.dtsi | 12 +++
arch/arm/boot/dts/socfpga_cyclone5.dtsi | 12 +++
arch/arm/boot/dts/socfpga_vt.dts | 12 +++
arch/arm/configs/socfpga_defconfig | 2 +
drivers/clk/socfpga/clk.c | 25 ++++++
drivers/mmc/host/Kconfig | 8 --
drivers/mmc/host/dw_mmc-socfpga.c | 138 -------------------------------
8 files changed, 74 insertions(+), 146 deletions(-)
delete mode 100644 drivers/mmc/host/dw_mmc-socfpga.c
--
1.7.9.5
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCHv5 1/4] arm: dts: Add support for SD/MMC on SOCFPGA
2013-12-09 13:57 ` dinguyen at altera.com
@ 2013-12-09 13:57 ` dinguyen at altera.com
-1 siblings, 0 replies; 14+ messages in thread
From: dinguyen @ 2013-12-09 13:57 UTC (permalink / raw)
To: dinh.linux, arnd, mturquette, cjb, jh80.chung, tgih.jun
Cc: linux-mmc, linux-arm-kernel, Dinh Nguyen
From: Dinh Nguyen <dinguyen@altera.com>
Use the standard "snps,dw-mshc" binding that will support SD/MMC on
Altera's SOCFPGA platform.
Signed-off-by: Dinh Nguyen <dinguyen@altera.com>
---
v5: Use the standard "snps,dw-mshc" binding
v4: Re-use "rockchip,rk2928-dw-mshc" binding
v3: none
v2: none
---
arch/arm/boot/dts/socfpga.dtsi | 11 +++++++++++
arch/arm/boot/dts/socfpga_arria5.dtsi | 12 ++++++++++++
arch/arm/boot/dts/socfpga_cyclone5.dtsi | 12 ++++++++++++
arch/arm/boot/dts/socfpga_vt.dts | 12 ++++++++++++
4 files changed, 47 insertions(+)
diff --git a/arch/arm/boot/dts/socfpga.dtsi b/arch/arm/boot/dts/socfpga.dtsi
index f936476..9e78c1d 100644
--- a/arch/arm/boot/dts/socfpga.dtsi
+++ b/arch/arm/boot/dts/socfpga.dtsi
@@ -469,6 +469,17 @@
cache-level = <2>;
};
+ mmc: dwmmc0@ff704000 {
+ compatible = "snps,dw-mshc";
+ reg = <0xff704000 0x1000>;
+ interrupts = <0 139 4>;
+ fifo-depth = <0x400>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ clocks = <&l4_mp_clk>, <&sdmmc_clk>;
+ clock-names = "biu", "ciu";
+ };
+
/* Local timer */
timer@fffec600 {
compatible = "arm,cortex-a9-twd-timer";
diff --git a/arch/arm/boot/dts/socfpga_arria5.dtsi b/arch/arm/boot/dts/socfpga_arria5.dtsi
index a85b404..112b7e2 100644
--- a/arch/arm/boot/dts/socfpga_arria5.dtsi
+++ b/arch/arm/boot/dts/socfpga_arria5.dtsi
@@ -27,6 +27,18 @@
};
};
+ dwmmc0@ff704000 {
+ num-slots = <1>;
+ supports-highspeed;
+ broken-cd;
+ samsung,dw-mshc-sdr-timing = <0 3>;
+
+ slot@0 {
+ reg = <0>;
+ bus-width = <4>;
+ };
+ };
+
serial0@ffc02000 {
clock-frequency = <100000000>;
};
diff --git a/arch/arm/boot/dts/socfpga_cyclone5.dtsi b/arch/arm/boot/dts/socfpga_cyclone5.dtsi
index a8716f6..52b1501 100644
--- a/arch/arm/boot/dts/socfpga_cyclone5.dtsi
+++ b/arch/arm/boot/dts/socfpga_cyclone5.dtsi
@@ -28,6 +28,18 @@
};
};
+ dwmmc0@ff704000 {
+ num-slots = <1>;
+ supports-highspeed;
+ broken-cd;
+ samsung,dw-mshc-sdr-timing = <0 3>;
+
+ slot@0 {
+ reg = <0>;
+ bus-width = <4>;
+ };
+ };
+
ethernet@ff702000 {
phy-mode = "rgmii";
phy-addr = <0xffffffff>; /* probe for phy addr */
diff --git a/arch/arm/boot/dts/socfpga_vt.dts b/arch/arm/boot/dts/socfpga_vt.dts
index d1ec0ca..7dc709b 100644
--- a/arch/arm/boot/dts/socfpga_vt.dts
+++ b/arch/arm/boot/dts/socfpga_vt.dts
@@ -41,6 +41,18 @@
};
};
+ dwmmc0@ff704000 {
+ num-slots = <1>;
+ supports-highspeed;
+ broken-cd;
+ samsung,dw-mshc-sdr-timing = <0 3>;
+
+ slot@0 {
+ reg = <0>;
+ bus-width = <4>;
+ };
+ };
+
ethernet@ff700000 {
phy-mode = "gmii";
status = "okay";
--
1.7.9.5
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCHv5 1/4] arm: dts: Add support for SD/MMC on SOCFPGA
@ 2013-12-09 13:57 ` dinguyen at altera.com
0 siblings, 0 replies; 14+ messages in thread
From: dinguyen at altera.com @ 2013-12-09 13:57 UTC (permalink / raw)
To: linux-arm-kernel
From: Dinh Nguyen <dinguyen@altera.com>
Use the standard "snps,dw-mshc" binding that will support SD/MMC on
Altera's SOCFPGA platform.
Signed-off-by: Dinh Nguyen <dinguyen@altera.com>
---
v5: Use the standard "snps,dw-mshc" binding
v4: Re-use "rockchip,rk2928-dw-mshc" binding
v3: none
v2: none
---
arch/arm/boot/dts/socfpga.dtsi | 11 +++++++++++
arch/arm/boot/dts/socfpga_arria5.dtsi | 12 ++++++++++++
arch/arm/boot/dts/socfpga_cyclone5.dtsi | 12 ++++++++++++
arch/arm/boot/dts/socfpga_vt.dts | 12 ++++++++++++
4 files changed, 47 insertions(+)
diff --git a/arch/arm/boot/dts/socfpga.dtsi b/arch/arm/boot/dts/socfpga.dtsi
index f936476..9e78c1d 100644
--- a/arch/arm/boot/dts/socfpga.dtsi
+++ b/arch/arm/boot/dts/socfpga.dtsi
@@ -469,6 +469,17 @@
cache-level = <2>;
};
+ mmc: dwmmc0 at ff704000 {
+ compatible = "snps,dw-mshc";
+ reg = <0xff704000 0x1000>;
+ interrupts = <0 139 4>;
+ fifo-depth = <0x400>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ clocks = <&l4_mp_clk>, <&sdmmc_clk>;
+ clock-names = "biu", "ciu";
+ };
+
/* Local timer */
timer at fffec600 {
compatible = "arm,cortex-a9-twd-timer";
diff --git a/arch/arm/boot/dts/socfpga_arria5.dtsi b/arch/arm/boot/dts/socfpga_arria5.dtsi
index a85b404..112b7e2 100644
--- a/arch/arm/boot/dts/socfpga_arria5.dtsi
+++ b/arch/arm/boot/dts/socfpga_arria5.dtsi
@@ -27,6 +27,18 @@
};
};
+ dwmmc0 at ff704000 {
+ num-slots = <1>;
+ supports-highspeed;
+ broken-cd;
+ samsung,dw-mshc-sdr-timing = <0 3>;
+
+ slot at 0 {
+ reg = <0>;
+ bus-width = <4>;
+ };
+ };
+
serial0 at ffc02000 {
clock-frequency = <100000000>;
};
diff --git a/arch/arm/boot/dts/socfpga_cyclone5.dtsi b/arch/arm/boot/dts/socfpga_cyclone5.dtsi
index a8716f6..52b1501 100644
--- a/arch/arm/boot/dts/socfpga_cyclone5.dtsi
+++ b/arch/arm/boot/dts/socfpga_cyclone5.dtsi
@@ -28,6 +28,18 @@
};
};
+ dwmmc0 at ff704000 {
+ num-slots = <1>;
+ supports-highspeed;
+ broken-cd;
+ samsung,dw-mshc-sdr-timing = <0 3>;
+
+ slot at 0 {
+ reg = <0>;
+ bus-width = <4>;
+ };
+ };
+
ethernet at ff702000 {
phy-mode = "rgmii";
phy-addr = <0xffffffff>; /* probe for phy addr */
diff --git a/arch/arm/boot/dts/socfpga_vt.dts b/arch/arm/boot/dts/socfpga_vt.dts
index d1ec0ca..7dc709b 100644
--- a/arch/arm/boot/dts/socfpga_vt.dts
+++ b/arch/arm/boot/dts/socfpga_vt.dts
@@ -41,6 +41,18 @@
};
};
+ dwmmc0 at ff704000 {
+ num-slots = <1>;
+ supports-highspeed;
+ broken-cd;
+ samsung,dw-mshc-sdr-timing = <0 3>;
+
+ slot at 0 {
+ reg = <0>;
+ bus-width = <4>;
+ };
+ };
+
ethernet at ff700000 {
phy-mode = "gmii";
status = "okay";
--
1.7.9.5
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCHv5 2/4] clk: socfpga: Add a hook for SD/MMC driver to control CIU clock settings
2013-12-09 13:57 ` dinguyen at altera.com
@ 2013-12-09 13:57 ` dinguyen at altera.com
-1 siblings, 0 replies; 14+ messages in thread
From: dinguyen @ 2013-12-09 13:57 UTC (permalink / raw)
To: dinh.linux, arnd, mturquette, cjb, jh80.chung, tgih.jun
Cc: linux-mmc, linux-arm-kernel, Dinh Nguyen
From: Dinh Nguyen <dinguyen@altera.com>
Populate the .prepare function in the clk-ops for the "sdmmc_clk" that represents
the "ciu" clock for the SD/MMC driver. The prepare function will handle setting
the correct clock-phase for the CIU clock of the SD/MMC IP.
Signed-off-by: Dinh Nguyen <dinguyen@altera.com>
---
v5: Use the "snps,dw-mshc" binding
v4: none
v3: none
v2: none
---
drivers/clk/socfpga/clk.c | 25 +++++++++++++++++++++++++
1 file changed, 25 insertions(+)
diff --git a/drivers/clk/socfpga/clk.c b/drivers/clk/socfpga/clk.c
index 60cb2f5..fe2670e 100644
--- a/drivers/clk/socfpga/clk.c
+++ b/drivers/clk/socfpga/clk.c
@@ -55,7 +55,13 @@
#define div_mask(width) ((1 << (width)) - 1)
#define streq(a, b) (strcmp((a), (b)) == 0)
+#define SYSMGR_SDMMCGRP_CTRL_OFFSET 0x108
+/* SDMMC Group for System Manager defines */
+#define SYSMGR_SDMMC_CTRL_SET(smplsel, drvsel) \
+ ((((smplsel) & 0x7) << 3) | (((drvsel) & 0x7) << 0))
+
extern void __iomem *clk_mgr_base_addr;
+extern void __iomem *sys_manager_base_addr;
struct socfpga_clk {
struct clk_gate hw;
@@ -68,6 +74,22 @@ struct socfpga_clk {
};
#define to_socfpga_clk(p) container_of(p, struct socfpga_clk, hw.hw)
+static int sdmmc_ciuclk_prepare(struct clk_hw *hwclk)
+{
+ struct device_node *np;
+ u32 timing[2];
+ u32 hs_timing;
+
+ np = of_find_compatible_node(NULL, NULL, "snps,dw-mshc");
+ if (of_property_read_u32_array(np, "samsung,dw-mshc-sdr-timing", timing, 2)) {
+ pr_err("SDMMC: cannot find samsung,dw-mshc-sdr-timing!\n");
+ return -ENODATA;
+ }
+ hs_timing = SYSMGR_SDMMC_CTRL_SET(timing[0], timing[1]);
+ writel(hs_timing, sys_manager_base_addr + SYSMGR_SDMMCGRP_CTRL_OFFSET);
+ return 0;
+}
+
static unsigned long clk_pll_recalc_rate(struct clk_hw *hwclk,
unsigned long parent_rate)
{
@@ -274,6 +296,9 @@ static void __init socfpga_gate_clk_init(struct device_node *node,
socfpga_clk->hw.reg = clk_mgr_base_addr + clk_gate[0];
socfpga_clk->hw.bit_idx = clk_gate[1];
+ if (streq(clk_name, "sdmmc_clk"))
+ gateclk_ops.prepare = sdmmc_ciuclk_prepare;
+
gateclk_ops.enable = clk_gate_ops.enable;
gateclk_ops.disable = clk_gate_ops.disable;
}
--
1.7.9.5
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCHv5 2/4] clk: socfpga: Add a hook for SD/MMC driver to control CIU clock settings
@ 2013-12-09 13:57 ` dinguyen at altera.com
0 siblings, 0 replies; 14+ messages in thread
From: dinguyen at altera.com @ 2013-12-09 13:57 UTC (permalink / raw)
To: linux-arm-kernel
From: Dinh Nguyen <dinguyen@altera.com>
Populate the .prepare function in the clk-ops for the "sdmmc_clk" that represents
the "ciu" clock for the SD/MMC driver. The prepare function will handle setting
the correct clock-phase for the CIU clock of the SD/MMC IP.
Signed-off-by: Dinh Nguyen <dinguyen@altera.com>
---
v5: Use the "snps,dw-mshc" binding
v4: none
v3: none
v2: none
---
drivers/clk/socfpga/clk.c | 25 +++++++++++++++++++++++++
1 file changed, 25 insertions(+)
diff --git a/drivers/clk/socfpga/clk.c b/drivers/clk/socfpga/clk.c
index 60cb2f5..fe2670e 100644
--- a/drivers/clk/socfpga/clk.c
+++ b/drivers/clk/socfpga/clk.c
@@ -55,7 +55,13 @@
#define div_mask(width) ((1 << (width)) - 1)
#define streq(a, b) (strcmp((a), (b)) == 0)
+#define SYSMGR_SDMMCGRP_CTRL_OFFSET 0x108
+/* SDMMC Group for System Manager defines */
+#define SYSMGR_SDMMC_CTRL_SET(smplsel, drvsel) \
+ ((((smplsel) & 0x7) << 3) | (((drvsel) & 0x7) << 0))
+
extern void __iomem *clk_mgr_base_addr;
+extern void __iomem *sys_manager_base_addr;
struct socfpga_clk {
struct clk_gate hw;
@@ -68,6 +74,22 @@ struct socfpga_clk {
};
#define to_socfpga_clk(p) container_of(p, struct socfpga_clk, hw.hw)
+static int sdmmc_ciuclk_prepare(struct clk_hw *hwclk)
+{
+ struct device_node *np;
+ u32 timing[2];
+ u32 hs_timing;
+
+ np = of_find_compatible_node(NULL, NULL, "snps,dw-mshc");
+ if (of_property_read_u32_array(np, "samsung,dw-mshc-sdr-timing", timing, 2)) {
+ pr_err("SDMMC: cannot find samsung,dw-mshc-sdr-timing!\n");
+ return -ENODATA;
+ }
+ hs_timing = SYSMGR_SDMMC_CTRL_SET(timing[0], timing[1]);
+ writel(hs_timing, sys_manager_base_addr + SYSMGR_SDMMCGRP_CTRL_OFFSET);
+ return 0;
+}
+
static unsigned long clk_pll_recalc_rate(struct clk_hw *hwclk,
unsigned long parent_rate)
{
@@ -274,6 +296,9 @@ static void __init socfpga_gate_clk_init(struct device_node *node,
socfpga_clk->hw.reg = clk_mgr_base_addr + clk_gate[0];
socfpga_clk->hw.bit_idx = clk_gate[1];
+ if (streq(clk_name, "sdmmc_clk"))
+ gateclk_ops.prepare = sdmmc_ciuclk_prepare;
+
gateclk_ops.enable = clk_gate_ops.enable;
gateclk_ops.disable = clk_gate_ops.disable;
}
--
1.7.9.5
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCHv5 3/4] mmc: dw_mmc-socfpga: Remove the SOCFPGA specific platform for dw_mmc
2013-12-09 13:57 ` dinguyen at altera.com
@ 2013-12-09 13:57 ` dinguyen at altera.com
-1 siblings, 0 replies; 14+ messages in thread
From: dinguyen @ 2013-12-09 13:57 UTC (permalink / raw)
To: dinh.linux, arnd, mturquette, cjb, jh80.chung, tgih.jun
Cc: linux-mmc, linux-arm-kernel, Dinh Nguyen
From: Dinh Nguyen <dinguyen@altera.com>
It turns now that the only really platform specific code that is needed for
SOCFPGA is using the SDMMC_CMD_USE_HOLD_REG in the prepare_command function.
Now that the code to check for the usage of the SDMMC_CMD_USE_HOLD_REG bit
is done by checking the speed mode of the slot, we can remove platform specific
code for SOCFPGA.
Signed-off-by: Dinh Nguyen <dinguyen@altera.com>
---
v5: none
v4: Update commit message with updated code to check the hold reg bi
v3: none
v2: none
---
drivers/mmc/host/Kconfig | 8 ---
drivers/mmc/host/dw_mmc-socfpga.c | 138 -------------------------------------
2 files changed, 146 deletions(-)
delete mode 100644 drivers/mmc/host/dw_mmc-socfpga.c
diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig
index 7fc5099..6737a4f 100644
--- a/drivers/mmc/host/Kconfig
+++ b/drivers/mmc/host/Kconfig
@@ -567,14 +567,6 @@ config MMC_DW_EXYNOS
Synopsys DesignWare Memory Card Interface driver. Select this option
for platforms based on Exynos4 and Exynos5 SoC's.
-config MMC_DW_SOCFPGA
- tristate "SOCFPGA specific extensions for Synopsys DW Memory Card Interface"
- depends on MMC_DW && MFD_SYSCON
- select MMC_DW_PLTFM
- help
- This selects support for Altera SoCFPGA specific extensions to the
- Synopsys DesignWare Memory Card Interface driver.
-
config MMC_DW_PCI
tristate "Synopsys Designware MCI support on PCI bus"
depends on MMC_DW && PCI
diff --git a/drivers/mmc/host/dw_mmc-socfpga.c b/drivers/mmc/host/dw_mmc-socfpga.c
deleted file mode 100644
index 3e8e53a..0000000
--- a/drivers/mmc/host/dw_mmc-socfpga.c
+++ /dev/null
@@ -1,138 +0,0 @@
-/*
- * Altera SoCFPGA Specific Extensions for Synopsys DW Multimedia Card Interface
- * driver
- *
- * Copyright (C) 2012, Samsung Electronics Co., Ltd.
- * Copyright (C) 2013 Altera Corporation
- *
- * 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.
- *
- * Taken from dw_mmc-exynos.c
- */
-#include <linux/clk.h>
-#include <linux/mfd/syscon.h>
-#include <linux/mmc/host.h>
-#include <linux/mmc/dw_mmc.h>
-#include <linux/module.h>
-#include <linux/of.h>
-#include <linux/platform_device.h>
-#include <linux/regmap.h>
-
-#include "dw_mmc.h"
-#include "dw_mmc-pltfm.h"
-
-#define SYSMGR_SDMMCGRP_CTRL_OFFSET 0x108
-#define DRV_CLK_PHASE_SHIFT_SEL_MASK 0x7
-#define SYSMGR_SDMMC_CTRL_SET(smplsel, drvsel) \
- ((((smplsel) & 0x7) << 3) | (((drvsel) & 0x7) << 0))
-
-/* SOCFPGA implementation specific driver private data */
-struct dw_mci_socfpga_priv_data {
- u8 ciu_div; /* card interface unit divisor */
- u32 hs_timing; /* bitmask for CIU clock phase shift */
- struct regmap *sysreg; /* regmap for system manager register */
-};
-
-static int dw_mci_socfpga_priv_init(struct dw_mci *host)
-{
- return 0;
-}
-
-static int dw_mci_socfpga_setup_clock(struct dw_mci *host)
-{
- struct dw_mci_socfpga_priv_data *priv = host->priv;
-
- clk_disable_unprepare(host->ciu_clk);
- regmap_write(priv->sysreg, SYSMGR_SDMMCGRP_CTRL_OFFSET,
- priv->hs_timing);
- clk_prepare_enable(host->ciu_clk);
-
- host->bus_hz /= (priv->ciu_div + 1);
- return 0;
-}
-
-static void dw_mci_socfpga_prepare_command(struct dw_mci *host, u32 *cmdr)
-{
- struct dw_mci_socfpga_priv_data *priv = host->priv;
-
- if (priv->hs_timing & DRV_CLK_PHASE_SHIFT_SEL_MASK)
- *cmdr |= SDMMC_CMD_USE_HOLD_REG;
-}
-
-static int dw_mci_socfpga_parse_dt(struct dw_mci *host)
-{
- struct dw_mci_socfpga_priv_data *priv;
- struct device_node *np = host->dev->of_node;
- u32 timing[2];
- u32 div = 0;
- int ret;
-
- priv = devm_kzalloc(host->dev, sizeof(*priv), GFP_KERNEL);
- if (!priv) {
- dev_err(host->dev, "mem alloc failed for private data\n");
- return -ENOMEM;
- }
-
- priv->sysreg = syscon_regmap_lookup_by_compatible("altr,sys-mgr");
- if (IS_ERR(priv->sysreg)) {
- dev_err(host->dev, "regmap for altr,sys-mgr lookup failed.\n");
- return PTR_ERR(priv->sysreg);
- }
-
- ret = of_property_read_u32(np, "altr,dw-mshc-ciu-div", &div);
- if (ret)
- dev_info(host->dev, "No dw-mshc-ciu-div specified, assuming 1");
- priv->ciu_div = div;
-
- ret = of_property_read_u32_array(np,
- "altr,dw-mshc-sdr-timing", timing, 2);
- if (ret)
- return ret;
-
- priv->hs_timing = SYSMGR_SDMMC_CTRL_SET(timing[0], timing[1]);
- host->priv = priv;
- return 0;
-}
-
-static const struct dw_mci_drv_data socfpga_drv_data = {
- .init = dw_mci_socfpga_priv_init,
- .setup_clock = dw_mci_socfpga_setup_clock,
- .prepare_command = dw_mci_socfpga_prepare_command,
- .parse_dt = dw_mci_socfpga_parse_dt,
-};
-
-static const struct of_device_id dw_mci_socfpga_match[] = {
- { .compatible = "altr,socfpga-dw-mshc",
- .data = &socfpga_drv_data, },
- {},
-};
-MODULE_DEVICE_TABLE(of, dw_mci_socfpga_match);
-
-static int dw_mci_socfpga_probe(struct platform_device *pdev)
-{
- const struct dw_mci_drv_data *drv_data;
- const struct of_device_id *match;
-
- match = of_match_node(dw_mci_socfpga_match, pdev->dev.of_node);
- drv_data = match->data;
- return dw_mci_pltfm_register(pdev, drv_data);
-}
-
-static struct platform_driver dw_mci_socfpga_pltfm_driver = {
- .probe = dw_mci_socfpga_probe,
- .remove = __exit_p(dw_mci_pltfm_remove),
- .driver = {
- .name = "dwmmc_socfpga",
- .of_match_table = dw_mci_socfpga_match,
- .pm = &dw_mci_pltfm_pmops,
- },
-};
-
-module_platform_driver(dw_mci_socfpga_pltfm_driver);
-
-MODULE_DESCRIPTION("Altera SOCFPGA Specific DW-MSHC Driver Extension");
-MODULE_LICENSE("GPL v2");
-MODULE_ALIAS("platform:dwmmc-socfpga");
--
1.7.9.5
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCHv5 3/4] mmc: dw_mmc-socfpga: Remove the SOCFPGA specific platform for dw_mmc
@ 2013-12-09 13:57 ` dinguyen at altera.com
0 siblings, 0 replies; 14+ messages in thread
From: dinguyen at altera.com @ 2013-12-09 13:57 UTC (permalink / raw)
To: linux-arm-kernel
From: Dinh Nguyen <dinguyen@altera.com>
It turns now that the only really platform specific code that is needed for
SOCFPGA is using the SDMMC_CMD_USE_HOLD_REG in the prepare_command function.
Now that the code to check for the usage of the SDMMC_CMD_USE_HOLD_REG bit
is done by checking the speed mode of the slot, we can remove platform specific
code for SOCFPGA.
Signed-off-by: Dinh Nguyen <dinguyen@altera.com>
---
v5: none
v4: Update commit message with updated code to check the hold reg bi
v3: none
v2: none
---
drivers/mmc/host/Kconfig | 8 ---
drivers/mmc/host/dw_mmc-socfpga.c | 138 -------------------------------------
2 files changed, 146 deletions(-)
delete mode 100644 drivers/mmc/host/dw_mmc-socfpga.c
diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig
index 7fc5099..6737a4f 100644
--- a/drivers/mmc/host/Kconfig
+++ b/drivers/mmc/host/Kconfig
@@ -567,14 +567,6 @@ config MMC_DW_EXYNOS
Synopsys DesignWare Memory Card Interface driver. Select this option
for platforms based on Exynos4 and Exynos5 SoC's.
-config MMC_DW_SOCFPGA
- tristate "SOCFPGA specific extensions for Synopsys DW Memory Card Interface"
- depends on MMC_DW && MFD_SYSCON
- select MMC_DW_PLTFM
- help
- This selects support for Altera SoCFPGA specific extensions to the
- Synopsys DesignWare Memory Card Interface driver.
-
config MMC_DW_PCI
tristate "Synopsys Designware MCI support on PCI bus"
depends on MMC_DW && PCI
diff --git a/drivers/mmc/host/dw_mmc-socfpga.c b/drivers/mmc/host/dw_mmc-socfpga.c
deleted file mode 100644
index 3e8e53a..0000000
--- a/drivers/mmc/host/dw_mmc-socfpga.c
+++ /dev/null
@@ -1,138 +0,0 @@
-/*
- * Altera SoCFPGA Specific Extensions for Synopsys DW Multimedia Card Interface
- * driver
- *
- * Copyright (C) 2012, Samsung Electronics Co., Ltd.
- * Copyright (C) 2013 Altera Corporation
- *
- * 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.
- *
- * Taken from dw_mmc-exynos.c
- */
-#include <linux/clk.h>
-#include <linux/mfd/syscon.h>
-#include <linux/mmc/host.h>
-#include <linux/mmc/dw_mmc.h>
-#include <linux/module.h>
-#include <linux/of.h>
-#include <linux/platform_device.h>
-#include <linux/regmap.h>
-
-#include "dw_mmc.h"
-#include "dw_mmc-pltfm.h"
-
-#define SYSMGR_SDMMCGRP_CTRL_OFFSET 0x108
-#define DRV_CLK_PHASE_SHIFT_SEL_MASK 0x7
-#define SYSMGR_SDMMC_CTRL_SET(smplsel, drvsel) \
- ((((smplsel) & 0x7) << 3) | (((drvsel) & 0x7) << 0))
-
-/* SOCFPGA implementation specific driver private data */
-struct dw_mci_socfpga_priv_data {
- u8 ciu_div; /* card interface unit divisor */
- u32 hs_timing; /* bitmask for CIU clock phase shift */
- struct regmap *sysreg; /* regmap for system manager register */
-};
-
-static int dw_mci_socfpga_priv_init(struct dw_mci *host)
-{
- return 0;
-}
-
-static int dw_mci_socfpga_setup_clock(struct dw_mci *host)
-{
- struct dw_mci_socfpga_priv_data *priv = host->priv;
-
- clk_disable_unprepare(host->ciu_clk);
- regmap_write(priv->sysreg, SYSMGR_SDMMCGRP_CTRL_OFFSET,
- priv->hs_timing);
- clk_prepare_enable(host->ciu_clk);
-
- host->bus_hz /= (priv->ciu_div + 1);
- return 0;
-}
-
-static void dw_mci_socfpga_prepare_command(struct dw_mci *host, u32 *cmdr)
-{
- struct dw_mci_socfpga_priv_data *priv = host->priv;
-
- if (priv->hs_timing & DRV_CLK_PHASE_SHIFT_SEL_MASK)
- *cmdr |= SDMMC_CMD_USE_HOLD_REG;
-}
-
-static int dw_mci_socfpga_parse_dt(struct dw_mci *host)
-{
- struct dw_mci_socfpga_priv_data *priv;
- struct device_node *np = host->dev->of_node;
- u32 timing[2];
- u32 div = 0;
- int ret;
-
- priv = devm_kzalloc(host->dev, sizeof(*priv), GFP_KERNEL);
- if (!priv) {
- dev_err(host->dev, "mem alloc failed for private data\n");
- return -ENOMEM;
- }
-
- priv->sysreg = syscon_regmap_lookup_by_compatible("altr,sys-mgr");
- if (IS_ERR(priv->sysreg)) {
- dev_err(host->dev, "regmap for altr,sys-mgr lookup failed.\n");
- return PTR_ERR(priv->sysreg);
- }
-
- ret = of_property_read_u32(np, "altr,dw-mshc-ciu-div", &div);
- if (ret)
- dev_info(host->dev, "No dw-mshc-ciu-div specified, assuming 1");
- priv->ciu_div = div;
-
- ret = of_property_read_u32_array(np,
- "altr,dw-mshc-sdr-timing", timing, 2);
- if (ret)
- return ret;
-
- priv->hs_timing = SYSMGR_SDMMC_CTRL_SET(timing[0], timing[1]);
- host->priv = priv;
- return 0;
-}
-
-static const struct dw_mci_drv_data socfpga_drv_data = {
- .init = dw_mci_socfpga_priv_init,
- .setup_clock = dw_mci_socfpga_setup_clock,
- .prepare_command = dw_mci_socfpga_prepare_command,
- .parse_dt = dw_mci_socfpga_parse_dt,
-};
-
-static const struct of_device_id dw_mci_socfpga_match[] = {
- { .compatible = "altr,socfpga-dw-mshc",
- .data = &socfpga_drv_data, },
- {},
-};
-MODULE_DEVICE_TABLE(of, dw_mci_socfpga_match);
-
-static int dw_mci_socfpga_probe(struct platform_device *pdev)
-{
- const struct dw_mci_drv_data *drv_data;
- const struct of_device_id *match;
-
- match = of_match_node(dw_mci_socfpga_match, pdev->dev.of_node);
- drv_data = match->data;
- return dw_mci_pltfm_register(pdev, drv_data);
-}
-
-static struct platform_driver dw_mci_socfpga_pltfm_driver = {
- .probe = dw_mci_socfpga_probe,
- .remove = __exit_p(dw_mci_pltfm_remove),
- .driver = {
- .name = "dwmmc_socfpga",
- .of_match_table = dw_mci_socfpga_match,
- .pm = &dw_mci_pltfm_pmops,
- },
-};
-
-module_platform_driver(dw_mci_socfpga_pltfm_driver);
-
-MODULE_DESCRIPTION("Altera SOCFPGA Specific DW-MSHC Driver Extension");
-MODULE_LICENSE("GPL v2");
-MODULE_ALIAS("platform:dwmmc-socfpga");
--
1.7.9.5
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCHv5 4/4] ARM: socfpga_defconfig: enable SD/MMC support
2013-12-09 13:57 ` dinguyen at altera.com
@ 2013-12-09 13:57 ` dinguyen at altera.com
-1 siblings, 0 replies; 14+ messages in thread
From: dinguyen @ 2013-12-09 13:57 UTC (permalink / raw)
To: dinh.linux, arnd, mturquette, cjb, jh80.chung, tgih.jun
Cc: linux-mmc, linux-arm-kernel, Dinh Nguyen
From: Dinh Nguyen <dinguyen@altera.com>
Enables the dw_mmc driver for SOCFPGA.
Signed-off-by: Dinh Nguyen <dinguyen@altera.com>
---
v5: none
v4: Added patch
v3: none
v3: none
---
arch/arm/configs/socfpga_defconfig | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm/configs/socfpga_defconfig b/arch/arm/configs/socfpga_defconfig
index 4e1ce21..8fff96ba 100644
--- a/arch/arm/configs/socfpga_defconfig
+++ b/arch/arm/configs/socfpga_defconfig
@@ -82,3 +82,5 @@ CONFIG_DEBUG_INFO=y
CONFIG_ENABLE_DEFAULT_TRACERS=y
CONFIG_DEBUG_USER=y
CONFIG_XZ_DEC=y
+CONFIG_MMC=y
+CONFIG_MMC_DW=y
--
1.7.9.5
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCHv5 4/4] ARM: socfpga_defconfig: enable SD/MMC support
@ 2013-12-09 13:57 ` dinguyen at altera.com
0 siblings, 0 replies; 14+ messages in thread
From: dinguyen at altera.com @ 2013-12-09 13:57 UTC (permalink / raw)
To: linux-arm-kernel
From: Dinh Nguyen <dinguyen@altera.com>
Enables the dw_mmc driver for SOCFPGA.
Signed-off-by: Dinh Nguyen <dinguyen@altera.com>
---
v5: none
v4: Added patch
v3: none
v3: none
---
arch/arm/configs/socfpga_defconfig | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm/configs/socfpga_defconfig b/arch/arm/configs/socfpga_defconfig
index 4e1ce21..8fff96ba 100644
--- a/arch/arm/configs/socfpga_defconfig
+++ b/arch/arm/configs/socfpga_defconfig
@@ -82,3 +82,5 @@ CONFIG_DEBUG_INFO=y
CONFIG_ENABLE_DEFAULT_TRACERS=y
CONFIG_DEBUG_USER=y
CONFIG_XZ_DEC=y
+CONFIG_MMC=y
+CONFIG_MMC_DW=y
--
1.7.9.5
^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCHv5 2/4] clk: socfpga: Add a hook for SD/MMC driver to control CIU clock settings
2013-12-09 13:57 ` dinguyen at altera.com
@ 2013-12-09 16:28 ` Arnd Bergmann
-1 siblings, 0 replies; 14+ messages in thread
From: Arnd Bergmann @ 2013-12-09 16:28 UTC (permalink / raw)
To: dinguyen
Cc: dinh.linux, mturquette, cjb, jh80.chung, tgih.jun, linux-mmc,
linux-arm-kernel
On Monday 09 December 2013, dinguyen@altera.com wrote:
> From: Dinh Nguyen <dinguyen@altera.com>
>
> Populate the .prepare function in the clk-ops for the "sdmmc_clk" that represents
> the "ciu" clock for the SD/MMC driver. The prepare function will handle setting
> the correct clock-phase for the CIU clock of the SD/MMC IP.
>
> Signed-off-by: Dinh Nguyen <dinguyen@altera.com>
Please see my comments for this patch for v4, it still looks wrong to me.
The other three patches are good though.
Arnd
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCHv5 2/4] clk: socfpga: Add a hook for SD/MMC driver to control CIU clock settings
@ 2013-12-09 16:28 ` Arnd Bergmann
0 siblings, 0 replies; 14+ messages in thread
From: Arnd Bergmann @ 2013-12-09 16:28 UTC (permalink / raw)
To: linux-arm-kernel
On Monday 09 December 2013, dinguyen at altera.com wrote:
> From: Dinh Nguyen <dinguyen@altera.com>
>
> Populate the .prepare function in the clk-ops for the "sdmmc_clk" that represents
> the "ciu" clock for the SD/MMC driver. The prepare function will handle setting
> the correct clock-phase for the CIU clock of the SD/MMC IP.
>
> Signed-off-by: Dinh Nguyen <dinguyen@altera.com>
Please see my comments for this patch for v4, it still looks wrong to me.
The other three patches are good though.
Arnd
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCHv5 2/4] clk: socfpga: Add a hook for SD/MMC driver to control CIU clock settings
2013-12-09 16:28 ` Arnd Bergmann
@ 2013-12-09 19:41 ` Dinh Nguyen
-1 siblings, 0 replies; 14+ messages in thread
From: Dinh Nguyen @ 2013-12-09 19:41 UTC (permalink / raw)
To: Arnd Bergmann
Cc: dinh.linux, mturquette, cjb, jh80.chung, tgih.jun, linux-mmc,
linux-arm-kernel
On Mon, 2013-12-09 at 17:28 +0100, Arnd Bergmann wrote:
> On Monday 09 December 2013, dinguyen@altera.com wrote:
> > From: Dinh Nguyen <dinguyen@altera.com>
> >
> > Populate the .prepare function in the clk-ops for the "sdmmc_clk" that represents
> > the "ciu" clock for the SD/MMC driver. The prepare function will handle setting
> > the correct clock-phase for the CIU clock of the SD/MMC IP.
> >
> > Signed-off-by: Dinh Nguyen <dinguyen@altera.com>
>
> Please see my comments for this patch for v4, it still looks wrong to me.
> The other three patches are good though.
I apologize for missing your V4 comments. It somehow never ended in my
Altera mbox but is there at gmail. I will address them.
Dinh
>
> Arnd
>
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCHv5 2/4] clk: socfpga: Add a hook for SD/MMC driver to control CIU clock settings
@ 2013-12-09 19:41 ` Dinh Nguyen
0 siblings, 0 replies; 14+ messages in thread
From: Dinh Nguyen @ 2013-12-09 19:41 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, 2013-12-09 at 17:28 +0100, Arnd Bergmann wrote:
> On Monday 09 December 2013, dinguyen at altera.com wrote:
> > From: Dinh Nguyen <dinguyen@altera.com>
> >
> > Populate the .prepare function in the clk-ops for the "sdmmc_clk" that represents
> > the "ciu" clock for the SD/MMC driver. The prepare function will handle setting
> > the correct clock-phase for the CIU clock of the SD/MMC IP.
> >
> > Signed-off-by: Dinh Nguyen <dinguyen@altera.com>
>
> Please see my comments for this patch for v4, it still looks wrong to me.
> The other three patches are good though.
I apologize for missing your V4 comments. It somehow never ended in my
Altera mbox but is there at gmail. I will address them.
Dinh
>
> Arnd
>
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2013-12-09 19:43 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-12-09 13:57 [PATCHv5 0/4]socfpga: Enable SD/MMC support dinguyen
2013-12-09 13:57 ` dinguyen at altera.com
2013-12-09 13:57 ` [PATCHv5 1/4] arm: dts: Add support for SD/MMC on SOCFPGA dinguyen
2013-12-09 13:57 ` dinguyen at altera.com
2013-12-09 13:57 ` [PATCHv5 2/4] clk: socfpga: Add a hook for SD/MMC driver to control CIU clock settings dinguyen
2013-12-09 13:57 ` dinguyen at altera.com
2013-12-09 16:28 ` Arnd Bergmann
2013-12-09 16:28 ` Arnd Bergmann
2013-12-09 19:41 ` Dinh Nguyen
2013-12-09 19:41 ` Dinh Nguyen
2013-12-09 13:57 ` [PATCHv5 3/4] mmc: dw_mmc-socfpga: Remove the SOCFPGA specific platform for dw_mmc dinguyen
2013-12-09 13:57 ` dinguyen at altera.com
2013-12-09 13:57 ` [PATCHv5 4/4] ARM: socfpga_defconfig: enable SD/MMC support dinguyen
2013-12-09 13:57 ` dinguyen at altera.com
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.