| Message ID | 20170913160405.17855-1-srinivas.kandagatla@linaro.org (mailing list archive) |
|---|---|
| State | Rejected |
| Headers | show |
On Wed 13 Sep 09:04 PDT 2017, srinivas.kandagatla@linaro.org wrote: > From: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> > > All Qualcomm PIL drivers require SMP2P module to get a functional PIL, > Currently user has to explicitly select SMP2P module, after looking > at failure logs. > > Fix this by selecting SMP2P as part of dependency of pil kconfig itself. > The Kconfig should describe dependencies needed by the code, not by the system. I agree with you that this leads to wasted time trying figure out if we're missing drivers/modules in the kernel, but I think that needs to be solved in a more generic way. > Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> > --- > drivers/remoteproc/Kconfig | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/remoteproc/Kconfig b/drivers/remoteproc/Kconfig > index 8891a8e50f12..052c9e20abb6 100644 > --- a/drivers/remoteproc/Kconfig > +++ b/drivers/remoteproc/Kconfig > @@ -87,6 +87,7 @@ config QCOM_ADSP_PIL > select QCOM_MDT_LOADER > select QCOM_RPROC_COMMON > select QCOM_SCM > + select QCOM_SMP2P NB. QCOM_SMP2P is a user-selectable config option, so you are not allowed to "select" it, only "depends on" it. Regards, Bjorn -- To unsubscribe from this list: send the line "unsubscribe linux-remoteproc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/remoteproc/Kconfig b/drivers/remoteproc/Kconfig index 8891a8e50f12..052c9e20abb6 100644 --- a/drivers/remoteproc/Kconfig +++ b/drivers/remoteproc/Kconfig @@ -87,6 +87,7 @@ config QCOM_ADSP_PIL select QCOM_MDT_LOADER select QCOM_RPROC_COMMON select QCOM_SCM + select QCOM_SMP2P help Say y here to support the TrustZone based Peripherial Image Loader for the Qualcomm ADSP remote processors. @@ -102,6 +103,7 @@ config QCOM_Q6V5_PIL select MFD_SYSCON select QCOM_RPROC_COMMON select QCOM_SCM + select QCOM_SMP2P help Say y here to support the Qualcomm Peripherial Image Loader for the Hexagon V5 based remote processors. @@ -114,6 +116,7 @@ config QCOM_WCNSS_PIL select QCOM_MDT_LOADER select QCOM_RPROC_COMMON select QCOM_SCM + select QCOM_SMP2P help Say y here to support the Peripheral Image Loader for the Qualcomm Wireless Connectivity Subsystem.