Skip to content

Commit

Permalink
Remove mutex in CAN driver to fix read in CAN RxIrq
Browse files Browse the repository at this point in the history
  • Loading branch information
Flowm committed Mar 7, 2019
1 parent 2fd0c5c commit c8a37ff
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions drivers/CAN.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -75,9 +75,9 @@ int CAN::write(CANMessage msg)

int CAN::read(CANMessage &msg, int handle)
{
lock();
// XXX: Remove mutex to fix read in CAN RxIrq
// https://github.com/ARMmbed/mbed-os/issues/9495
int ret = can_read(&_can, &msg, handle);
unlock();
return ret;
}

Expand Down

1 comment on commit c8a37ff

@TacoGrandeTX
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Flowm Thank you for the commit. Have you tested this and were you planning on making a PR for this? I have looped in the ST team to review.

Please sign in to comment.