diff options
author | Krzysztof Kozlowski <krzk@kernel.org> | 2024-04-10 20:45:22 +0200 |
---|---|---|
committer | Bjorn Andersson <andersson@kernel.org> | 2024-04-21 11:22:11 -0500 |
commit | 77706838f837785189578bd8fd768e646e63f8c2 (patch) | |
tree | eef1488b40447d43e28675e00a262687733545cc | |
parent | e478c5fb6aa10af7b7edbff69bc8aef6fbb5f0ed (diff) | |
download | linux-77706838f837785189578bd8fd768e646e63f8c2.tar.gz linux-77706838f837785189578bd8fd768e646e63f8c2.tar.bz2 linux-77706838f837785189578bd8fd768e646e63f8c2.zip |
soc: qcom: mention intentionally broken module autoloading
Qualcomm PMIC ChargerPD ULOG and RPM Master Statistics drivers are
solely for debugging purposes and should not be autoloaded as modules.
Add comments to annotate missing MODULE_DEVICE_TABLE.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Link: https://lore.kernel.org/r/20240410184522.271889-1-krzk@kernel.org
Signed-off-by: Bjorn Andersson <andersson@kernel.org>
-rw-r--r-- | drivers/soc/qcom/pmic_pdcharger_ulog.c | 4 | ||||
-rw-r--r-- | drivers/soc/qcom/rpm_master_stats.c | 4 |
2 files changed, 8 insertions, 0 deletions
diff --git a/drivers/soc/qcom/pmic_pdcharger_ulog.c b/drivers/soc/qcom/pmic_pdcharger_ulog.c index 238cd38589dc..39f412bbf2c1 100644 --- a/drivers/soc/qcom/pmic_pdcharger_ulog.c +++ b/drivers/soc/qcom/pmic_pdcharger_ulog.c @@ -150,6 +150,10 @@ static const struct rpmsg_device_id pmic_pdcharger_ulog_rpmsg_id_match[] = { { "PMIC_LOGS_ADSP_APPS" }, {} }; +/* + * No MODULE_DEVICE_TABLE intentionally: that's a debugging module, to be + * loaded manually only. + */ static struct rpmsg_driver pmic_pdcharger_ulog_rpmsg_driver = { .probe = pmic_pdcharger_ulog_rpmsg_probe, diff --git a/drivers/soc/qcom/rpm_master_stats.c b/drivers/soc/qcom/rpm_master_stats.c index 9ca13bcf67d3..086fe4ba6707 100644 --- a/drivers/soc/qcom/rpm_master_stats.c +++ b/drivers/soc/qcom/rpm_master_stats.c @@ -148,6 +148,10 @@ static const struct of_device_id rpm_master_table[] = { { .compatible = "qcom,rpm-master-stats" }, { }, }; +/* + * No MODULE_DEVICE_TABLE intentionally: that's a debugging module, to be + * loaded manually only. + */ static struct platform_driver master_stats_driver = { .probe = master_stats_probe, |