On Fri, 10 Nov 2023 at 07:55, Richard Cochran <richardcoch...@gmail.com>
wrote:

> On Thu, Oct 26, 2023 at 02:40:11PM +0200, Miroslav Lichvar wrote:
>
> > @@ -942,8 +957,8 @@ static int auto_init_ports(struct domain *domain)
> >       }
> >
> >       err = pmc_agent_subscribe(domain->agent, 1000,
> > -                               (60 > domain->phc_interval ?
> > -                                60 : domain->phc_interval) * 3);
> > +                               (10 > domain->phc_interval ?
> > +                                10 : domain->phc_interval) * 3);
>

I think it is better to use
MAX(10, domain->phc_interval)
Instead of
(10 > domain->phc_interval ? 10 : domain->phc_interval)
You may add
#define MAX(a, b) ((a) > (b) ? (a) : (b))

Erez



> Would it simplify things just to change the 180,60 constants
> into 30,10 ?
>
> Thanks,
> Richard
>
>
> _______________________________________________
> Linuxptp-devel mailing list
> Linuxptp-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/linuxptp-devel
>
_______________________________________________
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel

Reply via email to