On Sat, 2018-08-18 at 00:07 -0400, Richard Cochran wrote:
> On Thu, Aug 16, 2018 at 10:42:12AM -0700, Vedang Patel wrote:
> > 
> > diff --git a/fsm.c b/fsm.c
> > index ce6efad30937..a0c98891e39f 100644
> > --- a/fsm.c
> > +++ b/fsm.c
> > @@ -335,3 +335,103 @@ enum port_state ptp_slave_fsm(enum port_state
> > state, enum fsm_event event,
> >  
> >     return next;
> >  }
> > +
> > +enum port_state designated_master_fsm(enum port_state state,
> > +                                 enum fsm_event event,
> > +                                 int mdiff)
> > +{
>       ...
> > 
> > +}
> > +
> > +enum port_state designated_slave_fsm(enum port_state state,
> > +                                enum fsm_event event,
> > +                                int mdiff)
> > +{
>       ...
> > 
> > +}
> Can these two go into their own, new file?
> 
Yeah will move them out and name the file designated_fsm.c/.h.
> > 
> > @@ -2853,10 +2868,24 @@ struct port *port_open(int phc_index,
> >             goto err_port;
> >     }
> >  
> > -   p->state_machine = clock_slave_only(clock) ? ptp_slave_fsm
> > : ptp_fsm;
> >     p->phc_index = phc_index;
> >     p->jbod = config_get_int(cfg, interface->name,
> > "boundary_clock_jbod");
> >     transport = config_get_int(cfg, interface->name,
> > "network_transport");
> > +   p->master_only = config_get_int(cfg, p->name,
> > "masterOnly");
> > +   p->bmca = config_get_int(cfg, p->name, "BMCA");
> > +
> > +   if (p->bmca == BMCA_NOOP && transport != TRANS_UDS) {
> > +           if (p->master_only) {
> > +                   p->state_machine = designated_master_fsm;
> > +           } else if (clock_slave_only(clock)) {
> > +                   p->state_machine = designated_slave_fsm;
> > +           } else {
> > +                   pr_err("Please enable atleast one of
> > masterOnly or slaveOnly when BMCA == noop.\n");
> "at least" -----------------------------------^
> 
Will fix this.

Thanks,
Vedang
> 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
[email protected]
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel

Reply via email to