On Tue Aug 18 2020, Russell King - ARM Linux admin wrote:
> On Tue, Aug 18, 2020 at 12:32:45PM +0200, Kurt Kanzenbach wrote:
>> -static int is_pdelay_resp(u8 *msgtype)
>> +static int is_pdelay_resp(const struct ptp_header *hdr)
>> {
>> - return (*msgtype & 0xf) == 3;
>> + return (hdr->tsmt & 0xf) == 3;
>
> Forgive my ignorance about PTPv1, but does PTPv1 have these as well?pdelay is PTP v2 specific. > Is there a reason not to use the helper introduced in patch 2 here? Yes, it doesn't have to be. This driver only deals with PTP v2. > Should we have definitions for the message types? That is something one can think of. At the moment there are not many users though. Thanks, Kurt
signature.asc
Description: PGP signature
