From: "Wu, Hao A" <hao.a.wu@intel.com>
To: "Albecki, Mateusz" <mateusz.albecki@intel.com>,
"devel@edk2.groups.io" <devel@edk2.groups.io>
Subject: Re: [PATCH v3 2/2] MdeModulePkg/SdMmcHcDxe: Implement revision 3 of SdMmcOverrideProtocol
Date: Wed, 26 Jun 2019 01:26:43 +0000 [thread overview]
Message-ID: <B80AF82E9BFB8E4FBD8C89DA810C6A093C8F4422@SHSMSX104.ccr.corp.intel.com> (raw)
In-Reply-To: <92CF190FF2351747A47C1708F0E09C0875E7341C@IRSMSX102.ger.corp.intel.com>
> -----Original Message-----
> From: Albecki, Mateusz
> Sent: Tuesday, June 25, 2019 10:14 PM
> To: Wu, Hao A; devel@edk2.groups.io
> Subject: RE: [PATCH v3 2/2] MdeModulePkg/SdMmcHcDxe: Implement
> revision 3 of SdMmcOverrideProtocol
>
...
>
> You were clear in previous mails but as I responded I am not sure if you can
> be in default 1-bit mode when the SdCardSwitch function is executed. I am
> not sure if this command uses DAT lanes to transfer the data(I couldn't find
> information in spec) and I am unable to test it on real hardware. Could you
> test it on your end? If it works I see no reason not to combine those 2.
>
I understand now.
Though combining the two SdCardSwitchBusWidth() calls does work for the
UHS-I card on my side, I still prefer your proposed logic with regard to
performance consideration.
Please help to keep this chunk of logic as is in the next series. Thanks
for the explanation.
Best Regards,
Hao Wu
prev parent reply other threads:[~2019-06-26 1:26 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-06-21 15:12 [PATCH v3 0/2] Add GetOperatingParam notify phase to SdMmcOverride protocol Albecki, Mateusz
2019-06-21 15:12 ` [PATCH v3 1/2] MdeModulePkg/SdMmcOverride: Add GetOperatingParam notify phase Albecki, Mateusz
2019-06-24 8:14 ` Wu, Hao A
2019-06-25 14:15 ` Albecki, Mateusz
2019-06-21 15:12 ` [PATCH v3 2/2] MdeModulePkg/SdMmcHcDxe: Implement revision 3 of SdMmcOverrideProtocol Albecki, Mateusz
2019-06-24 8:15 ` Wu, Hao A
[not found] ` <B80AF82E9BFB8E4FBD8C89DA810C6A093C8F3A12@SHSMSX104.ccr.corp.intel.com>
2019-06-24 8:46 ` Wu, Hao A
2019-06-25 14:13 ` Albecki, Mateusz
2019-06-26 1:26 ` Wu, Hao A [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-list 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=B80AF82E9BFB8E4FBD8C89DA810C6A093C8F4422@SHSMSX104.ccr.corp.intel.com \
--to=devel@edk2.groups.io \
/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