mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Michael Grzeschik <m.grzeschik@pengutronix.de>
To: ptxdist@pengutronix.de
Subject: [ptxdist] [PATCH v4] linuxptp: Update package and configs
Date: Wed, 29 Nov 2017 12:24:28 +0100	[thread overview]
Message-ID: <20171129112428.3404-1-m.grzeschik@pengutronix.de> (raw)

Usualy we can not ensure only to use transparent clock switches.
Therefor we stick to the default E2E request-response mechanism.

The following options are highly depending on the usecase
so we remove them for the defaults.

path_trace_enabled
follow_up_info
tx_timestamp_timeout

With all other options being simple defaults the whole
config can be removed.

We also add some service files to start the ptp4l service on
dev eth0. In case the ptp4l daemon is configured
we add the option to install the phc2sys daemon to adjust the
system time by the ptp master.

Signed-off-by: Michael Grzeschik <m.grzeschik@pengutronix.de>
---
v1 -> v2: - removed install stage for gPTP.conf from rules file
          - limited to run ptp4l only on eth0 for the default setup case

v2 -> v3: - removed second -r option in phc2sys rule
          - changed systemd rule for ptp4l to static configuration

v3 -> v4: - fixed missing "." in link to service file of ptp4l
          - changed to default E2E, since P2P got mixed up to be the
            mechanism not needing transparent switches.

 projectroot/etc/gPTP.conf                          | 28 ----------------------
 projectroot/usr/lib/systemd/system/phc2sys.service | 10 ++++++++
 projectroot/usr/lib/systemd/system/ptp4l.service   | 12 ++++++++++
 rules/linuxptp.in                                  | 12 ++++++++++
 rules/linuxptp.make                                | 20 +++++++++++++---
 5 files changed, 51 insertions(+), 31 deletions(-)
 delete mode 100644 projectroot/etc/gPTP.conf
 create mode 100644 projectroot/usr/lib/systemd/system/phc2sys.service
 create mode 100644 projectroot/usr/lib/systemd/system/ptp4l.service

diff --git a/projectroot/etc/gPTP.conf b/projectroot/etc/gPTP.conf
deleted file mode 100644
index 5bbbaf288..000000000
--- a/projectroot/etc/gPTP.conf
+++ /dev/null
@@ -1,28 +0,0 @@
-[global]
-#
-# Run time options
-#
-assume_two_step         1
-logging_level           6
-path_trace_enabled      1
-follow_up_info          1
-tx_timestamp_timeout    40
-use_syslog              1
-verbose                 0
-summary_interval        0
-#
-# Transport options
-#
-transportSpecific       0x1
-ptp_dst_mac             01:80:C2:00:00:0E
-p2p_dst_mac             01:80:C2:00:00:0E
-uds_address             /var/run/ptp4l
-#
-# Default interface options
-#
-network_transport       L2
-delay_mechanism         P2P
-time_stamping           hardware
-
-[eth0]
-
diff --git a/projectroot/usr/lib/systemd/system/phc2sys.service b/projectroot/usr/lib/systemd/system/phc2sys.service
new file mode 100644
index 000000000..992a95484
--- /dev/null
+++ b/projectroot/usr/lib/systemd/system/phc2sys.service
@@ -0,0 +1,10 @@
+[Unit]
+Description=Precision Time Protocol system clock synchronization
+Wants=ptp4l@.service
+
+[Service]
+ExecStart=/usr/sbin/phc2sys -a -r
+Restart=always
+
+[Install]
+WantedBy=multi-user.target
diff --git a/projectroot/usr/lib/systemd/system/ptp4l.service b/projectroot/usr/lib/systemd/system/ptp4l.service
new file mode 100644
index 000000000..93cc89519
--- /dev/null
+++ b/projectroot/usr/lib/systemd/system/ptp4l.service
@@ -0,0 +1,12 @@
+[Unit]
+Description=Precision Time Protocol (PTP) service
+BindsTo = sys-subsystem-net-devices-eth0.device
+After = sys-subsystem-net-devices-eth0.device
+
+[Service]
+Type=simple
+ExecStart=/usr/sbin/ptp4l -i eth0
+Restart=always
+
+[Install]
+WantedBy=multi-user.target
diff --git a/rules/linuxptp.in b/rules/linuxptp.in
index 641d844c7..5418d2ad0 100644
--- a/rules/linuxptp.in
+++ b/rules/linuxptp.in
@@ -7,3 +7,15 @@ config LINUXPTP
 	  The Linux PTP project implements the Precision Time protocol (PTP),
 	  standardized as IEEE1588.
 
