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

[Marek Vasut]

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 ?