Thread (28 messages) 28 messages, 5 authors, 2026-03-19

Re: [PATCH v1 0/8] Group QMI service IDs into the QMI header

From: Daniel Lezcano <hidden>
Date: 2026-03-10 00:09:12
Also in: ath11k, ath12k, linux-arm-msm, linux-remoteproc, linux-sound, linux-wireless, lkml

On 3/10/26 00:50, Jeff Johnson wrote:
On 3/9/2026 4:03 PM, Daniel Lezcano wrote:
quoted
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
quoted
   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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help