Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
Message-ID: <515e00ec-fdfb-0a45-f504-4dd2dce60254@infradead.org>
Date:   Fri, 25 Feb 2022 12:52:16 -0800
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.6.1
Subject: Re: [PATCH net] net: ipa: fix a build dependency
Content-Language: en-US
To:     Alex Elder <elder@linaro.org>, davem@davemloft.net, kuba@kernel.org
Cc:     bjorn.andersson@linaro.org, mka@chromium.org, evgreen@chromium.org,
        cpratapa@codeaurora.org, avuyyuru@codeaurora.org,
        jponduru@codeaurora.org, subashab@codeaurora.org, elder@kernel.org,
        netdev@vger.kernel.org, linux-arm-msm@vger.kernel.org,
        linux-kernel@vger.kernel.org
References: <20220225201530.182085-1-elder@linaro.org>
From:   Randy Dunlap <rdunlap@infradead.org>
In-Reply-To: <20220225201530.182085-1-elder@linaro.org>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk


On 2/25/22 12:15, Alex Elder wrote:
> An IPA build problem arose in the linux-next tree the other day.
> The problem is that a recent commit adds a new dependency on some
> code, and the Kconfig file for IPA doesn't reflect that dependency.
> As a result, some configurations can fail to build (particularly
> when COMPILE_TEST is enabled).
> 
> The recent patch adds calls to qmp_get(), qmp_put(), and qmp_send(),
> and those are built based on the QCOM_AOSS_QMP config option.  If
> that symbol is not defined, stubs are defined, so we just need to
> ensure QCOM_AOSS_QMP is compatible with QCOM_IPA, or it's not
> defined.
> 
> Reported-by: Randy Dunlap <rdunlap@infradead.org>
> Fixes: 34a081761e4e3 ("net: ipa: request IPA register values be retained")
> Signed-off-by: Alex Elder <elder@linaro.org>

Tested-by: Randy Dunlap <rdunlap@infradead.org>
Acked-by: Randy Dunlap <rdunlap@infradead.org>

thanks.

> ---
>  drivers/net/ipa/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/net/ipa/Kconfig b/drivers/net/ipa/Kconfig
> index d037682fb7adb..e0164a55c1e66 100644
> --- a/drivers/net/ipa/Kconfig
> +++ b/drivers/net/ipa/Kconfig
> @@ -3,6 +3,7 @@ config QCOM_IPA
>  	depends on NET && QCOM_SMEM
>  	depends on ARCH_QCOM || COMPILE_TEST
>  	depends on QCOM_RPROC_COMMON || (QCOM_RPROC_COMMON=n && COMPILE_TEST)
> +	depends on QCOM_AOSS_QMP || QCOM_AOSS_QMP=n
>  	select QCOM_MDT_LOADER if ARCH_QCOM
>  	select QCOM_SCM
>  	select QCOM_QMI_HELPERS

-- 
~Randy