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

Controlling memory location of od data #256

Open
wants to merge 2 commits into
base: xdd
Choose a base branch
from
Open

Controlling memory location of od data #256

wants to merge 2 commits into from

Conversation

cfr-mir
Copy link
Contributor

@cfr-mir cfr-mir commented May 28, 2021

Generate placeholders for storage attributes on OD data structures. make the "ROM" option matter.

CANopenNode added a commit to CANopenNode/CANopenEditor that referenced this pull request Jun 22, 2021
@CANopenNode
Copy link
Contributor

libedssharp has been forked to https://github.com/CANopenNode/CANopenEditor, I'm updating there. (In agreement with Robin Cornelius, updates will continue there, he is there also a maintainer.)

So I added this PR in https://github.com/CANopenNode/CANopenEditor, branch main. I also renamed attribute and added default definition, like so:

#ifndef OD_ATTR_PERSIST_COMM
#define OD_ATTR_PERSIST_COMM
#endif
extern OD_ATTR_PERSIST_COMM OD_PERSIST_COMM_t OD_PERSIST_COMM;
...

@CANopenNode
Copy link
Contributor

This PR is not relevant any more, because it is included in https://github.com/CANopenNode/CANopenEditor.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants