-
Notifications
You must be signed in to change notification settings - Fork 57
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
Multi-device support #6
Comments
Let me start with making clear that english is not my native language. :-)
Okay, but why?
Again, for what purpose?
What wait-while-busy loops? Ah okay, so "Multi-device support" would be support to use more than one display? On the one hand I am intrigued. I designed this for a project, it fits on the backside of a EVE2-70G. If the customer would ask for a second screen to display additional information on I would just build annother unit with different software. I am afraid that adding multi-display support will make everything even slower than it already is. I could agree to a few things like putting the display paramters into a structure. Maybe there is a benefit for very special aplications like forming a cube of six modules. But this is, well, very special and should not be allowed to have a negative impact on everything else. Anyways, this is how I see it right now. :-) |
Alright, as promised the code as I have it is at https://github.com/alexvlockwood-bes/FT800-FT813/tree/4.0-bes-multiple. I see you're not particularly interested in this changeset so I'm not going to press further — not really worth arguing about the merits of supporting multiple displays from one master microcontroller or having timeouts in busy-loops. Thanks for the code, it works very smoothly and was a great starting point for our needs. 👍 |
Thank you for the feedback! |
Hi! I'm Alexis Lockwood, a software engineer at Boulder Engineering Studio; I found this driver useful for a recent project with some admittedly heavy modifications, thought I'd drop by and try to contribute something back.
Changes I've made
#define
s for display timing setup, instead allow a struct of values to be provided at runtime.eve_context_t *
, with that struct containing all the necessary per-instance state as well as a collection of function pointers implementing the underlying lower-level routines.Still planned
This is definitely not ready for immediate upstreaming. I've definitely broken all the example code, and some of these changes will balloon the generated code for small targets like AVR and PIC.
Additionally needed to properly merge upstream
PROGMEM
on AVR.When I've finished the "still planned" changes, I will certainly share with you the modified code — just the four files
EVE.h
,EVE_commands.h
,EVE_commands.c
,EVE_config.h
— no matter what. Let me know if this sounds like something you'd be willing to merge, as I may be able to spend some time making the rest of the changes needed to get this upstreamed.cc @austinbes
The text was updated successfully, but these errors were encountered: