trem wrote: > Hi > > I've written a very simple example for rtdm. And I would be pleased to > get a feedback about it. You can found it here : > > http://zarb.org/users/svn/trem/trem/kernel/xenomai/rtdm/ >
Looks good and (almost) clean. A few remarks and some suggestion: o Use kernel coding style for the driver (linux/scripts/Lindent + manual tweaking) o Put your driver under RTDM_CLASS_EXPERIMENTAL, TESTING is for benchmarks and regression test drivers. And you don't need rttesting.h. o As RTDM is mainly about "RT", why not switching to .read_rt and .write_rt? That means of course creating a shadow thread in read_simple_rtdm. o Once you are real-time with read/write, you could demonstrate synchronisation. One FAQ is "How do I wait with my application on something inside a driver?" You could do rtdm_sem_up on write and rtdm_sem_[timed]down on read. Alternatively: o What about creating a small series of RTDM examples, consisting of two demos so far: the first one a reduced version of you current code, just showing device registration and handler invocation, the second one about read/write and synchronisation. And if you feel like doing more, one could certainly continue this series with further aspects... Are you interested that we merge your example(s) into the Xenomai repos (examples/rtdm/driver-api)? I would welcome this and support your effort! Jan
signature.asc
Description: OpenPGP digital signature
_______________________________________________ Xenomai-help mailing list [email protected] https://mail.gna.org/listinfo/xenomai-help
