On Tue 05 Jun 05:56 PDT 2018, Sricharan R wrote: > Hi Vinod, > > On 6/5/2018 11:49 AM, Vinod wrote: > > On 05-06-18, 11:12, Sricharan R wrote: > > > >> +config QCOM_Q6V5_WCSS > >> + tristate "Qualcomm Hexagon based WCSS Peripheral Image Loader" > >> + depends on OF && ARCH_QCOM > >> + depends on QCOM_SMEM > >> + depends on RPMSG_QCOM_SMD || (COMPILE_TEST && RPMSG_QCOM_SMD=n) > >> + depends on RPMSG_QCOM_GLINK_SMEM || RPMSG_QCOM_GLINK_SMEM=n > > > > Is there a reason why it depends on RPMSG_QCOM_GLINK_SMEM=n? What would > > happen if distro wants both this and RPMSG_QCOM_GLINK_SMEM > > It says that QCOM_Q6V5_WCSS either must have a compatible state (i.e. builtin vs builtin, module vs builtin, but not builtin vs module) or that it's disabled, in which case we will hit the stub functions in qcom_glink.h. I.e. this prevents QCOM_Q6V5_WCSS to be compiled builtin when RPMSG_QCOM_GLINK_SMEM is module, as this would give us both stubs and the module. > RPMSG_QCOM_GLINK_SMEM=n should be for the COMPILE_TEST. Probably that > means that it should be corrected here and for ADSP, Q6V5_PIL as well. > Bjorn, is that correct ?, should it be, below ? > There are platforms with SMD, those with GLINK-SMEM and those with both. For the two first we want it to be possible only compile the specific transport being used and the other being stubbed. As Sricharan's particular platform uses GLINK for communicating with the WCSS it's perfectly fine to run this particular driver with RPMSG_QCOM_SMD=n RPMSG_QCOM_GLINK_SMEM=y/m As such I would recommend that you drop COMPILE_TEST from above. Regards, Bjorn -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
