On 07/04/2022 16:05, gabriel.moy...@dlr.de wrote:
On 07/04/2022 14:51, gabriel.moy...@dlr.de wrote:
On 07/04/2022 11:56, gabriel.moy...@dlr.de wrote:
On 07/04/2022 10:36, Gabriel Moyano wrote:
---
cpukit/score/src/kern_ntptime.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/cpukit/score/src/kern_ntptime.c
b/cpukit/score/src/kern_ntptime.c index d4a233f67e..d6ea739f5b
100644
--- a/cpukit/score/src/kern_ntptime.c
+++ b/cpukit/score/src/kern_ntptime.c
@@ -58,6 +58,9 @@ __FBSDID("$FreeBSD$");
#include <sys/time.h>
#include <sys/timex.h>
#include <sys/timetc.h>
+#ifdef __rtems__
+#define _KERNEL
+#endif /* __rtems__ */
#include <sys/timepps.h>
#ifndef __rtems__
#include <sys/syscallsubr.h>
Who is supposed to use the _KERNEL <sys/timepps.h.> API?
hardpps()
The pps_*() API should be accessible for RTEMS drivers without having to define
_KERNEL.
The API functions start with time_pps_*().
This is the user space API. It uses a file descriptor.
PPS device drivers should use the kernel space API. Since we don't have a user
and kernel space in RTEMS, the kerne space API should
be available also if __rtems__ is defined.
So, you mean to remove the #ifdef _KERNEL using #ifndef__rtems__.
It is an
#ifdef _KERNEL
...
#else /* !_KERNEL */
...
#endif /* KERNEL */
In RTEMS we need both APIs, one for PPS drivers and the other for
applications.
If we don't want to change the header, then PPS drivers have to do
something like this:
#define _KERNEL
#include <sys/timepps.h>
There should be a new chapter in
https://docs.rtems.org/branches/master/bsp-howto/index.html
for the PPS drivers.
hardpps() is declared if _KERNEL is defined. That's why I added it
before including timepps.h in kern_ntptime.c It was for removing a warning.
The hardpps() needs to be added to the _ namespace, for example
_NTP_Pulse_per_second_event().
Here I didn't follow you
The FreeBSD kernel has its own symbol namespace which is separated from the
application. In RTEMS, the kernel and application
share a symbol namespace. Kernel functions should use the _ namespace.
And here you mean to do something similar to what have to be done for
tc_getfrequency() ( declare _Timecounter_Get_frequency())?
Yes, maybe add the declaration and documentation to
<rtems/score/timecounter.h> and name the function
_Timecouter_Process_pulse_per_second_event().
--
embedded brains GmbH
Herr Sebastian HUBER
Dornierstr. 4
82178 Puchheim
Germany
email: sebastian.hu...@embedded-brains.de
phone: +49-89-18 94 741 - 16
fax: +49-89-18 94 741 - 08
Registergericht: Amtsgericht München
Registernummer: HRB 157899
Vertretungsberechtigte Geschäftsführer: Peter Rasmussen, Thomas Dörfler
Unsere Datenschutzerklärung finden Sie hier:
https://embedded-brains.de/datenschutzerklaerung/
_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel
