On Thu, Jul 13, 2023 at 12:08:02PM -0700, Tuan Phan wrote: > On Tue, Jul 4, 2023 at 12:01 AM Sunil V L <suni...@ventanamicro.com> wrote: > > > On Mon, Jul 03, 2023 at 11:45:45PM -0700, Tuan Phan wrote: > > > As i said, VirtNorFlashDxe needed to be loaded before VariableRuntimeDxe > > so > > > your suggestion will not work. > > > > > Okay, at least for me, by removing APRIORI patch and adding this depex, > > edk2 boots fine with your series. I am not sure what won't work. > > > > Hi Ard, any thoughts? If no better way, may be we have to use APRIORI. > > > It doesn't work as your workaround trying to make CpuDxe depends on > variable protocol which has nothing to do with it. Also, CpuDxe is an > essential module and should not depend on anything, what happens if the > variable driver before generating the protocol tries to use CPU protocol? > It is worse than having APRIORI workaround. > Okay. Thanks!. Let's go with this solution for now. But let me do some tests to ensure this builds fine with CLANG and then merge.
Thanks, Sunil -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#106929): https://edk2.groups.io/g/devel/message/106929 Mute This Topic: https://groups.io/mt/99724477/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
