> > + Status = EnumeratePciExpressFeatures (
1. "enumerate" means "visit". But I think this function is not just visiting
the features but also
programming them. So, How about "ProgramPciExpressFeatures"?
(I gave a similar review comment in last time review in Dec.)
2. In mail https://edk2.groups.io/g/devel/message/52399 I proposed to simplify
to 4 phases.
Did you find any issue with my proposal?
Thanks,
Ray
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#54114): https://edk2.groups.io/g/devel/message/54114
Mute This Topic: https://groups.io/mt/71063383/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
