On Sun, Oct 06, 2024 at 05:50:22PM +0800, [email protected] wrote: > From: Ye Li <[email protected]> > > When work with System Manager, System Manager provides vendor specific > protocol, so the protocol array size is not enough. > > Signed-off-by: Ye Li <[email protected]> > Signed-off-by: Alice Guo <[email protected]> > --- > 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
signature.asc
Description: PGP signature
