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

Support timebase-frequency in the cpus node of the Device Tree. #2782

Merged
merged 2 commits into from
Feb 11, 2021
Merged

Support timebase-frequency in the cpus node of the Device Tree. #2782

merged 2 commits into from
Feb 11, 2021

Conversation

mbs0221
Copy link
Contributor

@mbs0221 mbs0221 commented Feb 2, 2021

Related issue: per-hart timebase-frequency

Type of change: feature request

Impact: API addition (no impact on existing code)
Currently the GNU/Linux kernel only support timebase-frequency in cpus node of the Device Tree. The compiled *.dtb file can not work well with the kernel. It will produce Kernel panic - not syncing: 4RISC-V system with no 'timebase-frequency' in DTS. Thus we add timebase-frequency property in cpus node of the Device-Tree in addition to the cpu node. The panic do not appear in the booting stage now.

Development Phase: implementation

Release Notes
To work well with the GNU/Linux kernel in booting stage, we add timebase-frequency in cpus node of the Device-Tree in addition to the cpu node.

	modified:   src/main/scala/diplomacy/Resources.scala
	modified:   src/main/scala/subsystem/BaseSubsystem.scala
@mbs0221 mbs0221 changed the title Timebase Support timebase-frequency in the cpus node Feb 2, 2021
@mbs0221 mbs0221 changed the title Support timebase-frequency in the cpus node Support timebase-frequency in the cpus node of the Device Tree. Feb 2, 2021
@hcook hcook requested review from hcook and terpstra February 11, 2021 19:45
@hcook hcook merged commit da5edcb into chipsalliance:master Feb 11, 2021
@mbs0221 mbs0221 deleted the timebase branch February 20, 2021 16:04
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