mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Madhusudhan c" <cr.madhusudhan@gmail.com>
To: "Pierre Ossman" <drzeus@drzeus.cx>
Cc: linux-kernel@vger.kernel.org, philipl@overt.org
Subject: Re: MMCv4 support (8-bit support missing)
Date: Fri, 20 Apr 2007 16:51:24 +0530	[thread overview]
Message-ID: <ff1613620704200421w1e1aeea5r5acc3a7c0c548b96@mail.gmail.com> (raw)
In-Reply-To: <462797EE.2050808@drzeus.cx>

Hi Perre,

> Until 8-bit SD shows up, at which point things go to hell again.
>
> You need to present a valid case for why we need this bus testing stuff. The
> spec never requires it, and neither does any cards. So this is just crud that we
> know causes problems and doesn't give us anything back.
Okay. Let me try to explain a scenario where it is required.

Suppose a host controller is capable of suporting 8-bit and it tells
the core that it can support 8-bit. Now the card that is plugged in
might or might not support 8-bit based on the type of the card. There
is no field in the ext_csd which will tell you what bus width the card
can support.

So you need first try to setup 8-bit and send a patteren on the data
lines and read it back. If you get back the expected data then you are
good to go in 8-bit. Else, shift to 4-bit and try again with the data
patteren to see if four bit works. This is how the patch is
implemented. Also the spec says that you need to perform bus test
procedure before sending the switch command to set the bus width.

By doing this you can figure out whether the card supports 8-bit or
not. And this is the only way to support 8-bit cards, which are widely
available in the market today.

Also if the host controler says it can only support 4-bit then no need
of bus testing procedure.Ignoring the bus test procedure does not
affect you to functionally support 4-bit but there is no way you can
support 8-bit cards without it correctly.

Regards,
Madhu

On 4/19/07, Pierre Ossman <drzeus@drzeus.cx> wrote:
> Madhusudhan c wrote:
> >
> > The bus test procedure from this patch can be adopted to the MMCv4
> > support in the MMC core with small changes to do bus testing procedure
> > only if the host sets the capability to support 8-bit. That way we
> > dont break the legacy code. What do you think?
> >
>
> Until 8-bit SD shows up, at which point things go to hell again.
>
> You need to present a valid case for why we need this bus testing stuff. The
> spec never requires it, and neither does any cards. So this is just crud that we
> know causes problems and doesn't give us anything back.
>
> Rgds
> Pierre
>
>
>
>

  reply	other threads:[~2007-04-20 11:21 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-17  6:59 Madhusudhan c
2007-04-17 15:15 ` Philip Langdale
2007-04-18  8:46 ` Pierre Ossman
2007-04-18 11:53   ` Madhusudhan c
2007-04-18 14:21     ` Pierre Ossman
2007-04-19 14:04       ` Madhusudhan c
2007-04-19 16:25         ` Pierre Ossman
2007-04-20 11:21           ` Madhusudhan c [this message]
2007-04-20 16:12             ` Philip Langdale
2007-04-24  5:03             ` Pierre Ossman
2007-04-26  3:34               ` Madhusudhan c
2007-04-27  6:20                 ` Pierre Ossman

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=ff1613620704200421w1e1aeea5r5acc3a7c0c548b96@mail.gmail.com \
    --to=cr.madhusudhan@gmail.com \
    --cc=drzeus@drzeus.cx \
    --cc=linux-kernel@vger.kernel.org \
    --cc=philipl@overt.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

all inboxes | Powered by JetHome®