diff mbox series

[OSSTEST] ts-kernel-build: Disable CONFIG_ARCH_QCOM in Xen Project CI

Message ID 20190530165123.22593-1-ian.jackson@eu.citrix.com (mailing list archive)
State New, archived
Headers show
Series [OSSTEST] ts-kernel-build: Disable CONFIG_ARCH_QCOM in Xen Project CI | expand

Commit Message

Ian Jackson May 30, 2019, 4:51 p.m. UTC
drivers/firmware/qcom_scm.c:469:47: error: passing argument 3 of `dma_alloc_coherent' from incompatible pointer type [-Werror=incompatible-pointer-types]

This is fixed by

  firmware: qcom_scm: Use proper types for dma mappings

but this is not present in all relevant stable branches.

We currently have no Qualcomm hardware in the Xen Project test lab so
we do not need this enabled.

CC: Julien Grall <julien.grall@arm.com
CC: Stefano Stabellini <sstabellini@kernel.org>
CC: linux-arm-msm@vger.kernel.org
CC: linux-kernel@vger.kernel.org
CC: Stephen Boyd <swboyd@chromium.org>
CC: Andy Gross <agross@kernel.org>
CC: Bjorn Andersson <bjorn.andersson@linaro.org>
CC: Avaneesh Kumar Dwivedi <akdwived@codeaurora.org>
Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>
---
 ts-kernel-build | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Julien Grall May 31, 2019, 3:52 p.m. UTC | #1
Hi Ian,

On 30/05/2019 17:51, Ian Jackson wrote:
>    drivers/firmware/qcom_scm.c:469:47: error: passing argument 3 of `dma_alloc_coherent' from incompatible pointer type [-Werror=incompatible-pointer-types]
> 
> This is fixed by
> 
>    firmware: qcom_scm: Use proper types for dma mappings
> 
> but this is not present in all relevant stable branches.
> 
> We currently have no Qualcomm hardware in the Xen Project test lab so
> we do not need this enabled.
> 
> CC: Julien Grall <julien.grall@arm.com
> CC: Stefano Stabellini <sstabellini@kernel.org>
> CC: linux-arm-msm@vger.kernel.org
> CC: linux-kernel@vger.kernel.org
> CC: Stephen Boyd <swboyd@chromium.org>
> CC: Andy Gross <agross@kernel.org>
> CC: Bjorn Andersson <bjorn.andersson@linaro.org>
> CC: Avaneesh Kumar Dwivedi <akdwived@codeaurora.org>
> Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>

FWIW:

Acked-by: Julien Grall <julien.grall@arm.com>

> ---
>   ts-kernel-build | 4 ++++
>   1 file changed, 4 insertions(+)
> 
> diff --git a/ts-kernel-build b/ts-kernel-build
> index f7d059b0..5536586f 100755
> --- a/ts-kernel-build
> +++ b/ts-kernel-build
> @@ -274,6 +274,10 @@ setopt CONFIG_MDIO_THUNDER=m
>   setopt CONFIG_I2C_THUNDERX=m
>   setopt CONFIG_SPI_THUNDERX=m
>   
> +# Some Linux branches we care about, including 4.19, still lack
> +# firmware: qcom_scm: Use proper types for dma mappings
> +CONFIG_ARCH_QCOM=n
> +
>   ####
>   
>   END
>
diff mbox series

Patch

diff --git a/ts-kernel-build b/ts-kernel-build
index f7d059b0..5536586f 100755
--- a/ts-kernel-build
+++ b/ts-kernel-build
@@ -274,6 +274,10 @@  setopt CONFIG_MDIO_THUNDER=m
 setopt CONFIG_I2C_THUNDERX=m
 setopt CONFIG_SPI_THUNDERX=m
 
+# Some Linux branches we care about, including 4.19, still lack
+# firmware: qcom_scm: Use proper types for dma mappings
+CONFIG_ARCH_QCOM=n
+
 ####
 
 END