[PATCH v1] scmi_protocols: Update discover list protocol message

[Tom Rini]

On Sun, Oct 06, 2024 at 05:50:22PM +0800, alice.guo at oss.nxp.com wrote:
> From: Ye Li <ye.li at nxp.com>
> 
> When work with System Manager, System Manager provides vendor specific
> protocol, so the protocol array size is not enough.
> 
> Signed-off-by: Ye Li <ye.li at nxp.com>
> Signed-off-by: Alice Guo <alice.guo at nxp.com>
> ---
>  include/scmi_protocols.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/scmi_protocols.h b/include/scmi_protocols.h
> index 43b66be00a..6204ce076e 100644
> --- a/include/scmi_protocols.h
> +++ b/include/scmi_protocols.h
> @@ -146,7 +146,7 @@ struct scmi_base_discover_impl_version_out {
>  struct scmi_base_discover_list_protocols_out {
>  	s32 status;
>  	u32 num_protocols;
> -	u32 protocols[3];
> +	u32 protocols[4];
>  };
>  
>  /**

Changing a constant from 3 to 4 isn't good, please define something and
add a comment explaining what the constant means so it'll be clear
when/why someone changes it to 5. Thanks.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20241007/abd3426b/attachment.sig>