diff options
| author | Arnd Bergmann | 2017-07-17 10:56:49 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab | 2017-07-20 21:52:23 +0200 |
| commit | 0399b696f7f4ad4c48d59850d0e6adbed43c1d88 (patch) | |
| tree | 7454fc3073eaba48c5d900195fa49c395f98d112 /drivers/media/platform/qcom | |
| parent | media: venus: mark PM functions as __maybe_unused (diff) | |
| download | kernel-qcow2-linux-0399b696f7f4ad4c48d59850d0e6adbed43c1d88.tar.gz kernel-qcow2-linux-0399b696f7f4ad4c48d59850d0e6adbed43c1d88.tar.xz kernel-qcow2-linux-0399b696f7f4ad4c48d59850d0e6adbed43c1d88.zip | |
media: venus: fix compile-test build on non-qcom ARM platform
If QCOM_MDT_LOADER is enabled, but ARCH_QCOM is not, we run into
a build error:
ERROR: "qcom_mdt_load" [drivers/media/platform/qcom/venus/venus-core.ko] undefined!
ERROR: "qcom_mdt_get_size" [drivers/media/platform/qcom/venus/venus-core.ko] undefined!
This changes the 'select' statement again, so we only try to enable
those symbols when the drivers will actually get built, and explicitly
test for QCOM_MDT_LOADER to be enabled before calling into it.
Fixes: 76724b30f222 ("[media] media: venus: enable building with COMPILE_TEST")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Reviewed-by: Stanimir Varbanov <stanimir.varbanov@linaro.org>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
Diffstat (limited to 'drivers/media/platform/qcom')
| -rw-r--r-- | drivers/media/platform/qcom/venus/firmware.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/platform/qcom/venus/firmware.c b/drivers/media/platform/qcom/venus/firmware.c index 1b1a4f355918..1ede92300dd6 100644 --- a/drivers/media/platform/qcom/venus/firmware.c +++ b/drivers/media/platform/qcom/venus/firmware.c @@ -40,7 +40,7 @@ int venus_boot(struct device *parent, struct device *fw_dev, const char *fwname) void *mem_va; int ret; - if (!qcom_scm_is_available()) + if (!IS_ENABLED(CONFIG_QCOM_MDT_LOADER) || !qcom_scm_is_available()) return -EPROBE_DEFER; fw_dev->parent = parent; |