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.

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

include rmem with environment configs #54

Open
gtxaspec opened this issue Apr 22, 2024 · 2 comments
Open

include rmem with environment configs #54

gtxaspec opened this issue Apr 22, 2024 · 2 comments
Labels
pending needs more thought

Comments

@gtxaspec
Copy link
Collaborator

The current U-Boot environment is configured with default memory (rmem) values optimized for 1080p resolutions. This setup might not be ideal for sensors with resolutions that differ from 1080p. When the sensor resolution is either lower or higher than 1080p, this can result in either excessive memory usage or insufficient memory allocation, respectively. This leads to system instability or even prevents the streamer from starting.

We should define and include recommended rmem values in the device profiles for sensors with their respective resolutions. This approach will ensure more efficient memory usage and enhance system stability across different sensor configurations.

@gtxaspec
Copy link
Collaborator Author

maybe default values based off sensor resolution/sensor?

@gtxaspec
Copy link
Collaborator Author

added ISP reserved ram to config.soc.in; need to implement logic to update the env on build.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pending needs more thought
Projects
None yet
Development

No branches or pull requests

1 participant