Re: [PATCH 1/1] LinuxPPS core support.

[Andrew Morton]

On Fri, 22 May 2009 08:12:33 +0200
Rodolfo Giometti <giometti@xxxxxxxx> wrote:

> This patch adds the kernel side of the PPS support currently named
> "LinuxPPS".
> 
> PPS means "pulse per second" and a PPS source is just a device which
> provides a high precision signal each second so that an application
> can use it to adjust system clock time.
> 
> Common use is the combination of the NTPD as userland program with a
> GPS receiver as PPS source to obtain a wallclock-time with
> sub-millisecond synchronisation to UTC.
> 
> To obtain this goal the userland programs shoud use the PPS API
> specification (RFC 2783 - Pulse-Per-Second API for UNIX-like Operating
> Systems, Version 1.0) which in part is implemented by this patch. It
> provides a set of chars devices, one per PPS source, which can be used
> to get the time signal. The RFC's functions can be implemented by
> accessing to these char devices.
>
> ...
>
>  Documentation/ABI/testing/sysfs-pps  |   73 ++++++++
>  Documentation/ioctl/ioctl-number.txt |    2 +
>  Documentation/pps/pps.txt            |  172 ++++++++++++++++++
>  MAINTAINERS                          |    7 +
>  drivers/Kconfig                      |    2 +
>  drivers/Makefile                     |    1 +
>  drivers/pps/Kconfig                  |   33 ++++
>  drivers/pps/Makefile                 |    8 +
>  drivers/pps/kapi.c                   |  329 ++++++++++++++++++++++++++++++++++
>  drivers/pps/pps.c                    |  312 ++++++++++++++++++++++++++++++++
>  drivers/pps/sysfs.c                  |   98 ++++++++++
>  include/linux/Kbuild                 |    1 +
>  include/linux/pps.h                  |  122 +++++++++++++
>  include/linux/pps_kernel.h           |   89 +++++++++

Well it seems that Christoph's header file suggestion was addressed.

It's been a long time and there have been many reviews and comments. 
Have you been keeping track of them all?

If so, can you please let us know which review comments remain
unaddressed, or were deliberately not addressed because you disagreed
with them?

Thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/