Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork30
Portenta H7: Enable Camera clock#184
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
KurtE commentedAug 25, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
@iabdalkader@pillo79@facchinm and others: As I have asked in issue#164 wondering best way to do this. For example: currently it is setup like:
Note: this includes my one line change in this PR to enable it on Portenta H7. But I keep wondering on ArduinoCore-zephyr, that Not all user sketches use Cameras, and yet we always start up Also with this, maybe define some way to allow the sketch to specify which pins and/or which Carrier/Shield they Thoughts? Edit: Note: there currently is no approved way to actually access the PK_1 pin on Zephyr. Except if it is called out |
3d8bddd
to75ccdda
CompareWe need to start the clock in the same way as we do for the Arduino GIGA.As without the clock started, most cameras will not even make the cameravisible to I2C
Note: - I believe that my PR#195 is a better solution. It does not start the clocks until/if the sketch calls camera.begin But to do this the cameras and DCIM need to be marked as defered init in the overlay, as to not have them try to initialize things |
We need to start the clock in the same way as we do for the Arduino GIGA.
As without the clock started, most cameras will not even make the camera visible to I2C
This unrelated change was in the branch of#165
And has now been pulled out of it.
This PR is only a change of a #if statement that adds the Portenta H7 to the line that enables the clock for the Giga.
And only if Video (Camera) is configured in the .conf files.
Currently the overlay for the PortentaH7 is setup with the configuration to run on the Mid Carrier board, which is
different than the Portenta Vision shield. At some point probably need to configure it for either. Not sure
what Arduino wishes to do for this. Define them as Shields in Zephyr and have multiple variants? But that
is beyond the scope of this simple change.