+if LINUXPTP
+
+config LINUXPTP_SYSTEMD_SERVICE
+	bool "install ptp4l and phc2sys systemd service files"
+	default y
+
+config LINUXPTP_PHC2SYS_SYSTEMD_SERVICE
+	bool "Enable phc2sys systemd service file"
+	default y
+	depends on LINUXPTP_SYSTEMD_SERVICE
+
+endif
diff --git a/rules/linuxptp.make b/rules/linuxptp.make
index 59cbb5123..d74be4250 100644
--- a/rules/linuxptp.make
+++ b/rules/linuxptp.make
@@ -17,8 +17,8 @@ PACKAGES-$(PTXCONF_LINUXPTP) += linuxptp
 #
 # Paths and names
 #
-LINUXPTP_VERSION	:= 1.6
-LINUXPTP_MD5		:= 6aa15d83f5a35f1fd076ba9adc4e7285
+LINUXPTP_VERSION	:= 1.8
+LINUXPTP_MD5		:= 5688cdfe57932273e1dbf35b3b97b9a0
 LINUXPTP		:= linuxptp-$(LINUXPTP_VERSION)
 LINUXPTP_SUFFIX		:= tgz
 LINUXPTP_URL		:= $(call ptx/mirror, SF, linuxptp/$(LINUXPTP).$(LINUXPTP_SUFFIX))
@@ -64,7 +64,21 @@ $(STATEDIR)/linuxptp.targetinstall:
 	@$(call install_copy, linuxptp, 0, 0, 0755, -, /usr/sbin/phc2sys)
 	@$(call install_copy, linuxptp, 0, 0, 0755, -, /usr/sbin/pmc)
 	@$(call install_copy, linuxptp, 0, 0, 0755, -, /usr/sbin/hwstamp_ctl)
-	@$(call install_alternative, linuxptp, 0, 0, 0644, /etc/gPTP.conf)
+	@$(call install_copy, linuxptp, 0, 0, 0755, -, /usr/sbin/phc_ctl)
+	@$(call install_copy, linuxptp, 0, 0, 0755, -, /usr/sbin/timemaster)
+
+ifdef PTXCONF_LINUXPTP_SYSTEMD_SERVICE
+	@$(call install_alternative, linuxptp, 0, 0, 0644, /usr/lib/systemd/system/ptp4l.service)
+	@$(call install_alternative, linuxptp, 0, 0, 0644, /usr/lib/systemd/system/phc2sys.service)
+
+	@$(call install_link, linuxptp, ../ptp4l.service, \
+		/usr/lib/systemd/system/multi-user.target.wants/ptp4l.service)
+
+ifdef PTXCONF_LINUXPTP_PHC2SYS_SYSTEMD_SERVICE
+	@$(call install_link, linuxptp, ../phc2sys.service, \
+		/usr/lib/systemd/system/multi-user.target.wants/phc2sys.service)
+endif
+endif
 
 	@$(call install_finish, linuxptp)
 
-- 
2.11.0


_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

                 reply	other threads:[~2017-11-29 11:24 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20171129112428.3404-1-m.grzeschik@pengutronix.de \
    --to=m.grzeschik@pengutronix.de \
    --cc=ptxdist@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox