Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
45 commits
Select commit Hold shift + click to select a range
d8e1197
Add shared PendSV deferred service dispatcher
crabel99 Jun 20, 2026
b04854c
define sercom transaction structure for enqueueing async transactions.
crabel99 Jan 28, 2026
60af7ce
provide unified structure for maintaining and accessing sercom protoc…
crabel99 Jan 28, 2026
042458f
add support for mux pin deconfliction during development.
crabel99 Jan 28, 2026
755e70a
integrate needed dma sercom structure into the SERCOM class
crabel99 Jan 28, 2026
b6768fa
refactored initMaster/SlaveWIRE() to support rapid swapping of roles …
crabel99 Jan 29, 2026
d16a831
generalize RingBuffer to support any type not just uint8_t while pres…
crabel99 Jan 29, 2026
57c89a0
refactored stopTransmissionWIRE to support DMA and async operations, …
crabel99 Jan 29, 2026
8d1b6b5
add 10-bit address support for client operations
crabel99 Jan 29, 2026
c9d1336
finish implementing wire async and dma callback structure and inlinin…
crabel99 Jan 30, 2026
a789b2e
add async and automatic dma support for client mode operations.
crabel99 Jan 31, 2026
0e91a3f
refactor SPI to use SERCOM dma implementation
crabel99 Feb 1, 2026
37bca6e
stage more UART DMA structure
crabel99 Feb 15, 2026
639e539
unify structure between different sercoms to mirror each other more c…
crabel99 Feb 15, 2026
fedf6db
get sercom's wire protocol working in master mode
crabel99 Feb 15, 2026
f1ffbcc
clean up SPI
crabel99 Feb 15, 2026
d186cab
finish refactor of Wire to fully use async SERCOM
crabel99 Feb 15, 2026
b81a58a
master wire fully tested sync/async
crabel99 Feb 17, 2026
8decc12
test SPI sync and async operations
crabel99 Feb 17, 2026
cc3657a
high level testing no hardware testing done on UART
crabel99 Feb 17, 2026
fdd11fd
refactor to use async sercom api's
crabel99 Feb 17, 2026
7ce665e
samd datasheet multiplexing tables as csv
crabel99 Feb 17, 2026
3228488
script to generate wire sercom specific pin assignments can be used …
crabel99 Feb 17, 2026
cb94be4
Fix CI build errors
crabel99 Feb 17, 2026
c543bb2
Make SPI interrupt handlers weak and fix example ambiguity
crabel99 Feb 17, 2026
b841afb
add getters for sercom class object
crabel99 Feb 20, 2026
edac060
add support to allow transaction chaining for uart/spi/i2c
crabel99 Feb 21, 2026
e97d102
finish support for chaining transactions and formatting cleanup
crabel99 Feb 22, 2026
29728a6
allow readDataWIRE() to allow passing deferStopWIRE() on the final read.
crabel99 Apr 2, 2026
0d615d8
Revert "allow readDataWIRE() to allow passing deferStopWIRE() on the …
crabel99 Apr 6, 2026
b84405f
Reapply "allow readDataWIRE() to allow passing deferStopWIRE() on the…
crabel99 Apr 13, 2026
6b5dd11
generate SPI and Wire pin mux arrays
crabel99 Apr 14, 2026
260da0b
document sercom_txn.h purpose more clearly
crabel99 Apr 14, 2026
dec082e
warn user if SPI or Wire pins are not properly assigned with a boolea…
crabel99 Apr 14, 2026
fff3c38
fix UART test regression
crabel99 Apr 14, 2026
5beb170
fix(spi): derive IRQ handlers from PERIPH_SPIx and add SPI6/SPI7 support
crabel99 Apr 17, 2026
eff0511
Preserve Wire sync compatibility while tightening async transaction h…
crabel99 May 17, 2026
ef295ff
Normalize async branch whitespace
crabel99 Jun 21, 2026
f4dc420
Route SERCOM deferred services through shared PendSV
crabel99 Jun 21, 2026
8cfeeaf
Add PendSV peripheral channel map
crabel99 Jun 24, 2026
1a13c8f
Add PHY PendSV channel allocation
crabel99 Jun 26, 2026
12eeb02
Add RTC PendSV channel allocation
crabel99 Jul 4, 2026
e9e81f7
fix(wire): restore multimaster role transitions
crabel99 Jul 16, 2026
58f6e68
fix(pendsv): coalesce TinyUSB deferred service
crabel99 Jul 18, 2026
485229a
Merge remote-tracking branch 'origin/pendsv-support-clean' into serco…
crabel99 Jul 18, 2026
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
3 changes: 3 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -4,3 +4,6 @@
bootloaders/*/build/
*~
/libraries/**/build/
.ci-home
.ci_home
*.log
161 changes: 161 additions & 0 deletions cores/arduino/PendSV.cpp
Original file line number Diff line number Diff line change
@@ -0,0 +1,161 @@
#include "PendSV.h"

#include <Arduino.h>
#include <limits.h>

#if defined(USE_TINYUSB)
extern "C" void TinyUSB_Device_Task(void);
#endif

namespace {
PendSV pendSv;

#if defined(USE_TINYUSB)
void tinyUsbDeviceTaskService(uint8_t serviceId, void *context) {
(void)serviceId;
(void)context;
// One invocation drains TinyUSB's queued controller events under OPT_OS_NONE.
TinyUSB_Device_Task();
}
#endif
}

PendSV &PendSV::instance() {
return pendSv;
}

#if defined(USE_TINYUSB)
extern "C" void tud_event_hook_cb(uint8_t rhport, uint32_t eventId, bool inIsr) {
(void)rhport;
(void)eventId;
(void)inIsr;
// TinyUSB owns the event queue. PendSV is only its deferred wake signal, so
// multiple controller events before dispatch require only one service call.
PendSV::instance().setPendingOnce(PendSVChannels::Usb);
}
#endif

bool PendSV::initializeCoreServices() {
#if defined(USE_TINYUSB)
return instance().registerService(PendSVChannels::Usb, tinyUsbDeviceTaskService, nullptr);
#else
return true;
#endif
}

uint32_t PendSV::enterCritical() {
const uint32_t primask = __get_PRIMASK();
__disable_irq();
return primask;
}

void PendSV::exitCritical(uint32_t primask) {
__set_PRIMASK(primask);
}

bool PendSV::registerService(uint8_t serviceId, ServiceFn fn, void *context) {
if (serviceId >= kMaxServices || fn == nullptr)
return false;

const uint32_t primask = enterCritical();
pendingCount_[serviceId] = 0;
pendingMask_ &= ~(1u << serviceId);
services_[serviceId].fn = fn;
services_[serviceId].context = context;
exitCritical(primask);

return true;
}

void PendSV::clearService(uint8_t serviceId) {
if (serviceId >= kMaxServices)
return;

const uint32_t primask = enterCritical();
services_[serviceId].fn = nullptr;
services_[serviceId].context = nullptr;
pendingCount_[serviceId] = 0;
pendingMask_ &= ~(1u << serviceId);
exitCritical(primask);
}

void PendSV::setPending(uint8_t serviceId) {
if (serviceId >= kMaxServices)
return;

const uint32_t primask = enterCritical();
uint16_t &pendingCount = pendingCount_[serviceId];
if (pendingCount < UINT16_MAX)
++pendingCount;
pendingMask_ |= (1u << serviceId);
exitCritical(primask);

__DMB();
SCB->ICSR = SCB_ICSR_PENDSVSET_Msk;
}

void PendSV::setPendingOnce(uint8_t serviceId) {
if (serviceId >= kMaxServices)
return;

const uint32_t primask = enterCritical();
pendingCount_[serviceId] = 1;
pendingMask_ |= (1u << serviceId);
exitCritical(primask);

__DMB();
SCB->ICSR = SCB_ICSR_PENDSVSET_Msk;
}

void PendSV::dispatchPending() {
// Bound one PendSV entry so high-rate producers do not monopolize return to
// thread mode. Remaining work re-pends PendSV below.
uint8_t dispatched = 0;

while (dispatched < kDispatchBudget) {
uint32_t primask = enterCritical();
const uint32_t pending = pendingMask_;
if (pending == 0) {
exitCritical(primask);
return;
}

const uint8_t serviceId = static_cast<uint8_t>(__builtin_ctz(pending));
uint16_t &pendingCount = pendingCount_[serviceId];

if (pendingCount == 0) {
// Defensive scrub in case mask and count drift out of sync.
pendingMask_ &= ~(1u << serviceId);
exitCritical(primask);
continue;
}

--pendingCount;
if (pendingCount == 0)
pendingMask_ &= ~(1u << serviceId);

ServiceEntry entry = services_[serviceId];
exitCritical(primask);

// Pending work without a registered service is intentionally dropped.
// Producers are expected to register before calling setPending(), and
// clearService() cancels queued work for that service.
if (entry.fn != nullptr)
entry.fn(serviceId, entry.context);

++dispatched;
}

const uint32_t primask = enterCritical();
const bool hasRemaining = (pendingMask_ != 0);
exitCritical(primask);

if (hasRemaining) {
__DMB();
SCB->ICSR = SCB_ICSR_PENDSVSET_Msk;
}
}

extern "C" void PendSV_Handler(void) {
PendSV::instance().dispatchPending();
}
129 changes: 129 additions & 0 deletions cores/arduino/PendSV.h
Original file line number Diff line number Diff line change
@@ -0,0 +1,129 @@
#pragma once

#include "PendSVChannelMap.h"
#include "sam.h"

#include <array>
#include <stdint.h>

#if defined(SERCOM0) || defined(SERCOM0_REGS)
#define PENDSV_HAS_SERCOM0 true
#else
#define PENDSV_HAS_SERCOM0 false
#endif
#if defined(SERCOM1) || defined(SERCOM1_REGS)
#define PENDSV_HAS_SERCOM1 true
#else
#define PENDSV_HAS_SERCOM1 false
#endif
#if defined(SERCOM2) || defined(SERCOM2_REGS)
#define PENDSV_HAS_SERCOM2 true
#else
#define PENDSV_HAS_SERCOM2 false
#endif
#if defined(SERCOM3) || defined(SERCOM3_REGS)
#define PENDSV_HAS_SERCOM3 true
#else
#define PENDSV_HAS_SERCOM3 false
#endif
#if defined(SERCOM4) || defined(SERCOM4_REGS)
#define PENDSV_HAS_SERCOM4 true
#else
#define PENDSV_HAS_SERCOM4 false
#endif
#if defined(SERCOM5) || defined(SERCOM5_REGS)
#define PENDSV_HAS_SERCOM5 true
#else
#define PENDSV_HAS_SERCOM5 false
#endif
#if defined(SERCOM6) || defined(SERCOM6_REGS)
#define PENDSV_HAS_SERCOM6 true
#else
#define PENDSV_HAS_SERCOM6 false
#endif
#if defined(SERCOM7) || defined(SERCOM7_REGS)
#define PENDSV_HAS_SERCOM7 true
#else
#define PENDSV_HAS_SERCOM7 false
#endif

#if defined(USB) || defined(USB_REGS)
#define PENDSV_HAS_USB true
#else
#define PENDSV_HAS_USB false
#endif

#if defined(GMAC) || defined(GMAC_REGS)
#define PENDSV_HAS_GMAC true
#define PENDSV_HAS_PHY true
#else
#define PENDSV_HAS_GMAC false
#define PENDSV_HAS_PHY false
#endif
#if defined(ADC) || defined(ADC_REGS) || defined(ADC0) || defined(ADC0_REGS)
#define PENDSV_HAS_ADC true
#else
#define PENDSV_HAS_ADC false
#endif
#if defined(AES) || defined(AES_REGS)
#define PENDSV_HAS_AES true
#else
#define PENDSV_HAS_AES false
#endif
#if defined(PUKCC) || defined(PUKCC_REGS) || defined(ID_PUKCC) || \
defined(PUKCC_INSTANCE_ID) || defined(PUKCC_IRQn)
#define PENDSV_HAS_PUKCC true
#else
#define PENDSV_HAS_PUKCC false
#endif
#if defined(TRNG) || defined(TRNG_REGS)
#define PENDSV_HAS_TRNG true
#else
#define PENDSV_HAS_TRNG false
#endif
#if defined(RTC) || defined(RTC_REGS)
#define PENDSV_HAS_RTC true
#else
#define PENDSV_HAS_RTC false
#endif

using PendSVChannels = PendSVChannelMap<
PENDSV_HAS_SERCOM0, PENDSV_HAS_SERCOM1, PENDSV_HAS_SERCOM2,
PENDSV_HAS_SERCOM3, PENDSV_HAS_SERCOM4, PENDSV_HAS_SERCOM5,
PENDSV_HAS_SERCOM6, PENDSV_HAS_SERCOM7, PENDSV_HAS_USB, PENDSV_HAS_GMAC,
PENDSV_HAS_PHY, PENDSV_HAS_ADC, PENDSV_HAS_AES, PENDSV_HAS_PUKCC,
PENDSV_HAS_TRNG, PENDSV_HAS_RTC>;

class PendSV {
public:
using ServiceFn = void (*)(uint8_t serviceId, void *context);
static constexpr uint8_t kMaxServices = PendSVChannels::kMaxServices;
static constexpr uint8_t kDispatchBudget = 16;
static_assert(kMaxServices <= 32, "PendSV pending mask supports at most 32 services");
static_assert(PendSVChannels::Count <= kMaxServices,
"PendSV channel allocation exceeds dispatcher capacity");

static PendSV &instance();
static bool initializeCoreServices();

bool registerService(uint8_t serviceId, ServiceFn fn, void *context = nullptr);
// Cancels queued work that has not started dispatching. If a callback was
// already copied for dispatch, its context must remain valid until it returns.
void clearService(uint8_t serviceId);
void dispatchPending();
void setPending(uint8_t serviceId);
void setPendingOnce(uint8_t serviceId);

private:
static uint32_t enterCritical();
static void exitCritical(uint32_t primask);

struct ServiceEntry {
ServiceFn fn = nullptr;
void *context = nullptr;
};

std::array<ServiceEntry, kMaxServices> services_{};
std::array<uint16_t, kMaxServices> pendingCount_{};
volatile uint32_t pendingMask_ = 0;
};
61 changes: 61 additions & 0 deletions cores/arduino/PendSVChannelMap.h
Original file line number Diff line number Diff line change
@@ -0,0 +1,61 @@
#pragma once

#include <stdint.h>

template <bool HasSercom0, bool HasSercom1, bool HasSercom2, bool HasSercom3,
bool HasSercom4, bool HasSercom5, bool HasSercom6, bool HasSercom7,
bool HasUsb, bool HasGmac, bool HasPhy, bool HasAdc, bool HasAes,
bool HasPukcc, bool HasTrng, bool HasRtc>
struct PendSVChannelMap {
static constexpr uint8_t kMaxServices = 32;
static constexpr uint8_t kUnavailable = 0xFF;

private:
static constexpr uint8_t assign(bool present, uint8_t next) {
return present ? next : kUnavailable;
}

static constexpr uint8_t advance(bool present, uint8_t next) {
return present ? static_cast<uint8_t>(next + 1) : next;
}

static constexpr uint8_t kBase = 0;
static constexpr uint8_t kAfterSercom0 = advance(HasSercom0, kBase);
static constexpr uint8_t kAfterSercom1 = advance(HasSercom1, kAfterSercom0);
static constexpr uint8_t kAfterSercom2 = advance(HasSercom2, kAfterSercom1);
static constexpr uint8_t kAfterSercom3 = advance(HasSercom3, kAfterSercom2);
static constexpr uint8_t kAfterSercom4 = advance(HasSercom4, kAfterSercom3);
static constexpr uint8_t kAfterSercom5 = advance(HasSercom5, kAfterSercom4);
static constexpr uint8_t kAfterSercom6 = advance(HasSercom6, kAfterSercom5);
static constexpr uint8_t kAfterSercom7 = advance(HasSercom7, kAfterSercom6);
static constexpr uint8_t kAfterUsb = advance(HasUsb, kAfterSercom7);
static constexpr uint8_t kAfterGmac = advance(HasGmac, kAfterUsb);
static constexpr uint8_t kAfterPhy = advance(HasPhy, kAfterGmac);
static constexpr uint8_t kAfterAdc = advance(HasAdc, kAfterPhy);
static constexpr uint8_t kAfterAes = advance(HasAes, kAfterAdc);
static constexpr uint8_t kAfterPukcc = advance(HasPukcc, kAfterAes);
static constexpr uint8_t kAfterTrng = advance(HasTrng, kAfterPukcc);

public:
static constexpr uint8_t Sercom0 = assign(HasSercom0, kBase);
static constexpr uint8_t Sercom1 = assign(HasSercom1, kAfterSercom0);
static constexpr uint8_t Sercom2 = assign(HasSercom2, kAfterSercom1);
static constexpr uint8_t Sercom3 = assign(HasSercom3, kAfterSercom2);
static constexpr uint8_t Sercom4 = assign(HasSercom4, kAfterSercom3);
static constexpr uint8_t Sercom5 = assign(HasSercom5, kAfterSercom4);
static constexpr uint8_t Sercom6 = assign(HasSercom6, kAfterSercom5);
static constexpr uint8_t Sercom7 = assign(HasSercom7, kAfterSercom6);
static constexpr uint8_t Usb = assign(HasUsb, kAfterSercom7);
static constexpr uint8_t Gmac = assign(HasGmac, kAfterUsb);
static constexpr uint8_t Phy = assign(HasPhy, kAfterGmac);
static constexpr uint8_t Adc = assign(HasAdc, kAfterPhy);
static constexpr uint8_t Aes = assign(HasAes, kAfterAdc);
static constexpr uint8_t Pukcc = assign(HasPukcc, kAfterAes);
static constexpr uint8_t Trng = assign(HasTrng, kAfterPukcc);
static constexpr uint8_t Rtc = assign(HasRtc, kAfterTrng);
static constexpr uint8_t Count = advance(HasRtc, kAfterTrng);

static constexpr bool isAvailable(uint8_t channel) {
return channel != kUnavailable;
}
};
Loading
Loading