devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jaehoon Chung <jh80.chung-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
To: Vivek Gautam <gautamvivek1987-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: Alim Akhtar <alim.akhtar-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>,
	"linux-mmc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-mmc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	Chris Ball <chris-OsFVWbfNK3isTnJN9+BGXg@public.gmane.org>,
	Ulf Hansson <ulf.hansson-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	Seungwon Jeon <tgih.jun-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>,
	Doug Anderson <dianders-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>,
	Alim Akhtar <alim.akhtar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Kukjin Kim <kgene-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
	"linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org"
	<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>,
	"devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"linux-samsung-soc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-samsung-soc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	Abhilash Kesavan
	<a.kesavan-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
Subject: Re: [PATCH v5 0/2] mmc: dw_mmc: exynos: Add HS400 support
Date: Fri, 06 Feb 2015 15:03:41 +0900	[thread overview]
Message-ID: <54D4593D.3000007@samsung.com> (raw)
In-Reply-To: <CAFp+6iEGCEiGf5WooHJyh=cnJF_c7j=n_YhHLowfwjpLiXUPiQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>

Hi, Vivek.

On 02/06/2015 02:52 PM, Vivek Gautam wrote:
> Hi Jaehoon,
> 
> 
> On Fri, Feb 6, 2015 at 6:48 AM, Jaehoon Chung <jh80.chung-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org> wrote:
>> Hi, Alim.
>>
>> On my board, this patch isn't working fine.
>> So when i complete to test on my board(exynos5), i will reply with comments.
>> Sorry for late testing.
> 
> I tested this series on linux-next, on Exynos5800-peach-pi board, and HS400
> seems to be working fine.
> I can see the card getting detected as HS400, and then i ran iozone as well.
> The iozone numbers are also as required.
> 
> On peach-pit board too, the HS200 mode seems to be working good.
> 
> What are the observations you saw on your board ?

Thanks for testing on your board. I have exynos5422 and exynos5433.(One is eMMC5.0, other is eMMC5.1)
I want to see that it's working fine on my board.
I should miss something..so i will check more detail after complete my other job, or on this weekend.
(It failed for hs400 tuning sequence..I needs to check that it has the board dependency)

Best Regards,
Jaehoon Chung

> 
>>
>> Best Regards,
>> Jaehoon Chung
>>
>> On 01/29/2015 11:41 AM, Alim Akhtar wrote:
>>> This adds HS400 mode support for exynos dw_mmc host controller.
>>>
>>> Currently tested on Exynos5800-peach-pi and Exyons7 platform for HS400 mode.
>>> Tested HS200 mode with this series applied, HS200 still works.
>>>
>>> Appreciate testing on other exynos5/7 platform which supports emmc5.0
>>>
>>> Changes in V5:
>>>       * Enable HS400 on Exynos5800-peach-pi boards
>>>       * Addressed other review comments from Jaehoon Chung
>>>
>>> Changes in V4:
>>>       * drop the idea of changing existing binding for ciu_div as per [1]
>>>         * addressed comments from Jaehoon Chung [2]
>>>
>>> [1] http://www.spinics.net/lists/linux-samsung-soc/msg40923.html
>>> [2] http://www.spinics.net/lists/devicetree/msg64373.html
>>>
>>> Changes in V3:
>>>       rebased on ulf's next (commit: 607b448 mmc: core: Make tuning block patterns static)
>>>
>>> Seungwon Jeon (2):
>>>   mmc: dw_mmc: exynos: Support eMMC's HS400 mode
>>>   ARM: dts: Add HS400 support for exynos5420 and exynos5800
>>>
>>>  .../devicetree/bindings/mmc/exynos-dw-mshc.txt     |    7 +
>>>  arch/arm/boot/dts/exynos5420-peach-pit.dts         |    4 +-
>>>  arch/arm/boot/dts/exynos5420-pinctrl.dtsi          |    7 +
>>>  arch/arm/boot/dts/exynos5420-smdk5420.dts          |    4 +-
>>>  arch/arm/boot/dts/exynos5800-peach-pi.dts          |    7 +-
>>>  drivers/mmc/host/dw_mmc-exynos.c                   |  185 ++++++++++++++++----
>>>  drivers/mmc/host/dw_mmc-exynos.h                   |   19 +-
>>>  drivers/mmc/host/dw_mmc.c                          |   16 +-
>>>  drivers/mmc/host/dw_mmc.h                          |    2 +
>>>  9 files changed, 213 insertions(+), 38 deletions(-)
>>>
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
>> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> 
> 

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  parent reply	other threads:[~2015-02-06  6:03 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-01-29  2:41 [PATCH v5 0/2] mmc: dw_mmc: exynos: Add HS400 support Alim Akhtar
2015-01-29  2:41 ` [PATCH V5 1/2] mmc: dw_mmc: exynos: Support eMMC's HS400 mode Alim Akhtar
2015-02-25  6:34   ` Jaehoon Chung
2015-02-25  8:12     ` Alim Akhtar
2015-02-25 14:38   ` Rob Herring
2015-02-26  1:22     ` Jaehoon Chung
2015-01-29  2:41 ` [PATCH V5 2/2] ARM: dts: Add HS400 support for exynos5420 and exynos5800 Alim Akhtar
2015-02-25  6:35   ` Jaehoon Chung
2015-03-15 11:28     ` Alim Akhtar
2015-03-17  2:12       ` Kukjin Kim
2015-03-17 23:26         ` Alim Akhtar
2015-02-06  1:18 ` [PATCH v5 0/2] mmc: dw_mmc: exynos: Add HS400 support Jaehoon Chung
     [not found]   ` <54D41654.8030702-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2015-02-06  5:52     ` Vivek Gautam
     [not found]       ` <CAFp+6iEGCEiGf5WooHJyh=cnJF_c7j=n_YhHLowfwjpLiXUPiQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-02-06  6:03         ` Jaehoon Chung [this message]
2015-02-06  6:07           ` Vivek Gautam

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=54D4593D.3000007@samsung.com \
    --to=jh80.chung-sze3o3uu22jbdgjk7y7tuq@public.gmane.org \
    --cc=a.kesavan-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org \
    --cc=alim.akhtar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=alim.akhtar-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org \
    --cc=chris-OsFVWbfNK3isTnJN9+BGXg@public.gmane.org \
    --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=dianders-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org \
    --cc=gautamvivek1987-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=kgene-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=linux-mmc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-samsung-soc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=tgih.jun-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org \
    --cc=ulf.hansson-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
    /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;
as well as URLs for NNTP newsgroup(s).