From: Marcin Wojtas <mw@semihalf.com>
To: Leif Lindholm <leif.lindholm@linaro.org>
Cc: edk2-devel-01 <edk2-devel@lists.01.org>,
Ard Biesheuvel <ard.biesheuvel@linaro.org>,
nadavh@marvell.com, "jsd@semihalf.com" <jsd@semihalf.com>,
Grzegorz Jaszczyk <jaz@semihalf.com>,
Kostya Porotchkin <kostap@marvell.com>
Subject: Re: [platforms: PATCH v3 00/12] Armada 7k8k GPIO support
Date: Thu, 17 Jan 2019 13:26:11 +0100 [thread overview]
Message-ID: <CAPv3WKeKV5+_Xzx6wwqojOLtBo_7ZLTCOJupZLN2m89T6eWVCA@mail.gmail.com> (raw)
In-Reply-To: <20190117120356.wabcsghw2tfu5qra@bivouac.eciton.net>
Hi Leif,
czw., 17 sty 2019 o 13:04 Leif Lindholm <leif.lindholm@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
> >
prev parent reply other threads:[~2019-01-17 12:26 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-01-17 11:15 [platforms: PATCH v3 00/12] Armada 7k8k GPIO support Marcin Wojtas
2019-01-17 11:15 ` [platforms: PATCH v3 01/12] Marvell/Library: ArmadaSoCDescLib: Add GPIO information Marcin Wojtas
2019-01-17 11:15 ` [platforms: PATCH v3 02/12] Marvell/Library: ArmadaBoardDescLib: " Marcin Wojtas
2019-01-17 11:15 ` [platforms: PATCH v3 03/12] Marvell/Library: Introduce MvGpioLib class Marcin Wojtas
2019-01-17 11:33 ` Leif Lindholm
2019-01-17 12:19 ` Marcin Wojtas
2019-01-17 14:18 ` Leif Lindholm
2019-01-17 14:39 ` Marcin Wojtas
2019-01-17 11:15 ` [platforms: PATCH v3 04/12] SolidRun/Armada80x0McBin: Extend board description library with GPIO Marcin Wojtas
2019-01-17 11:15 ` [platforms: PATCH v3 05/12] Marvell/Armada70x0Db: " Marcin Wojtas
2019-01-17 11:34 ` Leif Lindholm
2019-01-17 11:15 ` [platforms: PATCH v3 06/12] Marvell/Armada80x0Db: " Marcin Wojtas
2019-01-17 11:34 ` Leif Lindholm
2019-01-17 11:15 ` [platforms: PATCH v3 07/12] Marvell/Drivers: MvBoardDesc: Extend protocol with GPIO support Marcin Wojtas
2019-01-17 11:50 ` Leif Lindholm
2019-01-17 11:15 ` [platforms: PATCH v3 08/12] Marvell/Drivers: MvGpioDxe: Introduce platform GPIO driver Marcin Wojtas
2019-01-17 11:52 ` Leif Lindholm
2019-01-17 11:15 ` [platforms: PATCH v3 09/12] Marvell/Drivers: I2c: Use common header for macros Marcin Wojtas
2019-01-17 11:15 ` [platforms: PATCH v3 10/12] Marvell/Drivers: MvPca95xxDxe: Introduce GPIO expander driver Marcin Wojtas
2019-01-17 11:55 ` Leif Lindholm
2019-01-17 11:15 ` [platforms: PATCH v3 11/12] Marvell/Armada7k8k: Enable GPIO drivers compilation Marcin Wojtas
2019-01-17 11:15 ` [platforms: PATCH v3 12/12] Marvell/Armada7k8k: Introduce NonDiscoverable device init routines Marcin Wojtas
2019-01-17 12:03 ` [platforms: PATCH v3 00/12] Armada 7k8k GPIO support Leif Lindholm
2019-01-17 12:26 ` Marcin Wojtas [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAPv3WKeKV5+_Xzx6wwqojOLtBo_7ZLTCOJupZLN2m89T6eWVCA@mail.gmail.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox