> -----Original Message-----
> From: Geva, Erez <erez.geva....@siemens.com>
> Sent: Thursday, March 11, 2021 2:23 AM
> To: Richard Cochran <richardcoch...@gmail.com>
> Cc: linuxptp-devel@lists.sourceforge.net
> Subject: Re: [Linuxptp-devel] [PATCH 1/2] Explicit length byte order swap
> functions.
>
> How do you want to call the 64 bits?
> I think that naming should be consistence.
>
> Erez
I think the usual scheme is "htonll" or "ntohll" for "long long".
Thanks,
Jake
>
> -----Original Message-----
> From: Richard Cochran <richardcoch...@gmail.com>
> Sent: Thursday, 11 March 2021 04:12
> To: Geva, Erez (ext) (DI PA DCP R&D 3) <erez.geva....@siemens.com>
> Cc: linuxptp-devel@lists.sourceforge.net
> Subject: Re: [Linuxptp-devel] [PATCH 1/2] Explicit length byte order swap
> functions.
>
> On Wed, Mar 10, 2021 at 11:17:48PM +0100, Erez Geva wrote:
> > Replace byte order with explicit length.
> >
> > Add function for byte order for 64 bits.
> >
> > Signed-off-by: Erez Geva <erez.geva....@siemens.com>
> > ---
> > clock.c | 4 +-
> > msg.c | 51 ++++++-------
> > nsm.c | 2 +-
> > port.c | 5 +-
> > raw.c | 10 +--
> > tc.c | 8 +-
> > tlv.c | 207 ++++++++++++++++++++++++++--------------------------
> > transport.c | 9 ++-
> > udp.c | 6 +-
> > udp6.c | 4 +-
> > util.h | 67 +++++++++++++++++
> > 11 files changed, 221 insertions(+), 152 deletions(-)
> >
> > diff --git a/clock.c b/clock.c
> > index 7005636..5b3b4d0 100644
> > --- a/clock.c
> > +++ b/clock.c
> > @@ -255,12 +255,12 @@ void clock_send_notification(struct clock *c, struct
> ptp_message *msg,
> > if (!event_bitmask_get(s->events, event))
> > continue;
> > /* send event */
> > - msg->header.sequenceId = htons(s->sequenceId);
> > + msg->header.sequenceId = hton16(s->sequenceId);
>
> I really don't see any improvement here.
>
> Sorry,
> 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