Thanks Rebecca. I fixed in the next version. Regards, Nhi ________________________________ From: Rebecca Cran OS <[email protected]> Sent: Wednesday, August 14, 2024 8:53 PM To: [email protected] <[email protected]>; Nhi Pham OS <[email protected]> Cc: [email protected] <[email protected]>; Chuong Tran OS <[email protected]>; Rebecca Cran OS <[email protected]> Subject: Re: [edk2-devel] [edk2-platforms][PATCH 1/1] JadePkg: Add ACPI SPMI table
On 8/13/2024 9:33 PM, Nhi Pham via groups.io wrote: This could be a static function. > +EFI_STATUS > +EFIAPI > +AcpiInstallSpmiTable ( > + VOID > + ); > + > #endif /* ACPI_PLATFORM_H_ */ > diff --git > a/Platform/Ampere/JadePkg/Drivers/AcpiPlatformDxe/AcpiPlatformDxe.c > b/Platform/Ampere/JadePkg/Drivers/AcpiPlatformDxe/AcpiPlatformDxe.c > index 28c422dff166..a82a93d23fa2 100644 > --- a/Platform/Ampere/JadePkg/Drivers/AcpiPlatformDxe/AcpiPlatformDxe.c > +++ b/Platform/Ampere/JadePkg/Drivers/AcpiPlatformDxe/AcpiPlatformDxe.c I believe the project now prefers the use of the C keyword "static". > + // BCD Format > + SpmiTable->SpecificationRevision = DeviceId.SpecificationVersion & 0xF0; > + SpmiTable->SpecificationRevision |= (DeviceId.SpecificationVersion & > 0x0F) << 8; It took a second to understand this. Maybe combine it into a single line, or change the '0x0F' to just '0xF'? -- Rebecca -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#120346): https://edk2.groups.io/g/devel/message/120346 Mute This Topic: https://groups.io/mt/107889268/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
