| Message ID | 20220912232526.27427-4-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 qcom-ngd-ctrl. > > Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com> Reviewed-by: Sibi Sankar <quic_sibis@quicinc.com> > --- > drivers/slimbus/qcom-ngd-ctrl.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/slimbus/qcom-ngd-ctrl.c b/drivers/slimbus/qcom-ngd-ctrl.c > index 0aa8408464ad..931ab6317467 100644 > --- a/drivers/slimbus/qcom-ngd-ctrl.c > +++ b/drivers/slimbus/qcom-ngd-ctrl.c > @@ -220,7 +220,7 @@ struct slimbus_power_resp_msg_v01 { > struct qmi_response_type_v01 resp; > }; > > -static struct qmi_elem_info slimbus_select_inst_req_msg_v01_ei[] = { > +static const struct qmi_elem_info slimbus_select_inst_req_msg_v01_ei[] = { > { > .data_type = QMI_UNSIGNED_4_BYTE, > .elem_len = 1, > @@ -262,7 +262,7 @@ static struct qmi_elem_info slimbus_select_inst_req_msg_v01_ei[] = { > }, > }; > > -static struct qmi_elem_info slimbus_select_inst_resp_msg_v01_ei[] = { > +static const struct qmi_elem_info slimbus_select_inst_resp_msg_v01_ei[] = { > { > .data_type = QMI_STRUCT, > .elem_len = 1, > @@ -284,7 +284,7 @@ static struct qmi_elem_info slimbus_select_inst_resp_msg_v01_ei[] = { > }, > }; > > -static struct qmi_elem_info slimbus_power_req_msg_v01_ei[] = { > +static const struct qmi_elem_info slimbus_power_req_msg_v01_ei[] = { > { > .data_type = QMI_UNSIGNED_4_BYTE, > .elem_len = 1, > @@ -324,7 +324,7 @@ static struct qmi_elem_info slimbus_power_req_msg_v01_ei[] = { > }, > }; > > -static struct qmi_elem_info slimbus_power_resp_msg_v01_ei[] = { > +static const struct qmi_elem_info slimbus_power_resp_msg_v01_ei[] = { > { > .data_type = QMI_STRUCT, > .elem_len = 1, >
On 13/09/2022 00:25, Jeff Johnson wrote: > Commit ff6d365898d ("soc: qcom: qmi: use con SHA ID should be at least 12 chars long. Same comment for all the patches in the series. st for struct > qmi_elem_info") allows QMI message encoding/decoding rules to be > const, so do that for qcom-ngd-ctrl. > > Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com> Other than that it LGTM, Once fixed: Acked-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> --srini > --- > drivers/slimbus/qcom-ngd-ctrl.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/slimbus/qcom-ngd-ctrl.c b/drivers/slimbus/qcom-ngd-ctrl.c > index 0aa8408464ad..931ab6317467 100644 > --- a/drivers/slimbus/qcom-ngd-ctrl.c > +++ b/drivers/slimbus/qcom-ngd-ctrl.c > @@ -220,7 +220,7 @@ struct slimbus_power_resp_msg_v01 { > struct qmi_response_type_v01 resp; > }; > > -static struct qmi_elem_info slimbus_select_inst_req_msg_v01_ei[] = { > +static const struct qmi_elem_info slimbus_select_inst_req_msg_v01_ei[] = { > { > .data_type = QMI_UNSIGNED_4_BYTE, > .elem_len = 1, > @@ -262,7 +262,7 @@ static struct qmi_elem_info slimbus_select_inst_req_msg_v01_ei[] = { > }, > }; > > -static struct qmi_elem_info slimbus_select_inst_resp_msg_v01_ei[] = { > +static const struct qmi_elem_info slimbus_select_inst_resp_msg_v01_ei[] = { > { > .data_type = QMI_STRUCT, > .elem_len = 1, > @@ -284,7 +284,7 @@ static struct qmi_elem_info slimbus_select_inst_resp_msg_v01_ei[] = { > }, > }; > > -static struct qmi_elem_info slimbus_power_req_msg_v01_ei[] = { > +static const struct qmi_elem_info slimbus_power_req_msg_v01_ei[] = { > { > .data_type = QMI_UNSIGNED_4_BYTE, > .elem_len = 1, > @@ -324,7 +324,7 @@ static struct qmi_elem_info slimbus_power_req_msg_v01_ei[] = { > }, > }; > > -static struct qmi_elem_info slimbus_power_resp_msg_v01_ei[] = { > +static const struct qmi_elem_info slimbus_power_resp_msg_v01_ei[] = { > { > .data_type = QMI_STRUCT, > .elem_len = 1,
On 9/16/2022 6:06 AM, Srinivas Kandagatla wrote: > > > On 13/09/2022 00:25, Jeff Johnson wrote: >> Commit ff6d365898d ("soc: qcom: qmi: use con > > SHA ID should be at least 12 chars long. > > Same comment for all the patches in the series. > > > st for struct >> qmi_elem_info") allows QMI message encoding/decoding rules to be >> const, so do that for qcom-ngd-ctrl. >> >> Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com> > > Other than that it LGTM, > Once fixed: > > Acked-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> > > > --srini that was corrected in v2. thx for the ack
diff --git a/drivers/slimbus/qcom-ngd-ctrl.c b/drivers/slimbus/qcom-ngd-ctrl.c index 0aa8408464ad..931ab6317467 100644 --- a/drivers/slimbus/qcom-ngd-ctrl.c +++ b/drivers/slimbus/qcom-ngd-ctrl.c @@ -220,7 +220,7 @@ struct slimbus_power_resp_msg_v01 { struct qmi_response_type_v01 resp; }; -static struct qmi_elem_info slimbus_select_inst_req_msg_v01_ei[] = { +static const struct qmi_elem_info slimbus_select_inst_req_msg_v01_ei[] = { { .data_type = QMI_UNSIGNED_4_BYTE, .elem_len = 1, @@ -262,7 +262,7 @@ static struct qmi_elem_info slimbus_select_inst_req_msg_v01_ei[] = { }, }; -static struct qmi_elem_info slimbus_select_inst_resp_msg_v01_ei[] = { +static const struct qmi_elem_info slimbus_select_inst_resp_msg_v01_ei[] = { { .data_type = QMI_STRUCT, .elem_len = 1, @@ -284,7 +284,7 @@ static struct qmi_elem_info slimbus_select_inst_resp_msg_v01_ei[] = { }, }; -static struct qmi_elem_info slimbus_power_req_msg_v01_ei[] = { +static const struct qmi_elem_info slimbus_power_req_msg_v01_ei[] = { { .data_type = QMI_UNSIGNED_4_BYTE, .elem_len = 1, @@ -324,7 +324,7 @@ static struct qmi_elem_info slimbus_power_req_msg_v01_ei[] = { }, }; -static struct qmi_elem_info slimbus_power_resp_msg_v01_ei[] = { +static const struct qmi_elem_info slimbus_power_resp_msg_v01_ei[] = { { .data_type = QMI_STRUCT, .elem_len = 1,
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 qcom-ngd-ctrl. Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com> --- drivers/slimbus/qcom-ngd-ctrl.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)