Hi Leif, czw., 17 sty 2019 o 13:04 Leif Lindholm <leif.lindh...@linaro.org> napisaĆ(a): > > So, I was about to say that apart from the "class" to be deleted from > 3/12 subject line, things were fine so I was going to fix that locally > and push. > > However. 3/12 is missing the Contributed-under tag. > Can you please separately confirm that it is intended to be > contributed under TianoCore Contribution Agreement 1.1? >
Sure, done. Please let know if I should resubmit. Best regards, Marcin > / > Leif > > On Thu, Jan 17, 2019 at 12:15:03PM +0100, Marcin Wojtas wrote: > > Hi, > > > > I submit v3 of the GPIO patchset for the Marvell SoCs. > > The changes are not significant and they address all the comments > > from the previous version. Please find the details in the changelog > > below. > > > > Patches are available in the github: > > https://github.com/MarvellEmbeddedProcessors/edk2-open-platform/commits/gpio-upstream-r20190117 > > > > I am looking forward to your feedback. > > > > Best regards, > > Marcin > > > > Changelog: > > v2->v3: > > * 1,2,4,12/12 > > - Add Leif's RB > > > > * 3/12 > > - Change MvGpio.h to be library header (Library/MvGpioLib.h) > > - Introduce MvGpioLib library > > - Add PCA_ID enums in this patch (instead from PCA9xxx addition) > > > > * 5,6/12 > > - Use PCA_ID enum from the beginning > > > > * 7/12 > > - Use static global variable and allocate it only only once > > - s/I2C/GPIO/ > > > > * 8/12 > > - Simplify error path after changing approach to not freeing > > already allocated GpioDescription in MvBoardDescDxe > > - Change signature to non-generic > > > > * 10/12 > > - Simplify error path after changing approach to not freeing > > already allocated GpioDescription in MvBoardDescDxe > > - No need to update PCA_ID in this patch > > > > v1->v2: > > * All > > - rebase on top of the master branch and Xenon changes > > > > * 1/12 > > - s/AP/AP806/ in order to avoid confusion > > - drop desc/DESC usage and improve variables naming > > - use GPIO_CONTROLLER from EmbeddedGpio.h instead of custom MV_SOC_GPIO > > - update commit message > > > > * 2,3,4,5/12 > > - drop desc/DESC usage and improve variables naming > > - drop reference to I2c and use GpioExpander > > - improve commit logs > > > > * 6/12 > > - drop desc/DESC usage and improve variables naming > > - drop reference to I2c and use GpioExpander > > - remove global variable - same as for other interfaces the consumer > > is responsible for avoiding memory leak > > > > * 7/12 > > - use MV_GPIO_ prefix > > - switch to EMBEDDED_GPIO > > - remove custom Marvell GPIO Protocol and types > > - add subroutine to MvGpioGetProtocol and more coments to clarify the code > > - add driver types and related enums in other commits > > > > * 8/12 > > - extend error path in the entry point, rename variables > > - use MV_GPIO_ prefix > > - switch to EMBEDDED_GPIO > > - remove custom Marvell GPIO Protocol and types > > - replace '!!' expression for obtaining pin state > > - remove BIT() macro > > - use generic mDevicePathTemplate name > > - put validate function under '#if !defined(MDEPKG_NDEBUG)' and > > call from inside ASSERT_EFI_ERROR, add comment > > - update commit message > > - add common enum for describing controllers > > - change authorship > > > > - 9/12 > > - Add Leif's RB > > > > * 10/12 > > - same modifications as 8/12 > > - use I2C flags directly and remove PCA95XX_OPERATION enum > > - add common enum for describing controllers > > > > * 11/12 > > - Add Leif's RB > > > > * 12/12 > > - Add local headers > > - Make STATIC names more generic > > - s/GetInitializerForType/NonDiscoverableDeviceInitializerGet/ > > - Use global enum for McBin GPIO controller > > > > > > Marcin Wojtas (12): > > Marvell/Library: ArmadaSoCDescLib: Add GPIO information > > Marvell/Library: ArmadaBoardDescLib: Add GPIO information > > Marvell/Library: Introduce MvGpioLib class > > SolidRun/Armada80x0McBin: Extend board description library with GPIO > > Marvell/Armada70x0Db: Extend board description library with GPIO > > Marvell/Armada80x0Db: Extend board description library with GPIO > > Marvell/Drivers: MvBoardDesc: Extend protocol with GPIO support > > Marvell/Drivers: MvGpioDxe: Introduce platform GPIO driver > > Marvell/Drivers: I2c: Use common header for macros > > Marvell/Drivers: MvPca95xxDxe: Introduce GPIO expander driver > > Marvell/Armada7k8k: Enable GPIO drivers compilation > > Marvell/Armada7k8k: Introduce NonDiscoverable device init routines > > > > Silicon/Marvell/Marvell.dec > > | 2 + > > Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc > > | 3 + > > Platform/Marvell/Armada70x0Db/Armada70x0Db.dsc > > | 7 +- > > Platform/Marvell/Armada80x0Db/Armada80x0Db.dsc > > | 3 + > > Platform/SolidRun/Armada80x0McBin/Armada80x0McBin.dsc > > | 3 + > > > > Platform/Marvell/Armada70x0Db/Armada70x0DbBoardDescLib/Armada70x0DbBoardDescLib.inf > > | 1 + > > > > Platform/Marvell/Armada70x0Db/NonDiscoverableInitLib/NonDiscoverableInitLib.inf > > | 49 ++ > > > > Platform/Marvell/Armada80x0Db/Armada80x0DbBoardDescLib/Armada80x0DbBoardDescLib.inf > > | 1 + > > > > Platform/Marvell/Armada80x0Db/NonDiscoverableInitLib/NonDiscoverableInitLib.inf > > | 50 ++ > > > > Platform/SolidRun/Armada80x0McBin/Armada80x0McBinBoardDescLib/Armada80x0McBinBoardDescLib.inf > > | 1 + > > > > Platform/SolidRun/Armada80x0McBin/NonDiscoverableInitLib/NonDiscoverableInitLib.inf > > | 50 ++ > > Silicon/Marvell/Armada7k8k/Drivers/PlatInitDxe/PlatInitDxe.inf > > | 1 + > > > > Silicon/Marvell/Armada7k8k/Library/Armada7k8kSoCDescLib/Armada7k8kSoCDescLib.inf > > | 1 + > > Silicon/Marvell/Drivers/BoardDesc/MvBoardDescDxe.inf > > | 1 + > > Silicon/Marvell/Drivers/Gpio/MvGpioDxe/MvGpioDxe.inf > > | 44 ++ > > Silicon/Marvell/Drivers/Gpio/MvPca95xxDxe/MvPca95xxDxe.inf > > | 45 ++ > > Silicon/Marvell/Drivers/I2c/MvI2cDxe/MvI2cDxe.inf > > | 1 + > > Silicon/Marvell/Drivers/Net/MvMdioDxe/MvMdioDxe.inf > > | 1 + > > Silicon/Marvell/Drivers/NonDiscoverableDxe/NonDiscoverableDxe.inf > > | 1 + > > Silicon/Marvell/Drivers/SdMmc/XenonDxe/XenonDxe.inf > > | 1 + > > Silicon/Marvell/Library/ComPhyLib/ComPhyLib.inf > > | 1 + > > Silicon/Marvell/Library/IcuLib/IcuLib.inf > > | 1 + > > Silicon/Marvell/Library/MvGpioLib/MvGpioLib.inf > > | 32 + > > Silicon/Marvell/Library/UtmiPhyLib/UtmiPhyLib.inf > > | 1 + > > > > Platform/Marvell/Armada70x0Db/NonDiscoverableInitLib/NonDiscoverableInitLib.h > > | 23 + > > > > Platform/Marvell/Armada80x0Db/NonDiscoverableInitLib/NonDiscoverableInitLib.h > > | 28 + > > > > Platform/SolidRun/Armada80x0McBin/NonDiscoverableInitLib/NonDiscoverableInitLib.h > > | 20 + > > > > Silicon/Marvell/Armada7k8k/Library/Armada7k8kSoCDescLib/Armada7k8kSoCDescLib.h > > | 10 + > > Silicon/Marvell/Drivers/Gpio/MvGpioDxe/MvGpioDxe.h > > | 49 ++ > > Silicon/Marvell/Drivers/Gpio/MvPca95xxDxe/MvPca95xxDxe.h > > | 65 ++ > > Silicon/Marvell/Drivers/I2c/MvEepromDxe/MvEepromDxe.h > > | 10 - > > Silicon/Marvell/Drivers/I2c/MvI2cDxe/MvI2cDxe.h > > | 17 +- > > Silicon/Marvell/Include/Library/ArmadaBoardDescLib.h > > | 23 + > > Silicon/Marvell/Include/Library/ArmadaSoCDescLib.h > > | 12 + > > Silicon/Marvell/Include/Library/MvGpioLib.h > > | 68 +++ > > Silicon/Marvell/Include/Library/NonDiscoverableInitLib.h > > | 28 + > > Silicon/Marvell/Include/Protocol/BoardDesc.h > > | 8 + > > Silicon/Marvell/Include/Protocol/MvI2c.h > > | 31 + > > > > Platform/Marvell/Armada70x0Db/Armada70x0DbBoardDescLib/Armada70x0DbBoardDescLib.c > > | 24 + > > > > Platform/Marvell/Armada70x0Db/NonDiscoverableInitLib/NonDiscoverableInitLib.c > > | 96 +++ > > > > Platform/Marvell/Armada80x0Db/Armada80x0DbBoardDescLib/Armada80x0DbBoardDescLib.c > > | 30 + > > > > Platform/Marvell/Armada80x0Db/NonDiscoverableInitLib/NonDiscoverableInitLib.c > > | 106 ++++ > > > > Platform/SolidRun/Armada80x0McBin/Armada80x0McBinBoardDescLib/Armada80x0McBinBoardDescLib.c > > | 17 + > > > > Platform/SolidRun/Armada80x0McBin/NonDiscoverableInitLib/NonDiscoverableInitLib.c > > | 71 +++ > > Silicon/Marvell/Applications/EepromCmd/EepromCmd.c > > | 5 +- > > > > Silicon/Marvell/Armada7k8k/Library/Armada7k8kSoCDescLib/Armada7k8kSoCDescLib.c > > | 39 ++ > > Silicon/Marvell/Drivers/BoardDesc/MvBoardDescDxe.c > > | 50 ++ > > Silicon/Marvell/Drivers/Gpio/MvGpioDxe/MvGpioDxe.c > > | 353 +++++++++++ > > Silicon/Marvell/Drivers/Gpio/MvPca95xxDxe/MvPca95xxDxe.c > > | 645 ++++++++++++++++++++ > > Silicon/Marvell/Drivers/I2c/MvEepromDxe/MvEepromDxe.c > > | 3 +- > > Silicon/Marvell/Drivers/I2c/MvI2cDxe/MvI2cDxe.c > > | 4 +- > > Silicon/Marvell/Drivers/NonDiscoverableDxe/NonDiscoverableDxe.c > > | 13 +- > > Silicon/Marvell/Library/MvGpioLib/MvGpioLib.c > > | 132 ++++ > > Platform/Marvell/Armada70x0Db/Armada70x0Db.fdf.inc > > | 2 + > > Platform/Marvell/Armada80x0Db/Armada80x0Db.fdf.inc > > | 2 + > > Platform/SolidRun/Armada80x0McBin/Armada80x0McBin.fdf.inc > > | 2 + > > 56 files changed, 2249 insertions(+), 38 deletions(-) > > create mode 100644 > > Platform/Marvell/Armada70x0Db/NonDiscoverableInitLib/NonDiscoverableInitLib.inf > > create mode 100644 > > Platform/Marvell/Armada80x0Db/NonDiscoverableInitLib/NonDiscoverableInitLib.inf > > create mode 100644 > > Platform/SolidRun/Armada80x0McBin/NonDiscoverableInitLib/NonDiscoverableInitLib.inf > > create mode 100644 Silicon/Marvell/Drivers/Gpio/MvGpioDxe/MvGpioDxe.inf > > create mode 100644 > > Silicon/Marvell/Drivers/Gpio/MvPca95xxDxe/MvPca95xxDxe.inf > > create mode 100644 Silicon/Marvell/Library/MvGpioLib/MvGpioLib.inf > > create mode 100644 > > Platform/Marvell/Armada70x0Db/NonDiscoverableInitLib/NonDiscoverableInitLib.h > > create mode 100644 > > Platform/Marvell/Armada80x0Db/NonDiscoverableInitLib/NonDiscoverableInitLib.h > > create mode 100644 > > Platform/SolidRun/Armada80x0McBin/NonDiscoverableInitLib/NonDiscoverableInitLib.h > > create mode 100644 Silicon/Marvell/Drivers/Gpio/MvGpioDxe/MvGpioDxe.h > > create mode 100644 Silicon/Marvell/Drivers/Gpio/MvPca95xxDxe/MvPca95xxDxe.h > > create mode 100644 Silicon/Marvell/Include/Library/MvGpioLib.h > > create mode 100644 Silicon/Marvell/Include/Library/NonDiscoverableInitLib.h > > create mode 100644 Silicon/Marvell/Include/Protocol/MvI2c.h > > create mode 100644 > > Platform/Marvell/Armada70x0Db/NonDiscoverableInitLib/NonDiscoverableInitLib.c > > create mode 100644 > > Platform/Marvell/Armada80x0Db/NonDiscoverableInitLib/NonDiscoverableInitLib.c > > create mode 100644 > > Platform/SolidRun/Armada80x0McBin/NonDiscoverableInitLib/NonDiscoverableInitLib.c > > create mode 100644 Silicon/Marvell/Drivers/Gpio/MvGpioDxe/MvGpioDxe.c > > create mode 100644 Silicon/Marvell/Drivers/Gpio/MvPca95xxDxe/MvPca95xxDxe.c > > create mode 100644 Silicon/Marvell/Library/MvGpioLib/MvGpioLib.c > > > > -- > > 2.7.4 > > _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel