Re: 回复: [EXT] Re: [PATCH v3 08/17] scmi_protocols: update struct scmi_base_discover_list_protocols_out

6 Jan 2025


      On 1/6/25 11:41 AM, Alice Guo (OSS) wrote:
...
...
...
@@ -141,11 +141,14 @@ struct scmi_base_discover_impl_version_out {
    * @status:         SCMI command status
    * @num_protocols:  Number of SCMI protocols in @protocol
    * @protocols:              Array of packed SCMI protocol ID's






The number of elements of @protocols is specified by callee-side.


Currently,

set it to 4 is enough for i.MX95.
*/

struct scmi_base_discover_list_protocols_out {
     s32 status;
     u32 num_protocols;


u32 protocols[3];




u32 protocols[4];



Can this be made somehow future-proof? For example, figure out how many
protocols are supported, and then malloc this structure with the right size and
query again ?
Hi Marek,
Currently, 4 is enough. 10 protocols are defined in Arm® System Control and Management Interface Platform Design Document Version 3.2. I can make a new patch as you said in the future.
Or we can future-proof it to 10 right now. Are there any downsides of that ?

    

Marek Vasut