[v1,2/2] media: tegra-video: Allow building driver with COMPILE_TEST

Message ID 1601434958-29305-3-git-send-email-skomatineni@nvidia.com (mailing list archive)
State Obsoleted, archived
Delegated to: Hans Verkuil
Headers
Series Allow building Tegra video driver with COMPILE_TEST |

Commit Message

Sowjanya Komatineni Sept. 30, 2020, 3:02 a.m. UTC
  This patch adds COMPILE_TEST option to Kconfig to allow building
it with COMPILE_TEST option.

Signed-off-by: Sowjanya Komatineni <skomatineni@nvidia.com>
---
 drivers/staging/media/tegra-video/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Michał Mirosław Sept. 30, 2020, 5:34 p.m. UTC | #1
On Tue, Sep 29, 2020 at 08:02:38PM -0700, Sowjanya Komatineni wrote:
> This patch adds COMPILE_TEST option to Kconfig to allow building
> it with COMPILE_TEST option.

Does it build without TEGRA_HOST1X selected? Isn't the previous patch
enough to allow the build with COMPILE_TEST?

Best Regards,
Michał Mirosław
  
Sowjanya Komatineni Sept. 30, 2020, 5:59 p.m. UTC | #2
On 9/30/20 10:34 AM, Michał Mirosław wrote:
> On Tue, Sep 29, 2020 at 08:02:38PM -0700, Sowjanya Komatineni wrote:
>> This patch adds COMPILE_TEST option to Kconfig to allow building
>> it with COMPILE_TEST option.
> Does it build without TEGRA_HOST1X selected? Isn't the previous patch
> enough to allow the build with COMPILE_TEST?
>
> Best Regards,
> Michał Mirosław

No, it does not build without selecting CONFIG_TEGRA_HOST1X along with 
CONFIG_VIDEO_TEGRA
  

Patch

diff --git a/drivers/staging/media/tegra-video/Kconfig b/drivers/staging/media/tegra-video/Kconfig
index 1f35da4..3e03ddd 100644
--- a/drivers/staging/media/tegra-video/Kconfig
+++ b/drivers/staging/media/tegra-video/Kconfig
@@ -1,7 +1,7 @@ 
 # SPDX-License-Identifier: GPL-2.0-only
 config VIDEO_TEGRA
 	tristate "NVIDIA Tegra VI driver"
-	depends on TEGRA_HOST1X
+	depends on TEGRA_HOST1X || COMPILE_TEST
 	depends on VIDEO_V4L2
 	select MEDIA_CONTROLLER
 	select VIDEOBUF2_DMA_CONTIG