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