Re: [PATCH] MAINTAINERS: add self as reviewer to INTEL STRATIX10 FIRMWARE DRIVERS

[Richard Gong <richard.gong@linux.intel.com>]

Hi Tom,

On 3/27/21 10:19 AM, trix@redhat.com wrote:
> From: Tom Rix <trix@redhat.com>
> 
> The Intel stratix 10 is a fpga.  I review fpga's. So I want to help
> in this related subsystem.
> 

Intel Stratix10 service layer driver is not a FPGA.

Intel FPGA SoC is composed of ARM Cortex hard processor system (HPS) and 
Secure Device Manager (SDM). SDM is the hardware which does FPGA 
configuration, remote system update (RSU), crypto service (FCS), warm 
reset and other features.

To meet the whole system security needs and support virtual machine 
requesting communication with SDM, only the secure world of software 
(EL3) can interfaces with SDM. All software entities running on the 
other exception level must channel through the EL3 software whenever it 
needs service from SDM.

Intel Stratix 10 service layer driver, added into Opensource kernel 
mainlines since version 5.0, interfaces with the service provides at EL1 
(FPGA manager, RSU and FPGA crypto service drivers as today) and 
managers secure monitor call (SMC) to communicate with the secure 
monitor software at the secure world (EL3).

Regards,
Richard

> Signed-off-by: Tom Rix <trix@redhat.com>
> ---
>   MAINTAINERS | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 67b104202602..00828de0a7bc 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -9266,6 +9266,7 @@ F:	tools/power/x86/intel-speed-select/
>   
>   INTEL STRATIX10 FIRMWARE DRIVERS
>   M:	Richard Gong <richard.gong@linux.intel.com>
> +R:	Tom Rix <trix@redhat.com >   L:	linux-kernel@vger.kernel.org
>   S:	Maintained
>   F:	Documentation/ABI/testing/sysfs-devices-platform-stratix10-rsu
>