Skip to content

Fix issue loading RM2 + Sync on a Pico1W#389

Merged
erichelgeson merged 4 commits into
mainfrom
eric/stinkerton-debug
Jul 15, 2026
Merged

Fix issue loading RM2 + Sync on a Pico1W#389
erichelgeson merged 4 commits into
mainfrom
eric/stinkerton-debug

Conversation

@erichelgeson

@erichelgeson erichelgeson commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Rework PIO loading a bit to support dynamic loading and tracking - also turn down tud_task to avoid any conflicts with hot path.

Thanks to @stinkerton18 for reporting/debugging with me.

@erichelgeson erichelgeson requested a review from androda July 14, 2026 23:45
@github-actions

Copy link
Copy Markdown

BlueSCSI Memory Report

Compared against release v2026.04.27

Memory Usage

Target FLASH RAM SCRATCH_X SCRATCH_Y
Pico_2_Audio_SPDIF 679.0 KB (+4.3 KB) [33%] 511.7 KB [100%] 3.5 KB [88%] 3.8 KB [94%]
Pico_2_DaynaPORT 671.3 KB (+4.3 KB) [33%] 511.7 KB [100%] 3.5 KB [88%] 3.0 KB [75%]
Pico_Audio_SPDIF 701.0 KB (+4.6 KB) [34%] 252.4 KB [99%] 1.5 KB [38%] 768 B [19%]
Pico_DaynaPORT 692.9 KB (+4.7 KB) [34%] 246.1 KB [96%] 1.5 KB [38%]
Ultra 688.3 KB (+4.3 KB) [34%] 511.7 KB [100%] 3.5 KB [88%] 3.0 KB [75%]
Ultra_Wide 447.9 KB (+3.9 KB) [22%] 511.7 KB [100%] 2.0 KB [50%] 3.0 KB [75%]

Symbol Region Changes

Pico_Audio_SPDIF: 1 symbols moved RAM → FLASH (-96 B RAM)
Symbol Size
scsiDiskFolderIsTapeFolder(FsFile*) 96 B
Pico_DaynaPORT: 1 symbols moved RAM → FLASH (-96 B RAM)
Symbol Size
scsiDiskFolderIsTapeFolder(FsFile*) 96 B

@androda androda 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.

Assuming this has been tested on the 1W and a few other types just to be sure, the change looks good to me

@stinkerton18

Copy link
Copy Markdown

Indeed, I was testing/validating on my 1W with an Adaptec 2740, a couple Future Mark ISA cards, a BusLogic ISA card, and an even older 8-bit NCR card, all working now.

@erichelgeson erichelgeson merged commit d065e71 into main Jul 15, 2026
1 check passed
@erichelgeson erichelgeson deleted the eric/stinkerton-debug branch July 15, 2026 19:57
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