On Sat, Aug 19, 2017 at 12:31:40AM +0000, Keller, Jacob E wrote: > This has me curious what's in the first telecom profile....
FWIW, both profiles are publicly available documents. > I like the overall method, and functionalizing the comparison > algorithm is pretty straight forward. I only had one comment > regarding the portDS and defaultDS, which I assume refer to "Domain > Specific"? So the DS in the names stands for "Data Set", and I added the prefix because the profile has both a per-port and a per-clock attribute called localPriority. In fact the profile refers to these are "defaultDS.localPriority" and "portDS.localPriority". [ BTW, all of our configuration options are simple strings, and so the dot carries no special meaning. We don't really have subclasses. ] > Would it make sense to subclass this into .telecom. so > it's more obvious this is very specific to the telecom profile? Hm, that wouldn't be worst idea, especially if we accumulate more and more profile specific options over time. G8275.defaultDS.localPriority 128 G8275.portDS.localPriority 128 G8275.notSlave 0 How does that strike you? Thanks, Richard ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot _______________________________________________ Linuxptp-devel mailing list Linuxptp-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linuxptp-devel