Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [PATCH v1 1/2] gpu: host1x: Allow COMPILE_TEST to build host1x
 driver on all platforms
To:     Sowjanya Komatineni <skomatineni@nvidia.com>,
        thierry.reding@gmail.com, jonathanh@nvidia.com
Cc:     linux-media@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-tegra@vger.kernel.org
References: <1601434958-29305-1-git-send-email-skomatineni@nvidia.com>
 <1601434958-29305-2-git-send-email-skomatineni@nvidia.com>
From:   Hans Verkuil <hverkuil@xs4all.nl>
Message-ID: <b8dc62e4-902b-9b72-8b51-5f7480d01760@xs4all.nl>
Date:   Fri, 30 Oct 2020 12:02:17 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101
 Thunderbird/68.12.0
MIME-Version: 1.0
In-Reply-To: <1601434958-29305-2-git-send-email-skomatineni@nvidia.com>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Precedence: bulk

Thierry,

On 30/09/2020 05:02, Sowjanya Komatineni wrote:
> Tegra V4L2 driver has dependency on this host1x driver and currently
> host1x driver is allowed to build with COMPILE_TEST only on ARM and
> this also limits Tegra V4L2 driver building.
> 
> So, this patch allows building host1x driver with COMPILE_TEST on
> all platforms.

Can you Ack/Review this so I can merge both patches for v5.11?
If you prefer to merge this on your side, then I can Ack patch 2/2
as well, but I think it makes more sense if this goes through the
media subsystem.

Regards,

	Hans

> 
> Signed-off-by: Sowjanya Komatineni <skomatineni@nvidia.com>
> ---
>  drivers/gpu/host1x/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/host1x/Kconfig b/drivers/gpu/host1x/Kconfig
> index 6dab94a..977a0ac 100644
> --- a/drivers/gpu/host1x/Kconfig
> +++ b/drivers/gpu/host1x/Kconfig
> @@ -1,7 +1,7 @@
>  # SPDX-License-Identifier: GPL-2.0-only
>  config TEGRA_HOST1X
>  	tristate "NVIDIA Tegra host1x driver"
> -	depends on ARCH_TEGRA || (ARM && COMPILE_TEST)
> +	depends on ARCH_TEGRA || COMPILE_TEST
>  	select IOMMU_IOVA
>  	help
>  	  Driver for the NVIDIA Tegra host1x hardware.
>