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

updateScreen() support using clip rectangle #33

Merged
merged 3 commits into from Apr 22, 2023

Conversation

KurtE
Copy link
Contributor

@KurtE KurtE commented Apr 10, 2023

Hi @PaulStoffregen @mjs513,

The ST77xx code did not have the code in place to support doing updateScreen from Frame buffer and have it only update the area within current clip rectangle, like some of the others do.

So added it. This does not yet include the code that figures out what areas of the display have been updated (dirty) and use that setting. not sure yet if that was added into all of the graphic primitives. Will check and see if I can simply add a few checks and sets, else need to merge in some more code.

Using the clip code to limit how much of the display gets updated can speed up things a lot in some cases.

Can merge nor or wait and see if we try to add in additional stuff.

Longer term may update these drivers to allow the clip code to work in the Asyn updates using DMA.

Now off to play

The code was not in place to allow the output to be clipped to the area in the current clip rectangle.

Can speed up output considerably if used right.
Copied over the function calls from the ILI9341_t3n.  There is maybe some more functions missing the updates.

But also added in new code from ILI9341_t3n the code now that when you do writeRect it looks and only marks the dirty area for the bounding rectangle of changed pixels (which may be the whole thing
Use similar settings to what the Adafruit library uses
@PaulStoffregen PaulStoffregen merged commit 25978c8 into PaulStoffregen:master Apr 22, 2023
@KurtE KurtE deleted the clipped_updateScreen branch April 22, 2023 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants