Re: [PATCH v1 0/8] Group QMI service IDs into the QMI header
Daniel Lezcano <[email protected]>
| Newsgroups | org.infradead.lists.ath10k,org.infradead.lists.ath11k,org.infradead.lists.ath12k,org.kernel.vger.linux-arm-msm,org.kernel.vger.linux-kernel,org.kernel.vger.linux-remoteproc,org.kernel.vger.linux-sound,org.kernel.vger.linux-wireless,org.kernel.vger.netdev |
|---|---|
| Message-ID | <[email protected]> |
On 3/10/26 00:50, Jeff Johnson wrote: > On 3/9/2026 4:03 PM, Daniel Lezcano wrote: >> The different subsystems implementing the QMI service protocol are >> using their own definition of the service id. It is not a problem but >> it results on having those duplicated with different names but the >> same value and without consistency in their name. >> >> It makes more sense to unify their names and move the definitions in [ ... ] > prefix for wireless drivers is simply wifi: <driver>: > so s/net: drivers: wireless:/wifi:/ Noted, thanks >> remoteproc: qcom: Use the unified QMI service ID instead of defining >> it locally >> slimbus: qcom-ngd-ctrl: Use the unified QMI service ID instead of >> defining it locally >> soc: qcom: pdr: Use the unified QMI service ID instead of defining it >> locally >> ALSA: usb-audio: qcom: Use the unified QMI service ID instead of >> defining it locally >> samples: qmi: Use the unified QMI service ID instead of defining it >> locally >> >> drivers/net/ipa/ipa_qmi.c | 6 ++---- >> drivers/net/wireless/ath/ath10k/qmi.c | 2 +- >> drivers/net/wireless/ath/ath10k/qmi_wlfw_v01.h | 1 - >> drivers/net/wireless/ath/ath11k/qmi.c | 2 +- >> drivers/net/wireless/ath/ath11k/qmi.h | 1 - >> drivers/net/wireless/ath/ath12k/qmi.c | 2 +- >> drivers/net/wireless/ath/ath12k/qmi.h | 1 - >> drivers/remoteproc/qcom_sysmon.c | 2 +- >> drivers/slimbus/qcom-ngd-ctrl.c | 5 ++--- >> drivers/soc/qcom/pdr_interface.c | 4 ++-- >> drivers/soc/qcom/pdr_internal.h | 3 --- >> drivers/soc/qcom/qcom_pd_mapper.c | 2 +- >> include/linux/soc/qcom/qmi.h | 12 ++++++++++++ >> samples/qmi/qmi_sample_client.c | 2 +- >> sound/usb/qcom/qc_audio_offload.c | 2 +- >> sound/usb/qcom/usb_audio_qmi_v01.h | 1 - > > You are touching a lot of subsystems with a single series. > How do you plan on having these land? > Do you have a maintainer who will take all of these through their tree? Yes I thought Bjorn or Konrad would take them with the acked-by from the different subsystems