Skip to content

Fix CO5300 updates when using 180-degree rotation#1623

Closed
kitazaki wants to merge 1 commit into
Moddable-OpenSource:publicfrom
kitazaki:public
Closed

Fix CO5300 updates when using 180-degree rotation#1623
kitazaki wants to merge 1 commit into
Moddable-OpenSource:publicfrom
kitazaki:public

Conversation

@kitazaki

@kitazaki kitazaki commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

Summary

This change removes the separate synchronous color-transfer path for rotation == 2 and routes it through the same color send flow used by the normal orientation. For 180-degree rotation, the update window is still adjusted per chunk based on the remaining update lines.

Rotation Moddable SDK 8.2.0 Updated driver Result
0 8 ms/frame 8 ms/frame no change
180 10 ms/frame 8 ms/frame 1.25x faster
90 full width 15 ms/frame 15 ms/frame no change
270 full width 15 ms/frame 15 ms/frame no change
90 narrow 5 ms/frame 5 ms/frame no change
270 narrow 5 ms/frame 5 ms/frame no change

Changes

  • Set the CO5300 update window per transferred chunk for rotation == 2
  • Reuse the existing async color transfer / queue handling in co5300Send
  • Remove the dedicated co5300SendHardwareRotated180 helper

## Summary

This change removes the separate synchronous color-transfer path for `rotation == 2` and routes it through the same color send flow used by the normal orientation. For 180-degree rotation, the update window is  still adjusted per chunk based on the remaining update lines.

## Changes

  - Set the CO5300 update window per transferred chunk for `rotation == 2`
  - Reuse the existing async color transfer / queue handling in `co5300Send`
  - Remove the dedicated `co5300SendHardwareRotated180` helper
@mkellner

mkellner commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator

This has been merged and will be available in the next release.

@mkellner mkellner closed this Jun 8, 2026
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.

2 participants