Hi Ard, 2018-06-11 13:00 GMT+02:00 Ard Biesheuvel <ard.biesheu...@linaro.org>: > Marcin, > > I am a bit reluctant to review another huge set of Armada patches > while we are still waiting for MacchiatoBin support to land. The only > hardware i have access to is MacchiatoBin, and it has been well over a > year now that MacchiatoBin support has been between 'under > construction' and 'about to land any day now'. > > The patches in this series actually carry traces of McBin support, > which suggests to me that they were taken from an internal tree that > does support MacchiatoBin, but it has been removed. What is that > about?
MacchiatoBin addition is on top of those patches in the branch I prepared solely for upstream. Anyway, I understand your point and let's skip this big patchset for now. I will submit support for other boards, once it's accepted, let's return to this one. Best regards, Marcin > > edk2-platforms is not intended as a baseline for Marvell customers to > base their private trees on, we actually prefer code that runs on > systems that are widely available (such as MacchiatoBin) > > Thanks, > Ard. > > > On 8 June 2018 at 17:33, Marcin Wojtas <m...@semihalf.com> wrote: >> Hi, >> >> This big patchset reworks handling of the SoC and Boards' description >> to the final version. Hitherto mechanism of hardcoded structures >> and parsing PCDs inside the drivers was unfortunate - it didn't >> allow to support different SoC families with different number >> of controllers or base addresses. >> >> The main concept is introducing a hardware description layer >> with the new protocol, which is responsible for providing >> data to the consumer platforms drivers. Additionally a new >> SoC and Board description libraries allow to move information >> from overly used PCDs to C code in an organized manner: >> >> ArmadaSoCDescLib + ArmadaBoardDescLib >> (per SoC family) (per Board, in next steps more of description >> | of ComPhy, Mpp and others can go there) >> | | >> | | >> |-> MV_BOARD_DESC <-| >> | >> | >> | >> Driver/Library (e.g. ComPhy, I2c, Pp2Dxe) >> >> Please don't be discouraged by big amount of patches, >> they are sort of repeatable: update protocol, libraries and >> the consumer drivers/libraries until the MvHwDescLib.h >> header could be completely removed. More details can >> be found in the commit logs. >> >> The patches are available in the github: >> https://github.com/MarvellEmbeddedProcessors/edk2-open-platform/commits/boarddesc-upstream-r20180608 >> >> I'm looking forward to review and any comments/remarks. >> >> Best regards, >> Marcin >> >> Marcin Wojtas (21): >> Marvell/Library: Introduce ArmadaBoardDescLib class >> Marvell/Library: UtmiPhyLib: Switch to use MARVELL_BOARD_DESC protocol >> Marvell/Library: RealTimeClockLib: Simplify obtaining base address >> Marvell/Armada7k8k: Extend ArmadaSoCDescLib with PP2 information >> Marvell/Drivers: MvBoardDesc: Extend protocol with PP2 support >> Marvell/Drivers: Pp2Dxe: Switch to use MARVELL_BOARD_DESC protocol >> Marvell/Armada7k8k: Extend ArmadaSoCDescLib with AHCI/SDMMC/XHCI >> Marvell/Drivers: MvBoardDesc: Extend protocol with AHCI/SDMMC/XHCI >> Marvell/Drivers: NonDiscoverable: Switch to use MARVELL_BOARD_DESC >> Marvell/Library: ComPhyLib: Get AHCI data with MARVELL_BOARD_DESC >> Marvell/Armada7k8k: Extend ArmadaSoCDescLib with ComPhy information >> Marvell/Drivers: MvBoardDesc: Extend protocol with COMPHY support >> Marvell/Library: ComPhyLib: Switch library to use MARVELL_BOARD_DESC >> Marvell/Armada7k8k: Extend ArmadaSoCDescLib with MDIO information >> Marvell/Drivers: MvBoardDesc: Extend protocol with MDIO support >> Marvell/Drivers: MvMdioDxe: Enable 64bit addressing >> Marvell/Drivers: MvMdioDxe: Switch driver to use MARVELL_BOARD_DESC >> Marvell/Armada7k8k: Extend ArmadaSoCDescLib with I2C information >> Marvell/Drivers: MvBoardDesc: Extend protocol with I2C support >> Marvell/Drivers: MvI2cDxe: Switch driver to use MARVELL_BOARD_DESC >> Marvell/Drivers: MvPhyDxe: Remove MvHwDescLib.h dependency >> >> jinghua (4): >> Marvell/Library: Introduce ArmadaSoCDescLib class >> Marvell: Introduce MARVELL_BOARD_DESC_PROTOCOL >> Marvell/Drivers: MvBoardDescDxe: Introduce board description driver >> Marvell/Armada7k8k: Enable board description driver compilation >> >> Platform/Marvell/Armada70x0Db/Armada70x0Db.dsc >> | 2 +- >> Platform/Marvell/Armada70x0Db/Armada70x0Db.fdf >> | 1 + >> Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc >> | 2 + >> Silicon/Marvell/Armada7k8k/Library/Armada7k8kLib/Armada7k8kLib.inf >> | 2 - >> >> Silicon/Marvell/Armada7k8k/Library/Armada7k8kSoCDescLib/Armada7k8kSoCDescLib.c >> | 338 ++++++++++++ >> >> Silicon/Marvell/Armada7k8k/Library/Armada7k8kSoCDescLib/Armada7k8kSoCDescLib.inf >> | 37 ++ >> Silicon/Marvell/Armada7k8k/Library/RealTimeClockLib/RealTimeClockLib.c >> | 29 +- >> Silicon/Marvell/Armada7k8k/Library/RealTimeClockLib/RealTimeClockLib.inf >> | 2 +- >> Silicon/Marvell/Drivers/BoardDesc/MvBoardDescDxe.c >> | 573 ++++++++++++++++++++ >> Silicon/Marvell/Drivers/BoardDesc/MvBoardDescDxe.h >> | 59 ++ >> Silicon/Marvell/Drivers/BoardDesc/MvBoardDescDxe.inf >> | 70 +++ >> Silicon/Marvell/Drivers/I2c/MvI2cDxe/MvI2cDxe.c >> | 37 +- >> Silicon/Marvell/Drivers/I2c/MvI2cDxe/MvI2cDxe.inf >> | 1 + >> Silicon/Marvell/Drivers/Net/MvMdioDxe/MvMdioDxe.c >> | 41 +- >> Silicon/Marvell/Drivers/Net/MvMdioDxe/MvMdioDxe.inf >> | 1 + >> Silicon/Marvell/Drivers/Net/MvPhyDxe/MvPhyDxe.c >> | 3 +- >> Silicon/Marvell/Drivers/Net/Pp2Dxe/Pp2Dxe.c >> | 43 +- >> Silicon/Marvell/Drivers/Net/Pp2Dxe/Pp2Dxe.inf >> | 2 +- >> Silicon/Marvell/Drivers/NonDiscoverableDxe/NonDiscoverableDxe.c >> | 100 ++-- >> Silicon/Marvell/Drivers/NonDiscoverableDxe/NonDiscoverableDxe.inf >> | 6 +- >> Silicon/Marvell/Include/Library/ArmadaBoardDescLib.h >> | 99 ++++ >> Silicon/Marvell/Include/Library/ArmadaSoCDescLib.h >> | 146 +++++ >> Silicon/Marvell/Include/Library/MvHwDescLib.h >> | 290 ---------- >> Silicon/Marvell/Include/Library/UtmiPhyLib.h >> | 2 + >> Silicon/Marvell/Include/Protocol/BoardDesc.h >> | 118 ++++ >> Silicon/Marvell/Include/Protocol/Mdio.h >> | 4 +- >> Silicon/Marvell/Library/ComPhyLib/ComPhyCp110.c >> | 50 +- >> Silicon/Marvell/Library/ComPhyLib/ComPhyLib.c >> | 74 ++- >> Silicon/Marvell/Library/ComPhyLib/ComPhyLib.h >> | 4 + >> Silicon/Marvell/Library/ComPhyLib/ComPhyLib.inf >> | 6 +- >> Silicon/Marvell/Library/UtmiPhyLib/UtmiPhyLib.c >> | 65 +-- >> Silicon/Marvell/Library/UtmiPhyLib/UtmiPhyLib.h >> | 5 + >> Silicon/Marvell/Library/UtmiPhyLib/UtmiPhyLib.inf >> | 9 +- >> Silicon/Marvell/Marvell.dec >> | 8 +- >> 34 files changed, 1694 insertions(+), 535 deletions(-) >> create mode 100644 >> Silicon/Marvell/Armada7k8k/Library/Armada7k8kSoCDescLib/Armada7k8kSoCDescLib.c >> create mode 100644 >> Silicon/Marvell/Armada7k8k/Library/Armada7k8kSoCDescLib/Armada7k8kSoCDescLib.inf >> create mode 100644 Silicon/Marvell/Drivers/BoardDesc/MvBoardDescDxe.c >> create mode 100644 Silicon/Marvell/Drivers/BoardDesc/MvBoardDescDxe.h >> create mode 100644 Silicon/Marvell/Drivers/BoardDesc/MvBoardDescDxe.inf >> create mode 100644 Silicon/Marvell/Include/Library/ArmadaBoardDescLib.h >> create mode 100644 Silicon/Marvell/Include/Library/ArmadaSoCDescLib.h >> delete mode 100644 Silicon/Marvell/Include/Library/MvHwDescLib.h >> create mode 100644 Silicon/Marvell/Include/Protocol/BoardDesc.h >> >> -- >> 2.7.4 >> _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel