Re: [Linuxptp-devel] [PATCH] Add E2E automotive profile example configs.

2022-03-07 Thread Richard Cochran
On Mon, Mar 07, 2022 at 05:14:52PM -0800, Hal Murray wrote:
> 
> richardcoch...@gmail.com said:
> > I just mean any kind of published standard or profile, like from the IEEE,
> > ITU, AVnu, etc.
> 
> > I'm not saying that your config isn't useful, but I just want to limit the
> > scope of the examples. 
> 
> If they might be useful, you could add a page with links to examples that 
> don't meet your threshold for full inclusiion.  That would probably take a 
> sentence or paragraph to describe the target of the link.

Sure.

Even better will be having mailing list archives that are actually
usable/searchable.  Like lore.kernel.org

Working on that...

Thanks,
Richard


___
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel


Re: [Linuxptp-devel] [PATCH] Add E2E automotive profile example configs.

2022-03-07 Thread Hal Murray


richardcoch...@gmail.com said:
> I just mean any kind of published standard or profile, like from the IEEE,
> ITU, AVnu, etc.

> I'm not saying that your config isn't useful, but I just want to limit the
> scope of the examples. 

If they might be useful, you could add a page with links to examples that 
don't meet your threshold for full inclusiion.  That would probably take a 
sentence or paragraph to describe the target of the link.

-- 
These are my opinions.  I hate spam.





___
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel


Re: [Linuxptp-devel] [PATCH v5 00/13] Dynamic sync direction for ts2phc

2022-03-07 Thread Richard Cochran
On Mon, Mar 07, 2022 at 11:27:02PM +0200, Vladimir Oltean wrote:
> Would you mind taking another look at these patches?

I'll do it, but I put it last on my list, because this is more complex
than the other open stuff.  I usually order the work from easy to hard.

Thanks,
Richard


___
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel


Re: [Linuxptp-devel] [PATCH] Add E2E automotive profile example configs.

2022-03-07 Thread Martin Pecka

Got it, thanks anyways.

Martin

Dne 07. 03. 22 v 22:14 Richard Cochran napsal(a):


On Mon, Mar 07, 2022 at 03:57:54PM +0100, Martin Pecka wrote:

Ah, okay. By "published" you mean by an industrial standard?

I just mean any kind of published standard or profile, like from the
IEEE, ITU, AVnu, etc.

I'm not saying that your config isn't useful, but I just want to limit
the scope of the examples.

Thanks,
Richard





smime.p7s
Description: Elektronicky podpis S/MIME
___
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel


Re: [Linuxptp-devel] [PATCH v5 00/13] Dynamic sync direction for ts2phc

2022-03-07 Thread Vladimir Oltean
Hello Richard,

On Tue, Nov 23, 2021 at 02:14:08AM +0200, Vladimir Oltean wrote:
> As discussed in this email thread:
> https://sourceforge.net/p/linuxptp/mailman/message/37047555/
> there is a desire to synchronize multiple DSA switches in a
> boundary_clock_jbod setup, using a PPS signal, and the ts2phc program
> already offers a solid base.

Would you mind taking another look at these patches?


___
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel


Re: [Linuxptp-devel] [PATCH] Add E2E automotive profile example configs.

2022-03-07 Thread Richard Cochran
On Mon, Mar 07, 2022 at 03:57:54PM +0100, Martin Pecka wrote:
> Ah, okay. By "published" you mean by an industrial standard?

I just mean any kind of published standard or profile, like from the
IEEE, ITU, AVnu, etc.

I'm not saying that your config isn't useful, but I just want to limit
the scope of the examples.

Thanks,
Richard


___
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel


[Linuxptp-devel] [PATCH 1/1] Fix management TLV print.

2022-03-07 Thread Erez Geva
Use proper management TLV name in print.
Use a simple 16 unsigned bits format.
Follow standard name for number of records in a table.

Signed-off-by: Erez Geva 
---
 pmc.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/pmc.c b/pmc.c
index 7916153..ab36856 100644
--- a/pmc.c
+++ b/pmc.c
@@ -545,8 +545,8 @@ static void pmc_show(struct ptp_message *msg, FILE *fp)
break;
case MID_UNICAST_MASTER_TABLE_NP:
umtn = (struct unicast_master_table_np *) mgt->data;
-   fprintf(fp, "MID_UNICAST_MASTER_TABLE_NP "
-   IFMT "actual_table_size %" PRIu16,
+   fprintf(fp, "UNICAST_MASTER_TABLE_NP "
+   IFMT "actualTableSize %hu",
umtn->actual_table_size);
buf = (uint8_t *) umtn->unicast_masters;
// table header
-- 
2.30.2



___
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel


[Linuxptp-devel] [PATCH 0/1] Fix new management TLV print

2022-03-07 Thread Erez Geva
Use proper TLV name, follow standard name for table size.
Use simple standard print format for unsigned 16 bits.

Erez Geva (1):
  Fix management TLV print.

 pmc.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

-- 
2.30.2



___
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel


Re: [Linuxptp-devel] [PATCH] Add E2E automotive profile example configs.

2022-03-07 Thread Martin Pecka

Ah, okay. By "published" you mean by an industrial standard?

Dne 07. 03. 22 v 15:43 Richard Cochran napsal(a):


On Mon, Mar 07, 2022 at 03:17:23PM +0100, Martin Pecka wrote:

I think automotive profile is defined here: 
https://avnu.org/wp-content/uploads/2014/05/Auto-Ethernet-AVB-Func-Interop-Spec_v1.6.pdf
. The document doesn't say anything about transport layer or delay
mechanism. So I just took the existing automotive profile and switched it to
E2E.

Okay, but I'm limiting the configs to published profiles.

Thanks,
Richard





smime.p7s
Description: Elektronicky podpis S/MIME
___
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel


Re: [Linuxptp-devel] [PATCH] Add E2E automotive profile example configs.

2022-03-07 Thread Richard Cochran
On Mon, Mar 07, 2022 at 03:17:23PM +0100, Martin Pecka wrote:
> I think automotive profile is defined here: 
> https://avnu.org/wp-content/uploads/2014/05/Auto-Ethernet-AVB-Func-Interop-Spec_v1.6.pdf
> . The document doesn't say anything about transport layer or delay
> mechanism. So I just took the existing automotive profile and switched it to
> E2E.

Okay, but I'm limiting the configs to published profiles.

Thanks,
Richard


___
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel


Re: [Linuxptp-devel] [PATCH] Add E2E automotive profile example configs.

2022-03-07 Thread Martin Pecka
I think automotive profile is defined here: 
https://avnu.org/wp-content/uploads/2014/05/Auto-Ethernet-AVB-Func-Interop-Spec_v1.6.pdf 
. The document doesn't say anything about transport layer or delay 
mechanism. So I just took the existing automotive profile and switched 
it to E2E.


This has been practically verified to work on our mobile robots to sync 
time between several computers and sensors (one of which is considered 
time master). In mobile robotics, space and weight are limited, so the 
ability to use a TC-only switch is crucial. Otherwise, the mobile robot 
behaves very similar to a car in the way that the devices are known 
a-priori, and the whole system is mainly intended to run as an isolated 
time domain (i.e. time-consistency between robots is not as crucial as 
time-consistency inside one robot).


Martin

Dne 07. 03. 22 v 14:27 Richard Cochran napsal(a):


On Mon, Mar 07, 2022 at 01:39:31PM +0100, Martin Pecka wrote:

This allows running an automotive-like network without a time-aware bridge. All 
clients will sync directly with the master e.g. via a simple switch with 
Transparent Clock.

I didn't know about this profile... can you share a link?

Thanks,
Richard





smime.p7s
Description: Elektronicky podpis S/MIME
___
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel


Re: [Linuxptp-devel] [PATCH] Add E2E automotive profile example configs.

2022-03-07 Thread Richard Cochran
On Mon, Mar 07, 2022 at 01:39:31PM +0100, Martin Pecka wrote:
> This allows running an automotive-like network without a time-aware bridge. 
> All clients will sync directly with the master e.g. via a simple switch with 
> Transparent Clock.

I didn't know about this profile... can you share a link?

Thanks,
Richard


___
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel


[Linuxptp-devel] [PATCH] Add E2E automotive profile example configs.

2022-03-07 Thread Martin Pecka
This allows running an automotive-like network without a time-aware bridge. All 
clients will sync directly with the master e.g. via a simple switch with 
Transparent Clock.

Signed-off-by: Martin Pecka 
---
 configs/automotive-master-e2e.cfg | 31 +
 configs/automotive-slave-e2e.cfg  | 38 +++
 2 files changed, 69 insertions(+)
 create mode 100644 configs/automotive-master-e2e.cfg
 create mode 100644 configs/automotive-slave-e2e.cfg

diff --git a/configs/automotive-master-e2e.cfg 
b/configs/automotive-master-e2e.cfg
new file mode 100644
index 000..ffeafed
--- /dev/null
+++ b/configs/automotive-master-e2e.cfg
@@ -0,0 +1,31 @@
+#
+# Automotive Profile example configuration for master containing those
+# attributes which differ from the defaults.  See the file, default.cfg, for
+# the complete list of available options.
+#
+[global]
+#
+# Options carried over from gPTP.
+#
+gmCapable  1
+priority1  248
+priority2  248
+logSyncInterval-3
+syncReceiptTimeout 3
+neighborPropDelayThresh800
+min_neighbor_prop_delay-2000
+assume_two_step1
+path_trace_enabled 1
+follow_up_info 1
+transportSpecific  0x1
+ptp_dst_mac01:80:C2:00:00:0E
+network_transport  L2
+#
+# Automotive Profile specific options
+#
+delay_mechanismE2E
+BMCA   noop
+serverOnly 1
+inhibit_announce   1
+asCapable   true
+inhibit_delay_req   1
diff --git a/configs/automotive-slave-e2e.cfg b/configs/automotive-slave-e2e.cfg
new file mode 100644
index 000..ea273a5
--- /dev/null
+++ b/configs/automotive-slave-e2e.cfg
@@ -0,0 +1,38 @@
+#
+# Automotive Profile example configuration for clients containing those
+# attributes which differ from the defaults.  See the file, default.cfg, for
+# the complete list of available options.
+#
+[global]
+#
+# Options carried over from gPTP.
+#
+gmCapable  1
+priority1  248
+priority2  248
+logSyncInterval-3
+syncReceiptTimeout 3
+neighborPropDelayThresh800
+min_neighbor_prop_delay-2000
+assume_two_step1
+path_trace_enabled 1
+follow_up_info 1
+transportSpecific  0x1
+ptp_dst_mac01:80:C2:00:00:0E
+network_transport  L2
+#
+# Automotive Profile specific options
+#
+delay_mechanismE2E
+BMCA   noop
+clientOnly 1
+inhibit_announce   1
+asCapable   true
+ignore_source_id   1
+# Required to quickly correct Time Jumps in master
+step_threshold  1
+operLogSyncInterval 0
+operLogPdelayReqInterval 2
+msg_interval_request 1
+servo_offset_threshold   30
+servo_num_offset_values  10
-- 
2.17.1



___
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel