Hi Ziyuan, On 05/16/2017 10:15 AM, Ziyuan wrote: > hi Simon & Jaehoon, > > On 05/16/2017 08:18 AM, Simon Glass wrote: >> Hi Ziyuan, >> >> On 15 May 2017 at 00:06, Ziyuan Xu <xzy...@rock-chips.com> wrote: >>> The original implementation select HS timing by default, add available >>> type selection for higher speed mode compatibility, such as hs200, >>> hs400, hs400es. >>> >>> By the way, we assume that card run at 1.8V or 1.2V I/O when its timing >>> is ddr52/hs200/hs400(es). >>> >>> Signed-off-by: Ziyuan Xu <xzy...@rock-chips.com> >>> --- >>> >>> drivers/mmc/mmc.c | 59 >>> ++++++++++++++++++++++++++++++++++++++++++++++++++++++- >>> include/mmc.h | 16 +++++++++++++++ >>> 2 files changed, 74 insertions(+), 1 deletion(-) >>> >> Is there a cover letter for this series, please? > > This patchset is used for hs200/hs400/ddr52 mode of eMMC device, and fixes > some bug for dw_mmc & sdhci controller. > It's only valid in U-Boot stage, we still use 'High Speed' in SPL. > > I tested it on evb-rk3288 board(eMMC 4.5) and evb-rk3388 board(eMMC 5.0). >> >> I just reviewed an MMC series at add higher speed support. I'm not >> sure but I suspect these overlap. > > Ha, I just reviewed Vignesh's patches, it focuses on uhs mode of sd card. It > looks to me. > But some details are not the same as mine. Anyway, what do you think?
I didn't review both yet..After checking, i will share my opinion. how about? > >> >> Regards, >> Simon >> >> >> > > > > > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot