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

pkg/libcsp: initial commit #18009

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

fjmolinas
Copy link
Contributor

Contribution description

This PR adds libCSP as a package. This network stack was designed for embedded systems in smaller networks, such as Cubesats. More details on it can be found on its GitHub.

This work was started by @millotp while working with @thingsat (CubeSat project with a RIOT-based payload in space). I'm adding this as a PR since it seems this network stack is used in CubeSat projects, so might be of interest for others.

Some things that could be improved:

  • use of conn_can, although it makes for easy code leads IMO for duplicate stack usage, I think candev should probably be directly used.

Testing procedure

A test is provided with a README, an automatic test is included as well.

@github-actions github-actions bot added Area: build system Area: Build system Area: doc Area: Documentation Area: pkg Area: External package ports Area: sys Area: System Area: tests Area: tests and testing framework labels Apr 26, 2022
@fjmolinas
Copy link
Contributor Author

@donsez might be interested.

@@ -23,11 +23,11 @@
#define CANDEV_NUMOF (ARRAY_SIZE(candev_params))

#ifndef CANDEV_STACKSIZE
#define CANDEV_STACKSIZE (THREAD_STACKSIZE_DEFAULT + THREAD_EXTRA_STACKSIZE_PRINTF)
#define CANDEV_STACKSIZE (THREAD_STACKSIZE_LARGE + THREAD_EXTRA_STACKSIZE_PRINTF)
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is this needed?

@stale
Copy link

stale bot commented Nov 2, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If you want me to ignore this issue, please mark it with the "State: don't stale" label. Thank you for your contributions.

@stale stale bot added the State: stale State: The issue / PR has no activity for >185 days label Nov 2, 2022
@Teufelchen1
Copy link
Contributor

How can we move this forward? @fjmolinas still interested?

@stale stale bot removed the State: stale State: The issue / PR has no activity for >185 days label Mar 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: build system Area: Build system Area: doc Area: Documentation Area: pkg Area: External package ports Area: sys Area: System Area: tests Area: tests and testing framework
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants