Skip to content
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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Dummy Display driver for native_posix #12001

Closed
vanwinkeljan opened this issue Dec 11, 2018 · 1 comment · Fixed by #12223
Closed

Dummy Display driver for native_posix #12001

vanwinkeljan opened this issue Dec 11, 2018 · 1 comment · Fixed by #12223
Assignees
@vanwinkeljan
Labels
area: Display area: native port Host native arch port (native_sim) Feature Request A request for a new feature

Comments

@vanwinkeljan
Copy link
Member

Is your feature request related to a problem? Please describe.
The current native_posix SDL based display driver is not cleaning up its used memory, this is caused mainly by the SDL and OpenGL libraries and as such can not be corrected. The net results is that it is very hard to use valgrind once the SDL based display is enabled due to the large amount, ~3MB, of memory that is not cleaned up.

Describe the solution you'd like
Add a dummy display driver for native_posix that only implements the API

Describe alternatives you've considered
None

Additional context
See discussion #9149 (comment) for more context

This feature could also be useful for running display related code in CI without including SDL2 (and all it dependencies) into the docker CI container
@vanwinkeljan vanwinkeljan added Feature Request A request for a new feature area: native port Host native arch port (native_sim) area: Display labels Dec 11, 2018
@vanwinkeljan vanwinkeljan self-assigned this Dec 11, 2018
@vanwinkeljan
Copy link
Member Author

@aescolar fyi

vanwinkeljan added a commit to vanwinkeljan/zephyr that referenced this issue Dec 23, 2018
@vanwinkeljan
driver: display: Added dummy display driver
1e88ccd 
Added dummy display driver.

fixes: zephyrproject-rtos#12001

Signed-off-by: Jan Van Winkel <[email protected]>
@vanwinkeljan vanwinkeljan mentioned this issue Dec 23, 2018
Merged
nashif pushed a commit that referenced this issue Dec 30, 2018
@vanwinkeljan @nashif
driver: display: Added dummy display driver
848b084 
Added dummy display driver.

fixes: #12001

Signed-off-by: Jan Van Winkel <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vanwinkeljan vanwinkeljan

Labels
area: Display area: native port Host native arch port (native_sim) Feature Request A request for a new feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Added dummy display driver vanwinkeljan/zephyr
1 participant
@vanwinkeljan