Re: [PATCH 1/1] firmware: stratix10-svc: add missing comment for fcs client

[Dinh Nguyen <dinguyen@kernel.org>]

Hi Khairul,

On 3/14/26 22:18, Khairul Anuar Romli wrote:
> Add the missing comment describing the fcs client as "fpga crypto services"
> in the SVC client list. This keeps the documentation consistent with other
> clients (fpga, rsu, hwmon) and improves readibility and mantainability.
> 
> Signed-off-by: Khairul Anuar Romli <karom.9560@gmail.com>
> ---
>   include/linux/firmware/intel/stratix10-svc-client.h | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/include/linux/firmware/intel/stratix10-svc-client.h b/include/linux/firmware/intel/stratix10-svc-client.h
> index d290060f4c73..74adf34d6142 100644
> --- a/include/linux/firmware/intel/stratix10-svc-client.h
> +++ b/include/linux/firmware/intel/stratix10-svc-client.h
> @@ -12,6 +12,7 @@
>    *
>    * fpga: for FPGA configuration
>    * rsu: for remote status update
> + * fcs: for fpga crypto services


Thanks for your patch, but for now there is no fcs in the driver yet. In 
order to minimize churn, we'll add this when there is an fcs driver.

Dinh