mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Pierre Ossman <pierre@ossman.eu>
To: Tobias Diedrich <ranma+kernel@tdiedrich.de>
Cc: SDHCI development <sdhci-devel@lists.ossman.eu>,
	linux-kernel@vger.kernel.org
Subject: Re: QUIRK_FORCE_HISPD (was: Re: Ricoh R5C822 and QUIRK_FORCE_DMA)
Date: Sat, 13 Jun 2009 12:53:22 +0200	[thread overview]
Message-ID: <20090613125322.750a1180@mjolnir.ossman.eu> (raw)
In-Reply-To: <20090609224145.GA11001@yamamaya.is-a-geek.org>

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

On Wed, 10 Jun 2009 00:41:45 +0200
Tobias Diedrich <ranma+kernel@tdiedrich.de> wrote:

> Pierre Ossman wrote:
> > Might be a system problem so it's Lenovo that disabled it. You're the
> > first report I've seen on this since the DMA logic was reworked so that
> > DMA didn't have to be forced for most cases.
> 
> FWIW, so far I haven't had any errors due to dma being enabled.
> 
> I tried changing the CAPS using setpci, but they seem to be
> read-only (unless theres are write-protect bit somewhere in the
> config space).

They tend to be write-protected, yes. Usually there are some other
registers you need to fiddle with first.

> On a related note, I had a little time to play a bit more with my
> R5C822 and found that I can also force-enable HISPD mode, which
> boost performance further.
> 

This seems odd. What frequency is the controller reporting?

It could be that they originally wanted the controller to be
high-speed, but that they had stability problems and turned it off.
There is at least one other controller that went through that scenario.

> I tried improving the write speed by adding support for
> APP_CMD_SET_WR_BLOCK_COUNT to the block driver, but did not see any
> improvement.
> It's still in the patch, in case you are interested.

I've tried this before, and I got the same results you did.

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: 198 bytes --]

  reply	other threads:[~2009-06-13 10:53 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-30 22:45 Ricoh R5C822 and QUIRK_FORCE_DMA Tobias Diedrich
2009-05-13 17:59 ` Pierre Ossman
2009-05-13 18:09   ` [Sdhci-devel] " Garrett D'Amore
2009-06-09 22:41   ` QUIRK_FORCE_HISPD (was: Re: Ricoh R5C822 and QUIRK_FORCE_DMA) Tobias Diedrich
2009-06-13 10:53     ` Pierre Ossman [this message]
2009-08-01  5:17       ` Tobias Diedrich
2009-09-06 13:30         ` 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=20090613125322.750a1180@mjolnir.ossman.eu \
    --to=pierre@ossman.eu \
    --cc=linux-kernel@vger.kernel.org \
    --cc=ranma+kernel@tdiedrich.de \
    --cc=sdhci-devel@lists.ossman.eu \
    /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®