lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 26 Aug 2020 20:58:24 -0500 From: Bjorn Andersson <bjorn.andersson@...aro.org> To: Maulik Shah <mkshah@...eaurora.org> Cc: andy.gross@...aro.org, linux-kernel@...r.kernel.org, linux-arm-msm@...r.kernel.org, ulf.hansson@...aro.org, swboyd@...omium.org, dianders@...omium.org, rnayak@...eaurora.org, ilina@...eaurora.org, lsrao@...eaurora.org, John Stultz <john.stultz@...aro.org>, Todd Kjos <tkjos@...gle.com>, Saravana Kannan <saravanak@...gle.com>, Andy Gross <agross@...nel.org> Subject: Re: [PATCH v2 2/2] soc: qcom: rpmh: Allow RPMH driver to be loaded as a module On Tue 25 Aug 06:21 CDT 2020, Maulik Shah wrote: > From: John Stultz <john.stultz@...aro.org> > > This patch allow the rpmh driver to be loaded as a permenent > module. Meaning it can be loaded from a module, but then cannot > be unloaded. > > Ideally, it would include a remove hook and related logic, but > the rpmh driver is fairly core to the system, so once its loaded > with almost anything else to get the system to go, the dependencies > are not likely to ever also be removed. > > So making it a permanent module at least improves things slightly > over requiring it to be a built in driver. > > Cc: Todd Kjos <tkjos@...gle.com> > Cc: Saravana Kannan <saravanak@...gle.com> > Cc: Andy Gross <agross@...nel.org> > Cc: Bjorn Andersson <bjorn.andersson@...aro.org> > Cc: Rajendra Nayak <rnayak@...eaurora.org> > Cc: linux-arm-msm@...r.kernel.org > Signed-off-by: John Stultz <john.stultz@...aro.org> > Signed-off-by: Bjorn Andersson <bjorn.andersson@...aro.org> > [mkshah: Fix typos in commit message, send after removing _rcuidle trace] > Signed-off-by: Maulik Shah <mkshah@...eaurora.org> Reviewed-by: Bjorn Andersson <bjorn.andersson@...aro.org> Regards, Bjorn > --- > drivers/soc/qcom/Kconfig | 2 +- > drivers/soc/qcom/rpmh-rsc.c | 5 +++++ > 2 files changed, 6 insertions(+), 1 deletion(-) > > diff --git a/drivers/soc/qcom/Kconfig b/drivers/soc/qcom/Kconfig > index 3dc3e3d..892bdc7 100644 > --- a/drivers/soc/qcom/Kconfig > +++ b/drivers/soc/qcom/Kconfig > @@ -92,7 +92,7 @@ config QCOM_RMTFS_MEM > Say y here if you intend to boot the modem remoteproc. > > config QCOM_RPMH > - bool "Qualcomm RPM-Hardened (RPMH) Communication" > + tristate "Qualcomm RPM-Hardened (RPMH) Communication" > depends on ARCH_QCOM || COMPILE_TEST > help > Support for communication with the hardened-RPM blocks in > diff --git a/drivers/soc/qcom/rpmh-rsc.c b/drivers/soc/qcom/rpmh-rsc.c > index fabe390d..5fce87c 100644 > --- a/drivers/soc/qcom/rpmh-rsc.c > +++ b/drivers/soc/qcom/rpmh-rsc.c > @@ -13,6 +13,7 @@ > #include <linux/iopoll.h> > #include <linux/kernel.h> > #include <linux/list.h> > +#include <linux/module.h> > #include <linux/of.h> > #include <linux/of_irq.h> > #include <linux/of_platform.h> > @@ -1025,6 +1026,7 @@ static const struct of_device_id rpmh_drv_match[] = { > { .compatible = "qcom,rpmh-rsc", }, > { } > }; > +MODULE_DEVICE_TABLE(of, rpmh_drv_match); > > static struct platform_driver rpmh_driver = { > .probe = rpmh_rsc_probe, > @@ -1040,3 +1042,6 @@ static int __init rpmh_driver_init(void) > return platform_driver_register(&rpmh_driver); > } > arch_initcall(rpmh_driver_init); > + > +MODULE_DESCRIPTION("Qualcomm Technologies, Inc. RPMh Driver"); > +MODULE_LICENSE("GPL v2"); > -- > QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member > of Code Aurora Forum, hosted by The Linux Foundation >
Powered by blists - more mailing lists