* [PATCH v3 0/3] mmc: mediatek: Use data tune for CMD line tune @ 2017-01-19 10:19 Yong Mao 2017-01-19 10:19 ` [PATCH v3 1/3] mmc: dt-bindings: update Mediatek MMC bindings Yong Mao ` (2 more replies) 0 siblings, 3 replies; 11+ messages in thread From: Yong Mao @ 2017-01-19 10:19 UTC (permalink / raw) To: Ulf Hansson Cc: Linus Walleij, Chaotian Jing, yong mao, Eddie Huang, Chunfeng Yun, linux-mmc, srv_heupstream, linux-mediatek, linux-kernel yong mao (3): mmc: dt-bindings: update Mediatek MMC bindings dts: mediatek: configure some fixed mmc parameters mmc: mediatek: Use data tune for CMD line tune Documentation/devicetree/bindings/mmc/mtk-sd.txt | 9 ++ arch/arm64/boot/dts/mediatek/mt8173-evb.dts | 3 + drivers/mmc/host/mtk-sd.c | 170 ++++++++++++++++++++--- 3 files changed, 165 insertions(+), 17 deletions(-) -- 1.8.1.1.dirty ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH v3 1/3] mmc: dt-bindings: update Mediatek MMC bindings 2017-01-19 10:19 [PATCH v3 0/3] mmc: mediatek: Use data tune for CMD line tune Yong Mao @ 2017-01-19 10:19 ` Yong Mao 2017-01-20 7:43 ` Daniel Kurtz 2017-01-20 7:52 ` Ulf Hansson 2017-01-19 10:19 ` [PATCH v3 2/3] dts: mediatek: configure some fixed mmc parameters Yong Mao 2017-01-19 10:19 ` [PATCH v3 3/3] mmc: mediatek: Use data tune for CMD line tune Yong Mao 2 siblings, 2 replies; 11+ messages in thread From: Yong Mao @ 2017-01-19 10:19 UTC (permalink / raw) To: Ulf Hansson Cc: Linus Walleij, Chaotian Jing, yong mao, Eddie Huang, Chunfeng Yun, linux-mmc, srv_heupstream, linux-mediatek, linux-kernel From: yong mao <yong.mao@mediatek.com> Add description for mtk-hs200-cmd-int-delay Add description for mtk-hs400-cmd-int-delay Add description for mtk-hs400-cmd-resp-sel Signed-off-by: Yong Mao <yong.mao@mediatek.com> --- Documentation/devicetree/bindings/mmc/mtk-sd.txt | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/Documentation/devicetree/bindings/mmc/mtk-sd.txt b/Documentation/devicetree/bindings/mmc/mtk-sd.txt index 0120c7f..149f472 100644 --- a/Documentation/devicetree/bindings/mmc/mtk-sd.txt +++ b/Documentation/devicetree/bindings/mmc/mtk-sd.txt @@ -21,6 +21,12 @@ Optional properties: - assigned-clocks: PLL of the source clock - assigned-clock-parents: parent of source clock, used for HS400 mode to get 400Mhz source clock - hs400-ds-delay: HS400 DS delay setting +- mtk-hs200-cmd-int-delay: HS200 command internal delay setting. + The value is an integer from 0 to 31 +- mtk-hs400-cmd-int-delay: HS400 command internal delay setting + The value is an integer from 0 to 31 +- mtk-hs400-cmd-resp-sel: HS400 command response sample selection + The value is an integer from 0 to 1 Examples: mmc0: mmc@11230000 { @@ -38,4 +44,7 @@ mmc0: mmc@11230000 { assigned-clocks = <&topckgen CLK_TOP_MSDC50_0_SEL>; assigned-clock-parents = <&topckgen CLK_TOP_MSDCPLL_D2>; hs400-ds-delay = <0x14015>; + mtk-hs200-cmd-int-delay = <26>; + mtk-hs400-cmd-int-delay = <14>; + mtk-hs400-cmd-resp-sel = <0>; /* 0: rising, 1: falling */ }; -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH v3 1/3] mmc: dt-bindings: update Mediatek MMC bindings 2017-01-19 10:19 ` [PATCH v3 1/3] mmc: dt-bindings: update Mediatek MMC bindings Yong Mao @ 2017-01-20 7:43 ` Daniel Kurtz 2017-01-20 7:52 ` Ulf Hansson 1 sibling, 0 replies; 11+ messages in thread From: Daniel Kurtz @ 2017-01-20 7:43 UTC (permalink / raw) To: Yong Mao Cc: Ulf Hansson, Linus Walleij, Chaotian Jing, Eddie Huang, Chunfeng Yun, linux-mmc, srv_heupstream, moderated list:ARM/Mediatek SoC support, linux-kernel On Thu, Jan 19, 2017 at 6:19 PM, Yong Mao <yong.mao@mediatek.com> wrote: > > From: yong mao <yong.mao@mediatek.com> > > Add description for mtk-hs200-cmd-int-delay > Add description for mtk-hs400-cmd-int-delay > Add description for mtk-hs400-cmd-resp-sel > > Signed-off-by: Yong Mao <yong.mao@mediatek.com> > --- > Documentation/devicetree/bindings/mmc/mtk-sd.txt | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/Documentation/devicetree/bindings/mmc/mtk-sd.txt b/Documentation/devicetree/bindings/mmc/mtk-sd.txt > index 0120c7f..149f472 100644 > --- a/Documentation/devicetree/bindings/mmc/mtk-sd.txt > +++ b/Documentation/devicetree/bindings/mmc/mtk-sd.txt > @@ -21,6 +21,12 @@ Optional properties: > - assigned-clocks: PLL of the source clock > - assigned-clock-parents: parent of source clock, used for HS400 mode to get 400Mhz source clock > - hs400-ds-delay: HS400 DS delay setting > +- mtk-hs200-cmd-int-delay: HS200 command internal delay setting. > + The value is an integer from 0 to 31 > +- mtk-hs400-cmd-int-delay: HS400 command internal delay setting > + The value is an integer from 0 to 31 I think last time Rob meant that you should add a vendor prefixes like this: mediatek,hs200-cmd-int-delay > +- mtk-hs400-cmd-resp-sel: HS400 command response sample selection > + The value is an integer from 0 to 1 Last time Rob stated this looks like a boolean, and to state that and make the default the more common case. So, this could be something like: mediatek,hs200-cmd-resp-sample-rising: A boolean. If present, HS400 command responses are sampled on rising edges. If not present, HS400 command responses are sampled on falling edges. > > Examples: > mmc0: mmc@11230000 { > @@ -38,4 +44,7 @@ mmc0: mmc@11230000 { > assigned-clocks = <&topckgen CLK_TOP_MSDC50_0_SEL>; > assigned-clock-parents = <&topckgen CLK_TOP_MSDCPLL_D2>; > hs400-ds-delay = <0x14015>; > + mtk-hs200-cmd-int-delay = <26>; > + mtk-hs400-cmd-int-delay = <14>; > + mtk-hs400-cmd-resp-sel = <0>; /* 0: rising, 1: falling */ > }; > -- > 1.7.9.5 > ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v3 1/3] mmc: dt-bindings: update Mediatek MMC bindings 2017-01-19 10:19 ` [PATCH v3 1/3] mmc: dt-bindings: update Mediatek MMC bindings Yong Mao 2017-01-20 7:43 ` Daniel Kurtz @ 2017-01-20 7:52 ` Ulf Hansson 1 sibling, 0 replies; 11+ messages in thread From: Ulf Hansson @ 2017-01-20 7:52 UTC (permalink / raw) To: Yong Mao Cc: Linus Walleij, Chaotian Jing, Eddie Huang, Chunfeng Yun, linux-mmc, srv_heupstream, linux-mediatek, linux-kernel, devicetree, Rob Herring +devicetree, Rob On 19 January 2017 at 11:19, Yong Mao <yong.mao@mediatek.com> wrote: > From: yong mao <yong.mao@mediatek.com> > > Add description for mtk-hs200-cmd-int-delay > Add description for mtk-hs400-cmd-int-delay > Add description for mtk-hs400-cmd-resp-sel > > Signed-off-by: Yong Mao <yong.mao@mediatek.com> > --- > Documentation/devicetree/bindings/mmc/mtk-sd.txt | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/Documentation/devicetree/bindings/mmc/mtk-sd.txt b/Documentation/devicetree/bindings/mmc/mtk-sd.txt > index 0120c7f..149f472 100644 > --- a/Documentation/devicetree/bindings/mmc/mtk-sd.txt > +++ b/Documentation/devicetree/bindings/mmc/mtk-sd.txt > @@ -21,6 +21,12 @@ Optional properties: > - assigned-clocks: PLL of the source clock > - assigned-clock-parents: parent of source clock, used for HS400 mode to get 400Mhz source clock > - hs400-ds-delay: HS400 DS delay setting > +- mtk-hs200-cmd-int-delay: HS200 command internal delay setting. > + The value is an integer from 0 to 31 Please change to: mediatek,hs200-cmd-delay ... and if there is a unit, like ns or us, please add that a suffix. > +- mtk-hs400-cmd-int-delay: HS400 command internal delay setting > + The value is an integer from 0 to 31 mediatek,hs400-cmd-delay and add unit if applicable. > +- mtk-hs400-cmd-resp-sel: HS400 command response sample selection > + The value is an integer from 0 to 1 mediatek,hs400-cmd-resp-sel And make it a boolean value instead! > > Examples: > mmc0: mmc@11230000 { > @@ -38,4 +44,7 @@ mmc0: mmc@11230000 { > assigned-clocks = <&topckgen CLK_TOP_MSDC50_0_SEL>; > assigned-clock-parents = <&topckgen CLK_TOP_MSDCPLL_D2>; > hs400-ds-delay = <0x14015>; > + mtk-hs200-cmd-int-delay = <26>; > + mtk-hs400-cmd-int-delay = <14>; > + mtk-hs400-cmd-resp-sel = <0>; /* 0: rising, 1: falling */ The rising/falling information belongs in description of the binding a few lines above. Please move it there. > }; > -- > 1.7.9.5 > Kind regards Uffe ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH v3 2/3] dts: mediatek: configure some fixed mmc parameters 2017-01-19 10:19 [PATCH v3 0/3] mmc: mediatek: Use data tune for CMD line tune Yong Mao 2017-01-19 10:19 ` [PATCH v3 1/3] mmc: dt-bindings: update Mediatek MMC bindings Yong Mao @ 2017-01-19 10:19 ` Yong Mao 2017-01-20 7:59 ` Ulf Hansson 2017-01-19 10:19 ` [PATCH v3 3/3] mmc: mediatek: Use data tune for CMD line tune Yong Mao 2 siblings, 1 reply; 11+ messages in thread From: Yong Mao @ 2017-01-19 10:19 UTC (permalink / raw) To: Ulf Hansson Cc: Linus Walleij, Chaotian Jing, yong mao, Eddie Huang, Chunfeng Yun, linux-mmc, srv_heupstream, linux-mediatek, linux-kernel From: yong mao <yong.mao@mediatek.com> configure some fixed mmc parameters Signed-off-by: Yong Mao <yong.mao@mediatek.com> Signed-off-by: Chaotian Jing <chaotian.jing@mediatek.com> --- arch/arm64/boot/dts/mediatek/mt8173-evb.dts | 3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/arm64/boot/dts/mediatek/mt8173-evb.dts b/arch/arm64/boot/dts/mediatek/mt8173-evb.dts index 0ecaad4..403705e 100644 --- a/arch/arm64/boot/dts/mediatek/mt8173-evb.dts +++ b/arch/arm64/boot/dts/mediatek/mt8173-evb.dts @@ -134,6 +134,9 @@ bus-width = <8>; max-frequency = <50000000>; cap-mmc-highspeed; + mtk-hs200-cmd-int-delay=<26>; + mtk-hs400-cmd-int-delay=<14>; + mtk-hs400-cmd-resp-sel=<0>; /* 0:rising, 1:falling */ vmmc-supply = <&mt6397_vemc_3v3_reg>; vqmmc-supply = <&mt6397_vio18_reg>; non-removable; -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH v3 2/3] dts: mediatek: configure some fixed mmc parameters 2017-01-19 10:19 ` [PATCH v3 2/3] dts: mediatek: configure some fixed mmc parameters Yong Mao @ 2017-01-20 7:59 ` Ulf Hansson 2017-01-24 10:36 ` Matthias Brugger 0 siblings, 1 reply; 11+ messages in thread From: Ulf Hansson @ 2017-01-20 7:59 UTC (permalink / raw) To: Yong Mao Cc: Linus Walleij, Chaotian Jing, Eddie Huang, Chunfeng Yun, linux-mmc, srv_heupstream, linux-mediatek, linux-kernel On 19 January 2017 at 11:19, Yong Mao <yong.mao@mediatek.com> wrote: > From: yong mao <yong.mao@mediatek.com> > > configure some fixed mmc parameters > > Signed-off-by: Yong Mao <yong.mao@mediatek.com> > Signed-off-by: Chaotian Jing <chaotian.jing@mediatek.com> Please change the prefix of the commit message header to "ARM64: dts: mt8173:". Also make sure you send this to the ARM mailing list as well. In general, I think it's better DTS changes go via the ARM SoC tree, but I can pick it up if it's trivial and I get an ack from the SoC maintainer. > --- > arch/arm64/boot/dts/mediatek/mt8173-evb.dts | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/arch/arm64/boot/dts/mediatek/mt8173-evb.dts b/arch/arm64/boot/dts/mediatek/mt8173-evb.dts > index 0ecaad4..403705e 100644 > --- a/arch/arm64/boot/dts/mediatek/mt8173-evb.dts > +++ b/arch/arm64/boot/dts/mediatek/mt8173-evb.dts > @@ -134,6 +134,9 @@ > bus-width = <8>; > max-frequency = <50000000>; > cap-mmc-highspeed; > + mtk-hs200-cmd-int-delay=<26>; > + mtk-hs400-cmd-int-delay=<14>; > + mtk-hs400-cmd-resp-sel=<0>; /* 0:rising, 1:falling */ > vmmc-supply = <&mt6397_vemc_3v3_reg>; > vqmmc-supply = <&mt6397_vio18_reg>; > non-removable; > -- > 1.7.9.5 > Kind regards Uffe ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v3 2/3] dts: mediatek: configure some fixed mmc parameters 2017-01-20 7:59 ` Ulf Hansson @ 2017-01-24 10:36 ` Matthias Brugger 0 siblings, 0 replies; 11+ messages in thread From: Matthias Brugger @ 2017-01-24 10:36 UTC (permalink / raw) To: Ulf Hansson, Yong Mao Cc: srv_heupstream, Linus Walleij, linux-mmc, linux-kernel, linux-mediatek, Chunfeng Yun, Eddie Huang, Chaotian Jing Hi Ulf, On 01/20/2017 08:59 AM, Ulf Hansson wrote: > On 19 January 2017 at 11:19, Yong Mao <yong.mao@mediatek.com> wrote: >> From: yong mao <yong.mao@mediatek.com> >> >> configure some fixed mmc parameters >> >> Signed-off-by: Yong Mao <yong.mao@mediatek.com> >> Signed-off-by: Chaotian Jing <chaotian.jing@mediatek.com> > > Please change the prefix of the commit message header to "ARM64: dts: mt8173:". > > Also make sure you send this to the ARM mailing list as well. > > In general, I think it's better DTS changes go via the ARM SoC tree, > but I can pick it up if it's trivial and I get an ack from the SoC > maintainer. > I can take them through my tree, let's wait at Rob's opinion of v4. Regards, Matthias >> --- >> arch/arm64/boot/dts/mediatek/mt8173-evb.dts | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/mediatek/mt8173-evb.dts b/arch/arm64/boot/dts/mediatek/mt8173-evb.dts >> index 0ecaad4..403705e 100644 >> --- a/arch/arm64/boot/dts/mediatek/mt8173-evb.dts >> +++ b/arch/arm64/boot/dts/mediatek/mt8173-evb.dts >> @@ -134,6 +134,9 @@ >> bus-width = <8>; >> max-frequency = <50000000>; >> cap-mmc-highspeed; >> + mtk-hs200-cmd-int-delay=<26>; >> + mtk-hs400-cmd-int-delay=<14>; >> + mtk-hs400-cmd-resp-sel=<0>; /* 0:rising, 1:falling */ >> vmmc-supply = <&mt6397_vemc_3v3_reg>; >> vqmmc-supply = <&mt6397_vio18_reg>; >> non-removable; >> -- >> 1.7.9.5 >> > > Kind regards > Uffe > > _______________________________________________ > Linux-mediatek mailing list > Linux-mediatek@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-mediatek > ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH v3 3/3] mmc: mediatek: Use data tune for CMD line tune 2017-01-19 10:19 [PATCH v3 0/3] mmc: mediatek: Use data tune for CMD line tune Yong Mao 2017-01-19 10:19 ` [PATCH v3 1/3] mmc: dt-bindings: update Mediatek MMC bindings Yong Mao 2017-01-19 10:19 ` [PATCH v3 2/3] dts: mediatek: configure some fixed mmc parameters Yong Mao @ 2017-01-19 10:19 ` Yong Mao 2017-01-20 4:49 ` [PATCH] mmc: mediatek: fix semicolon.cocci warnings kbuild test robot ` (2 more replies) 2 siblings, 3 replies; 11+ messages in thread From: Yong Mao @ 2017-01-19 10:19 UTC (permalink / raw) To: Ulf Hansson Cc: Linus Walleij, Chaotian Jing, yong mao, Eddie Huang, Chunfeng Yun, linux-mmc, srv_heupstream, linux-mediatek, linux-kernel From: yong mao <yong.mao@mediatek.com> If we don't select a set of better parameters for our emmc host, It may easily occur CMD response CRC error. And also it may cause cannot boot up issue. Fot getting a set of better parameters, our emmc host supports data tune mechanism.Therefore, our emmc driver also should change to use data tune for CMD line. Because our emmc host use the different clock source to sample the CMD signal between HS200 and HS400 mode, the parameters are also different between these two modes. Separate cmd internal delay for HS200/HS400 mode. This change can fix "System can not boot up" issue. Signed-off-by: Yong Mao <yong.mao@mediatek.com> Signed-off-by: Chaotian Jing <chaotian.jing@mediatek.com> --- drivers/mmc/host/mtk-sd.c | 170 ++++++++++++++++++++++++++++++++++++++++----- 1 file changed, 153 insertions(+), 17 deletions(-) diff --git a/drivers/mmc/host/mtk-sd.c b/drivers/mmc/host/mtk-sd.c index 80ba034..b4e1c43 100644 --- a/drivers/mmc/host/mtk-sd.c +++ b/drivers/mmc/host/mtk-sd.c @@ -75,6 +75,7 @@ #define MSDC_PATCH_BIT1 0xb4 #define MSDC_PAD_TUNE 0xec #define PAD_DS_TUNE 0x188 +#define PAD_CMD_TUNE 0x18c #define EMMC50_CFG0 0x208 /*--------------------------------------------------------------------------*/ @@ -210,13 +211,18 @@ #define MSDC_PATCH_BIT_SPCPUSH (0x1 << 29) /* RW */ #define MSDC_PATCH_BIT_DECRCTMO (0x1 << 30) /* RW */ +#define MSDC_PAD_TUNE_DATWRDLY (0x1f << 0) /* RW */ #define MSDC_PAD_TUNE_DATRRDLY (0x1f << 8) /* RW */ #define MSDC_PAD_TUNE_CMDRDLY (0x1f << 16) /* RW */ +#define MSDC_PAD_TUNE_CMDRRDLY (0x1f << 22) /* RW */ +#define MSDC_PAD_TUNE_CLKTDLY (0x1f << 27) /* RW */ #define PAD_DS_TUNE_DLY1 (0x1f << 2) /* RW */ #define PAD_DS_TUNE_DLY2 (0x1f << 7) /* RW */ #define PAD_DS_TUNE_DLY3 (0x1f << 12) /* RW */ +#define PAD_CMD_TUNE_RX_DLY3 (0x1f << 1) /* RW */ + #define EMMC50_CFG_PADCMD_LATCHCK (0x1 << 0) /* RW */ #define EMMC50_CFG_CRCSTS_EDGE (0x1 << 3) /* RW */ #define EMMC50_CFG_CFCSTS_SEL (0x1 << 4) /* RW */ @@ -284,12 +290,14 @@ struct msdc_save_para { u32 patch_bit0; u32 patch_bit1; u32 pad_ds_tune; + u32 pad_cmd_tune; u32 emmc50_cfg0; }; struct msdc_tune_para { u32 iocon; u32 pad_tune; + u32 pad_cmd_tune; }; struct msdc_delay_phase { @@ -331,6 +339,9 @@ struct msdc_host { unsigned char timing; bool vqmmc_enabled; u32 hs400_ds_delay; + u32 hs200_cmd_int_delay; /* cmd internal delay for HS200/SDR104 */ + u32 hs400_cmd_int_delay; /* cmd internal delay for HS400 */ + u32 hs400_cmd_resp_sel; /* cmd response sample selection for HS400 */ bool hs400_mode; /* current eMMC will run at hs400 mode */ struct msdc_save_para save_para; /* used when gate HCLK */ struct msdc_tune_para def_tune_para; /* default tune setting */ @@ -600,8 +611,14 @@ static void msdc_set_mclk(struct msdc_host *host, unsigned char timing, u32 hz) } else { writel(host->saved_tune_para.iocon, host->base + MSDC_IOCON); writel(host->saved_tune_para.pad_tune, host->base + MSDC_PAD_TUNE); + writel(host->saved_tune_para.pad_cmd_tune, + host->base + PAD_CMD_TUNE); } + if (timing == MMC_TIMING_MMC_HS400) + sdr_set_field(host->base + PAD_CMD_TUNE, + MSDC_PAD_TUNE_CMDRRDLY, + host->hs400_cmd_int_delay); dev_dbg(host->dev, "sclk: %d, timing: %d\n", host->sclk, timing); } @@ -1302,7 +1319,7 @@ static struct msdc_delay_phase get_best_delay(struct msdc_host *host, u32 delay) len_final = len; } start += len ? len : 1; - if (len >= 8 && start_final < 4) + if (len >= 12 && start_final < 4) break; } @@ -1325,36 +1342,67 @@ static int msdc_tune_response(struct mmc_host *mmc, u32 opcode) struct msdc_host *host = mmc_priv(mmc); u32 rise_delay = 0, fall_delay = 0; struct msdc_delay_phase final_rise_delay, final_fall_delay = { 0,}; + struct msdc_delay_phase internal_delay_phase; u8 final_delay, final_maxlen; + u32 internal_delay = 0; int cmd_err; - int i; + int i, j; + + if (mmc->ios.timing == MMC_TIMING_MMC_HS200 || + mmc->ios.timing == MMC_TIMING_UHS_SDR104) + sdr_set_field(host->base + MSDC_PAD_TUNE, + MSDC_PAD_TUNE_CMDRRDLY, + host->hs200_cmd_int_delay); sdr_clr_bits(host->base + MSDC_IOCON, MSDC_IOCON_RSPL); for (i = 0 ; i < PAD_DELAY_MAX; i++) { sdr_set_field(host->base + MSDC_PAD_TUNE, MSDC_PAD_TUNE_CMDRDLY, i); - mmc_send_tuning(mmc, opcode, &cmd_err); - if (!cmd_err) - rise_delay |= (1 << i); + /* + * Using the same parameters, it may sometimes pass the test, + * but sometimes it may fail. To make sure the parameters are + * more stable, we test each set of parameters 3 times. + */ + for (j = 0; j < 3; j++) { + mmc_send_tuning(mmc, opcode, &cmd_err); + if (!cmd_err) { + rise_delay |= (1 << i); + } else { + rise_delay &= ~(1 << i); + break; + } + } } final_rise_delay = get_best_delay(host, rise_delay); /* if rising edge has enough margin, then do not scan falling edge */ - if (final_rise_delay.maxlen >= 10 || - (final_rise_delay.start == 0 && final_rise_delay.maxlen >= 4)) + if (final_rise_delay.maxlen >= 12 && final_rise_delay.start < 4) goto skip_fall; sdr_set_bits(host->base + MSDC_IOCON, MSDC_IOCON_RSPL); for (i = 0; i < PAD_DELAY_MAX; i++) { sdr_set_field(host->base + MSDC_PAD_TUNE, MSDC_PAD_TUNE_CMDRDLY, i); - mmc_send_tuning(mmc, opcode, &cmd_err); - if (!cmd_err) - fall_delay |= (1 << i); + /* + * Using the same parameters, it may sometimes pass the test, + * but sometimes it may fail. To make sure the parameters are + * more stable, we test each set of parameters 3 times. + */ + for (j = 0; j < 3; j++) { + mmc_send_tuning(mmc, opcode, &cmd_err); + if (!cmd_err) { + fall_delay |= (1 << i); + } else { + fall_delay &= ~(1 << i); + break; + }; + } } final_fall_delay = get_best_delay(host, fall_delay); skip_fall: final_maxlen = max(final_rise_delay.maxlen, final_fall_delay.maxlen); + if (final_fall_delay.maxlen >= 12 && final_fall_delay.start < 4) + final_maxlen = final_fall_delay.maxlen; if (final_maxlen == final_rise_delay.maxlen) { sdr_clr_bits(host->base + MSDC_IOCON, MSDC_IOCON_RSPL); sdr_set_field(host->base + MSDC_PAD_TUNE, MSDC_PAD_TUNE_CMDRDLY, @@ -1366,7 +1414,68 @@ static int msdc_tune_response(struct mmc_host *mmc, u32 opcode) final_fall_delay.final_phase); final_delay = final_fall_delay.final_phase; } + if (host->hs200_cmd_int_delay) + goto skip_internal; + for (i = 0; i < PAD_DELAY_MAX; i++) { + sdr_set_field(host->base + MSDC_PAD_TUNE, + MSDC_PAD_TUNE_CMDRRDLY, i); + mmc_send_tuning(mmc, opcode, &cmd_err); + if (!cmd_err) + internal_delay |= (1 << i); + } + dev_dbg(host->dev, "Final internal delay: 0x%x\n", internal_delay); + internal_delay_phase = get_best_delay(host, internal_delay); + sdr_set_field(host->base + MSDC_PAD_TUNE, MSDC_PAD_TUNE_CMDRRDLY, + internal_delay_phase.final_phase); +skip_internal: + dev_dbg(host->dev, "Final cmd pad delay: %x\n", final_delay); + return final_delay == 0xff ? -EIO : 0; +} + +static int hs400_tune_response(struct mmc_host *mmc, u32 opcode) +{ + struct msdc_host *host = mmc_priv(mmc); + u32 cmd_delay = 0; + struct msdc_delay_phase final_cmd_delay = { 0,}; + u8 final_delay; + int cmd_err; + int i, j; + + /* select EMMC50 PAD CMD tune */ + sdr_set_bits(host->base + PAD_CMD_TUNE, BIT(0)); + + if (mmc->ios.timing == MMC_TIMING_MMC_HS200 || + mmc->ios.timing == MMC_TIMING_UHS_SDR104) + sdr_set_field(host->base + MSDC_PAD_TUNE, + MSDC_PAD_TUNE_CMDRRDLY, + host->hs200_cmd_int_delay); + sdr_set_field(host->base + MSDC_IOCON, MSDC_IOCON_RSPL, + host->hs400_cmd_resp_sel); + for (i = 0 ; i < PAD_DELAY_MAX; i++) { + sdr_set_field(host->base + PAD_CMD_TUNE, + PAD_CMD_TUNE_RX_DLY3, i); + /* + * Using the same parameters, it may sometimes pass the test, + * but sometimes it may fail. To make sure the parameters are + * more stable, we test each set of parameters 3 times. + */ + for (j = 0; j < 3; j++) { + mmc_send_tuning(mmc, opcode, &cmd_err); + if (!cmd_err) { + cmd_delay |= (1 << i); + } else { + cmd_delay &= ~(1 << i); + break; + } + } + } + final_cmd_delay = get_best_delay(host, cmd_delay); + sdr_set_field(host->base + PAD_CMD_TUNE, PAD_CMD_TUNE_RX_DLY3, + final_cmd_delay.final_phase); + final_delay = final_cmd_delay.final_phase; + + dev_dbg(host->dev, "Final cmd pad delay: %x\n", final_delay); return final_delay == 0xff ? -EIO : 0; } @@ -1389,7 +1498,7 @@ static int msdc_tune_data(struct mmc_host *mmc, u32 opcode) } final_rise_delay = get_best_delay(host, rise_delay); /* if rising edge has enough margin, then do not scan falling edge */ - if (final_rise_delay.maxlen >= 10 || + if (final_rise_delay.maxlen >= 12 || (final_rise_delay.start == 0 && final_rise_delay.maxlen >= 4)) goto skip_fall; @@ -1422,6 +1531,7 @@ static int msdc_tune_data(struct mmc_host *mmc, u32 opcode) final_delay = final_fall_delay.final_phase; } + dev_dbg(host->dev, "Final data pad delay: %x\n", final_delay); return final_delay == 0xff ? -EIO : 0; } @@ -1430,7 +1540,10 @@ static int msdc_execute_tuning(struct mmc_host *mmc, u32 opcode) struct msdc_host *host = mmc_priv(mmc); int ret; - ret = msdc_tune_response(mmc, opcode); + if (host->hs400_mode) + ret = hs400_tune_response(mmc, opcode); + else + ret = msdc_tune_response(mmc, opcode); if (ret == -EIO) { dev_err(host->dev, "Tune response fail!\n"); return ret; @@ -1443,6 +1556,7 @@ static int msdc_execute_tuning(struct mmc_host *mmc, u32 opcode) host->saved_tune_para.iocon = readl(host->base + MSDC_IOCON); host->saved_tune_para.pad_tune = readl(host->base + MSDC_PAD_TUNE); + host->saved_tune_para.pad_cmd_tune = readl(host->base + PAD_CMD_TUNE); return ret; } @@ -1477,6 +1591,30 @@ static void msdc_hw_reset(struct mmc_host *mmc) .hw_reset = msdc_hw_reset, }; +static void msdc_of_property_parse(struct platform_device *pdev, + struct msdc_host *host) +{ + if (!of_property_read_u32(pdev->dev.of_node, "hs400-ds-delay", + &host->hs400_ds_delay)) + dev_dbg(&pdev->dev, "hs400-ds-delay: %x\n", + host->hs400_ds_delay); + + if (!of_property_read_u32(pdev->dev.of_node, "mtk-hs200-cmd-int-delay", + &host->hs200_cmd_int_delay)) + dev_dbg(&pdev->dev, "host->hs200-cmd-int-delay: %x\n", + host->hs200_cmd_int_delay); + + if (!of_property_read_u32(pdev->dev.of_node, "mtk-hs400-cmd-int-delay", + &host->hs400_cmd_int_delay)) + dev_dbg(&pdev->dev, "host->hs400-cmd-int-delay: %x\n", + host->hs400_cmd_int_delay); + + if (!of_property_read_u32(pdev->dev.of_node, "mtk-hs400-cmd-resp-sel", + &host->hs400_cmd_resp_sel)) + dev_dbg(&pdev->dev, "host->hs200_cmd-resp-sel: %x\n", + host->hs400_cmd_resp_sel); +} + static int msdc_drv_probe(struct platform_device *pdev) { struct mmc_host *mmc; @@ -1497,7 +1635,6 @@ static int msdc_drv_probe(struct platform_device *pdev) ret = mmc_of_parse(mmc); if (ret) goto host_free; - res = platform_get_resource(pdev, IORESOURCE_MEM, 0); host->base = devm_ioremap_resource(&pdev->dev, res); if (IS_ERR(host->base)) { @@ -1548,10 +1685,7 @@ static int msdc_drv_probe(struct platform_device *pdev) goto host_free; } - if (!of_property_read_u32(pdev->dev.of_node, "hs400-ds-delay", - &host->hs400_ds_delay)) - dev_dbg(&pdev->dev, "hs400-ds-delay: %x\n", - host->hs400_ds_delay); + msdc_of_property_parse(pdev, host); host->dev = &pdev->dev; host->mmc = mmc; @@ -1663,6 +1797,7 @@ static void msdc_save_reg(struct msdc_host *host) host->save_para.patch_bit0 = readl(host->base + MSDC_PATCH_BIT); host->save_para.patch_bit1 = readl(host->base + MSDC_PATCH_BIT1); host->save_para.pad_ds_tune = readl(host->base + PAD_DS_TUNE); + host->save_para.pad_cmd_tune = readl(host->base + PAD_CMD_TUNE); host->save_para.emmc50_cfg0 = readl(host->base + EMMC50_CFG0); } @@ -1675,6 +1810,7 @@ static void msdc_restore_reg(struct msdc_host *host) writel(host->save_para.patch_bit0, host->base + MSDC_PATCH_BIT); writel(host->save_para.patch_bit1, host->base + MSDC_PATCH_BIT1); writel(host->save_para.pad_ds_tune, host->base + PAD_DS_TUNE); + writel(host->save_para.pad_cmd_tune, host->base + PAD_CMD_TUNE); writel(host->save_para.emmc50_cfg0, host->base + EMMC50_CFG0); } -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH] mmc: mediatek: fix semicolon.cocci warnings 2017-01-19 10:19 ` [PATCH v3 3/3] mmc: mediatek: Use data tune for CMD line tune Yong Mao @ 2017-01-20 4:49 ` kbuild test robot 2017-01-20 4:49 ` [PATCH v3 3/3] mmc: mediatek: Use data tune for CMD line tune kbuild test robot 2017-01-20 8:22 ` Ulf Hansson 2 siblings, 0 replies; 11+ messages in thread From: kbuild test robot @ 2017-01-20 4:49 UTC (permalink / raw) To: Yong Mao Cc: kbuild-all, Ulf Hansson, Linus Walleij, Chaotian Jing, yong mao, Eddie Huang, Chunfeng Yun, linux-mmc, srv_heupstream, linux-mediatek, linux-kernel drivers/mmc/host/mtk-sd.c:1400:4-5: Unneeded semicolon Remove unneeded semicolon. Generated by: scripts/coccinelle/misc/semicolon.cocci CC: yong mao <yong.mao@mediatek.com> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com> --- mtk-sd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- a/drivers/mmc/host/mtk-sd.c +++ b/drivers/mmc/host/mtk-sd.c @@ -1397,7 +1397,7 @@ static int msdc_tune_response(struct mmc } else { fall_delay &= ~(1 << i); break; - }; + } } } final_fall_delay = get_best_delay(host, fall_delay); ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v3 3/3] mmc: mediatek: Use data tune for CMD line tune 2017-01-19 10:19 ` [PATCH v3 3/3] mmc: mediatek: Use data tune for CMD line tune Yong Mao 2017-01-20 4:49 ` [PATCH] mmc: mediatek: fix semicolon.cocci warnings kbuild test robot @ 2017-01-20 4:49 ` kbuild test robot 2017-01-20 8:22 ` Ulf Hansson 2 siblings, 0 replies; 11+ messages in thread From: kbuild test robot @ 2017-01-20 4:49 UTC (permalink / raw) To: Yong Mao Cc: kbuild-all, Ulf Hansson, Linus Walleij, Chaotian Jing, yong mao, Eddie Huang, Chunfeng Yun, linux-mmc, srv_heupstream, linux-mediatek, linux-kernel Hi yong, [auto build test WARNING on robh/for-next] [also build test WARNING on v4.10-rc4 next-20170119] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Yong-Mao/mmc-mediatek-Use-data-tune-for-CMD-line-tune/20170120-103149 base: https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next coccinelle warnings: (new ones prefixed by >>) >> drivers/mmc/host/mtk-sd.c:1400:4-5: Unneeded semicolon Please review and possibly fold the followup patch. --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v3 3/3] mmc: mediatek: Use data tune for CMD line tune 2017-01-19 10:19 ` [PATCH v3 3/3] mmc: mediatek: Use data tune for CMD line tune Yong Mao 2017-01-20 4:49 ` [PATCH] mmc: mediatek: fix semicolon.cocci warnings kbuild test robot 2017-01-20 4:49 ` [PATCH v3 3/3] mmc: mediatek: Use data tune for CMD line tune kbuild test robot @ 2017-01-20 8:22 ` Ulf Hansson 2 siblings, 0 replies; 11+ messages in thread From: Ulf Hansson @ 2017-01-20 8:22 UTC (permalink / raw) To: Yong Mao Cc: Linus Walleij, Chaotian Jing, Eddie Huang, Chunfeng Yun, linux-mmc, srv_heupstream, linux-mediatek, linux-kernel [...] > struct msdc_delay_phase { > @@ -331,6 +339,9 @@ struct msdc_host { > unsigned char timing; > bool vqmmc_enabled; > u32 hs400_ds_delay; > + u32 hs200_cmd_int_delay; /* cmd internal delay for HS200/SDR104 */ > + u32 hs400_cmd_int_delay; /* cmd internal delay for HS400 */ > + u32 hs400_cmd_resp_sel; /* cmd response sample selection for HS400 */ When you converted the DT binding, this should become bool instead. > bool hs400_mode; /* current eMMC will run at hs400 mode */ > struct msdc_save_para save_para; /* used when gate HCLK */ > struct msdc_tune_para def_tune_para; /* default tune setting */ [...] > +static void msdc_of_property_parse(struct platform_device *pdev, > + struct msdc_host *host) > +{ > + if (!of_property_read_u32(pdev->dev.of_node, "hs400-ds-delay", > + &host->hs400_ds_delay)) > + dev_dbg(&pdev->dev, "hs400-ds-delay: %x\n", > + host->hs400_ds_delay); Writing a dev_dbg for each parsed DT property, seems a bit silly. Can you please remove that. > + > + if (!of_property_read_u32(pdev->dev.of_node, "mtk-hs200-cmd-int-delay", > + &host->hs200_cmd_int_delay)) > + dev_dbg(&pdev->dev, "host->hs200-cmd-int-delay: %x\n", > + host->hs200_cmd_int_delay); > + > + if (!of_property_read_u32(pdev->dev.of_node, "mtk-hs400-cmd-int-delay", > + &host->hs400_cmd_int_delay)) > + dev_dbg(&pdev->dev, "host->hs400-cmd-int-delay: %x\n", > + host->hs400_cmd_int_delay); > + > + if (!of_property_read_u32(pdev->dev.of_node, "mtk-hs400-cmd-resp-sel", > + &host->hs400_cmd_resp_sel)) > + dev_dbg(&pdev->dev, "host->hs200_cmd-resp-sel: %x\n", > + host->hs400_cmd_resp_sel); > +} > + > static int msdc_drv_probe(struct platform_device *pdev) > { > struct mmc_host *mmc; > @@ -1497,7 +1635,6 @@ static int msdc_drv_probe(struct platform_device *pdev) > ret = mmc_of_parse(mmc); > if (ret) > goto host_free; > - Whitespace. > res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > host->base = devm_ioremap_resource(&pdev->dev, res); > if (IS_ERR(host->base)) { > @@ -1548,10 +1685,7 @@ static int msdc_drv_probe(struct platform_device *pdev) > goto host_free; > } > > - if (!of_property_read_u32(pdev->dev.of_node, "hs400-ds-delay", > - &host->hs400_ds_delay)) > - dev_dbg(&pdev->dev, "hs400-ds-delay: %x\n", > - host->hs400_ds_delay); > + msdc_of_property_parse(pdev, host); > > host->dev = &pdev->dev; > host->mmc = mmc; > @@ -1663,6 +1797,7 @@ static void msdc_save_reg(struct msdc_host *host) > host->save_para.patch_bit0 = readl(host->base + MSDC_PATCH_BIT); > host->save_para.patch_bit1 = readl(host->base + MSDC_PATCH_BIT1); > host->save_para.pad_ds_tune = readl(host->base + PAD_DS_TUNE); > + host->save_para.pad_cmd_tune = readl(host->base + PAD_CMD_TUNE); > host->save_para.emmc50_cfg0 = readl(host->base + EMMC50_CFG0); > } > > @@ -1675,6 +1810,7 @@ static void msdc_restore_reg(struct msdc_host *host) > writel(host->save_para.patch_bit0, host->base + MSDC_PATCH_BIT); > writel(host->save_para.patch_bit1, host->base + MSDC_PATCH_BIT1); > writel(host->save_para.pad_ds_tune, host->base + PAD_DS_TUNE); > + writel(host->save_para.pad_cmd_tune, host->base + PAD_CMD_TUNE); > writel(host->save_para.emmc50_cfg0, host->base + EMMC50_CFG0); > } > > -- > 1.7.9.5 > Please also fold in the change suggested/reported by the kbuild test robot. Besides the minor things, this looks good to me! Kind regards Uffe ^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2017-01-24 10:36 UTC | newest] Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2017-01-19 10:19 [PATCH v3 0/3] mmc: mediatek: Use data tune for CMD line tune Yong Mao 2017-01-19 10:19 ` [PATCH v3 1/3] mmc: dt-bindings: update Mediatek MMC bindings Yong Mao 2017-01-20 7:43 ` Daniel Kurtz 2017-01-20 7:52 ` Ulf Hansson 2017-01-19 10:19 ` [PATCH v3 2/3] dts: mediatek: configure some fixed mmc parameters Yong Mao 2017-01-20 7:59 ` Ulf Hansson 2017-01-24 10:36 ` Matthias Brugger 2017-01-19 10:19 ` [PATCH v3 3/3] mmc: mediatek: Use data tune for CMD line tune Yong Mao 2017-01-20 4:49 ` [PATCH] mmc: mediatek: fix semicolon.cocci warnings kbuild test robot 2017-01-20 4:49 ` [PATCH v3 3/3] mmc: mediatek: Use data tune for CMD line tune kbuild test robot 2017-01-20 8:22 ` Ulf Hansson
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).