Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

u-boot (Rockchip): roc-3328-cc: increase ddr4 clock rate at 800Mhz #9590

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ilmich
Copy link
Contributor

@ilmich ilmich commented Dec 18, 2024

The Libre Computer roc-3328-cc has dd4 ram.
By porting these two vendor patches

libre-computer-project/libretech-u-boot@cdb523f
libre-computer-project/libretech-u-boot@7c0723f

the clock frequency is increased to 800Mhz (actually the clock frequency is 330Mhz).
this makes the system much more responsive and solves problems related to the playback of 4k streams reported by users on the forum (https://forum.libreelec.tv/thread/27677-rk3328-4k-hevc-stutter-frame-drop/).

I will try as soon as possible to understand if they can be sent upstream

@ilmich
Copy link
Contributor Author

ilmich commented Dec 25, 2024

@Kwiboo @knaerzche please, when and if you can, would it be possible to have a review of this contribution?

@knaerzche
Copy link
Contributor

knaerzche commented Dec 25, 2024

@ilmich I'd say better send those patches upstream. @Kwiboo is very active at U-Boot mailing lists and can help reviewing them.

@Kwiboo
Copy link
Member

Kwiboo commented Dec 25, 2024

This looks fine, and agree please send patches upstream.
Will be able to runtime test in 1-2 weeks, on one board from initial launch with Firefly v1.0 branding and a second more recent board with Libre Computer v1.3 branding.

@Kwiboo
Copy link
Member

Kwiboo commented Jan 5, 2025

@ilmich I have now tested these patches on my two ROC-RK3328-CC V1.0 and V1.3 boards without issues and this can be merged.

Are you planning on sending this upstream to mainline U-Boot? If not I can send it, the unused 1866Mhz .dtsi-file is probably not needed for upstream.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants