Branch: refs/heads/master
  Home:   https://github.com/tianocore/edk2
  Commit: f56cc67f62b25f524d95d77696d9dab5514a798a
      
https://github.com/tianocore/edk2/commit/f56cc67f62b25f524d95d77696d9dab5514a798a
  Author: Albecki, Mateusz <mateusz.albe...@intel.com>
  Date:   2019-07-01 (Mon, 01 Jul 2019)

  Changed paths:
    M MdeModulePkg/Include/Protocol/SdMmcOverride.h

  Log Message:
  -----------
  MdeModulePkg/SdMmcOverride: Add GetOperatingParam notify phase

https://bugzilla.tianocore.org/show_bug.cgi?id=1882

The new notify phase allows platform to configure additional
bus parameters in addition to parameters that can already be configured
with capability override. Specifically we allow to configure bus width,
clock frequency and driver strength. If platform doesn't wish to configure
some of the parameters it can left it on default values and driver will
assume it's standard behavior with respect to those parameters.
The definition of the SD_MMC_BUS_MODE has been extended to
incorporate SD card default speed and high speed.

Tests on Marvell boards were also performed by Marcin Wojtas
<m...@semihalf.com>:

https://edk2.groups.io/g/devel/message/42999

Board 1 (out of tree): SD - OK, MMC - OK
Board 2: (Armada80x0McBin): SD - OK, MMC - OK
Board 3: (Armada70x0Db): SD - problems, MMC - OK

Please note that the problem on Armada70x0Db SD devices are introduced by
adding new types of SD bus modes, a subsequent patch within edk2-platforms
repository will be proposed to address it.
(More details can be referred from the above link.)

Signed-off-by: Mateusz Albecki <mateusz.albe...@intel.com>
Reviewed-by: Hao A Wu <hao.a...@intel.com>
Regression-tested-by: Sumit Garg <sumit.g...@linaro.org>


  Commit: adec1f5deb89c68d82598074500006bd575e8f58
      
https://github.com/tianocore/edk2/commit/adec1f5deb89c68d82598074500006bd575e8f58
  Author: Albecki, Mateusz <mateusz.albe...@intel.com>
  Date:   2019-07-01 (Mon, 01 Jul 2019)

  Changed paths:
    M MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/EmmcDevice.c
    M MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdDevice.c
    M MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdMmcPciHcDxe.c
    M MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdMmcPciHcDxe.h
    M MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdMmcPciHci.c
    M MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdMmcPciHci.h

  Log Message:
  -----------
  MdeModulePkg/SdMmcHcDxe: Implement revision 3 of SdMmcOverrideProtocol

https://bugzilla.tianocore.org/show_bug.cgi?id=1882

Implement support for GetOperatingParamters notify phase
in SdMmcHcDxe driver. GetOperatingParameters notify phase
is signaled before we start card detection and initialization.
Code has been updated for both eMMC and SD card controllers to
take into consideration those new parameters. Initialization process
has been divided into 2 steps. In the first step we bring the link
up to the point where we can get card identification data(Extended
CSD in eMMC case and SWITCH command response in SD card case). This
data is later used along with controller capabilities and operating
parameters passed in GetOperatingParameters phase to choose preferred
bus settings in GetTargetBusSettings function. Those settings are later
on to start bus training to high speeds. If user passes incompatible
setting with selected bus timing driver will assume it's standard behavior
with respect to that setting. For instance if HS400 has been selected as a
target bus timing due to card and controller support bus width setting of
4 and 1 bit won't be respected and 8 bit setting will be chosen instead.

Tests on Marvell boards were also performed by Marcin Wojtas
<m...@semihalf.com>:

https://edk2.groups.io/g/devel/message/42999

Board 1 (out of tree): SD - OK, MMC - OK
Board 2: (Armada80x0McBin): SD - OK, MMC - OK
Board 3: (Armada70x0Db): SD - problems, MMC - OK

Please note that the problem on Armada70x0Db SD devices are introduced by
adding new types of SD bus modes, a subsequent patch within edk2-platforms
repository will be proposed to address it.
(More details can be referred from the above link.)

Signed-off-by: Mateusz Albecki <mateusz.albe...@intel.com>
Reviewed-by: Hao A Wu <hao.a...@intel.com>
Regression-tested-by: Sumit Garg <sumit.g...@linaro.org>


Compare: https://github.com/tianocore/edk2/compare/a37e18f6fc10...adec1f5deb89


_______________________________________________
edk2-commits mailing list
edk2-commits@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-commits

Reply via email to