Repository navigation
[Version 3] Raspberry CPU support - #791
benoit-cty wants to merge 1 commit into
Conversation
| B77,95 | ||
| B97,95 | ||
| B99,95 | ||
| Cortex-A72,7.5 |
There was a problem hiding this comment.
That's because the Cortex-A72 is integrated into the Broadcom BCM2711 System on a Chip (SoC) for the Raspberry Pi 4. The entire BCM2711 chip (which packages the quad-core Cortex-A72 CPU alongside the VideoCore VI GPU and I/O controllers) is broadly rated with a TDP of around 7.5 Watts.
8eac358 to
0792bc8
Compare
|
@benoit-cty you have some conflicts there, can you check? It would be interesting to merge this thanks! |
|
@benoit-cty heads-up: #797 covers the Raspberry Pi power reading through |
|
Hi David, |
d27aa53 to
0941389
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #791 +/- ##
=======================================
Coverage 92.07% 92.07%
=======================================
Files 49 49
Lines 5202 5202
=======================================
Hits 4790 4790
Misses 412 412 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Ready to merge. Can someone review this 2 lines PR ? |
davidberenstein1957
left a comment
There was a problem hiding this comment.
Two rows in cpu_power.csv, placed in order, and the values are in line with the Pi 4 (BCM2711, about 6.5 to 7.5 W under load) and Pi 5 (BCM2712, about 12 W) board figures, so constant mode gets a sane number instead of the default. One thing I could not check from here: that cpuinfo on a Pi reports the model as Cortex-A72 / Cortex-A76 so the lookup actually hits. If you have seen that match on your Pi, good to merge.

Following #790 I've added the support for Rpi 4 and 5 CPU.
Source : https://gadgetversus.com/processor/broadcom-bcm2712-vs-broadcom-bcm2711/