On Wed, Mar 22, 2017 at 02:34:15PM +0100, Marcin Wojtas wrote:
Hi,
Here's third version of the patchset comprising minor issues fixed pointed in v2.
The commits are also available in the github: https://github.com/MarvellEmbeddedProcessors/edk2-open-platform/commits/opp-...
Any remarks or comments will be very welcome.
Best regards, Marcin
Thanks, pushed as: 2d34e1d..63a7184.
/ Leif
Changelog v2 -> v3:
- Add reviewed-by's to 1/4 and 3/4
- 1/4 Add 'Status = EFI_INVALID_PARAMETER' in switch default and remove default variable initalization
- 3/4 For clarity leave NonDiscoverable header in PciEmulation as well
- 4/4 Fix whitespaces
v1 -> v2:
- 1/4: Change type of COMPHY_CHIP_INIT callback to VOID and rework Status usage
- 2/4: Add 'Reviewed-by'
- 3/4:
- Sort includes in header
- Rename 'storage' to 'nondiscoverable' in all macros and names
- Add defines for array sizes in definition
- Add defines for base addresses, so that they describe HW block and device
- Add comments for sections
- 4/4:
- Remove 'Ptr' suffixes
- Move and rename DEV_ENABLED macro from PciEmulation driver - use it in ComPhy code
- Simplify obtaining enabled Sata ports and do not return EFI_SUCCESS on PCD mismatch (lane enabled in ComPhy, despite the controller disabled)
- Add macros for controller ID's
- Remove unrelated style fixes
- Rewrite error print information in ComPhyCp110Init, because from now we have more root causes than PLL
Konstantin Porotchkin (4): Platforms/Marvell: ComPhyLib: Fix compilation warning Platforms/Marvell: Add support for COMPHY on CP slaves Platform/Marvell: Extend and share the platform description Platform/Marvell: ComPhyLib: Enable SATA PHY init for multiple devices
Platforms/Marvell/Armada/Armada70x0.dsc | 9 +- Platforms/Marvell/Include/Library/MvHwDescLib.h | 106 ++++++++++++++++++++++ Platforms/Marvell/Library/ComPhyLib/ComPhyCp110.c | 45 ++++++--- Platforms/Marvell/Library/ComPhyLib/ComPhyLib.c | 8 +- Platforms/Marvell/Library/ComPhyLib/ComPhyLib.h | 4 +- Platforms/Marvell/Library/ComPhyLib/ComPhyLib.inf | 3 +- Platforms/Marvell/Marvell.dec | 3 - Platforms/Marvell/PciEmulation/PciEmulation.c | 51 ++--------- 8 files changed, 156 insertions(+), 73 deletions(-) create mode 100644 Platforms/Marvell/Include/Library/MvHwDescLib.h
-- 1.8.3.1