Skip to content

video: allocate the video tile before the BSP - #6

Merged
finger563 merged 1 commit into
mainfrom
fix/video-tile-first
Oct 19, 2026
Merged

finger563 merged 1 commit into
mainfrom
fix/video-tile-first

Conversation

@finger563

Copy link
Copy Markdown
Contributor

Summary

tab5-emu's 64 KB video conversion tile needs one contiguous block of internal RAM. It was allocated in initialize_video(), after the BSP had already set up the display, audio DMA and USB.

With espp #738's proposed rotation tile (16–32 KB of internal RAM in the BSP), no 64 KB block was left. The tile then fell back to PSRAM (No internal RAM for the 65536 byte video tile; using PSRAM), which slows every game's video path.

initialize_tab5() now allocates the tile first. The call in initialize_video() stays and does nothing once the tile exists.

Testing

  • On the Tab5 with espp #738 plus the rotation tile (16 KB): the tile is in internal RAM again and the fallback warning is gone.
  • With the current espp BSP, the change has no effect: the 64 KB block was always available, just later.

🤖 Generated with Claude Code

The 64 KB conversion tile needs one contiguous block of internal RAM. It
was allocated in initialize_video(), after the BSP had set up the display,
audio DMA and USB; with espp #738's rotation tile (16-32 KB of internal
RAM in the BSP) no 64 KB block was left and the tile fell back to PSRAM,
slowing every game's video path. initialize_tab5() now allocates it first.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

ESP-IDF Size Report for 'Tab5 Emu'

Metric Base PR Delta
FLASH 0 bytes (0.00%) 0 bytes (0.00%) 0 bytes (0.00%)
DRAM 266,590 bytes (60.41%) 266,590 bytes (60.41%) 0 bytes (0.00%)
IRAM 0 bytes 0 bytes 0 bytes
RAM (DRAM+IRAM) 266,590 bytes 266,590 bytes 0 bytes (0.00%)

FLASH uses app .bin size or json2 flash sum. RAM sums DRAM+IRAM via idf_size. Percentages shown when totals are available.
DRAM/IRAM usage does not include memory used by the heap allocator at runtime.
This report was generated by esp-idf-size-delta.

@finger563
finger563 merged commit 42a7b51 into main Oct 19, 2026
1 check passed
@finger563
finger563 deleted the fix/video-tile-first branch October 19, 2026 23:10
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.

1 participant