Re: [edk2-devel] [staging/dynamictables-reorg PATCH 15/15] DynamicTablesPkg: FdtHwInfoParserLib: Create wrapper to get INTC addr cells

2024-07-03 Thread Sami Mujawar
Hi Sunil, Thank you for the review. > +EFI_STATUS > +EFIAPI > +FdtGetIntcAddressCells ( > + IN CONST VOID *Fdt, > + IN INT32 Node, > + OUT INT32 *AddressCells, OPTIONAL > + OUT INT32 *SizeCells OPTIONAL NIT: I might be wrong but alignment doesn't look correct. [SAMI] I wonder if this is an artifa

Re: [edk2-devel] [staging/dynamictables-reorg PATCH 00/15] Prepare libraries to support other archs

2024-07-03 Thread Sami Mujawar
Hi Pierre, Overall, this patch series looks good to me. I have some minor comments regarding the return value for the Arch hook functions their placement in Common folder. e.g. in Patch "DynamicTablesPkg: AcpiFadtLib: Prepare to support other archs" The file DynamicTablesPkg/Library/Acpi/Common

Re: [edk2-devel] [staging/dynamictables-reorg PATCH 00/15] Prepare libraries to support other archs

2024-07-03 Thread Sunil V L
Hi Pierre, Sami, Thanks a lot again for this work!. The series looks good to me as well. I agree with Sami's suggestions. However, I have a request for an additional change. The common ACPI tables still use ARMH/ARMLTD as the CREATOR_ID/OEM_ID. Can they be made architecture specific? Reviewed-b

Re: [edk2-devel] [staging/dynamictables-reorg PATCH 00/15] Prepare libraries to support other archs

2024-07-03 Thread Sami Mujawar
Hi Sunil, I think we can look into that. My initial thoughts are that this can be solved using a Pcd. However, we need a bit of investigation. Is it ok if we address that in a separate patch? Regards, Sami Mujawar On 03/07/2024, 10:37, "Sunil V L" mailto:suni...@ventanamicro.com>> wrote: Hi

Re: [edk2-devel] [staging/dynamictables-reorg PATCH 00/15] Prepare libraries to support other archs

2024-07-03 Thread Sunil V L
On Wed, Jul 03, 2024 at 09:39:22AM +, Sami Mujawar wrote: > Hi Sunil, > > I think we can look into that. My initial thoughts are that this can be > solved using a Pcd. However, we need a bit of investigation. > Is it ok if we address that in a separate patch? > Sure!. Thanks, Sunil > Regard

[edk2-devel] [staging/dynamictables-reorg PATCH 0/2] DynamicTablesPkg: CI related fixes

2024-07-03 Thread PierreGondois
Small fixes reported while running the CI. These patches should be applied on top of: - [staging/dynamictables-reorg PATCH 00/15] Prepare libraries to support other archs https://edk2.groups.io/g/devel/message/119632 Cc: AbdulLateef Attar Cc: Girish Mahadevan Cc: Jeff Brasen Cc: Jeshua Smit

[edk2-devel] [staging/dynamictables-reorg PATCH 1/2] DynamicTablesPkg: Fix conversion compiler warnings

2024-07-03 Thread PierreGondois
Some CM objects fields are wider than the targeted field in ACPI tables. Some assignments are also subject to data loss and trigger the following warnings: - '<': signed/unsigned mismatch - '=': conversion from 'UINTxx' to 'UINTyy', possible loss of data with xx > yy. Add checks/cast to remove the

[edk2-devel] [staging/dynamictables-reorg PATCH 2/2] DynamicTablesPkg: Add EFIAPI to generators hooks

2024-07-03 Thread PierreGondois
For X64 builds, the EFIAPI is replaced by '(__attribute__((ms_abi))'. This might lead to build error for some ACPI tablte generators due to function prototype mismatch. Add the EFIAPI to ACPI table generator hooks: - ACPI_TABLE_GENERATOR_BUILD_TABLEEX - ACPI_TABLE_GENERATOR_FREE_TABLEEX Signed-of

Re: [edk2-devel] [staging/dynamictables-reorg PATCH 00/15] Prepare libraries to support other archs

2024-07-03 Thread PierreGondois
Hello Sami, On 7/3/24 11:08, Sami Mujawar wrote: Hi Pierre, Overall, this patch series looks good to me. I have some minor comments regarding the return value for the Arch hook functions their placement in Common folder. e.g. in Patch "DynamicTablesPkg: AcpiFadtLib: Prepare to support other a

Re: [edk2-devel] [staging/dynamictables-reorg PATCH 1/2] DynamicTablesPkg: Fix conversion compiler warnings

2024-07-03 Thread Sami Mujawar
Hi Pierre, Thank you for these patches. On 03/07/2024, 10:54, "Pierre Gondois" mailto:pierre.gond...@arm.com>> wrote: Some CM objects fields are wider than the targeted field in ACPI tables. Some assignments are also subject to data loss and trigger the following warnings: - '<': signed/unsig

Re: [edk2-devel] [staging/dynamictables-reorg PATCH 2/2] DynamicTablesPkg: Add EFIAPI to generators hooks

2024-07-03 Thread Sami Mujawar
Hi Pierre, Thank you for this patch. These changes look good to me. Reviewed-by: Sami Mujawar Regards, Sami Mujawar On 03/07/2024, 10:54, "Pierre Gondois" mailto:pierre.gond...@arm.com>> wrote: For X64 builds, the EFIAPI is replaced by '(__attribute__((ms_abi))'. This might lead to build

Re: [edk2-devel] [staging/dynamictables-reorg PATCH 1/2] DynamicTablesPkg: Fix conversion compiler warnings

2024-07-03 Thread PierreGondois
Hi Sami, On 7/3/24 12:35, Sami Mujawar wrote: Hi Pierre, Thank you for these patches. On 03/07/2024, 10:54, "Pierre Gondois" mailto:pierre.gond...@arm.com>> wrote: Some CM objects fields are wider than the targeted field in ACPI tables. Some assignments are also subject to data loss and tr

[edk2-devel] How to optimize boot time?

2024-07-03 Thread Hamit Can Karaca
Hi, So I've a SBL+EDK2 customized firmware. This firmware enters the BIOS menu in approximately 17 seconds. I haven't seen this as a problem for a long time, but I've seen some CFL motherboards boot up the OS in 17 seconds. The motherboard I use is CFL-H. Is there a solution that will allow me

[edk2-devel] [PATCH edk2-platforms v2] SbsaQemu: use FEAT_RNG for EFI_RNG_PROTOCOL

2024-07-03 Thread Marcin Juszkiewicz
# FAT filesystem + GPT/MBR partitioning + UDF filesystem --- base-commit: c7ed8deaa8c1d7ee83af994b2c90d4490ef27bdc change-id: 20240703-efi-rng-protocol-be991536709a Best regards, -- Marcin Juszkiewicz -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Re

[edk2-devel] Event: TianoCore edk2-test Bug Triage Meeting - Thursday, July 4, 2024 #cal-reminder

2024-07-03 Thread Group Notification
*Reminder: TianoCore edk2-test Bug Triage Meeting* *When:* Thursday, July 4, 2024 10:00pm to 11:00pm (UTC+08:00) Asia/Shanghai *Where:* https://armltd.zoom.us/j/94348061758?pwd=Q3RDeFA5K2JFaU5jdWUxc1FnaGdyUT09&from=addon *Organizer:* Edhaya Chandran edhaya.chand...@arm.com ( edhaya.chand...@arm

Re: [edk2-devel] Moving edk2-platforms reviews to GitHub Pull Requests

2024-07-03 Thread Rebecca Cran
On 7/1/24 13:16, Rebecca Cran via groups.io wrote: Now that edk2 has been using PRs for a few weeks, I'd like to propose enabling the same workflow for edk2-platfoms. As maintainers or reviewers of platforms in the edk2-platforms repo, I'd like to get any feedback on moving from email-based re

Re: [edk2-devel] Moving edk2-platforms reviews to GitHub Pull Requests

2024-07-03 Thread Chang, Abner via groups.io
[AMD Official Use Only - AMD Internal Distribution Only] Hi Rebecca, Having GitHub CODEOWNERS to assign reviewers for the PR is fine to me, as well as only maintainers have the privilege to set "Push" label. I would suggest we use only one CODEOWNERS file on the repo so we can easily find the co