config: Remove redundant default clocks
Review Request #1878 - Created May 22, 2013 and submitted
| Information | |
|---|---|
| Andreas Hansson | |
| gem5 | |
| default | |
| Reviewers | |
| Default | |
Changeset 9736:a40008d23a51 --------------------------- config: Remove redundant default clocks This patch removes the clocks of certain instances of CoherentBus, NonCoherentBus and IOCache where the specified clock is same as the default value of the system clock. As all the values used are the defaults, there are no performance changes. There are similar cases where the toL2Bus is set to use the parent CPU clock which is already the default behaviour. The main motivation for these simplifications is to ease the introduction of clock domains.
All regressions pass without any changes
Ship It!
You might want to update the message to make it clear that you're just removing the explicit setting of the clock parameter values, not removing the clocks themselves (which had me confused until I looked at the diff).