Skip to content

Throw error when not using software spinlocks on RP2350 #99

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.

Sign up for GitHub

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

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

Throw error when not using software spinlocks on RP2350 #99

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
9743419
Throw error when not using software spinlocks on RP2350
will-v-pi Sep 10, 2025
2c6e8f4
scanvideo too
will-v-pi Sep 10, 2025
de5b8c6
Update error message
will-v-pi Sep 10, 2025
3c1fc67
Apply suggestions from code review
will-v-pi Sep 11, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
scanvideo too
  • Loading branch information
@will-v-pi
will-v-pi authored Sep 10, 2025
commit 2c6e8f4bbba66ede75f09da8e0a750c2985c33f9
16 changes: 16 additions & 0 deletions src/common/pico_scanvideo/include/pico/scanvideo/scanvideo_base.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -319,20 +319,36 @@ extern void scanvideo_default_configure_pio(pio_hw_t *pio, uint sm, uint offset,
#endif

#ifndef PICO_SPINLOCK_ID_VIDEO_SCANLINE_LOCK
#if PICO_RP2350 && !PICO_USE_SW_SPIN_LOCKS
#error "When not using software spin locks on RP2350, you must explicitly define the pico-extras spinlock IDs, accounting for Errata E2 and the SDK spin lock IDs"
#else
#define PICO_SPINLOCK_ID_VIDEO_SCANLINE_LOCK 2
#endif
#endif

#ifndef PICO_SPINLOCK_ID_VIDEO_FREE_LIST_LOCK
#if PICO_RP2350 && !PICO_USE_SW_SPIN_LOCKS
#error "When not using software spin locks on RP2350, you must explicitly define the pico-extras spinlock IDs, accounting for Errata E2 and the SDK spin lock IDs"
#else
#define PICO_SPINLOCK_ID_VIDEO_FREE_LIST_LOCK 3
#endif
#endif

#ifndef PICO_SPINLOCK_ID_VIDEO_DMA_LOCK
#if PICO_RP2350 && !PICO_USE_SW_SPIN_LOCKS
#error "When not using software spin locks on RP2350, you must explicitly define the pico-extras spinlock IDs, accounting for Errata E2 and the SDK spin lock IDs"
#else
#define PICO_SPINLOCK_ID_VIDEO_DMA_LOCK 4
#endif
#endif

#ifndef PICO_SPINLOCK_ID_VIDEO_IN_USE_LOCK
#if PICO_RP2350 && !PICO_USE_SW_SPIN_LOCKS
#error "When not using software spin locks on RP2350, you must explicitly define the pico-extras spinlock IDs, accounting for Errata E2 and the SDK spin lock IDs"
#else
#define PICO_SPINLOCK_ID_VIDEO_IN_USE_LOCK 5
#endif
#endif

// note this is not necessarily an absolute gpio pin mask, it is still shifted by PICO_SCANVIDEO_COLOR_PIN_BASE
#define PICO_SCANVIDEO_ALPHA_MASK (1u << PICO_SCANVIDEO_ALPHA_PIN)
Expand Down