diff options
| author | Kathiravan Thirumoorthy <kathiravan.thirumoorthy@oss.qualcomm.com> | 2025-10-31 11:21:01 +0530 |
|---|---|---|
| committer | Bjorn Andersson <andersson@kernel.org> | 2025-11-03 21:14:16 -0600 |
| commit | d403276969b2aae147f671506a6c69089587ddd7 (patch) | |
| tree | dac93707cb52aa4665b027af531b37672d0136b5 | |
| parent | 85d55d8cc3ef7f77b249c97e9fac6a0fc5f5daa7 (diff) | |
soc: qcom: smem: drop the WARN_ON() on SMEM item validation
When a SMEM item is allocated or retrieved, sanity check on the SMEM item
is performed and backtrace is printed if it is invalid. But there is no
benefit in dumping that information in the logs. Lets drop it.
Signed-off-by: Kathiravan Thirumoorthy <kathiravan.thirumoorthy@oss.qualcomm.com>
Link: https://lore.kernel.org/r/20251031-image-crm-part2-v2-1-c224c45c381a@oss.qualcomm.com
Signed-off-by: Bjorn Andersson <andersson@kernel.org>
| -rw-r--r-- | drivers/soc/qcom/smem.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/soc/qcom/smem.c b/drivers/soc/qcom/smem.c index f4a4acfefba9..fef840b54574 100644 --- a/drivers/soc/qcom/smem.c +++ b/drivers/soc/qcom/smem.c @@ -521,7 +521,7 @@ int qcom_smem_alloc(unsigned host, unsigned item, size_t size) return -EINVAL; } - if (WARN_ON(item >= __smem->item_count)) + if (item >= __smem->item_count) return -EINVAL; ret = hwspin_lock_timeout_irqsave(__smem->hwlock, @@ -694,7 +694,7 @@ void *qcom_smem_get(unsigned host, unsigned item, size_t *size) if (IS_ERR(__smem)) return __smem; - if (WARN_ON(item >= __smem->item_count)) + if (item >= __smem->item_count) return ERR_PTR(-EINVAL); if (host < SMEM_HOST_COUNT && __smem->partitions[host].virt_base) { |