[Buildroot] [PATCH] package/linuxptp: make initscript options configurable

Heiko Thiery heiko.thiery at gmail.com
Fri Mar 6 10:27:17 UTC 2020


Hi,

Am Fr., 6. März 2020 um 11:19 Uhr schrieb Michael Walle <michael at walle.cc>:
>
> Let the user supply its own options in /etc/default/linuxptp.
> Additionally, let the user decide if he wants to start php2sys.
>
> Signed-off-by: Michael Walle <michael at walle.cc>
> ---
>
> It might be better to have an own php2sys initscript. I'm not sure.
> Thoughts?

I think 2 (ptp4l and phc2sys) scripts like done e.g. in debian would be better.

>
>  package/linuxptp/S65linuxptp | 30 +++++++++++++++++++++---------
>  1 file changed, 21 insertions(+), 9 deletions(-)
>
> diff --git a/package/linuxptp/S65linuxptp b/package/linuxptp/S65linuxptp
> index 46b8921fdd..5f296a4651 100644
> --- a/package/linuxptp/S65linuxptp
> +++ b/package/linuxptp/S65linuxptp
> @@ -3,23 +3,35 @@
>  # Start linuxptp
>  #
>
> +# defaults
> +PTP4L_OPTS="-f /etc/linuxptp.cfg"
> +PHC2SYS_ENABLE="true"
> +PHC2SYS_OPTS="-s eth0 -c CLOCK_REALTIME -w -S 1.0"
> +
> +# Read configuration variable file if it is present
> +[ -r /etc/default/linuxptp ] && . /etc/default/linuxptp
> +
>  start() {
>         printf "Starting linuxptp daemon: "
>         start-stop-daemon -S -b -q -p /var/run/linuxptp-ptp4l.pid \
> -               -x /usr/sbin/ptp4l -- -f /etc/linuxptp.cfg
> +               -x /usr/sbin/ptp4l -- ${PTP4L_OPTS}
>         [ $? = 0 ] && echo "OK" || echo "FAIL"
>
> -       printf "Starting linuxptp system clock synchronization: "
> -       start-stop-daemon -S -b -q -p /var/run/linuxptp-phc2sys.pid \
> -               -x /usr/sbin/phc2sys -- -s eth0 -c CLOCK_REALTIME -w -S 1.0
> -       [ $? = 0 ] && echo "OK" || echo "FAIL"
> +       if [ "${PHC2SYS_ENABLE}" = "true" ]; then
> +               printf "Starting linuxptp system clock synchronization: "
> +               start-stop-daemon -S -b -q -p /var/run/linuxptp-phc2sys.pid \
> +                       -x /usr/sbin/phc2sys -- ${PHC2SYS_OPTS}
> +               [ $? = 0 ] && echo "OK" || echo "FAIL"
> +       fi
>  }
>
>  stop() {
> -       printf "Stopping linuxptp system clock synchronization: "
> -       start-stop-daemon -K -q -p /var/run/linuxptp-phc2sys.pid \
> -               -x /usr/sbin/phc2sys
> -       echo "OK"
> +       if [ "${PHC2SYS_ENABLE}" = "true" ]; then
> +               printf "Stopping linuxptp system clock synchronization: "
> +               start-stop-daemon -K -q -p /var/run/linuxptp-phc2sys.pid \
> +                       -x /usr/sbin/phc2sys
> +               echo "OK"
> +       fi
>
>         printf "Stopping linuxptp daemon: "
>         start-stop-daemon -K -q -p /var/run/linuxptp-ptp4l.pid \
> --
> 2.20.1
>

Heiko


More information about the buildroot mailing list