Modifications for using with Zephyr#34
Closed
mjs513 wants to merge 9 commits intoarduino-libraries:mainfrom
Closed
Modifications for using with Zephyr#34mjs513 wants to merge 9 commits intoarduino-libraries:mainfrom
mjs513 wants to merge 9 commits intoarduino-libraries:mainfrom
Conversation
|
|
|
Memory usage change @ af959eb
Click for full report table
Click for full report CSV |
|
Memory usage change @ e05eb4e
Click for full report table
Click for full report CSV |
Author
|
Yeah - unfortunately they are not using the st7701 display driver. They are using this driver. When I first started this had the same concern then found that it was different. What sketch are you trying to run? Mike Update: PS as a reminder arduino_H7_video does not work currently with zephyr |
Author
|
Here is a good example that should work: Sorry in the middle of something else .. havent synched up with the pre-release version yet. Probably later today. |
|
Thanks for the info, i may have fount the issue, now (zephyr release 4.2) the driver is using |
Author
Ok thats a change - thanks for the update. Right now in the gigsdisplay.h file I have: and this so it may still work: |
per1234
added
type: enhancement
Fixes issue with struct display_buffer_descriptor *buf_desc; never being allocated - see arduino/ArduinoCore-zephyr#274 (comment)
|
Memory usage change @ 23a01e1
Click for full report table
Click for full report CSV |
Contributor
|
@mjs513 Thanks for the great work, really appreciated! |
Author
|
Cant seem to get to that link - keep getting a 404 error. And if I do a
search for Arduino_video it comes up empty
Glad it helped though
…On Wed, Apr 1, 2026 at 5:18 AM Leonardo Cavagnis ***@***.***> wrote:
*leonardocavagnis* left a comment
(arduino-libraries/Arduino_GigaDisplay#34)
<#34?email_source=notifications&email_token=ABI6DRLZLVPWZ7XE6BXMZ7T4TTNGNA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTIMJWHA3TAOBTHA3KM4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJNLQOJPWG33NNVSW45C7N5YGK3S7MNWGSY3L#issuecomment-4168708386>
@mjs513 <https://github.com/mjs513> Thanks for the great work, really
appreciated!
We’ve decided not to bring the Arduino_H7_Video changes into this
repository, but instead to create a separate Arduino_Video
<https://github.com/arduino-libraries/Arduino_Video> library.
You can find the branch I’m currently working on, where I’ve integrated
your changes, here:
https://github.com/arduino-libraries/Arduino_Video/tree/feat/giga-display-shield-zephyr
—
Reply to this email directly, view it on GitHub
<#34?email_source=notifications&email_token=ABI6DRLZLVPWZ7XE6BXMZ7T4TTNGNA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTIMJWHA3TAOBTHA3KM4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJNLQOJPWG33NNVSW45C7N5YGK3S7MNWGSY3L#issuecomment-4168708386>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABI6DRPA5VXXTB2H2FMZLP34TTNGNAVCNFSM6AAAAACDBY4YJGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHM2DCNRYG4YDQMZYGY>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Contributor
The reason is that the repository is currently private. So it is only accessible to Arduino employees (or other users an administrator has granted access). @leonardocavagnis can the repository be made public? |
Contributor
Not yet! Sorry, it's still in progress |
Contributor
|
Hi @mjs513, thank you for your work on this! I have moved this implementation to #46, where I’ve updated the code to integrate with the new Arduino_Video library (WIP & private). Closing this in favor of #46. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Modifications made so can be used with the GigaDisplay whether using Zephyr or mbed.
The changes made for zephyr imply that Arduino_h7_portenta is no longer needed to run the display and gfx as well as the camera - does not support UVC.
In addition as mentioned here
Not sure if everyone prefers if they are deleted or just left as place holders?
@KurtE and I have been using the library since we created it.
Readme will probably have to be updated at some point. Would consider this first release