Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux
1
fork

Configure Feed

Select the types of activity you want to include in your feed.

soc: qcom: pdr: Use the unified QMI service ID instead of defining it locally

Instead of defining a local macro with a custom name for the QMI
service identifier, use the one provided in qmi.h and remove the
locally defined macro.

Signed-off-by: Daniel Lezcano <daniel.lezcano@oss.qualcomm.com>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
Link: https://lore.kernel.org/r/20260309230346.3584252-7-daniel.lezcano@oss.qualcomm.com
Signed-off-by: Bjorn Andersson <andersson@kernel.org>

authored by

Daniel Lezcano and committed by
Bjorn Andersson
24f80cbe 6fc1aa70

+3 -6
+2 -2
drivers/soc/qcom/pdr_interface.c
··· 523 523 if (!pds) 524 524 return ERR_PTR(-ENOMEM); 525 525 526 - pds->service = SERVREG_NOTIFIER_SERVICE; 526 + pds->service = QMI_SERVICE_ID_SERVREG_NOTIF; 527 527 strscpy(pds->service_name, service_name, sizeof(pds->service_name)); 528 528 strscpy(pds->service_path, service_path, sizeof(pds->service_path)); 529 529 pds->need_locator_lookup = true; ··· 678 678 if (ret < 0) 679 679 goto destroy_indack; 680 680 681 - ret = qmi_add_lookup(&pdr->locator_hdl, SERVREG_LOCATOR_SERVICE, 1, 1); 681 + ret = qmi_add_lookup(&pdr->locator_hdl, QMI_SERVICE_ID_SERVREG_LOC, 1, 1); 682 682 if (ret < 0) 683 683 goto release_qmi_handle; 684 684
-3
drivers/soc/qcom/pdr_internal.h
··· 4 4 5 5 #include <linux/soc/qcom/pdr.h> 6 6 7 - #define SERVREG_LOCATOR_SERVICE 0x40 8 - #define SERVREG_NOTIFIER_SERVICE 0x42 9 - 10 7 #define SERVREG_REGISTER_LISTENER_REQ 0x20 11 8 #define SERVREG_GET_DOMAIN_LIST_REQ 0x21 12 9 #define SERVREG_STATE_UPDATED_IND_ID 0x22
+1 -1
drivers/soc/qcom/qcom_pd_mapper.c
··· 661 661 goto err_stop; 662 662 } 663 663 664 - ret = qmi_add_server(&data->handle, SERVREG_LOCATOR_SERVICE, 664 + ret = qmi_add_server(&data->handle, QMI_SERVICE_ID_SERVREG_LOC, 665 665 SERVREG_QMI_VERSION, SERVREG_QMI_INSTANCE); 666 666 if (ret) { 667 667 pr_err("PDM: error adding server %d\n", ret);