public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "PierreGondois" <pierre.gondois@arm.com>
To: Rebecca Cran <rebecca@nuviainc.com>,
	Sami Mujawar <Sami.Mujawar@arm.com>,
	Ard Biesheuvel <ardb@kernel.org>
Cc: edk2-devel-groups-io <devel@edk2.groups.io>,
	Ard Biesheuvel <ardb+tianocore@kernel.org>,
	Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>,
	Leif Lindholm <leif@nuviainc.com>,
	Masami Hiramatsu <masami.hiramatsu@linaro.org>,
	Graeme Gregory <graeme@nuviainc.com>,
	Radoslaw Biernacki <rad@semihalf.com>,
	Marcin Wojtas <mw@semihalf.com>, nd <nd@arm.com>
Subject: Re: [edk2-platforms PATCH 0/8] Update Arm platforms following addition of EFI_MP_SERVICES_PROTOCOL support in edk2
Date: Thu, 16 Dec 2021 16:26:29 +0100	[thread overview]
Message-ID: <ec3508b6-bfad-b54e-9d5c-9c71d5b5535f@arm.com> (raw)
In-Reply-To: <a2bda28a-c7e6-ba82-737b-c09b8727d478@nuviainc.com>

Hi Rebecca,

On 12/16/21 3:59 PM, Rebecca Cran wrote:
> On 12/16/21 6:26 AM, Sami Mujawar wrote:
>
>> [SAMI] I posted the following patches to fix some of the issues you mention above:
>> Platform/Socionext: Fix build break for SynQuacerEvalBoard (https://edk2.groups.io/g/devel/message/83553)
>> Platform/LeMaker: Fix build break for CelloBoard (https://edk2.groups.io/g/devel/message/83556)
>> Platform/Socionext/SynQuacerEvalBoard: Add missing ArmSoftFloatLib (https://edk2.groups.io/g/devel/message/83554)
>> Platform/Socionext/SynQuacerEvalBoard: Disable EbcDxe for AArch32  Builds (https://edk2.groups.io/g/devel/message/83557)
>>
>> If it helps, I can carve out these patches in a separate series and post them to the list.
>>
>> I feel we need a CI for edk2-platforms. Without this it is going to be hard to keep track of such build breaks.
>> [/SAMI]
>
>
> Thanks, I'll take a look at those and post any comments.
>
> I agree, CI for edk2-platforms would be good. I'm hoping to find some time next week to work on it. A useful step, even if it's just the first of many, would be to simply run lots of 'build' commands for each platform.
>
>
The following patchset should enable a CI in edk2-platforms for the Juno platorm, but it was tested before uncrustify was enabled. The patch-set is also relying on a modification in edk2-pytool-extensions:
https://edk2.groups.io/g/devel/message/83803
https://github.com/tianocore/edk2-pytool-extensions/pull/279

There is a bugzilla on this topic at:
https://bugzilla.tianocore.org/show_bug.cgi?id=3509

Feel free to use the patches (or not) and to take the bugzilla (or not). It didn't seem the feature was really appealing until now,

Regards,

Pierre


  reply	other threads:[~2021-12-16 15:26 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-12-15  0:03 [edk2-platforms PATCH 0/8] Update Arm platforms following addition of EFI_MP_SERVICES_PROTOCOL support in edk2 Rebecca Cran
2021-12-15  0:03 ` [edk2-platforms PATCH 1/8] Platform/ARM: Add MpInitLib instance Rebecca Cran
2021-12-15  0:03 ` [edk2-platforms PATCH 2/8] Platform/Socionext: Add instance of MpInitLib Rebecca Cran
2021-12-15  0:03 ` [edk2-platforms PATCH 3/8] Silicon/Marvell: " Rebecca Cran
2021-12-16  7:14   ` Marcin Wojtas
2021-12-15  0:03 ` [edk2-platforms PATCH 4/8] Platform/Qemu: " Rebecca Cran
2021-12-15  0:03 ` [edk2-platforms PATCH 5/8] Platform/ARM: Update ARM_CORE_INFO initializer for MPIDR field change Rebecca Cran
2021-12-15  0:03 ` [edk2-platforms PATCH 6/8] Silicon/Marvell: " Rebecca Cran
2021-12-15  0:03 ` [edk2-platforms PATCH 7/8] Silicon/Socionext: " Rebecca Cran
2021-12-15  0:03 ` [edk2-platforms PATCH 8/8] Silicon/Qemu: " Rebecca Cran
2021-12-15  8:16 ` [edk2-platforms PATCH 0/8] Update Arm platforms following addition of EFI_MP_SERVICES_PROTOCOL support in edk2 Ard Biesheuvel
2021-12-16  9:32 ` Ard Biesheuvel
2021-12-16 13:13   ` Rebecca Cran
2021-12-16 13:16     ` Ard Biesheuvel
2021-12-16 13:38       ` [edk2-devel] " Marcin Juszkiewicz
2021-12-16 15:05         ` Ard Biesheuvel
2021-12-16 18:54           ` Sami Mujawar
2021-12-16 18:55             ` Ard Biesheuvel
2021-12-16 22:09       ` Rebecca Cran
2021-12-19 10:32         ` Marcin Wojtas
2022-01-10 20:33           ` Rebecca Cran
2021-12-16 13:26     ` Sami Mujawar
2021-12-16 14:59       ` Rebecca Cran
2021-12-16 15:26         ` PierreGondois [this message]
2021-12-16 15:44           ` Rebecca Cran
2021-12-16 15:27       ` Rebecca Cran

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=ec3508b6-bfad-b54e-9d5c-9c71d5b5535f@arm.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