From: "Marcin Wojtas" <mw@semihalf.com>
To: Leif Lindholm <leif.lindholm@linaro.org>
Cc: edk2-devel-groups-io <devel@edk2.groups.io>,
Ard Biesheuvel <ard.biesheuvel@linaro.org>,
"jsd@semihalf.com" <jsd@semihalf.com>,
Grzegorz Jaszczyk <jaz@semihalf.com>,
Kostya Porotchkin <kostap@marvell.com>,
Jici Gao <Jici.Gao@arm.com>
Subject: Re: [edk2-platforms: PATCH v3 0/4] Armada7k8k FVB improvements
Date: Mon, 29 Apr 2019 15:54:13 +0200 [thread overview]
Message-ID: <CAPv3WKdozS0TTswwuPEj_aeX10Bm4em4jLJqM47JT1Ne_sP2Gw@mail.gmail.com> (raw)
In-Reply-To: <CAPv3WKd9t1OJ9PP-c4mZY9Knj_4Af9np2r5izx_=Q16WZjFJEw@mail.gmail.com>
Hi Leif, Ard,
czw., 25 kwi 2019 o 14:35 Marcin Wojtas <mw@semihalf.com> napisał(a):
>
> Hi Leif,
>
> czw., 25 kwi 2019 o 14:01 Leif Lindholm <leif.lindholm@linaro.org> napisał(a):
> >
> > Hi Marcin,
> >
> > Just to be awkward - edk2-platforms (and edk2-non-osi) have not yet
> > been relicensed, so we still need "contributed-under" here.
> >
> > For this series, can you confirm to the list that this is indeed
> > Contributed-under: TianoCore Contribution Agreement 1.1
> > ? If so, we can fix that up before committing.
> >
>
> Yes, I confirm that - if possible fix-up. Actually I removed those
> lines right before sending :)
>
Do you have any more comments on the patchset?
Thanks,
Marcin
> Best regards,
> Marcin
>
> > I'll start looking into what we need to do for a license transition
> > for edk2-platforms master next week.
> >
> > Best Regards,
> >
> > Leif
> >
> > On Thu, Apr 25, 2019 at 01:28:20PM +0200, Marcin Wojtas wrote:
> > > Hi,
> > >
> > > The third version of the patchset cleans up the dependencies between
> > > the SPI drivers, so that now they explicitly rely on the protocols'
> > > instead of BEFORE statements and modules GUIDs. Also force dependency
> > > between FTW and MvFvbDxe using library NULL resolution.
> > >
> > > The patches are available in the github:
> > > https://github.com/MarvellEmbeddedProcessors/edk2-open-platform/commits/fvb-upstream-r20190425
> > >
> > > I'm looking forward to your comments or remarks.
> > >
> > > Best regards,
> > > Marcin
> > >
> > > Changelog:
> > > v2->v3
> > >
> > > 3/4 (new patch)
> > > * Clean-up MvSpiFlashDxe / MvSpiOrionDxe / MvFvbDxe dependencies and
> > > rely on the protocols' GUIDs
> > >
> > > 4/4
> > > * Force dependency for loading FTW driver
> > >
> > > v1->v2
> > > * Replace clock-enabling patch with PEI phase FV cleanup
> > >
> > > Hanna Hawa (1):
> > > Marvell/Drivers: MvFvbDxe: Change Pcd parameters to be 64 bit
> > >
> > > Kornel Duleba (1):
> > > Marvell/Drivers: MvFvbDxe: Introduce non-mmio mode
> > >
> > > Marcin Wojtas (2):
> > > Marvell/Armada7k8k: Cleanup PEI phase FV
> > > Marvell/Drivers/Spi: Improve modules dependencies
> > >
> > > Silicon/Marvell/Marvell.dec | 7 +-
> > > Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc | 18 ++-
> > > Silicon/Marvell/Armada7k8k/Armada7k8k.fdf | 3 -
> > > Silicon/Marvell/Drivers/Spi/MvFvbDxe/MvFvbDxe.inf | 14 +-
> > > Silicon/Marvell/Drivers/Spi/MvSpiFlashDxe/MvSpiFlashDxe.inf | 5 +-
> > > Silicon/Marvell/Drivers/Spi/MvSpiOrionDxe/MvSpiOrionDxe.inf | 5 +-
> > > Silicon/Marvell/Drivers/Spi/MvFvbDxe/MvFvbDxe.h | 1 +
> > > Silicon/Marvell/Drivers/Spi/MvFvbDxe/MvFvbDxe.c | 149 +++++++++++++++-----
> > > 8 files changed, 140 insertions(+), 62 deletions(-)
> > >
> > > --
> > > 2.7.4
> > >
next prev parent reply other threads:[~2019-04-29 13:54 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-04-25 11:28 [edk2-platforms: PATCH v3 0/4] Armada7k8k FVB improvements Marcin Wojtas
2019-04-25 11:28 ` [edk2-platforms: PATCH v3 1/4] Marvell/Drivers: MvFvbDxe: Change Pcd parameters to be 64 bit Marcin Wojtas
2019-04-25 11:28 ` [edk2-platforms: PATCH v3 2/4] Marvell/Armada7k8k: Cleanup PEI phase FV Marcin Wojtas
2019-04-25 11:28 ` [edk2-platforms: PATCH v3 3/4] Marvell/Drivers/Spi: Improve modules dependencies Marcin Wojtas
2019-04-25 11:28 ` [edk2-platforms: PATCH v3 4/4] Marvell/Drivers: MvFvbDxe: Introduce non-mmio mode Marcin Wojtas
2019-04-25 12:01 ` [edk2-platforms: PATCH v3 0/4] Armada7k8k FVB improvements Leif Lindholm
2019-04-25 12:35 ` Marcin Wojtas
2019-04-29 13:54 ` Marcin Wojtas [this message]
2019-05-02 11:07 ` Ard Biesheuvel
2019-05-02 11:28 ` Marcin Wojtas
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=CAPv3WKdozS0TTswwuPEj_aeX10Bm4em4jLJqM47JT1Ne_sP2Gw@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