On 08/11/2018 05:36, Chris Johns wrote:
On 08/11/2018 00:14, Sebastian Huber wrote:
the <rtems.h> header file still exposes a lot of implementation details via the
definition of the *_Control structures of the API objects. They are only
necessary for the application configuration. I would like to move them to
separate header files. Currently we have:

<rtems/rtems/XYZ.h>
<rtems/rtems/XYZimpl.h>

I need a new name for this header file. For example:

<rtems/rtems/XYZdata.h>

An example patch is attached. Comments?

  cpukit/include/rtems/confdefs.h          |   1 +
  cpukit/include/rtems/rtems/ratemon.h     | 111 +---------------------------
  cpukit/include/rtems/rtems/ratemondata.h | 123 +++++++++++++++++++++++++++++++
This is the ratemon "control struct" so does `ratemonctrl.h` work?

It contains also Rate_monotonic_Statistics, but most of the time it is just XYZ_Control.


Which ever way you go please make sure it will work for all the cases you wish
to change, for example these do not work `foodatadata.h`, or `fooctrlctrl.h`.

It would be:

rtems/posix/mqueuectrl.h
rtems/posix/psignalctrl.h
rtems/posix/semaphorectrl.h
rtems/posix/shmctrl.h
rtems/posix/timerctrl.h
rtems/rtems/asrctrl.h
rtems/rtems/attrctrl.h
rtems/rtems/barrierctrl.h
rtems/rtems/dpmemctrl.h
rtems/rtems/eventctrl.h
rtems/rtems/messagectrl.h
rtems/rtems/modesctrl.h
rtems/rtems/optionsctrl.h
rtems/rtems/partctrl.h
rtems/rtems/ratemonctrl.h
rtems/rtems/regionctrl.h
rtems/rtems/semctrl.h
rtems/rtems/signalctrl.h
rtems/rtems/statusctrl.h
rtems/rtems/tasksctrl.h
rtems/rtems/timerctrl.h

--
Sebastian Huber, embedded brains GmbH

Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone   : +49 89 189 47 41-16
Fax     : +49 89 189 47 41-09
E-Mail  : sebastian.hu...@embedded-brains.de
PGP     : Public key available on request.

Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.

_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel

Reply via email to