Skip to content

feat: Port missing IsoTpBehaviour flags #13

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

feat: Port missing IsoTpBehaviour flags #13

wants to merge 1 commit into from

Conversation

skullim
Copy link

@skullim skullim commented Apr 29, 2025 •
edited
Loading

Extends https://github.com/marcelbuesing/socketcan-isotp/pull/11/commits by adding additional flags.

  • Added support for setting functional addressing by exposing the necessary ISO-TP socket flags.
  • Extended the API to allow enabling individual flags without overwriting existing ones.
  • Performed small refactorings to decouple socket construction steps. The initial goal was to define a public API for opening broadcast sockets, but that turned out unnecessary. These changes can be reverted if preferred.

@skullim skullim force-pushed the feature/add-broadcast-config branch 4 times, most recently from 1754bcd to e85f410 Compare April 29, 2025 12:08
@skullim skullim force-pushed the feature/add-broadcast-config branch from e85f410 to 0510642 Compare April 29, 2025 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@skullim