From: Ziyuan <xzy.xu@rock-chips.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 01/33] mmc: select the available type from host_caps and card_caps
Date: Mon, 5 Jun 2017 08:50:58 +0800 [thread overview]
Message-ID: <5934AAF2.40807@rock-chips.com> (raw)
In-Reply-To: <6af9e904-1f9a-5b41-e300-891857913099@samsung.com>
hi Jaehoon,
On 05/25/2017 09:08 PM, Jaehoon Chung wrote:
> Hi Ziyuan,
>
> On 05/25/2017 05:12 PM, Ziyuan wrote:
>> hi Jaehoon,
>>
>> On 05/16/2017 09:55 AM, Jaehoon Chung wrote:
>>> 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.xu@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.xu@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?
>> *ping*...
>> Did you test this patchset on Samsung platform? As far as I know, Samsung SoCs also make use of dw_mmc and sdhci controllers.
> some details are not the same as yours..but some points are duplicated.
> DWMMC and sdhci controller side are not duplicated..
>
> As i am feeling..your patches are based on Linux kernel code..right?
Yup, the kernel's mmc driver is stabilized and it was tested on so many
various boards.
Simon had review Jean-jacques's patch, but a little glitch need to be
debugged. So I should wait for him, and rebase?
>>>>> Regards,
>>>>> Simon
>>>>>
>>>>>
>>>>>
>>>>
>>>>
>>>>
>>>
>>>
>>
>>
>>
>>
>
>
>
next prev parent reply other threads:[~2017-06-05 0:50 UTC|newest]
Thread overview: 45+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-05-15 6:06 [U-Boot] [PATCH 01/33] mmc: select the available type from host_caps and card_caps Ziyuan Xu
2017-05-15 6:06 ` [U-Boot] [PATCH 02/33] mmc: add set_timing entry for timing selection Ziyuan Xu
2017-05-15 6:06 ` [U-Boot] [PATCH 03/33] mmc: xenon_sdhci: drop redundant timing definitions Ziyuan Xu
2017-05-15 6:06 ` [U-Boot] [PATCH 04/33] mmc: rework high speed mode selection Ziyuan Xu
2017-05-15 6:06 ` [U-Boot] [PATCH 05/33] mmc: sdhci: fix HISPD bit setting Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 06/33] mmc: add card_busy to query card status Ziyuan Xu
2017-05-25 13:02 ` Jaehoon Chung
2017-05-15 6:07 ` [U-Boot] [PATCH 07/33] mmc: dw_mmc: implement card_busy detection Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 08/33] mmc: sdhci: " Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 09/33] mmc: rework mmc_switch for non-send_status scenario Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 10/33] mmc: add support for HS200 mode of eMMC4.5 Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 11/33] mmc: rework ddr mode judgement with timing Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 12/33] mmc: remove tran_speed from struct mmc Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 13/33] cmd: mmc: show the current speed mode Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 14/33] clk: introduce clk_phase get/set function & callback Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 15/33] rockchip: clk: rk3288: fix mmc clock setting Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 16/33] rockchip: clk: rk3288: add support for the clock phase Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 17/33] rockchip: clk: rk3399: fix emmc clock setting Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 18/33] mmc: dw_mmc: add the support for the tuning scheme Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 19/33] mmc: dw_mmc: rockchip: implement tuning with clock phase framework Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 20/33] mmc: dw_mmc: reset controller after data error Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 21/33] mmc: add DDR52 support for eMMC card Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 22/33] mmc: dw_mmc: rockchip: fix data crc error on ddr52 8bit mode Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 23/33] mmc: dw_mmc: fix bus width setting Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 24/33] mmc: sdhci: rockchip: " Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 25/33] mmc: sdhci: update host->clock after clock setting Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 26/33] mmc: sdhci: add support for UHS timing Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 27/33] mmc: sdhci: rename set_clock callback Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 28/33] mmc: sdhci: export sdhci_set_clock() Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 29/33] mmc: sdhci: rockchip: add phy support Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 30/33] mmc: sdhci: add the support for tuning Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 31/33] mmc: add support for HS400 mode of eMMC5.0 Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 32/33] dts: rk3399: change the maximum eMMC clock frequency to 150MHz Ziyuan Xu
2017-05-15 6:07 ` [U-Boot] [PATCH 33/33] SPL: tiny-printf: add "X" modifier Ziyuan Xu
2017-05-16 0:18 ` [U-Boot] [PATCH 01/33] mmc: select the available type from host_caps and card_caps Simon Glass
2017-05-16 1:15 ` Ziyuan
2017-05-16 1:55 ` Jaehoon Chung
2017-05-16 2:22 ` Ziyuan
2017-05-25 8:12 ` Ziyuan
2017-05-25 10:50 ` Jaehoon Chung
2017-05-25 13:08 ` Jaehoon Chung
2017-06-05 0:50 ` Ziyuan [this message]
2017-06-05 1:13 ` Jaehoon Chung
2017-05-17 1:38 ` Simon Glass
-- strict thread matches above, loose matches on Subject: below --
2017-05-15 5:56 Ziyuan Xu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5934AAF2.40807@rock-chips.com \
--to=xzy.xu@rock-chips.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox