arm64: defconfig: Add Meson DDR PMU as module

[PATCH] arm64: defconfig: Add Meson DDR PMU as module

Posted by Jiucheng Xu 3 years, 4 months ago

Add Meson DDR PMU to defconfig so that build errors are caught.

Signed-off-by: Jiucheng Xu <jiucheng.xu@amlogic.com>
---
 arch/arm64/configs/defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index 0b6af3348e79..f2324b54a6ba 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -1265,6 +1265,7 @@ CONFIG_ARM_DMC620_PMU=m
 CONFIG_QCOM_L2_PMU=y
 CONFIG_QCOM_L3_PMU=y
 CONFIG_HISI_PMU=y
+CONFIG_MESON_DDR_PMU=m
 CONFIG_NVMEM_IMX_OCOTP=y
 CONFIG_NVMEM_IMX_OCOTP_SCU=y
 CONFIG_NVMEM_MTK_EFUSE=y
-- 
2.25.1

Re: [PATCH] arm64: defconfig: Add Meson DDR PMU as module

Posted by Neil Armstrong 3 years, 4 months ago

+CC linux-amlogic@lists.infradead.org

On 04/12/2022 05:35, Jiucheng Xu wrote:
> Add Meson DDR PMU to defconfig so that build errors are caught.
> 
> Signed-off-by: Jiucheng Xu <jiucheng.xu@amlogic.com>
> ---
>   arch/arm64/configs/defconfig | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> index 0b6af3348e79..f2324b54a6ba 100644
> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> @@ -1265,6 +1265,7 @@ CONFIG_ARM_DMC620_PMU=m
>   CONFIG_QCOM_L2_PMU=y
>   CONFIG_QCOM_L3_PMU=y
>   CONFIG_HISI_PMU=y
> +CONFIG_MESON_DDR_PMU=m
>   CONFIG_NVMEM_IMX_OCOTP=y
>   CONFIG_NVMEM_IMX_OCOTP_SCU=y
>   CONFIG_NVMEM_MTK_EFUSE=y

Re: [PATCH] arm64: defconfig: Add Meson DDR PMU as module

Posted by Chris Healy 3 years, 4 months ago

On Mon, Dec 5, 2022 at 12:40 AM Neil Armstrong
<neil.armstrong@linaro.org> wrote:
>
> +CC linux-amlogic@lists.infradead.org
>
> On 04/12/2022 05:35, Jiucheng Xu wrote:
> > Add Meson DDR PMU to defconfig so that build errors are caught.
> >
> > Signed-off-by: Jiucheng Xu <jiucheng.xu@amlogic.com>
> > ---
> >   arch/arm64/configs/defconfig | 1 +
> >   1 file changed, 1 insertion(+)
> >
> > diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> > index 0b6af3348e79..f2324b54a6ba 100644
> > --- a/arch/arm64/configs/defconfig
> > +++ b/arch/arm64/configs/defconfig
> > @@ -1265,6 +1265,7 @@ CONFIG_ARM_DMC620_PMU=m
> >   CONFIG_QCOM_L2_PMU=y
> >   CONFIG_QCOM_L3_PMU=y
> >   CONFIG_HISI_PMU=y
> > +CONFIG_MESON_DDR_PMU=m

Given that the other vendor specific PMU counter drivers are being
configured as built-in, perhaps it makes more sense to do the same
with the new MESON DDR PMU driver and also set it to "y"?

> >   CONFIG_NVMEM_IMX_OCOTP=y
> >   CONFIG_NVMEM_IMX_OCOTP_SCU=y
> >   CONFIG_NVMEM_MTK_EFUSE=y
>