linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: b29396@freescale.com (Dong Aisheng)
To: linux-arm-kernel@lists.infradead.org
Subject: Adding esdhc4 eMMC support on mx6sabresd
Date: Thu, 8 May 2014 18:17:32 +0800	[thread overview]
Message-ID: <20140508101730.GA28787@shlinux1.ap.freescale.net> (raw)
In-Reply-To: <20140508091835.78daafbe@ipc1.ka-ro>

On Thu, May 08, 2014 at 09:18:35AM +0200, Lothar Wa?mann wrote:
> Hi,
> 
> Fabio Estevam wrote:
> > Hi Dong,
> > 
> > I have tested the following patch to add support for the emmc port on
> > mx6qsabresd:
> > 
> > --- a/arch/arm/boot/dts/imx6qdl-sabresd.dtsi
> > +++ b/arch/arm/boot/dts/imx6qdl-sabresd.dtsi
> > @@ -444,6 +444,21 @@
> >                                 MX6QDL_PAD_SD3_DAT7__SD3_DATA7          0x17059
> >                         >;
> >                 };
> > +
> > +               pinctrl_usdhc4: usdhc4grp {
> > +                       fsl,pins = <
> > +                               MX6QDL_PAD_SD4_CMD__SD4_CMD             0x17059
> > +                               MX6QDL_PAD_SD4_CLK__SD4_CLK             0x10059
> > +                               MX6QDL_PAD_SD4_DAT0__SD4_DATA0          0x17059
> > +                               MX6QDL_PAD_SD4_DAT1__SD4_DATA1          0x17059
> > +                               MX6QDL_PAD_SD4_DAT2__SD4_DATA2          0x17059
> > +                               MX6QDL_PAD_SD4_DAT3__SD4_DATA3          0x17059
> > +                               MX6QDL_PAD_SD4_DAT4__SD4_DATA4          0x17059
> > +                               MX6QDL_PAD_SD4_DAT5__SD4_DATA5          0x17059
> > +                               MX6QDL_PAD_SD4_DAT6__SD4_DATA6          0x17059
> > +                               MX6QDL_PAD_SD4_DAT7__SD4_DATA7          0x17059
> [...]
> > 
> > Is there any extra patch I need to apply? I am running linux-next 20140506.
> > 
> I'm using the pad settings 0x070b1 for our i.MX6 module with eMMC.
> At least you should probably drop the HYS setting (0x10000), since
> according to my experience the SD interface works much better with
> hysteresis disabled.
> 

I'm not very sure about HYS bit, but it should not be pad setting issue.
This pad setting was tested ok with FSL internal tree.

I just tested an old Chris/mmc-next tree which is 3.14 rc2 with the top commit is b42b9b.
The eMMC works well on SabreSD boards.
But it failed and got the same issue with latest chris/mmc-next tree.
I noticed there's no changes on sdhci-esdhc-imx.c
Probably some other changes breaks this function.

Fabio,
You may bisect it yourself or i will do it later when i'm a bit free.

Shawn,
I remember this patch should have been upstream through your tree long time ago.
http://lists.infradead.org/pipermail/linux-arm-kernel/2013-October/206446.html
It seems somehow missed.

Regards
Dong Aisheng

> 
> Lothar Wa?mann
> -- 
> ___________________________________________________________
> 
> Ka-Ro electronics GmbH | Pascalstra?e 22 | D - 52076 Aachen
> Phone: +49 2408 1402-0 | Fax: +49 2408 1402-10
> Gesch?ftsf?hrer: Matthias Kaussen
> Handelsregistereintrag: Amtsgericht Aachen, HRB 4996
> 
> www.karo-electronics.de | info at karo-electronics.de
> ___________________________________________________________

  reply	other threads:[~2014-05-08 10:17 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-07 20:14 Adding esdhc4 eMMC support on mx6sabresd Fabio Estevam
2014-05-08  7:18 ` Lothar Waßmann
2014-05-08 10:17   ` Dong Aisheng [this message]
2014-05-08 14:05     ` Fabio Estevam
2014-05-09  3:16     ` Shawn Guo
2014-05-09  2:24       ` Dong Aisheng
     [not found] <CAKVbgXpn43n-8Azjc9f2-RHw51uVAAKkv_dT7b_iXWxNm7q+Qw@mail.gmail.com>
2014-11-18 15:02 ` Fabio Estevam

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=20140508101730.GA28787@shlinux1.ap.freescale.net \
    --to=b29396@freescale.com \
    --cc=linux-arm-kernel@lists.infradead.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).