mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Pierre Ossman <drzeus@drzeus.cx>
To: Adrian Hunter <ext-adrian.hunter@nokia.com>
Cc: LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 1/2] mmc_block: print better data error message after timeout
Date: Sun, 21 Dec 2008 15:22:49 +0100	[thread overview]
Message-ID: <20081221152249.2085fe53@mjolnir.drzeus.cx> (raw)
In-Reply-To: <4937DCAF.3010806@nokia.com>

[-- Attachment #1: Type: text/plain, Size: 1348 bytes --]

On Thu, 04 Dec 2008 15:35:43 +0200
Adrian Hunter <ext-adrian.hunter@nokia.com> wrote:

> Pierre Ossman wrote:
> > On Wed, 29 Oct 2008 12:29:48 +0200
> > Adrian Hunter <ext-adrian.hunter@nokia.com> wrote:
> > 
> >> Waiting for the card to become ready after an error is also not included,
> >> as it is not in the standards and is a separate issue anyway.
> > 
> > What do you mean?
> 
> You had suggested
> 
> 	while (card not ready)
> 		status |= send_status();
> 
> and I didn't do it, because of the reasons given above.
> 

What confused me was that you sait "it is not in the standards", but
perhaps you were just referring to the read case? In the write case the
specs are clear that errors will be done during execution, which can be
delayed for writes.

> From: Adrian Hunter <ext-adrian.hunter@nokia.com>
> Date: Thu, 16 Oct 2008 12:55:25 +0300
> Subject: [PATCH] mmc_block: print better error messages
> 
> Add command response and card status to error
> messages.
> 
> Signed-off-by: Adrian Hunter <ext-adrian.hunter@nokia.com>
> ---

Queued. Thanks for all the work. :)

Rgds
-- 
     -- Pierre Ossman

  WARNING: This correspondence is being monitored by the
  Swedish government. Make sure your server uses encryption
  for SMTP traffic and consider using PGP for end-to-end
  encryption.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 197 bytes --]

      reply	other threads:[~2008-12-21 14:23 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-16 13:26 Adrian Hunter
2008-10-26 11:11 ` Pierre Ossman
2008-10-27 15:33   ` Adrian Hunter
2008-10-29 10:29   ` Adrian Hunter
2008-11-10  8:20     ` Adrian Hunter
2008-11-10  9:53       ` Pierre Ossman
2008-11-30 18:48     ` Pierre Ossman
2008-12-04 13:35       ` Adrian Hunter
2008-12-21 14:22         ` Pierre Ossman [this message]

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=20081221152249.2085fe53@mjolnir.drzeus.cx \
    --to=drzeus@drzeus.cx \
    --cc=ext-adrian.hunter@nokia.com \
    --cc=linux-kernel@vger.kernel.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®