[U-Boot] [PATCH] sunxi: lower the DRAM frequency of Nano Pi NEO2 to 504MHz

[U-Boot] [PATCH] sunxi: lower the DRAM frequency of Nano Pi NEO2 to 504MHz

[Icenowy Zheng]

The BSP of Nano Pi NEO2 provided by FriendlyARM uses 504MHz DRAM
frequency, which is much lower than the current 672MHz used in
mainline.

Switch to use the BSP-provided frequency 504MHz for DRAM.

Thanks to Thomas Kaiser, who pointed out the frequency problem.

Signed-off-by: Icenowy Zheng <icenowy@aosc.io>
---
 configs/nanopi_neo2_defconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/configs/nanopi_neo2_defconfig b/configs/nanopi_neo2_defconfig
index b8e089176f..e210af4f0e 100644
--- a/configs/nanopi_neo2_defconfig
+++ b/configs/nanopi_neo2_defconfig
@@ -1,7 +1,7 @@
 CONFIG_ARM=y
 CONFIG_ARCH_SUNXI=y
 CONFIG_MACH_SUN50I_H5=y
-CONFIG_DRAM_CLK=672
+CONFIG_DRAM_CLK=504
 CONFIG_DRAM_ZQ=3881977
 CONFIG_DEFAULT_DEVICE_TREE="sun50i-h5-nanopi-neo2"
 # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set
-- 
2.13.6

[U-Boot] [PATCH] sunxi: lower the DRAM frequency of Nano Pi NEO2 to 504MHz

[Jagan Teki]

On Thu, Nov 2, 2017 at 9:06 AM, Icenowy Zheng <icenowy@aosc.io> wrote:
> The BSP of Nano Pi NEO2 provided by FriendlyARM uses 504MHz DRAM
> frequency, which is much lower than the current 672MHz used in
> mainline.
>
> Switch to use the BSP-provided frequency 504MHz for DRAM.
>
> Thanks to Thomas Kaiser, who pointed out the frequency problem.
>
> Signed-off-by: Icenowy Zheng <icenowy@aosc.io>
> ---
>  configs/nanopi_neo2_defconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/configs/nanopi_neo2_defconfig b/configs/nanopi_neo2_defconfig
> index b8e089176f..e210af4f0e 100644
> --- a/configs/nanopi_neo2_defconfig
> +++ b/configs/nanopi_neo2_defconfig
> @@ -1,7 +1,7 @@
>  CONFIG_ARM=y
>  CONFIG_ARCH_SUNXI=y
>  CONFIG_MACH_SUN50I_H5=y
> -CONFIG_DRAM_CLK=672
> +CONFIG_DRAM_CLK=504

Can you point me the BSP, here what I refer to have [1]

[1] https://github.com/friendlyarm/u-boot/blob/sunxi-v2017.x/configs/nanopi_neo2_defconfig

thanks!
-- 
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.

[U-Boot] [PATCH] sunxi: lower the DRAM frequency of Nano Pi NEO2 to 504MHz

[Icenowy Zheng]

在 2017-11-02 16:25，Jagan Teki 写道：
> On Thu, Nov 2, 2017 at 9:06 AM, Icenowy Zheng <icenowy@aosc.io> wrote:
>> The BSP of Nano Pi NEO2 provided by FriendlyARM uses 504MHz DRAM
>> frequency, which is much lower than the current 672MHz used in
>> mainline.
>> 
>> Switch to use the BSP-provided frequency 504MHz for DRAM.
>> 
>> Thanks to Thomas Kaiser, who pointed out the frequency problem.
>> 
>> Signed-off-by: Icenowy Zheng <icenowy@aosc.io>
>> ---
>>  configs/nanopi_neo2_defconfig | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>> diff --git a/configs/nanopi_neo2_defconfig 
>> b/configs/nanopi_neo2_defconfig
>> index b8e089176f..e210af4f0e 100644
>> --- a/configs/nanopi_neo2_defconfig
>> +++ b/configs/nanopi_neo2_defconfig
>> @@ -1,7 +1,7 @@
>>  CONFIG_ARM=y
>>  CONFIG_ARCH_SUNXI=y
>>  CONFIG_MACH_SUN50I_H5=y
>> -CONFIG_DRAM_CLK=672
>> +CONFIG_DRAM_CLK=504
> 
> Can you point me the BSP, here what I refer to have [1]

The file at [1] is just from mainline.

The BSP configuration is at [2].

[2] 
https://github.com/friendlyarm/h5_lichee/blob/master/tools/pack/chips/sun50iw2p1/configs/cheetah-p1/board/sys_config_nanopi-neo2.fex

> 
> [1]
> https://github.com/friendlyarm/u-boot/blob/sunxi-v2017.x/configs/nanopi_neo2_defconfig
> 
> thanks!

[U-Boot] [PATCH] sunxi: lower the DRAM frequency of Nano Pi NEO2 to 504MHz

[Jagan Teki]

On Thu, Nov 2, 2017 at 1:57 PM, Icenowy Zheng <icenowy@aosc.io> wrote:
> 在 2017-11-02 16:25，Jagan Teki 写道：
>>
>> On Thu, Nov 2, 2017 at 9:06 AM, Icenowy Zheng <icenowy@aosc.io> wrote:
>>>
>>> The BSP of Nano Pi NEO2 provided by FriendlyARM uses 504MHz DRAM
>>> frequency, which is much lower than the current 672MHz used in
>>> mainline.
>>>
>>> Switch to use the BSP-provided frequency 504MHz for DRAM.
>>>
>>> Thanks to Thomas Kaiser, who pointed out the frequency problem.
>>>
>>> Signed-off-by: Icenowy Zheng <icenowy@aosc.io>
>>> ---
>>>  configs/nanopi_neo2_defconfig | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/configs/nanopi_neo2_defconfig
>>> b/configs/nanopi_neo2_defconfig
>>> index b8e089176f..e210af4f0e 100644
>>> --- a/configs/nanopi_neo2_defconfig
>>> +++ b/configs/nanopi_neo2_defconfig
>>> @@ -1,7 +1,7 @@
>>>  CONFIG_ARM=y
>>>  CONFIG_ARCH_SUNXI=y
>>>  CONFIG_MACH_SUN50I_H5=y
>>> -CONFIG_DRAM_CLK=672
>>> +CONFIG_DRAM_CLK=504

Did you find this abort[3] with existing master, look like autoboot corrupted

[3] https://paste.ubuntu.com/25872498/

thanks!
-- 
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.

[U-Boot] [PATCH] sunxi: lower the DRAM frequency of Nano Pi NEO2 to 504MHz

[Maxime Ripard]

On Thu, Nov 02, 2017 at 11:36:52AM +0800, Icenowy Zheng wrote:
> The BSP of Nano Pi NEO2 provided by FriendlyARM uses 504MHz DRAM
> frequency, which is much lower than the current 672MHz used in
> mainline.
> 
> Switch to use the BSP-provided frequency 504MHz for DRAM.
> 
> Thanks to Thomas Kaiser, who pointed out the frequency problem.

What issue does this patch fix? BSPs have often been way underclocked
in the past.

Did you encounter any corruption?

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20171102/9d04367a/attachment.sig>