Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix CAN filtering to allow mask matching #30

Open
heliochronix opened this issue Aug 13, 2020 · 0 comments
Open

Fix CAN filtering to allow mask matching #30

heliochronix opened this issue Aug 13, 2020 · 0 comments
Assignees
Labels
bug Something isn't working

Comments

@heliochronix
Copy link
Member

heliochronix commented Aug 13, 2020

Currently, CAN filtering implements a per-message-ID filtering scheme in order to maximize the available number of filters that can be used. However, in the case of CANopen Emergency Messages, we need to have at least one message mask filter in place to match all 0x80+NodeID messages. Right now the C3 card can't receive Emergency messages due to this issue.

The challenge is, the index of the CAN filter acts as the index into the internal CANopenNode buffer in order to process the message. We need to find a way that ensures the index remains correct.

@heliochronix heliochronix added bug Something isn't working Critical labels Aug 13, 2020
@heliochronix heliochronix added this to the OreSat 0 milestone Aug 13, 2020
@heliochronix heliochronix self-assigned this Aug 13, 2020
@andrewgreenberg andrewgreenberg removed this from the OreSat 0 milestone Nov 22, 2021
@wrh2 wrh2 added the C3 label Apr 22, 2022
@ryanpdx ryanpdx removed the C3 label Jun 2, 2024
@ryanpdx ryanpdx removed the critical label Jul 7, 2024
@ryanpdx ryanpdx moved this to Todo in Space Segment Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Backlog
Development

No branches or pull requests

4 participants