On 02/12/20 03:52, Ni, Ray wrote:
> 
>> +EFI_STATUS
>> +PlatformShadowMicrocode (
>> +  IN OUT CPU_MP_DATA             *CpuMpData
>> +  )
>> +{
>> +  return EFI_UNSUPPORTED;
>> +}
> 
> Siyuan,
> Better to have comments above "return EFI_UNSUPPORTED" to clarify the
> existing supporting scope.
> 
> With that, Reviewed-by: Ray Ni <ray...@intel.com>
> 
> + Mike for potential comments since he was involved in the offline discussion
> and the final patch doesn't exactly reflect the discussion output.

OK, I'll wait with my ACK until Mike's approval then. Otherwise the v2
changes look OK to me (including your request for more comments).

Thanks
Laszlo


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

View/Reply Online (#54276): https://edk2.groups.io/g/devel/message/54276
Mute This Topic: https://groups.io/mt/71166759/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to