Skip to content

[Version 3] Raspberry CPU support - #791

Open
benoit-cty wants to merge 1 commit into
masterfrom
cpu/arm_pi5
Open

benoit-cty wants to merge 1 commit into
masterfrom
cpu/arm_pi5

Conversation

@benoit-cty

Copy link
Copy Markdown
Contributor

Following #790 I've added the support for Rpi 4 and 5 CPU.

Source : https://gadgetversus.com/processor/broadcom-bcm2712-vs-broadcom-bcm2711/

@benoit-cty
benoit-cty changed the base branch from master to codecarbon_v3_rc February 28, 2025 07:26
@benoit-cty
benoit-cty changed the base branch from codecarbon_v3_rc to master April 14, 2025 18:18
@benoit-cty
benoit-cty changed the base branch from master to codecarbon_v3_rc April 14, 2025 18:19
@benoit-cty
benoit-cty requested a review from inimaz April 14, 2025 18:30
@benoit-cty benoit-cty changed the title Raspberry CPU support [Version 3] Raspberry CPU support Apr 14, 2025
B77,95
B97,95
B99,95
Cortex-A72,7.5

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see in the reference that this one is 8W
image

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@benoit-cty
benoit-cty changed the base branch from codecarbon_v3_rc to master June 22, 2025 08:42
@inimaz

inimaz commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

@benoit-cty you have some conflicts there, can you check? It would be interesting to merge this thanks!

@inimaz inimaz added enhancement New feature or request question Further information is requested labels Jul 30, 2026
@davidberenstein1957

Copy link
Copy Markdown
Collaborator

@benoit-cty heads-up: #797 covers the Raspberry Pi power reading through vcgencmd pmic_read_adc in 8 files with a test, and is mergeable. Worth checking whether the Pi part of this PR is still needed next to it, or whether this one can shrink to the datasets and notebooks.

@benoit-cty

Copy link
Copy Markdown
Contributor Author

Hi David,
It's complementary : cpu database is for fallback.
I will look to merge these 2 PR.

@benoit-cty
benoit-cty requested a review from a team as a code owner October 9, 2026 17:27
@codecov

codecov Bot commented Oct 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.07%. Comparing base (8a8776a) to head (0941389).

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.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@benoit-cty

Copy link
Copy Markdown
Contributor Author

Ready to merge. Can someone review this 2 lines PR ?

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request question Further information is requested size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants