Skip to content

Commit bd8cc59

Browse files
nordic-krchcarlescufi
authored andcommitted
lib: os: mpsc_pbuf: Add const to mpsc_pbuf_free argument
Added const qualifier to argument in a function. Signed-off-by: Krzysztof Chruscinski <krzysztof.chruscinski@nordicsemi.no>
1 parent 6d72d91 commit bd8cc59

File tree

2 files changed

+6
-5
lines changed

2 files changed

+6
-5
lines changed

include/sys/mpsc_pbuf.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -226,7 +226,7 @@ const union mpsc_pbuf_generic *mpsc_pbuf_claim(struct mpsc_pbuf_buffer *buffer);
226226
* @param packet Packet.
227227
*/
228228
void mpsc_pbuf_free(struct mpsc_pbuf_buffer *buffer,
229-
union mpsc_pbuf_generic *packet);
229+
const union mpsc_pbuf_generic *packet);
230230

231231
/** @brief Check if there are any message pending.
232232
*

lib/os/mpsc_pbuf.c

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -432,18 +432,19 @@ const union mpsc_pbuf_generic *mpsc_pbuf_claim(struct mpsc_pbuf_buffer *buffer)
432432
}
433433

434434
void mpsc_pbuf_free(struct mpsc_pbuf_buffer *buffer,
435-
union mpsc_pbuf_generic *item)
435+
const union mpsc_pbuf_generic *item)
436436
{
437437
uint32_t wlen = buffer->get_wlen(item);
438438
k_spinlock_key_t key = k_spin_lock(&buffer->lock);
439+
union mpsc_pbuf_generic *witem = (union mpsc_pbuf_generic *)item;
439440

440-
item->hdr.valid = 0;
441+
witem->hdr.valid = 0;
441442
if (!(buffer->flags & MPSC_PBUF_MODE_OVERWRITE) ||
442443
((uint32_t *)item == &buffer->buf[buffer->rd_idx])) {
443-
item->hdr.busy = 0;
444+
witem->hdr.busy = 0;
444445
buffer->rd_idx = idx_inc(buffer, buffer->rd_idx, wlen);
445446
} else {
446-
item->skip.len = wlen;
447+
witem->skip.len = wlen;
447448
}
448449
MPSC_PBUF_DBG(buffer, "freed: %p ", item);
449450

0 commit comments

Comments
 (0)