Skip to content

add cpu and mem supply to all big cores on rk3582 boards#357

Merged
amazingfate merged 3 commits intoarmbian:rk-6.1-rkr5.1from
amazingfate:rk3582-7core
May 6, 2025
Merged

add cpu and mem supply to all big cores on rk3582 boards#357
amazingfate merged 3 commits intoarmbian:rk-6.1-rkr5.1from
amazingfate:rk3582-7core

Conversation

@amazingfate
Copy link
Copy Markdown
Collaborator

A user has reported that his rock5c lite only runs at minimal cpu freq. His rock5c lite has a broken cpu6, which is disabled by uboot(cpu_b2). Now all rk3588 boards has the cpu-supply defined at cpu_b2, when the node is disabled, the cpu freq driver can't be initialized.

He has confirmed that this issue doesn't happend with mainline kernel because all cpu core nodes has defined cpu-supply in mainline devicetree, so I add cpu-supply and mem-supply to the two rk3582 boards.

@kwankiu
Copy link
Copy Markdown

kwankiu commented May 2, 2025

shouldn't this be applied to rk3588s-radxa-cm5.dtsi too?

@amazingfate
Copy link
Copy Markdown
Collaborator Author

shouldn't this be applied to rk3588s-radxa-cm5.dtsi too?

radxa cm5 is a rk3588s board which has all 8 cores, so it doesn't have the issue when one big core is disabled by uboot.

@kwankiu
Copy link
Copy Markdown

kwankiu commented May 2, 2025

shouldn't this be applied to rk3588s-radxa-cm5.dtsi too?

radxa cm5 is a rk3588s board which has all 8 cores, so it doesn't have the issue when one big core is disabled by uboot.

Radxa CM5 has RK3582 variants, I own a CM5 Lite myself, although this doesnt affect my board as i have an RK3582 that all cpu, gpu and video decoder seems to be good.

@amazingfate
Copy link
Copy Markdown
Collaborator Author

shouldn't this be applied to rk3588s-radxa-cm5.dtsi too?

radxa cm5 is a rk3588s board which has all 8 cores, so it doesn't have the issue when one big core is disabled by uboot.

Radxa CM5 has RK3582 variants, I own a CM5 Lite myself, although this doesnt affect my board as i have an RK3582 that all cpu, gpu and video decoder seems to be good.

I didn't know it has a lite version. I will add it later.

cooling-cells for all big cores on rock5c
cooling-cells for all big cores on radxa e52c
cooling-cells for all big cores on radxa cm5
@amazingfate
Copy link
Copy Markdown
Collaborator Author

I find that pwm-fan also need the cooling-cells property on big core. Missing this property will cause pwm fan always working at full speed.

@amazingfate amazingfate merged commit a631747 into armbian:rk-6.1-rkr5.1 May 6, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants