2 x changes to hal_buffers (1 bug, 1 add)

Report here problems in any of ChibiOS components. This forum is NOT for support.
User avatar
FXCoder
Posts: 116
Joined: Sun Jun 12, 2016 4:10 am
Location: Sydney, Australia
Has thanked: 30 times
Been thanked: 30 times

2 x changes to hal_buffers (1 bug, 1 add)

Postby FXCoder » Mon Sep 10, 2018 2:27 am

#1 Documentation fix in hal_buffers.c relating to output queue use case
#2 Add bqSetLinkX to allow updating of user defined link (same functionality that was added to hal_queues)

Code: Select all

Index: include/hal_buffers.h
===================================================================
--- include/hal_buffers.h   (revision 12246)
+++ include/hal_buffers.h   (working copy)
@@ -175,6 +175,16 @@
 #define bqGetLinkX(bqp) ((bqp)->link)
 
 /**
+ * @brief   Sets the queue application-defined link.
+ *
+ * @param[in] bqp       pointer to an @p io_buffers_queue_t structure
+ * @param[in] lk        The application-defined link.
+ *
+ * @special
+ */
+#define bqSetLinkX(bqp, lk) ((bqp)->link = lk)
+
+/**
  * @brief   Return the suspended state of the queue.
  *
  * @param[in] bqp       pointer to an @p io_buffers_queue_t structure
Index: src/hal_buffers.c
===================================================================
--- src/hal_buffers.c   (revision 12246)
+++ src/hal_buffers.c   (working copy)
@@ -28,7 +28,7 @@
  *          - <b>Input queue</b>, unidirectional queue where the writer is the
  *            ISR side and the reader is the thread side.
  *          - <b>Output queue</b>, unidirectional queue where the writer is the
- *            ISR side and the reader is the thread side.
+ *            thread side and the reader is the ISR side.
  *          - <b>Full duplex queue</b>, bidirectional queue. Full duplex queues
  *            are implemented by pairing an input queue and an output queue
  *            together.

Return to “Bug Reports”

Who is online

Users browsing this forum: No registered users and 1 guest