nxp-mcuxpresso / mcux-sdk

MCUXpresso SDK
BSD 3-Clause "New" or "Revised" License
323 stars 142 forks source link

mcux: mcux-sdk: driver: flexcan: propagate timestamp on RxOverflow #105

Closed henrikbrixandersen closed 1 year ago

henrikbrixandersen commented 1 year ago

Prerequisites

Describe the pull request Propagate timestamp on kStatus_FLEXCAN_RxOverflow status when using the FlexCAN transactional APIs. Originally submitted to the NXP HAL in Zephyr RTOS: https://github.com/zephyrproject-rtos/hal_nxp/pull/217

Type of change

Tests

henrikbrixandersen commented 1 year ago

@mmahadevan108 Can you help get this reviewed?

mmahadevan108 commented 1 year ago

@henrikbrixandersen, thank you for the PR. I have created a ticket in our internal SDK system as well to draw attention to this PR.

zejiang0jason commented 1 year ago

Hi @henrikbrixandersen , this patch looks reasonable. may I know your use case, or how this issue impacts your case? thank you very much.

henrikbrixandersen commented 1 year ago

Hi @henrikbrixandersen , this patch looks reasonable. may I know your use case, or how this issue impacts your case? thank you very much.

Sure, my use-case is detailed in this Zephyr PR: https://github.com/zephyrproject-rtos/zephyr/pull/53444 Without this patch. the Rx overflow count is not propagated to the upper layers, and thus not possible to expose as a statistic.

henrikbrixandersen commented 1 year ago

Title and description updated after discussion with @dleach02 @ ZDS 2023 in Prague.