diff options
author | Bjorn Andersson <quic_bjorande@quicinc.com> | 2024-01-19 09:13:37 -0800 |
---|---|---|
committer | Bjorn Andersson <andersson@kernel.org> | 2024-01-22 18:09:15 -0600 |
commit | b65a3fa38dbb5c84f360a48f0d5da3aed0011964 (patch) | |
tree | 6bfe23c1b66c781815eab2eccb3ee5c43985667f | |
parent | 6613476e225e090cc9aad49be7fa504e290dd33d (diff) | |
download | lwn-b65a3fa38dbb5c84f360a48f0d5da3aed0011964.tar.gz lwn-b65a3fa38dbb5c84f360a48f0d5da3aed0011964.zip |
soc: qcom: aoss: Mark qmp_send() __printf()
As reported by lkp, qmp_send() would benefit from a __printf() marker to
allow the compiler to further validate the passed parameters, fix this.
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202401100855.UYl3HPPt-lkp@intel.com/
Signed-off-by: Bjorn Andersson <quic_bjorande@quicinc.com>
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>
Link: https://lore.kernel.org/r/20240119-aoss-printf-annotation-v1-1-27e2ceb8937a@quicinc.com
Signed-off-by: Bjorn Andersson <andersson@kernel.org>
-rw-r--r-- | drivers/soc/qcom/qcom_aoss.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/soc/qcom/qcom_aoss.c b/drivers/soc/qcom/qcom_aoss.c index aff0cfb71482..9d7a74767008 100644 --- a/drivers/soc/qcom/qcom_aoss.c +++ b/drivers/soc/qcom/qcom_aoss.c @@ -214,7 +214,7 @@ static bool qmp_message_empty(struct qmp *qmp) * * Return: 0 on success, negative errno on failure */ -int qmp_send(struct qmp *qmp, const char *fmt, ...) +int __printf(2, 3) qmp_send(struct qmp *qmp, const char *fmt, ...) { char buf[QMP_MSG_LEN]; long time_left; |