[edk2] [platforms: PATCH v2 0/5] Armada 7k/8k - ParsePcdLib removal

Marcin Wojtas posted 5 patches 7 years, 2 months ago
Failed in applying to current master (apply log)
Platform/Marvell/Armada/Armada.dsc.inc                 |  68 +++++-
Platform/Marvell/Armada/Armada70x0.dsc                 |  30 +--
Platform/Marvell/Drivers/I2c/MvI2cDxe/MvI2cDxe.c       |  42 ++--
Platform/Marvell/Drivers/I2c/MvI2cDxe/MvI2cDxe.inf     |   3 +-
Platform/Marvell/Drivers/Net/MvMdioDxe/MvMdioDxe.c     |  35 ++-
Platform/Marvell/Drivers/Net/MvMdioDxe/MvMdioDxe.inf   |   3 -
Platform/Marvell/Drivers/Net/Phy/MvPhyDxe/MvPhyDxe.c   | 122 ++++++-----
Platform/Marvell/Drivers/Net/Phy/MvPhyDxe/MvPhyDxe.h   |   2 -
Platform/Marvell/Drivers/Net/Phy/MvPhyDxe/MvPhyDxe.inf |   4 +-
Platform/Marvell/Drivers/Net/Pp2Dxe/Pp2Dxe.c           |  16 +-
Platform/Marvell/Drivers/Net/Pp2Dxe/Pp2Dxe.h           |   2 +-
Platform/Marvell/Drivers/Net/Pp2Dxe/Pp2Dxe.inf         |   4 +-
Platform/Marvell/Include/Library/MvHwDescLib.h         |  95 ++++++++
Platform/Marvell/Include/Library/ParsePcdLib.h         |  46 ----
Platform/Marvell/Include/Protocol/Mdio.h               |   6 +
Platform/Marvell/Include/Protocol/MvPhy.h              |   1 +
Platform/Marvell/Library/ComPhyLib/ComPhyLib.c         |  65 +-----
Platform/Marvell/Library/ComPhyLib/ComPhyLib.h         |  25 +--
Platform/Marvell/Library/ComPhyLib/ComPhyLib.inf       |   1 -
Platform/Marvell/Library/ParsePcdLib/ParsePcdLib.c     | 228 --------------------
Platform/Marvell/Library/ParsePcdLib/ParsePcdLib.inf   |  50 -----
Platform/Marvell/Library/UtmiPhyLib/UtmiPhyLib.c       | 150 ++++++-------
Platform/Marvell/Library/UtmiPhyLib/UtmiPhyLib.h       |   1 -
Platform/Marvell/Library/UtmiPhyLib/UtmiPhyLib.inf     |  11 +-
Platform/Marvell/Marvell.dec                           |  17 +-
Silicon/Marvell/Documentation/PortingGuide.txt         | 165 ++++++++------
26 files changed, 528 insertions(+), 664 deletions(-)
delete mode 100644 Platform/Marvell/Include/Library/ParsePcdLib.h
delete mode 100644 Platform/Marvell/Library/ParsePcdLib/ParsePcdLib.c
delete mode 100644 Platform/Marvell/Library/ParsePcdLib/ParsePcdLib.inf
[edk2] [platforms: PATCH v2 0/5] Armada 7k/8k - ParsePcdLib removal
Posted by Marcin Wojtas 7 years, 2 months ago
Hi,

This is a second version of the patchset with the complete
removal of ParsePcdLib and cleanup of boards' PCD representation.
According to v1 remarks, I modified PCD names and introduced macros
in Armada.dsc.inc, so that the readability of boards .dsc could
increse.
More details can befound in the changelog and commits.

The patches are available in the github:
https://github.com/MarvellEmbeddedProcessors/edk2-open-platform/commits/parsepcd-upstream-r20171008

I'm looking forward to your comments or remarks.

Best regards,
Marcin

Changelog:
v1  -> v2
1/5
  * Add new [Defines] section was added to Armada.dsc.inc
    file in order to increase readability.

