Re: [Xenomai-help] very simple example for rtdm

[Jan Kiszka <jan.kiszka@domain.hid>]

[-- Attachment #1: Type: text/plain, Size: 1474 bytes --]

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


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 250 bytes --]