From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jaehoon Chung Subject: Re: [PATCH 3/3] mmc: dw_mmc: k3: support hi6220 Date: Mon, 04 May 2015 09:56:51 +0900 Message-ID: <5546C3D3.7080402@samsung.com> References: <1430444818-7740-1-git-send-email-zhangfei.gao@linaro.org> <1430444818-7740-4-git-send-email-zhangfei.gao@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: Received: from mailout4.samsung.com ([203.254.224.34]:24210 "EHLO mailout4.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751030AbbEDA4y (ORCPT ); Sun, 3 May 2015 20:56:54 -0400 In-reply-to: <1430444818-7740-4-git-send-email-zhangfei.gao@linaro.org> Sender: linux-mmc-owner@vger.kernel.org List-Id: linux-mmc@vger.kernel.org To: Zhangfei Gao , Ulf Hansson Cc: linux-mmc@vger.kernel.org, devicetree@vger.kernel.org, "Jorge A. Ramirez-Ortiz" , Dan Yuan Hi, Zhangfei. Could you add the commit message for this patch? On 05/01/2015 10:46 AM, Zhangfei Gao wrote: > Signed-off-by: Jorge A. Ramirez-Ortiz > Signed-off-by: Dan Yuan > Signed-off-by: Zhangfei Gao > --- > drivers/mmc/host/dw_mmc-k3.c | 104 ++++++++++++++++++++++++++++++++++++++++++- > 1 file changed, 102 insertions(+), 2 deletions(-) > > diff --git a/drivers/mmc/host/dw_mmc-k3.c b/drivers/mmc/host/dw_mmc-k3.c > index 650f9cc..ab3dc59 100644 > --- a/drivers/mmc/host/dw_mmc-k3.c > +++ b/drivers/mmc/host/dw_mmc-k3.c > @@ -8,16 +8,29 @@ > * (at your option) any later version. > */ > > -#include > -#include > #include > +#include > +#include > #include > #include > +#include > #include > +#include > +#include > +#include > > #include "dw_mmc.h" > #include "dw_mmc-pltfm.h" > > +#define AO_SCTRL_SEL18 BIT(10) > +#define AO_SCTRL_CTRL3 0x40C Does it use these at all K3 SoC? > + > +#define SDMMC_CMD_DISABLE_BOOT BIT(26) > + > +struct k3_priv { > + struct regmap *reg; > +}; > + > static void dw_mci_k3_set_ios(struct dw_mci *host, struct mmc_ios *ios) > { > int ret; > @@ -33,8 +46,95 @@ static const struct dw_mci_drv_data k3_drv_data = { > .set_ios = dw_mci_k3_set_ios, > }; > > +static int dw_mci_hi6220_parse_dt(struct dw_mci *host) > +{ > + struct k3_priv *priv; > + > + priv = devm_kzalloc(host->dev, sizeof(*priv), GFP_KERNEL); > + if (!priv) > + return -ENOMEM; > + > + priv->reg = syscon_regmap_lookup_by_phandle(host->dev->of_node, > + "hisilicon,peripheral-syscon"); > + if (IS_ERR(priv->reg)) > + priv->reg = NULL; > + > + host->priv = priv; > + return 0; > +} > + > +static int dw_mci_hi6220_switch_voltage(struct mmc_host *mmc, struct mmc_ios *ios) > +{ > + struct dw_mci_slot *slot = mmc_priv(mmc); > + struct k3_priv *priv; > + struct dw_mci *host; > + int min_uv, max_uv; > + int ret; > + > + host = slot->host; > + priv = host->priv; > + > + if (!priv || !priv->reg) > + return 0; If returned at this point, switch_voltage don't do nothing. See voltage_switch into dw_mmc.c. "return 0" is meaning that switch_voltage was done. How about using the error number? > + > + min_uv = 1800000; > + max_uv = 1800000; > + > + if (ios->signal_voltage == MMC_SIGNAL_VOLTAGE_330) { > + ret = regmap_update_bits(priv->reg, AO_SCTRL_CTRL3, > + AO_SCTRL_SEL18, 0); > + if (ret) { > + dev_dbg(host->dev, "switch voltage failed\n"); > + return ret; > + } > + min_uv = 3000000; > + max_uv = 3000000; > + } > + > + if (IS_ERR_OR_NULL(mmc->supply.vqmmc)) > + return 0; ditto. > + > + ret = regulator_set_voltage(mmc->supply.vqmmc, min_uv, max_uv); > + if (ret) { > + dev_dbg(host->dev, "Regulator set error %d: %d - %d\n", > + ret, min_uv, max_uv); > + return ret; > + } > + > + ret = regulator_enable(mmc->supply.vqmmc); > + if (ret) { > + dev_dbg(host->dev, "Regulator enable error %d\n", ret); > + return ret; > + } > + > + usleep_range(5000, 5500); What's delay? > + > + return 0; > +} > + > +static void dw_mci_hi6220_set_ios(struct dw_mci *host, struct mmc_ios *ios) > +{ > + int ret; > + unsigned int clock; > + > + clock = (ios->clock <= 25000000) ? 25000000 : ios->clock; Even though ios->clock is 400KHz, is it set to 25MHz? Best Regards, Jaehoon Chung > + > + ret = clk_set_rate(host->biu_clk, clock); > + if (ret) > + dev_warn(host->dev, "failed to set rate %uHz\n", clock); > + > + host->bus_hz = clk_get_rate(host->biu_clk); > +} > + > +static const struct dw_mci_drv_data hi6220_data = { > + .switch_voltage = dw_mci_hi6220_switch_voltage, > + .set_ios = dw_mci_hi6220_set_ios, > + .parse_dt = dw_mci_hi6220_parse_dt, > +}; > + > static const struct of_device_id dw_mci_k3_match[] = { > { .compatible = "hisilicon,hi4511-dw-mshc", .data = &k3_drv_data, }, > + { .compatible = "hisilicon,hi6220-dw-mshc", .data = &hi6220_data, }, > {}, > }; > MODULE_DEVICE_TABLE(of, dw_mci_k3_match); >