michaelh/zephyr
1
0
Fork 0
Code Releases Activity

drivers: can: loopback: rename receive function

Browse source 
Rename the CAN loopback driver receive function to make the code easier to
understand. Add proper check against NULL for RX callback function
pointers.

Signed-off-by: Henrik Brix Andersen <hebad@vestas.com>
This commit is contained in:
Henrik Brix Andersen 2022-10-31 12:54:18 +01:00 • committed by Carles Cufí
commit 728138023b
1 changed files with 5 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

10
drivers/can/can_loopback.c
View file

@ -46,9 +46,9 @@ struct can_loopback_data {
CONFIG_CAN_LOOPBACK_TX_THREAD_STACK_SIZE);
};
static void dispatch_frame(const struct device *dev,
const struct can_frame *frame,
struct can_loopback_filter *filter)
static void receive_frame(const struct device *dev,
const struct can_frame *frame,
struct can_loopback_filter *filter)
{
struct can_frame frame_tmp = *frame;
@ -82,9 +82,9 @@ static void tx_thread(void *arg1, void *arg2, void *arg3)
for (int i = 0; i < CONFIG_CAN_MAX_FILTER; i++) {
filter = &data->filters[i];
if (filter->rx_cb &&
if (filter->rx_cb != NULL &&
can_utils_filter_match(&frame.frame, &filter->filter)) {
dispatch_frame(dev, &frame.frame, filter);
receive_frame(dev, &frame.frame, filter);
}
}