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

drivers/sys : Initial import of high-level driver for reader/writers #752

Closed
wants to merge 1 commit into from

Conversation

miri64
Copy link
Member

@miri64 miri64 commented Feb 21, 2014

After some discussion with @haukepetersen a few weeks ago (and @thomaseichinger and @OlegHahm yesterday), I was thinking about a unified interface for radio chips, files, sockets, network stacks, serial interfaces, etc. a lot and now I finally got some time to formulate my idea (in the spirit of Hauke's #612, #613, #614, #615, and #616). It could completely replace the transceiver module, the (not merged from #460 yet) net_if module, and would greatly simplify POSIX wrappers, since this basically equalizes PIDs and file handlers.
This does not make #460 obsolete for now. In fact: #460 would greatly smooth the transition from the current state to the state of this proposal.

@miri64
Copy link
Member Author

miri64 commented Apr 1, 2014

Any opinions?

@LudwigKnuepfer
Copy link
Member

#460 has been merged, care to update the description?

@LudwigKnuepfer
Copy link
Member

Maybe you can also add a usage example to the description?

@Kijewski
Copy link
Contributor

Kijewski commented Apr 1, 2014

Looks incredibly complicated. The static inline functions should be proper functions in a .c file IMO.

@LudwigKnuepfer
Copy link
Member

We have errno?

@miri64
Copy link
Member Author

miri64 commented Apr 2, 2014

If the clib supplies it, yes. (And in my opinion we should use it more)

@LudwigKnuepfer
Copy link
Member

Is it thread safe?

@LudwigKnuepfer
Copy link
Member

Anyways - if not all stdlibs do support it we should not use it as part of an interface outside sys.

@miri64
Copy link
Member Author

miri64 commented May 15, 2014

Is there any strong opinion against closing (not merging) this PR?

@OlegHahm
Copy link
Member

For which reason?

@miri64
Copy link
Member Author

miri64 commented May 16, 2014

I've got the feeling, that it is not really wished for :-)

@OlegHahm
Copy link
Member

Let's discuss at the next after-release developer meeting.

@OlegHahm OlegHahm added this to the FIX ME FIRST milestone Jun 3, 2014
@miri64
Copy link
Member Author

miri64 commented Jul 29, 2014

Closed in favor of #1448

@miri64 miri64 closed this Jul 29, 2014
@miri64 miri64 deleted the stream-handler-driver branch July 29, 2014 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation Type: question The issue poses a question regarding usage of RIOT
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants