Re: [PATCH v12 2/3] Documentation: driver-api: pps: Add Intel Timed I/O PPS generator

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 25/09/24 23:55, Hall, Christopher S wrote:
Hi Rodolfo,

Hello.

From: Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx>
Sent: Tuesday, September 03, 2024 3:25 AM
To: Rodolfo Giometti <giometti@xxxxxxxxxxxx>

Subject: Re: [PATCH v12 2/3] Documentation: driver-api: pps: Add Intel Timed
I/O PPS generator


If you are willing to stop the inclusion due this fact maybe its time to add
such PPS generators interface... on the other hand, if you agree for
inclusion we can do this job as soon as the code has been included, in order
to fix this anomalous status.

Please make a generic pps subsystem for this, it would make it simpler
for everyone.

What is required to move this? We can certainly test the code and do some
of this work, but I would look to you to define the interface.

The problem is that a pps-generator has no related device, then no sysfs entries.

I think the right-thing(TM) to do is adding a new class named "pps-generator", so we will get the directory /sys/class/pps-generator with several devices as pps-generator0, pps-generator1, etc. For each device we should add at least these sysfs entries:

- system       : This file return "1" if the generator takes the timing from
                 the system clock, while it returns "0" if not (i.e. from a
                 peripheral device clock).

- time         : This file contains the current time stored into the generator
                 clock as two integers representing the current time seconds and
                 nanoseconds.

- enable       : This write-only file enables or disables generation of the
                 PPS signal.

- period       : This file defines the period for the generator signal. When
                 read, by default, it returns "1 0" which represent the period
                 second and nanoseconds (i.e. a PPS signal). When written, it
                 sets the period accordingly or ENOTSUPP if not supported.

- start_time   : This file defines the starting time for the the generator
                 signal. When read, by default, it returns "0 0" which means
                 "now". When written, it sets the starting time accordingly or
                 ENOTSUPP if not supported.

Please, let me know what do you think about this proposal. I'm going to provide a draft in the next weeks since I'm stuck on another project right now. :(

Ciao,

Rodolfo

--
GNU/Linux Solutions                  e-mail: giometti@xxxxxxxxxxxx
Linux Device Driver                          giometti@xxxxxxxx
Embedded Systems                     phone:  +39 349 2432127
UNIX programming





[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux