linux-kernel - Re: [PATCH v2 3/7] firmware: stratix10-svc: Add initial support for asynchronous communication with Stratix 10 service channel

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <ea381d6e-3554-429b-87ae-74451c72d4a2@altera.com>
Date: Thu, 22 May 2025 16:03:24 +0530
From: Mahesh Rao <mahesh.rao@...era.com>
To: Dinh Nguyen <dinguyen@...nel.org>, Rob Herring <robh@...nel.org>,
 Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>
Cc: Matthew Gerlach <matthew.gerlach@...era.com>,
 linux-kernel@...r.kernel.org, devicetree@...r.kernel.org
Subject: Re: [PATCH v2 3/7] firmware: stratix10-svc: Add initial support for
 asynchronous communication with Stratix 10 service channel



On 22-05-2025 01:48 am, Dinh Nguyen wrote:
> On 5/21/25 03:42, Mahesh Rao wrote:
>>
>>
>> On 19-05-2025 05:28 pm, Dinh Nguyen wrote:
>>> On 5/12/25 06:39, Mahesh Rao via B4 Relay wrote:
>>>> From: Mahesh Rao <mahesh.rao@...era.com>
>>>>
> 
> <snip>
> 
>>>> +
>>>> +/**
>>>> + * stratix10_svc_async_prepare_response - Prepare the response data 
>>>> for an asynchronous transaction.
>>>> + * @chan: Pointer to the service channel structure.
>>>> + * @handle: Pointer to the asynchronous handler structure.
>>>> + * @data: Pointer to the callback data structure.
>>>> + *
>>>> + * This function prepares the response data for an asynchronous 
>>>> transaction. It
>>>> + * extracts the response data from the SMC response structure and 
>>>> stores it in
>>>> + * the callback data structure. The function also logs the 
>>>> completion of the
>>>> + * asynchronous transaction.
>>>> + *
>>>> + * Return: 0 on success, -ENOENT if the command is invalid
>>>> + */
>>>> +static int stratix10_svc_async_prepare_response(struct 
>>>> stratix10_svc_chan *chan,
>>>> +                        struct stratix10_svc_async_handler *handle,
>>>> +                        struct stratix10_svc_cb_data *data)
>>>> +{
>>>> +    struct stratix10_svc_client_msg *p_msg =
>>>> +        (struct stratix10_svc_client_msg *)handle->msg;
>>>> +    struct stratix10_svc_controller *ctrl = chan->ctrl;
>>>> +
>>>> +    data->status = STRATIX10_GET_SDM_STATUS_CODE(handle->res.a1);
>>>> +
>>>> +    switch (p_msg->command) {
>>>> +    default:
>>>> +        dev_alert(ctrl->dev, "Invalid command\n ,%d", p_msg->command);
>>>> +        return -ENOENT;
>>>> +    }
>>>
>>> What is the above code doing?
>>
>> This function prepares the response  for clients after retriving the 
>> response from the Arm Trusted Firmware using the polling call. 
>> Currently only the negative scenario is shown for incvalid command, 
>> the last patch in series adds command for hwmon for the positive 
>> scenario.
>>
> 
> Okay, but this is confusing. Please just add this to the last patch then.
> 

Shall I combine the last patch for adding hwmon commands here so that 
the framework has some command usage?.