-
Notifications
You must be signed in to change notification settings - Fork 330
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
Sync with latest iio events #1268
Conversation
For traceability - we normally include the reference to the kernel git that includes the required change... your preference. |
and I think you are missing a update to |
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/include/uapi/linux/iio/types.h?id=34934d79965518548d33c0513a9572ae936d8c29 Signed-off-by: Dan Nechita <dan.nechita@analog.com>
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/include/uapi/linux/iio/types.h?id=34934d79965518548d33c0513a9572ae936d8c29 Signed-off-by: Dan Nechita <dan.nechita@analog.com>
7afeab9
to
d599e17
Compare
It looks that way, yes. I wasn't aware of it. Thanks! |
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/include/uapi/linux/iio/types.h?id=34934d79965518548d33c0513a9572ae936d8c29 Signed-off-by: Dan Nechita <dan.nechita@analog.com>
Sorry - what is #1269 and this one? (Looks like they are dups?) |
This PR is targeting libiio v1 while PR 1269 is targeting libiio v0. |
should those strings go into iio.h, or somewhere else (inside event.c - similar to channel.c - so it can be passed to an app in a libiio function) so applications don't need to maintain them themselves? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks - looks great.
PR Description
Update C, C#, Python to include the new:
PR Type
PR Checklist