Skip to content

remove fake brightness feature - #3296

Merged
zxkmm merged 1 commit into
portapack-mayhem:nextfrom
zxkmm:remove_brightness
Aug 20, 2026
Merged

remove fake brightness feature#3296
zxkmm merged 1 commit into
portapack-mayhem:nextfrom
zxkmm:remove_brightness

Conversation

@zxkmm

@zxkmm zxkmm commented Aug 20, 2026

Copy link
Copy Markdown
Member

TLDR:

  • we have a weird lagging issue years ago, caused by fake brightness.
  • it got "fixed" later by assuming the slow part seems from repetedly read from p.mem.
  • the fixes worked for years.
  • it's broken again now and the assume fix is probably wrong too.
  • we have to remove it cuz in current SPI flash XIP structure it is too expensive to fix.

@zxkmm
zxkmm enabled auto-merge (squash) August 20, 2026 04:47
@gullradriel

Copy link
Copy Markdown
Member

Ok, that's your call bro.

@gullradriel
gullradriel requested a balanced review from Copilot August 20, 2026 18:44

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes the performance-heavy fake brightness feature while preserving persistent-memory layout compatibility.

Changes:

  • Removes fake-brightness rendering and cached state.
  • Removes related settings and status-bar controls.
  • Retires persistent fields as reserved entries.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Show a summary per file
File Description
firmware/common/portapack_persistent_memory.hpp Removes fake-brightness APIs.
firmware/common/portapack_persistent_memory.cpp Retires stored fields while preserving layout.
firmware/common/portapack_io.hpp Removes per-pixel brightness transforms.
firmware/common/portapack_io.cpp Removes brightness cache accessors.
firmware/common/lcd_ili9341.cpp Removes brightness cache refresh.
firmware/application/ui_navigation.hpp Removes the status-bar brightness button.
firmware/application/ui_navigation.cpp Removes brightness status behavior.
firmware/application/external/debug_pmem/ui_debug_pmem.cpp Removes obsolete debug fields.
firmware/application/apps/ui_settings.hpp Removes brightness settings controls.
firmware/application/apps/ui_settings.cpp Removes brightness settings logic.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@zxkmm
zxkmm merged commit 0840756 into portapack-mayhem:next Aug 20, 2026
3 checks passed
@zxkmm
zxkmm deleted the remove_brightness branch August 21, 2026 03:17
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.

3 participants