Re: [PATCH 1/6] sunxi: Kconfig: Fix default order for V3s DRAM clock

[Andre Przywara <andre.przywara@arm.com>]

On Sun,  1 Jun 2025 17:39:38 +0200
Paul Kocialkowski <contact@paulk.fr> wrote:

Hi Paul,

> The V3s (using co-packaged DRAM) runs at 360 MHz, which is specified in
> the common platform Kconfig file. However the value for MACH_SUN8I will
> be picked up instead due to ordering.
> 
> Re-order the defaults to have MACH_SUN8I_V3S before MACH_SUN8I and let
> it select the correct default. Also update the LicheePi Zero Dock
> defconfig to remove the value, which is now correctly selected.

Looks alright, can confirm that all generated .config files are the
same before and after:

> Signed-off-by: Paul Kocialkowski <contact@paulk.fr>

Reviewed-by: Andre Przywara <andre.przywara@arm.com>

Thanks!
Andre

> ---
>  arch/arm/mach-sunxi/Kconfig     | 2 +-
>  configs/LicheePi_Zero_defconfig | 1 -
>  2 files changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/arch/arm/mach-sunxi/Kconfig b/arch/arm/mach-sunxi/Kconfig
> index 17179593913e..8b43fd176dc0 100644
> --- a/arch/arm/mach-sunxi/Kconfig
> +++ b/arch/arm/mach-sunxi/Kconfig
> @@ -591,9 +591,9 @@ config DRAM_CLK
>  	int "sunxi dram clock speed"
>  	default 792 if MACH_SUN9I
>  	default 648 if MACH_SUN8I_R40
> -	default 312 if MACH_SUN6I || MACH_SUN8I
>  	default 360 if MACH_SUN4I || MACH_SUN5I || MACH_SUN7I || \
>  		       MACH_SUN8I_V3S
> +	default 312 if MACH_SUN6I || MACH_SUN8I
>  	default 672 if MACH_SUN50I
>  	default 744 if MACH_SUN50I_H6
>  	default 720 if MACH_SUN50I_H616
> diff --git a/configs/LicheePi_Zero_defconfig b/configs/LicheePi_Zero_defconfig
> index c37c49ccbb16..b2cf0a018c7f 100644
> --- a/configs/LicheePi_Zero_defconfig
> +++ b/configs/LicheePi_Zero_defconfig
> @@ -3,6 +3,5 @@ CONFIG_ARCH_SUNXI=y
>  CONFIG_DEFAULT_DEVICE_TREE="sun8i-v3s-licheepi-zero"
>  CONFIG_SPL=y
>  CONFIG_MACH_SUN8I_V3S=y
> -CONFIG_DRAM_CLK=360
>  # CONFIG_HAS_ARMV7_SECURE_BASE is not set
>  CONFIG_NO_NET=y