On Wed, Mar 21, 2018 at 04:59:47PM +0100, Bastian Stender wrote:
> Use proper ptx/endis and ptx/wwo options, checked and corrected
> configure options using configure_helper.py.
> 
> Some configure options changed over time:
> 
> --enable-ARCRON_MSF is actually --enable-ARCRON-MSF
> 
> --enable-arlib is now --with-arlib
> 
> --enable-udp-wildcard removed in 4848b7bf9 ("Lose UDP_WILDCARD_DELIVERY")
> 
> --enable-MSFEES removed in ba7975983 ("[Bug 340, 342] Deprecate broken TRAK 
> and MSF EES refclocks")
> --enable-TRAK removed in ba7975983 ("[Bug 340, 342] Deprecate broken TRAK and 
> MSF EES refclocks")
> 
> --enable-PTBACTS removed in e0da3f545 ("Remove PTB and USNO lint")
> 
> Signed-off-by: Bastian Stender <b...@pengutronix.de>
> ---
> Changes since (implicit) v1:
> - use ptx/endis, ptx/wwo and ptx/ifdef where appropriate
> - revisit configure options with helper script
> - sorted options according to configure

Thanks. I've pushed it with a followup commit with some more cleanup.
Mostly removing options that where bogus or should not be configurable.
It would be great if you could test this to make sure I didn't break
anything.
Do you have any opinion on the remaining options? I think there are still
too many, but I don't know enough to make a good decision.

Michael

> ---
>  rules/ntp.in          |   6 -
>  rules/ntp.make        | 327 
> +++++++++++++-------------------------------------
>  rules/ntp_nonparse.in |  12 --
>  3 files changed, 82 insertions(+), 263 deletions(-)
> 
> diff --git a/rules/ntp.in b/rules/ntp.in
> index 9f85406ab..73120c961 100644
> --- a/rules/ntp.in
> +++ b/rules/ntp.in
> @@ -103,12 +103,6 @@ config NTP_SIMULATOR
>       help
>         FIXME: This item needs to be documented
>  
> -config NTP_UDP_WILDCARD
> -     bool
> -     prompt "use UDP wildcard delivery"
> -     help
> -       FIXME: This item needs to be documented
> -
>  config NTP_SLEW_ALWAYS
>       bool
>       prompt "always slew the time"
> diff --git a/rules/ntp.make b/rules/ntp.make
> index 67405c47b..efbd0fd9f 100644
> --- a/rules/ntp.make
> +++ b/rules/ntp.make
> @@ -39,255 +39,92 @@ NTP_ENV  := \
>  #
>  # autoconf
>  #
> +
> +# Note: Only if '--disable-all-clocks' is given, the additional clock driver
> +# switches makes sense (else most of the clock drivers are enabled by 
> default)
>  NTP_AUTOCONF := $(CROSS_AUTOCONF_USR) \
> -     $(GLOBAL_IPV6_OPTION) \
>       --bindir=/usr/sbin \
> +     --disable-nls \
>       --without-lineeditlibs \
> -     --without-net-snmp-config \
> +     --$(call ptx/endis, PTXCONF_NTP_DEBUGGING)-debugging \
> +     --enable-thread-support \
> +     --with-yielding-select=yes \
> +     --$(call ptx/endis, PTXCONF_NTP_CLOCKCTL)-clockctl \
>       --disable-linuxcaps \
> -     --with-yielding-select=yes
> -
> -ifdef PTXCONF_NTP_ALL_CLOCK_DRIVERS
> -NTP_AUTOCONF += --enable-all-clocks
> -else
> -NTP_AUTOCONF += --disable-all-clocks
> -endif
> -
> -#
> -# NTP: options, we need lots of options ;-)
> -# Note: Only if '--disable-all-clocks' is given, the additional clock driver
> -# switches makes sense (else most of the clock drivers are enabled
> -# by default)
> -#
> -ifdef PTXCONF_NTP_CLOCKCTL
> -NTP_AUTOCONF += --enable-clockctl
> -endif
> -ifdef PTXCONF_NTP_DEBUGGING
> -NTP_AUTOCONF += --enable-debugging
> -endif
> -ifdef PTXCONF_NTP_DST_MINUTES
> -NTP_AUTOCONF += --enable-dst-minutes=$(PTXCONF_NTP_DST_MINUTES)
> -endif
> -ifdef PTXCONF_NTP_BANCOMM
> -NTP_AUTOCONF += --enable-BANCOMM
> -endif
> -ifdef PTXCONF_NTP_GPSVME
> -NTP_AUTOCONF += --enable-GPSVME
> -endif
> -ifdef PTXCONF_NTP_ACTS
> -NTP_AUTOCONF += --enable-ACTS
> -endif
> -ifdef PTXCONF_NTP_ARBITER
> -NTP_AUTOCONF += --enable-ARBITER
> -endif
> -ifdef PTXCONF_NTP_ARCRON_MSF
> -NTP_AUTOCONF += --enable-ARCRON_MSF
> -endif
> -ifdef PTXCONF_NTP_AS2201
> -NTP_AUTOCONF += --enable-AS2201
> -endif
> -ifdef PTXCONF_NTP_ATOM
> -NTP_AUTOCONF += --enable-ATOM
> -endif
> -ifdef PTXCONF_NTP_CHRONOLOG
> -NTP_AUTOCONF += --enable-CHRONOLOG
> -endif
> -ifdef PTXCONF_NTP_CHU
> -NTP_AUTOCONF += --enable-CHU
> -endif
> -ifdef PTXCONF_NTP_AUDIO_CHU
> -NTP_AUTOCONF += --enable-AUDIO-CHU
> -endif
> -ifdef PTXCONF_NTP_DATUM
> -NTP_AUTOCONF += --enable-DATUM
> -endif
> -ifdef PTXCONF_NTP_DUMBCLOCK
> -NTP_AUTOCONF += --enable-DUMBCLOCK
> -endif
> -ifdef PTXCONF_NTP_FG
> -NTP_AUTOCONF += --enable-FG
> -endif
> -ifdef PTXCONF_NTP_GPSD
> -NTP_AUTOCONF += --enable-GPSD
> -endif
> -ifdef PTXCONF_NTP_HEATH
> -NTP_AUTOCONF += --enable-HEATH
> -endif
> -ifdef PTXCONF_NTP_HOPFSERIAL
> -NTP_AUTOCONF += --enable-HOPFSERIAL
> -endif
> -ifdef PTXCONF_NTP_HOPFPCI
> -NTP_AUTOCONF += --enable-HOPFPCI
> -endif
> -ifdef PTXCONF_NTP_HPGPS
> -NTP_AUTOCONF += --enable-HPGPS
> -endif
> -ifdef PTXCONF_NTP_IRIG
> -NTP_AUTOCONF += --enable-IRIG
> -endif
> -ifdef PTXCONF_NTP_JJY
> -NTP_AUTOCONF += --enable-JJY
> -endif
> -ifdef PTXCONF_NTP_JUPITER
> -NTP_AUTOCONF += --enable-JUPITER
> -endif
> -ifdef PTXCONF_NTP_LEITCH
> -NTP_AUTOCONF += --enable-LEITCH
> -endif
> -ifdef PTXCONF_NTP_LOCAL_CLOCK
> -NTP_AUTOCONF += --enable-LOCAL-CLOCK
> -endif
> -ifdef PTXCONF_NTP_MSFEES
> -NTP_AUTOCONF += --enable-MSFEES
> -endif
> -ifdef PTXCONF_NTP_MX4200
> -NTP_AUTOCONF += --enable-MX4200
> -endif
> -ifdef PTXCONF_NTP_NEOCLOCK4X
> -NTP_AUTOCONF += --enable-NEOCLOCK4X
> -endif
> -ifdef PTXCONF_NTP_NMEA
> -NTP_AUTOCONF += --enable-NMEA
> -endif
> -ifdef PTXCONF_NTP_ONCORE
> -NTP_AUTOCONF += --enable-ONCORE
> -endif
> -ifdef PTXCONF_NTP_PALISADE
> -NTP_AUTOCONF += --enable-PALISADE
> -endif
> -ifdef PTXCONF_NTP_PCF
> -NTP_AUTOCONF += --enable-PCF
> -endif
> -ifdef PTXCONF_NTP_PST
> -NTP_AUTOCONF += --enable-PST
> -endif
> -ifdef PTXCONF_NTP_PTBACTS
> -NTP_AUTOCONF += --enable-PTBACTS
> -endif
> -ifdef PTXCONF_NTP_RIPENCC
> -NTP_AUTOCONF += --enable-RIPENCC
> -endif
> -ifdef PTXCONF_NTP_SEL240X
> -NTP_AUTOCONF += --enable-SEL240X
> -endif
> -ifdef PTXCONF_NTP_SHM
> -NTP_AUTOCONF += --enable-SHM
> -endif
> -ifdef PTXCONF_NTP_SPECTRACOM
> -NTP_AUTOCONF += --enable-SPECTRACOM
> -endif
> -ifdef PTXCONF_NTP_TPRO
> -NTP_AUTOCONF += --enable-TPRO
> -endif
> -ifdef PTXCONF_NTP_TRAK
> -NTP_AUTOCONF += --enable-TRAK
> -endif
> -ifdef PTXCONF_NTP_TRUETIME
> -NTP_AUTOCONF += --enable-TRUETIME
> -endif
> -ifdef PTXCONF_NTP_TSYNCPCI
> -NTP_AUTOCONF += --enable-TSYNCPCI
> -endif
> -ifdef PTXCONF_NTP_TT560
> -NTP_AUTOCONF += --enable-TT560
> -endif
> -ifdef PTXCONF_NTP_ULINK
> -NTP_AUTOCONF += --enable-ULINK
> -endif
> -ifdef PTXCONF_NTP_WWV
> -NTP_AUTOCONF += --enable-WWV
> -endif
> -ifdef PTXCONF_NTP_ZYFER
> -NTP_AUTOCONF += --enable-ZYFER
> -endif
> -ifdef PTXCONF_NTP_COMPUTIME
> -NTP_AUTOCONF += --enable-COMPUTIME
> -endif
> -ifdef PTXCONF_NTP_DCF7000
> -NTP_AUTOCONF += --enable-DCF7000
> -endif
> -ifdef PTXCONF_NTP_HOPF6021
> -NTP_AUTOCONF += --enable-HOPF6021
> -endif
> -ifdef PTXCONF_NTP_MEINBERG
> -NTP_AUTOCONF += --enable-MEINBERG
> -endif
> -ifdef PTXCONF_NTP_RAWDCF
> -NTP_AUTOCONF += --enable-RAWDCF
> -endif
> -ifdef PTXCONF_NTP_RCC8000
> -NTP_AUTOCONF += --enable-RCC8000
> -endif
> -ifdef PTXCONF_NTP_SCHMID
> -NTP_AUTOCONF += --enable-SCHMID
> -endif
> -ifdef PTXCONF_NTP_TRIMTAIP
> -NTP_AUTOCONF += --enable-TRIMTAIP
> -endif
> -ifdef PTXCONF_NTP_TRIMTSIP
> -NTP_AUTOCONF += --enable-TRIMTSIP
> -endif
> -ifdef PTXCONF_NTP_WHARTON
> -NTP_AUTOCONF += --enable-WHARTON
> -endif
> -ifdef PTXCONF_NTP_VARITEXT
> -NTP_AUTOCONF += --enable-VARITEXT
> -endif
> -ifdef PTXCONF_NTP_KMEM
> -NTP_AUTOCONF += --enable-kmem
> -endif
> -ifdef PTXCONF_NTP_ACCURATE_ADJTIME
> -NTP_AUTOCONF += --enable-accurate-adjtime
> -endif
> -ifdef PTXCONF_NTP_TICK_FORCE
> -NTP_AUTOCONF += --enable-tick=$(PTXCONF_NTP_TICK)
> -endif
> -ifdef PTXCONF_NTP_TICKADJ_FORCE
> -NTP_AUTOCONF += --enable-tickadj=$(PTXCONF_NTP_TICKADJ)
> -endif
> -ifdef PTXCONF_NTP_SIMULATOR
> -NTP_AUTOCONF += --enable-simulator
> -endif
> -ifdef PTXCONF_NTP_UDP_WILDCARD
> -NTP_AUTOCONF += --enable-udp-wildcard
> -endif
> -ifdef PTXCONF_NTP_SLEW_ALWAYS
> -NTP_AUTOCONF += --enable-slew-always
> -endif
> -ifdef PTXCONF_NTP_STEP_SLEW
> -NTP_AUTOCONF += --enable-step-slew
> -endif
> -ifdef PTXCONF_NTP_NTPDATE_STEP
> -NTP_AUTOCONF += --enable-ntpdate-step
> -endif
> -ifdef PTXCONF_NTP_HOURLY_TODR_SYNC
> -NTP_AUTOCONF += --enable-hourly-todr-sync
> -endif
> -ifdef PTXCONF_NTP_KERNEL_FLL_BUG
> -NTP_AUTOCONF += --enable-kernel-fll-bug
> -endif
> -ifdef PTXCONF_NTP_IRIG_SAWTOOTH
> -NTP_AUTOCONF += --enable-irig-sawtooth
> -endif
> -ifdef PTXCONF_NTP_NIST
> -NTP_AUTOCONF += --enable-nist
> -endif
> -ifdef PTXCONF_NTP_CRYPTO
> -NTP_AUTOCONF += \
> -     --with-crypto \
> -     --with-openssl-libdir=$(PTXDIST_SYSROOT_TARGET)/usr/lib \
> -     --with-openssl-incdir=$(PTXDIST_SYSROOT_TARGET)/usr/include
> -else
> -NTP_AUTOCONF += --without-crypto \
> -     --without-openssl-libdir \
> -     --without-openssl-incdir
> -endif
> -ifdef PTXCONF_NTP_SNTP
> -NTP_AUTOCONF += --with-sntp
> -endif
> -ifdef PTXCONF_NTP_ARLIB
> -NTP_AUTOCONF += --enable-arlib
> -endif
> +     --$(call ptx/wwo, PTXCONF_NTP_ARLIB)-arlib \
> +     --without-net-snmp-config \
> +     --$(call ptx/endis, PTXCONF_NTP_DST_MINUTES)-dst-minutes$(call 
> ptx/ifdef, PTXCONF_NTP_DST_MINUTES, =$(PTXCONF_NTP_DST_MINUTES),) \
> +     --$(call ptx/endis, PTXCONF_NTP_BANCOMM)-BANCOMM \
> +     --$(call ptx/endis, PTXCONF_NTP_GPSVME)-GPSVME \
> +     --$(call ptx/endis, PTXCONF_NTP_ALL_CLOCK_DRIVERS)-all-clocks \
> +     --$(call ptx/endis, PTXCONF_NTP_ACTS)-ACTS \
> +     --$(call ptx/endis, PTXCONF_NTP_ARBITER)-ARBITER \
> +     --$(call ptx/endis, PTXCONF_NTP_ARCRON_MSF)-ARCRON-MSF \
> +     --$(call ptx/endis, PTXCONF_NTP_AS2201)-AS2201 \
> +     --$(call ptx/endis, PTXCONF_NTP_ATOM)-ATOM \
> +     --$(call ptx/endis, PTXCONF_NTP_CHRONOLOG)-CHRONOLOG \
> +     --$(call ptx/endis, PTXCONF_NTP_CHU)-CHU \
> +     --$(call ptx/endis, PTXCONF_NTP_AUDIO_CHU)-AUDIO-CHU \
> +     --$(call ptx/endis, PTXCONF_NTP_DATUM)-DATUM \
> +     --$(call ptx/endis, PTXCONF_NTP_DUMBCLOCK)-DUMBCLOCK \
> +     --$(call ptx/endis, PTXCONF_NTP_FG)-FG \
> +     --$(call ptx/endis, PTXCONF_NTP_HEATH)-HEATH \
> +     --$(call ptx/endis, PTXCONF_NTP_HOPFSERIAL)-HOPFSERIAL \
> +     --$(call ptx/endis, PTXCONF_NTP_HOPFPCI)-HOPFPCI \
> +     --$(call ptx/endis, PTXCONF_NTP_HPGPS)-HPGPS \
> +     --$(call ptx/endis, PTXCONF_NTP_IRIG)-IRIG \
> +     --$(call ptx/endis, PTXCONF_NTP_JJY)-JJY \
> +     --$(call ptx/endis, PTXCONF_NTP_JUPITER)-JUPITER \
> +     --$(call ptx/endis, PTXCONF_NTP_LEITCH)-LEITCH \
> +     --$(call ptx/endis, PTXCONF_NTP_LOCAL_CLOCK)-LOCAL-CLOCK \
> +     --$(call ptx/endis, PTXCONF_NTP_MX4200)-MX4200 \
> +     --$(call ptx/endis, PTXCONF_NTP_NEOCLOCK4X)-NEOCLOCK4X \
> +     --$(call ptx/endis, PTXCONF_NTP_NMEA)-NMEA \
> +     --$(call ptx/endis, PTXCONF_NTP_GPSD)-GPSD \
> +     --$(call ptx/endis, PTXCONF_NTP_ONCORE)-ONCORE \
> +     --$(call ptx/endis, PTXCONF_NTP_PALISADE)-PALISADE \
> +     --$(call ptx/endis, PTXCONF_NTP_PCF)-PCF \
> +     --$(call ptx/endis, PTXCONF_NTP_PST)-PST \
> +     --$(call ptx/endis, PTXCONF_NTP_RIPENCC)-RIPENCC \
> +     --$(call ptx/endis, PTXCONF_NTP_SHM)-SHM \
> +     --$(call ptx/endis, PTXCONF_NTP_SPECTRACOM)-SPECTRACOM \
> +     --$(call ptx/endis, PTXCONF_NTP_TPRO)-TPRO \
> +     --$(call ptx/endis, PTXCONF_NTP_TRUETIME)-TRUETIME \
> +     --$(call ptx/endis, PTXCONF_NTP_TT560)-TT560 \
> +     --$(call ptx/endis, PTXCONF_NTP_ULINK)-ULINK \
> +     --$(call ptx/endis, PTXCONF_NTP_TSYNCPCI)-TSYNCPCI \
> +     --$(call ptx/endis, PTXCONF_NTP_WWV)-WWV \
> +     --$(call ptx/endis, PTXCONF_NTP_ZYFER)-ZYFER \
> +     --$(call ptx/endis, PTXCONF_NTP_COMPUTIME)-COMPUTIME \
> +     --$(call ptx/endis, PTXCONF_NTP_DCF7000)-DCF7000 \
> +     --$(call ptx/endis, PTXCONF_NTP_HOPF6021)-HOPF6021 \
> +     --$(call ptx/endis, PTXCONF_NTP_MEINBERG)-MEINBERG \
> +     --$(call ptx/endis, PTXCONF_NTP_RAWDCF)-RAWDCF \
> +     --$(call ptx/endis, PTXCONF_NTP_RCC8000)-RCC8000 \
> +     --$(call ptx/endis, PTXCONF_NTP_SCHMID)-SCHMID \
> +     --$(call ptx/endis, PTXCONF_NTP_TRIMTAIP)-TRIMTAIP \
> +     --$(call ptx/endis, PTXCONF_NTP_TRIMTSIP)-TRIMTSIP \
> +     --$(call ptx/endis, PTXCONF_NTP_WHARTON)-WHARTON \
> +     --$(call ptx/endis, PTXCONF_NTP_VARITEXT)-VARITEXT \
> +     --$(call ptx/endis, PTXCONF_NTP_SEL240X)-SEL240X \
> +     --$(call ptx/wwo, PTXCONF_NTP_CRYPTO)-crypto \
> +     --$(call ptx/wwo, PTXCONF_NTP_CRYPTO)-openssl-libdir$(call ptx/ifdef, 
> PTXCONF_NTP_CRYPTO, =$(PTXDIST_SYSROOT_TARGET)/usr/lib,) \
> +     --$(call ptx/wwo, PTXCONF_NTP_CRYPTO)-openssl-incdir$(call ptx/ifdef, 
> PTXCONF_NTP_CRYPTO, =$(PTXDIST_SYSROOT_TARGET)/usr/include,) \
> +     --without-rpath \
> +     --$(call ptx/endis, PTXCONF_NTP_KMEM)-kmem \
> +     --$(call ptx/endis, PTXCONF_NTP_ACCURATE_ADJTIME)-accurate-adjtime \
> +     --$(call ptx/endis, PTXCONF_NTP_TICK_FORCE)-tick$(call ptx/ifdef, 
> PTXCONF_NTP_TICK_FORCE, =$(PTXCONF_NTP_TICK),) \
> +     --$(call ptx/endis, PTXCONF_NTP_TICKADJ_FORCE)-tickadj$(call ptx/ifdef, 
> PTXCONF_NTP_TICKADJ_FORCE, =$(PTXCONF_NTP_TICKADJ),) \
> +     --$(call ptx/endis, PTXCONF_NTP_SIMULATOR)-simulator \
> +     --$(call ptx/wwo, PTXCONF_NTP_SNTP)-sntp \
> +     --$(call ptx/endis, PTXCONF_NTP_SLEW_ALWAYS)-slew-always \
> +     --$(call ptx/endis, PTXCONF_NTP_STEP_SLEW)-step-slew \
> +     --$(call ptx/endis, PTXCONF_NTP_NTPDATE_STEP)-ntpdate-step \
> +     --$(call ptx/endis, PTXCONF_NTP_HOURLY_TODR_SYNC)-hourly-todr-sync \
> +     --$(call ptx/endis, PTXCONF_NTP_KERNEL_FLL_BUG)-kernel-fll-bug \
> +     --$(call ptx/endis, PTXCONF_NTP_IRIG_SAWTOOTH)-irig-sawtooth \
> +     --$(call ptx/endis, PTXCONF_NTP_NIST)-nist \
> +     $(GLOBAL_IPV6_OPTION)
>  
>  # 
> ----------------------------------------------------------------------------
>  # Target-Install
> diff --git a/rules/ntp_nonparse.in b/rules/ntp_nonparse.in
> index 397b6e00a..13c38d887 100644
> --- a/rules/ntp_nonparse.in
> +++ b/rules/ntp_nonparse.in
> @@ -98,10 +98,6 @@ config NTP_LOCAL_CLOCK
>         Enables the driver for the internal PC CMOS clock. This clock
>         is mostly unreliable, but suitable as a fall back time source.
>  
> -config NTP_MSFEES
> -     bool
> -     prompt "EES M201 MSF receiver"
> -
>  config NTP_MX4200
>       bool
>       prompt "Magnavox MX4200 GPS receiver "
> @@ -130,10 +126,6 @@ config NTP_PST
>       bool
>       prompt "PST/Traconex 1020 WWV/WWVH receiver"
>  
> -config NTP_PTBACTS
> -     bool
> -     prompt "PTB modem service"
> -
>  config NTP_RIPENCC
>       bool
>       prompt "RIPENCC specific Trimble driver"
> @@ -154,10 +146,6 @@ config NTP_TPRO
>       prompt "KSI/Odetics TPRO/S GPS receiver/IRIG interface [BROKEN]"
>       depends on BROKEN
>  
> -config NTP_TRAK
> -     bool
> -     prompt "TRAK 8810 GPS receiver"
> -
>  config NTP_TRUETIME
>       bool
>       prompt "Kinemetrics/TrueTime receivers"
> -- 
> 2.16.1
> 
> 
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

Reply via email to