Re: [v2 6/6] arm64: defconfig: Enable QCOM CAMCC, CAMSS and CCI drivers

From: Robert Foss
Date: Tue Mar 24 2020 - 11:39:15 EST


Hey Luca,

On Fri, 20 Mar 2020 at 20:52, Luca Weiss <luca@xxxxxxxxx> 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@xxxxxxxxxx>
> > Reviewed-by: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx>
> > ---
> > 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
>
>