On Thu, 12 Oct 2023 at 11:10, Laszlo Ersek <ler...@redhat.com> wrote: > > https://bugzilla.tianocore.org/show_bug.cgi?id=4564 > > The RealTimeClockLib class header in edk2's EmbeddedPkg mistakenly > declares a function called LibRtcVirtualNotifyEvent(). No component ever > calls this function across module boundaries; all RealTimeClockLib > instances in edk2 and edk2-platforms are supposed to register -- and do > register -- their SetVirtualAddressMap() notification functions. > > In a sibling series for edk2, we're going to remove the > LibRtcVirtualNotifyEvent() API from the RealTimeClockLib *class* header. > In edk2-platforms, that means the following: > > (1) Fix existent (independent) build failures / bitrot. This is a > pre-requisite for showing that the other changes don't regress the > build. Build regression testing is only possible if you have a build > that completes in the first place. > > Patch #1 enables the NOOPT target for Hisilicon platforms, so that they > can be more quickly built: Hisilicon needs several fixes for existent > build issues. Patches #2 through #7 fix those issues. > > In particular patch#4 ("Hisilicon: rename OemMiscLib class to > HisiOemMiscLib") requires an ek2-non-osi complement; please see that in > the sibling edk2-non-osi posting ("Hisilicon: rename OemMiscLib class > dependencies to HisiOemMiscLib"). > > Patch #12 fixes prior build breakage in AmpereAltraPkg. > > (2) In those RealTimeClockLib instances that define > LibRtcVirtualNotifyEvent(), demonstrate that either (a) the usage is > module-internal, or (b) there is no usage. > > In case (a), rename LibRtcVirtualNotifyEvent() to VirtualNotifyEvent(), > and make it static. If needed, hoist the function definition above the > reference. In case (b), remove the function definition. > > (3) Some affected RealTimeClockLib instances turn out to be entirely > unused / superfluous; drop those altogether. > > Cc: Ard Biesheuvel <ardb+tianoc...@kernel.org> > Cc: Bibo Mao <maob...@loongson.cn> > Cc: Chao Li <lic...@loongson.cn> > Cc: Chuong Tran <chu...@os.amperecomputing.com> > Cc: Leif Lindholm <quic_llind...@quicinc.com> > Cc: Ling Jia <jial...@phytium.com.cn> > Cc: Marcin Wojtas <m...@semihalf.com> > Cc: Masami Hiramatsu <masami.hirama...@linaro.org> > Cc: Meenakshi Aggarwal <meenakshi.aggar...@nxp.com> > Cc: Nhi Pham <n...@os.amperecomputing.com> > Cc: Peng Xie <xiep...@phytium.com.cn> > Cc: Rebecca Cran <rebe...@os.amperecomputing.com> > Cc: Wenyi Xie <xiewen...@huawei.com> > Cc: Xianglai li <lixiang...@loongson.cn> > Cc: Yiqi Shu <shuy...@phytium.com.cn> > > Thanks > Laszlo > > Laszlo Ersek (19): > Hisilicon: enable NOOPT builds > Hisilicon/FlashFvbDxe: fix PcdNorFlashCheckBlockLocked token space > GUID > Hisilicon.dsc.inc: resolve VariableFlashInfoLib > Hisilicon: rename OemMiscLib class to HisiOemMiscLib > Hisilicon: add missing include file to Pptt components > Hisilicon/Hi1620AcpiTables: fix up ASL > Hisilicon/ProcessorSubClassDxe: drop conflicting PROCESSOR_STATUS_DATA > type > Hisilicon: drop unused DS3231RealTimeClockLib instance > Hisilicon: drop unused VirtualRealTimeClockLib instance > Hisilicon/DS3231RealTimeClockLib: drop LibRtcVirtualNotifyEvent > Hisilicon/RX8900RealTimeClockLib: drop LibRtcVirtualNotifyEvent > AmpereAltraPkg/Ac01PcieLib: fix compilation error > JadePkg/PCF85063RealTimeClockLib: hide LibRtcVirtualNotifyEvent > LoongArchQemuPkg/LsRealTimeClockLib: hide LibRtcVirtualNotifyEvent > Styx/RealTimeClockLib: hide LibRtcVirtualNotifyEvent > Armada7k8k/RealTimeClockLib: hide LibRtcVirtualNotifyEvent > NXP/Pcf8563RealTimeClockLib: hide LibRtcVirtualNotifyEvent > FT2000-4Pkg/RealTimeClockLib: hide LibRtcVirtualNotifyEvent > Omap35xxPkg/RealTimeClockLib: drop LibRtcVirtualNotifyEvent >
Thanks a lot for cleaning this up! For the series, Reviewed-by: Ard Biesheuvel <a...@kernel.org> Pushed as d6e36a151ff8..06f6274d5642 edk2-non-osi patch pushed as d6e36a151ff8..06f6274d5642 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#109584): https://edk2.groups.io/g/devel/message/109584 Mute This Topic: https://groups.io/mt/101914650/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-