> +    //
> +    // retrain the link at Root Port level, if its link is active
> +    //
> +    if (Level == 1 && 
> PciIoDevice->PciExpressCapability.LinkStatus.Bits.DataLinkLayerLinkActive) {
Should use 
"PciIoDevice->PciExpressCapability.LinkStatus.Bits.DataLinkLayerLinkActive != 
0" to following EDK II coding style.
Do you want to retrain the link from the root port? Then we need compare the 
Level against 0 instead of 1.

> +    //
> +    if (Level == 1 && 
> PciIoDevice->PciExpressCapability.LinkStatus.Bits.DataLinkLayerLinkActive) {
Similar comments to above line.

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#59418): https://edk2.groups.io/g/devel/message/59418
Mute This Topic: https://groups.io/mt/74118527/21656
Group Owner: [email protected]
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to