mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Wolfgang Mües" <wolfgang.mues@auerswald.de>
To: Pierre Ossman <pierre@ossman.eu>
Cc: "Andrew Morton" <akpm@linux-foundation.org>,
	"Matt Fleming" <matt@console-pimps.org>,
	"David Brownell" <dbrownell@users.sourceforge.net>,
	"Mike Frysinger" <vapier.adi@gmail.com>,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH] mmc_spi: fail gracefully if host or card do not support the switch command
Date: Mon, 15 Jun 2009 10:36:05 +0200	[thread overview]
Message-ID: <200906151036.05884.wolfgang.mues@auerswald.de> (raw)
In-Reply-To: <20090613130829.49216355@mjolnir.ossman.eu>

Hello Pierre,

Am Samstag, 13. Juni 2009 schrieb Pierre Ossman:
> > The switch command (typically speed switching from 25 to 50 MHz clock)
> > can be rejected by _both_ the host controller/driver and by the card.
> > Elder revisions of the SD card do not support speed switching.
> > Fail gracefully if the reject is comming from the card.

> Have you ever seen this in practice?

Yes, I have. An old 1 GByte SD card.

> For SD there are several checks 
> before the switch command is actually issued.

But there are two cases not covered by these checks:

a) the switch command is supported, but the card is not capable of switching 
to 50 MHz.

b) the card rejects the switch command in SPI mode, because it is not 
mandatory here.

So yes, the checks can be improved. But only extending the error codes was a 
simple, fast and minimal solution.

regards

i. A. Wolfgang Mües
-- 
Auerswald GmbH & Co. KG
Hardware Development
Telefon: +49 (0)5306 9219 0
Telefax: +49 (0)5306 9219 94 
E-Mail: Wolfgang.Mues@Auerswald.de
Web: http://www.auerswald.de
 
--------------------------------------------------------------
Auerswald GmbH & Co. KG, Vor den Grashöfen 1, 38162 Cremlingen
Registriert beim AG Braunschweig HRA 13289
p.h.G Auerswald Geschäftsführungsges. mbH
Registriert beim AG Braunschweig HRB 7463
Geschäftsführer: Dipl-Ing. Gerhard Auerswald

  reply	other threads:[~2009-06-15  8:36 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-11 15:13 Wolfgang Mües
2009-06-13 11:08 ` Pierre Ossman
2009-06-15  8:36   ` Wolfgang Mües [this message]
2009-06-15 19:24     ` Pierre Ossman
2009-09-09 13:02 Wolfgang Mües

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=200906151036.05884.wolfgang.mues@auerswald.de \
    --to=wolfgang.mues@auerswald.de \
    --cc=akpm@linux-foundation.org \
    --cc=dbrownell@users.sourceforge.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=matt@console-pimps.org \
    --cc=pierre@ossman.eu \
    --cc=vapier.adi@gmail.com \
    /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®