2/5
  * s/PcdI2cControllers/PcdI2cControllersEnabled/

3/5
  * s/PcdUtmiControllers/PcdUtmiControllersEnabled/
  * Add defines for PcdUtmiPortType

4/5:
  * s/PcdMdioControllers/PcdMdioControllersEnabled/
  * Add macros for network PHY type and speed in Armada.dsc.inc

5/5
  * Add RBs

Marcin Wojtas (5):
  Marvell/Library: ComPhyLib: Remove PCD string parsing
  Marvell/Drivers: MvI2cDxe: Move devices description to MvHwDescLib
  Marvell/Library: UtmiLib: Move devices description to MvHwDescLib
  Marvell/Drivers: Pp2Dxe: Rework PHY handling
  Platform/Marvell/Armada: Remove ParsePcdLib

 Platform/Marvell/Armada/Armada.dsc.inc                 |  68 +++++-
 Platform/Marvell/Armada/Armada70x0.dsc                 |  30 +--
 Platform/Marvell/Drivers/I2c/MvI2cDxe/MvI2cDxe.c       |  42 ++--
 Platform/Marvell/Drivers/I2c/MvI2cDxe/MvI2cDxe.inf     |   3 +-
 Platform/Marvell/Drivers/Net/MvMdioDxe/MvMdioDxe.c     |  35 ++-
 Platform/Marvell/Drivers/Net/MvMdioDxe/MvMdioDxe.inf   |   3 -
 Platform/Marvell/Drivers/Net/Phy/MvPhyDxe/MvPhyDxe.c   | 122 ++++++-----
 Platform/Marvell/Drivers/Net/Phy/MvPhyDxe/MvPhyDxe.h   |   2 -
 Platform/Marvell/Drivers/Net/Phy/MvPhyDxe/MvPhyDxe.inf |   4 +-
 Platform/Marvell/Drivers/Net/Pp2Dxe/Pp2Dxe.c           |  16 +-
 Platform/Marvell/Drivers/Net/Pp2Dxe/Pp2Dxe.h           |   2 +-
 Platform/Marvell/Drivers/Net/Pp2Dxe/Pp2Dxe.inf         |   4 +-
 Platform/Marvell/Include/Library/MvHwDescLib.h         |  95 ++++++++
 Platform/Marvell/Include/Library/ParsePcdLib.h         |  46 ----
 Platform/Marvell/Include/Protocol/Mdio.h               |   6 +
 Platform/Marvell/Include/Protocol/MvPhy.h              |   1 +
 Platform/Marvell/Library/ComPhyLib/ComPhyLib.c         |  65 +-----
 Platform/Marvell/Library/ComPhyLib/ComPhyLib.h         |  25 +--
 Platform/Marvell/Library/ComPhyLib/ComPhyLib.inf       |   1 -
 Platform/Marvell/Library/ParsePcdLib/ParsePcdLib.c     | 228 --------------------
 Platform/Marvell/Library/ParsePcdLib/ParsePcdLib.inf   |  50 -----
 Platform/Marvell/Library/UtmiPhyLib/UtmiPhyLib.c       | 150 ++++++-------
 Platform/Marvell/Library/UtmiPhyLib/UtmiPhyLib.h       |   1 -
 Platform/Marvell/Library/UtmiPhyLib/UtmiPhyLib.inf     |  11 +-
 Platform/Marvell/Marvell.dec                           |  17 +-
 Silicon/Marvell/Documentation/PortingGuide.txt         | 165 ++++++++------
 26 files changed, 528 insertions(+), 664 deletions(-)
 delete mode 100644 Platform/Marvell/Include/Library/ParsePcdLib.h
 delete mode 100644 Platform/Marvell/Library/ParsePcdLib/ParsePcdLib.c
 delete mode 100644 Platform/Marvell/Library/ParsePcdLib/ParsePcdLib.inf

-- 
1.8.3.1

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel