| Message ID | 1389604469-8064-3-git-send-email-zhangfei.gao@linaro.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Monday, January 13, 2014, Zhangfei Gao wrote: > Remove clk_table and directly use ios->clock as clock source rate. > Abstract init clock rate and max clock limitation in clk.c > > Signed-off-by: Zhangfei Gao <zhangfei.gao@linaro.org> Acked-by: Seungwon Jeon <tgih.jun@samsung.com> Thanks, Seungwon Jeon -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
+ Chris Ball (New e-mail) On Tue, January 14, 2014, Seungwon Jeon wrote: > On Monday, January 13, 2014, Zhangfei Gao wrote: > > Remove clk_table and directly use ios->clock as clock source rate. > > Abstract init clock rate and max clock limitation in clk.c > > > > Signed-off-by: Zhangfei Gao <zhangfei.gao@linaro.org> > > Acked-by: Seungwon Jeon <tgih.jun@samsung.com> > > Thanks, > Seungwon Jeon -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi, On Tue, Jan 14 2014, Seungwon Jeon wrote: > On Monday, January 13, 2014, Zhangfei Gao wrote: >> Remove clk_table and directly use ios->clock as clock source rate. >> Abstract init clock rate and max clock limitation in clk.c >> >> Signed-off-by: Zhangfei Gao <zhangfei.gao@linaro.org> > > Acked-by: Seungwon Jeon <tgih.jun@samsung.com> It looks like patch 1/2 did not make 3.14. Do you want me to take this patch for 3.14 anyway? (Why?) Thanks, - Chris.
On Sat, Jan 18, 2014 at 12:01 AM, Chris Ball <chris@printf.net> wrote: > > Hi, > > On Tue, Jan 14 2014, Seungwon Jeon wrote: > > On Monday, January 13, 2014, Zhangfei Gao wrote: > >> Remove clk_table and directly use ios->clock as clock source rate. > >> Abstract init clock rate and max clock limitation in clk.c > >> > >> Signed-off-by: Zhangfei Gao <zhangfei.gao@linaro.org> > > > > Acked-by: Seungwon Jeon <tgih.jun@samsung.com> > > It looks like patch 1/2 did not make 3.14. Do you want me to take > this patch for 3.14 anyway? (Why?) Yes, I hope that it would be merged this time. I think that this change should have been applied to (036f29: mmc: dw_mmc: add dw_mmc-k3 for k3 platform) originally. Thanks, sw-j -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Quoting Seungwon Jeon (2014-01-17 08:10:23) > On Sat, Jan 18, 2014 at 12:01 AM, Chris Ball <chris@printf.net> wrote: > > > > Hi, > > > > On Tue, Jan 14 2014, Seungwon Jeon wrote: > > > On Monday, January 13, 2014, Zhangfei Gao wrote: > > >> Remove clk_table and directly use ios->clock as clock source rate. > > >> Abstract init clock rate and max clock limitation in clk.c > > >> > > >> Signed-off-by: Zhangfei Gao <zhangfei.gao@linaro.org> > > > > > > Acked-by: Seungwon Jeon <tgih.jun@samsung.com> > > > > It looks like patch 1/2 did not make 3.14. Do you want me to take > > this patch for 3.14 anyway? (Why?) > Yes, I hope that it would be merged this time. > I think that this change should have been applied to (036f29: mmc: > dw_mmc: add dw_mmc-k3 for k3 platform) originally. Oops, I just read this mail. I have taken in patch #1 through the clk tree but have not yet pushed it out to my public mirror. Chris, I'm happy to take it if you haven't yet. Regards, Mike > > Thanks, > sw-j -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Mike, On Fri, Jan 17 2014, Mike Turquette wrote: > Oops, I just read this mail. I have taken in patch #1 through the clk > tree but have not yet pushed it out to my public mirror. > > Chris, I'm happy to take it if you haven't yet. That's fine. I'd like to take 2/2, just pushed it to mmc-next for 3.14. Thanks, - Chris.
diff --git a/Documentation/devicetree/bindings/mmc/k3-dw-mshc.txt b/Documentation/devicetree/bindings/mmc/k3-dw-mshc.txt index d7e2d7f159bb..b8653ea97957 100644 --- a/Documentation/devicetree/bindings/mmc/k3-dw-mshc.txt +++ b/Documentation/devicetree/bindings/mmc/k3-dw-mshc.txt @@ -14,18 +14,6 @@ Required Properties: * compatible: should be one of the following. - "hisilicon,hi4511-dw-mshc": for controllers with hi4511 specific extentions. -* clock-freq-table: should be the frequency (in Hz) array of the ciu clock - in each supported mode. - 0. CIU clock rate in Hz for DS mode - 1. CIU clock rate in Hz for MMC HS mode - 2. CIU clock rate in Hz for SD HS mode - 3. CIU clock rate in Hz for SDR12 mode - 4. CIU clock rate in Hz for SDR25 mode - 5. CIU clock rate in Hz for SDR50 mode - 6. CIU clock rate in Hz for SDR104 mode - 7. CIU clock rate in Hz for DDR50 mode - 8. CIU clock rate in Hz for HS200 mode - Example: /* for Hi3620 */ @@ -39,8 +27,6 @@ Example: #size-cells = <0>; clocks = <&mmc_clock HI3620_SD_CIUCLK>, <&clock HI3620_DDRC_PER_CLK>; clock-names = "ciu", "biu"; - clock-freq-table = - <25000000 0 50000000 25000000 50000000 100000000 0 50000000>; }; /* Board portion */ diff --git a/drivers/mmc/host/dw_mmc-k3.c b/drivers/mmc/host/dw_mmc-k3.c index 68e5e428e8f6..84e1a670f8c7 100644 --- a/drivers/mmc/host/dw_mmc-k3.c +++ b/drivers/mmc/host/dw_mmc-k3.c @@ -18,56 +18,19 @@ #include "dw_mmc.h" #include "dw_mmc-pltfm.h" -#define MAX_NUMS 10 -struct dw_mci_k3_priv_data { - u32 clk_table[MAX_NUMS]; -}; - static void dw_mci_k3_set_ios(struct dw_mci *host, struct mmc_ios *ios) { - struct dw_mci_k3_priv_data *priv = host->priv; - u32 rate = priv->clk_table[ios->timing]; int ret; - if (!rate) { - dev_warn(host->dev, - "no specified rate in timing %u\n", ios->timing); - return; - } - - ret = clk_set_rate(host->ciu_clk, rate); + ret = clk_set_rate(host->ciu_clk, ios->clock); if (ret) - dev_warn(host->dev, "failed to set clock rate %uHz\n", rate); + dev_warn(host->dev, "failed to set rate %uHz\n", ios->clock); host->bus_hz = clk_get_rate(host->ciu_clk); } -static int dw_mci_k3_parse_dt(struct dw_mci *host) -{ - struct dw_mci_k3_priv_data *priv; - struct device_node *node = host->dev->of_node; - struct property *prop; - const __be32 *cur; - u32 val, num = 0; - - priv = devm_kzalloc(host->dev, sizeof(*priv), GFP_KERNEL); - if (!priv) { - dev_err(host->dev, "mem alloc failed for private data\n"); - return -ENOMEM; - } - host->priv = priv; - - of_property_for_each_u32(node, "clock-freq-table", prop, cur, val) { - if (num >= MAX_NUMS) - break; - priv->clk_table[num++] = val; - } - return 0; -} - static const struct dw_mci_drv_data k3_drv_data = { .set_ios = dw_mci_k3_set_ios, - .parse_dt = dw_mci_k3_parse_dt, }; static const struct of_device_id dw_mci_k3_match[] = {
Remove clk_table and directly use ios->clock as clock source rate. Abstract init clock rate and max clock limitation in clk.c Signed-off-by: Zhangfei Gao <zhangfei.gao@linaro.org> --- .../devicetree/bindings/mmc/k3-dw-mshc.txt | 14 ------- drivers/mmc/host/dw_mmc-k3.c | 41 +------------------- 2 files changed, 2 insertions(+), 53 deletions(-)