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: <40b939d6-9339-444f-a2e4-3e4cd4bcb317@kernel.org>
Date: Wed, 21 May 2025 15:18:48 -0500
From: Dinh Nguyen <dinguyen@...nel.org>
To: Mahesh Rao <mahesh.rao@...era.com>, 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 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.