[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAG3jFyuiVFHfNVwCAEynH0j8fK91k32m+nvZYYR79gju9cwPKQ@mail.gmail.com>
Date: Tue, 24 Mar 2020 16:38:54 +0100
From: Robert Foss <robert.foss@...aro.org>
To: Luca Weiss <luca@...tu.xyz>
Cc: agross@...nel.org, Bjorn Andersson <bjorn.andersson@...aro.org>,
Rob Herring <robh+dt@...nel.org>,
Mark Rutland <mark.rutland@....com>, catalin.marinas@....com,
will@...nel.org, shawnguo@...nel.org, olof@...om.net,
maxime@...no.tech, Anson.Huang@....com, dinguyen@...nel.org,
leonard.crestez@....com,
Marcin Juszkiewicz <marcin.juszkiewicz@...aro.org>,
linux-arm-msm@...r.kernel.org,
"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
<devicetree@...r.kernel.org>,
linux-kernel <linux-kernel@...r.kernel.org>,
"moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE"
<linux-arm-kernel@...ts.infradead.org>,
Loic Poulain <loic.poulain@...aro.org>
Subject: Re: [v2 6/6] arm64: defconfig: Enable QCOM CAMCC, CAMSS and CCI drivers
Hey Luca,
On Fri, 20 Mar 2020 at 20:52, Luca Weiss <luca@...tu.xyz> wrote:
>
> Hi Robert,
>
> On Dienstag, 17. März 2020 14:57:40 CET Robert Foss wrote:
> > Build camera clock, isp and controller drivers as modules.
> >
> > Signed-off-by: Robert Foss <robert.foss@...aro.org>
> > Reviewed-by: Bjorn Andersson <bjorn.andersson@...aro.org>
> > ---
> > arch/arm64/configs/defconfig | 4 ++++
> > 1 file changed, 4 insertions(+)
> >
> > diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> > index 4db223dbc549..7cb6989249ab 100644
> > --- a/arch/arm64/configs/defconfig
> > +++ b/arch/arm64/configs/defconfig
> > @@ -376,6 +376,7 @@ CONFIG_I2C_MESON=y
> > CONFIG_I2C_MV64XXX=y
> > CONFIG_I2C_OWL=y
> > CONFIG_I2C_PXA=y
> > +CONFIG_I2C_QCOM_CCI=m
> > CONFIG_I2C_QCOM_GENI=m
> > CONFIG_I2C_QUP=y
> > CONFIG_I2C_RK3X=y
> > @@ -530,6 +531,7 @@ CONFIG_VIDEO_SAMSUNG_S5P_MFC=m
> > CONFIG_VIDEO_SAMSUNG_EXYNOS_GSC=m
> > CONFIG_VIDEO_RENESAS_FCP=m
> > CONFIG_VIDEO_RENESAS_VSP1=m
> > +CONFIG_VIDEO_QCOM_CAMSS=m
> > CONFIG_DRM=m
> > CONFIG_DRM_I2C_NXP_TDA998X=m
> > CONFIG_DRM_NOUVEAU=m
> > @@ -732,6 +734,7 @@ CONFIG_MSM_GCC_8994=y
> > CONFIG_MSM_MMCC_8996=y
> > CONFIG_MSM_GCC_8998=y
> > CONFIG_QCS_GCC_404=y
> > +CONFIG_SDM_CAMCC_845=m
>
> You seem to have this option twice in this patch.
Thanks for catching this.
I'll send out a fix in v3.
>
> > CONFIG_SDM_GCC_845=y
> > CONFIG_SM_GCC_8150=y
> > CONFIG_QCOM_HFPLL=y
> > @@ -762,6 +765,7 @@ CONFIG_QCOM_COMMAND_DB=y
> > CONFIG_QCOM_GENI_SE=y
> > CONFIG_QCOM_GLINK_SSR=m
> > CONFIG_QCOM_RMTFS_MEM=m
> > +CONFIG_SDM_CAMCC_845=m
>
> ^
>
> > CONFIG_QCOM_RPMH=y
> > CONFIG_QCOM_RPMHPD=y
> > CONFIG_QCOM_SMEM=y
>
> Regards
> Luca
>
>
Powered by blists - more mailing lists