> -----Original Message-----
> From: Peter Schneider [mailto:pe...@psch.de]
> Sent: Wednesday, March 14, 2018 5:33 AM
> To: linuxptp-devel@lists.sourceforge.net
> Subject: [Linuxptp-devel] [PATCH 0/5] phc2sys: long options / cfg file /
> transportSpecific
> 
> As promised, this is a set of patches to enable long options, a
> specifying a configuration filele, and specifying the transportSpecific
> value.
> 
> Patch 1:      Add transportSpecific as parameter to init_pmc()
> Patch 2:      Enable configuration file on command line
> Patch 3:      Enable long options
> Patch 4:      Add transportSpecific functionality
> Patch 5:      Documentation
> 

The code looks reasonable, but it would be nice of each commit has a dedicated 
message explaining it rather than the same title for each one.

Thanks,
Jake

> Only specifying the 'transportSpecific' value is tested. Some other long
>  options from cfg-file should work out of the box, but these are
> untested (mentioned in the man page, Patch 5.
> I did neither add new elements to the option list nor changes to add
> other commandline parameters to the existing options (like, p.e.,
> domainNumber)
> Kind Regards,
> Peter
> 
> ------------------------------------------------------------------------------
> 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

------------------------------------------------------------------------------
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

Reply via email to