Re: [Linuxptp-devel] [PATCH] ptp4l: Accept any configuration option as a command line argument.

2016-12-13 Thread Miroslav Lichvar
On Tue, Dec 13, 2016 at 05:34:03PM +0100, Richard Cochran wrote: > On Tue, Dec 13, 2016 at 04:47:23PM +0100, Miroslav Lichvar wrote: > > > + if (commandline) { > > > + dst->flags &= CFG_ITEM_LOCKED; > > > > Should this be dst->flags |= CFG_ITEM_LOCKED ? > > Ouch. Yes of course. Ok,

Re: [Linuxptp-devel] [PATCH] ptp4l: Accept any configuration option as a command line argument.

2016-12-13 Thread Richard Cochran
On Tue, Dec 13, 2016 at 04:47:23PM +0100, Miroslav Lichvar wrote: > I'm testing the latest code from git which includes this feature. It > doesn't seem to work correctly for me. The program accepts long > options, but the behaviour doesn't change as if they were not > specified. Also, valgrind

Re: [Linuxptp-devel] [PATCH] ptp4l: Accept any configuration option as a command line argument.

2016-12-13 Thread Miroslav Lichvar
On Tue, Dec 06, 2016 at 07:49:56PM +0100, Richard Cochran wrote: > This patch provides a way to use the entire table of configuration options > as "long" command line switches. I'm testing the latest code from git which includes this feature. It doesn't seem to work correctly for me. The program

Re: [Linuxptp-devel] [PATCH] ptp4l: Accept any configuration option as a command line argument.

2016-12-06 Thread Keller, Jacob E
for esoteric configurations. Regards, Jake > -Original Message- > From: Richard Cochran [mailto:richardcoch...@gmail.com] > Sent: Tuesday, December 06, 2016 10:50 AM > To: linuxptp-devel@lists.sourceforge.net > Subject: [Linuxptp-devel] [PATCH] ptp4l: Accept any confi

[Linuxptp-devel] [PATCH] ptp4l: Accept any configuration option as a command line argument.

2016-12-06 Thread Richard Cochran
This patch provides a way to use the entire table of configuration options as "long" command line switches. Signed-off-by: Richard Cochran --- config.c | 64 ++-- config.h | 11 +++ ptp4l.c | 11