| Message ID | 20220912232526.27427-3-quic_jjohnson@quicinc.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | Make QMI message rules const | expand |
On 9/13/22 4:55 AM, Jeff Johnson wrote: > Commit ff6d365898d ("soc: qcom: qmi: use const for struct > qmi_elem_info") allows QMI message encoding/decoding rules to be > const, so do that for sysmon. > > Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com> Reviewed-by: Sibi Sankar <quic_sibis@quicinc.com> > --- > drivers/remoteproc/qcom_sysmon.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/remoteproc/qcom_sysmon.c b/drivers/remoteproc/qcom_sysmon.c > index 57dde2a69b9d..3992bb61d2ec 100644 > --- a/drivers/remoteproc/qcom_sysmon.c > +++ b/drivers/remoteproc/qcom_sysmon.c > @@ -190,7 +190,7 @@ struct ssctl_shutdown_resp { > struct qmi_response_type_v01 resp; > }; > > -static struct qmi_elem_info ssctl_shutdown_resp_ei[] = { > +static const struct qmi_elem_info ssctl_shutdown_resp_ei[] = { > { > .data_type = QMI_STRUCT, > .elem_len = 1, > @@ -211,7 +211,7 @@ struct ssctl_subsys_event_req { > u32 evt_driven; > }; > > -static struct qmi_elem_info ssctl_subsys_event_req_ei[] = { > +static const struct qmi_elem_info ssctl_subsys_event_req_ei[] = { > { > .data_type = QMI_DATA_LEN, > .elem_len = 1, > @@ -269,7 +269,7 @@ struct ssctl_subsys_event_resp { > struct qmi_response_type_v01 resp; > }; > > -static struct qmi_elem_info ssctl_subsys_event_resp_ei[] = { > +static const struct qmi_elem_info ssctl_subsys_event_resp_ei[] = { > { > .data_type = QMI_STRUCT, > .elem_len = 1, > @@ -283,7 +283,7 @@ static struct qmi_elem_info ssctl_subsys_event_resp_ei[] = { > {} > }; > > -static struct qmi_elem_info ssctl_shutdown_ind_ei[] = { > +static const struct qmi_elem_info ssctl_shutdown_ind_ei[] = { > {} > }; > >
diff --git a/drivers/remoteproc/qcom_sysmon.c b/drivers/remoteproc/qcom_sysmon.c index 57dde2a69b9d..3992bb61d2ec 100644 --- a/drivers/remoteproc/qcom_sysmon.c +++ b/drivers/remoteproc/qcom_sysmon.c @@ -190,7 +190,7 @@ struct ssctl_shutdown_resp { struct qmi_response_type_v01 resp; }; -static struct qmi_elem_info ssctl_shutdown_resp_ei[] = { +static const struct qmi_elem_info ssctl_shutdown_resp_ei[] = { { .data_type = QMI_STRUCT, .elem_len = 1, @@ -211,7 +211,7 @@ struct ssctl_subsys_event_req { u32 evt_driven; }; -static struct qmi_elem_info ssctl_subsys_event_req_ei[] = { +static const struct qmi_elem_info ssctl_subsys_event_req_ei[] = { { .data_type = QMI_DATA_LEN, .elem_len = 1, @@ -269,7 +269,7 @@ struct ssctl_subsys_event_resp { struct qmi_response_type_v01 resp; }; -static struct qmi_elem_info ssctl_subsys_event_resp_ei[] = { +static const struct qmi_elem_info ssctl_subsys_event_resp_ei[] = { { .data_type = QMI_STRUCT, .elem_len = 1, @@ -283,7 +283,7 @@ static struct qmi_elem_info ssctl_subsys_event_resp_ei[] = { {} }; -static struct qmi_elem_info ssctl_shutdown_ind_ei[] = { +static const struct qmi_elem_info ssctl_shutdown_ind_ei[] = { {} };
Commit ff6d365898d ("soc: qcom: qmi: use const for struct qmi_elem_info") allows QMI message encoding/decoding rules to be const, so do that for sysmon. Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com> --- drivers/remoteproc/qcom_sysmon.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)