All of lore.kernel.org
 help / color / mirror / Atom feed
From: Chris Ball <cjb@laptop.org>
To: Qiang Liu <b32616@freescale.com>
Cc: linux-mmc@vger.kernel.org, leoli@freescale.com, kumar.gala@freescale.com
Subject: Re: [PATCH] SD/MMC: fix the issue of SDHC performance regression
Date: Mon, 07 Nov 2011 08:20:10 -0500	[thread overview]
Message-ID: <m2ipmw133p.fsf@bob.laptop.org> (raw)
In-Reply-To: <1320658589-13143-1-git-send-email-b32616@freescale.com> (Qiang Liu's message of "Mon, 7 Nov 2011 17:36:29 +0800")

Hi,

On Mon, Nov 07 2011, Qiang Liu wrote:
> Low performance of SDHC (half of before) due to its frequency was set to 25MHz,
> but not 50MHz (involved by commit id 013909c4ffd16ded4895528b856fd8782df04dc6,
> add support for query function modes for uhs cards according to Physical Layer
> SPEC V3.01).
>
> Set high speed max frequency according to response status of CMD6, but
> not for SPEC Version. Response of switch command is first consideration
> factor when set SDHC max working frequency. TRAN_SPEED in CSD register
> will be seemed as the second factor.
>
> Signed-off-by: Qiang Liu <qiang.liu@freescale.com>

Have you seen:

http://git.kernel.org/?p=linux/kernel/git/torvalds/linux.git;a=commitdiff;h=f2815f68dabbb373fd1c9f0fd4a609d486697c2b 
("mmc: sd: Handle SD3.0 cards not supporting UHS-I bus speed mode")

which is already in mainline?  I think your patch is identical.

Thanks,

- Chris.
-- 
Chris Ball   <cjb@laptop.org>   <http://printf.net/>
One Laptop Per Child

  parent reply	other threads:[~2011-11-07 13:20 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-07  9:36 [PATCH] SD/MMC: fix the issue of SDHC performance regression Qiang Liu
2011-11-07 10:08 ` Li Yang-R58472
2011-11-07 13:20 ` Chris Ball [this message]
2011-11-08  2:17   ` Liu Qiang-B32616
2011-11-08  9:47   ` Aaron Lu
2011-11-08 13:37     ` Chris Ball
2011-11-13 16:14       ` Subhash Jadavani
2011-11-13 16:45         ` Chris Ball
2011-11-13 18:11           ` Subhash Jadavani

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=m2ipmw133p.fsf@bob.laptop.org \
    --to=cjb@laptop.org \
    --cc=b32616@freescale.com \
    --cc=kumar.gala@freescale.com \
    --cc=leoli@freescale.com \
    --cc=linux-mmc@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.