Please add some comments above "if" to explain why the it's only skipped in S3 boot.
With the updated comments, Reviewed-by: Ray Ni <ray...@intel.com>

> +    if ((BootMode == BOOT_ON_S3_RESUME) &&
> +        (NvmeS3SkipThisController (DevicePath, DevicePathLength))) {
> +      DEBUG ((
> +        DEBUG_ERROR, "%a: Controller %d is skipped during S3.\n",
> +        __FUNCTION__, Controller
> +        ));
> +      Controller++;
> +      continue;
> +    }
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to