AW: AW: [PATCH 12/12] spec/build/cpukit: Add option for enabling PPS synchronization

Sebastian Huber sebastian.huber at embedded-brains.de
Thu Apr 7 13:01:35 UTC 2022


On 07/04/2022 14:51, Gabriel.Moyano at dlr.de wrote:
>>>> In general, the patch set lacks test cases.
>>> I was thinking that the next step could be to add a generic device which is required to use the API (a file descriptor is needed). This is
>> something that wanted to ask in the mailing list first. When this device is added, also the test cases can be. Furthermore I wanted to
>> split the submitted changes in order to make a review easier.
>>
>> A file descriptor should be optional. It should be possible to add a PPS driver just using struct pps_state.
> The struct pps_state type is used only internally. In order to be able to use the API a file descriptor is required (API functions start with time_pps_*()).
> The "generic device" I have in mind will have an object of struct pps_state.

A PPS device driver needs access to the _KERNEL API defined by 
<sys/timepps.h>. It should not be required to use a file descriptor.

The user space time_pps_*() API is an optional component.

-- 
embedded brains GmbH
Herr Sebastian HUBER
Dornierstr. 4
82178 Puchheim
Germany
email: sebastian.huber at 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/


More information about the devel mailing list