Re: [PATCH net-next v2 2/2] net: dsa: mv88e6xxx: embedded PTP timestamp support

[Luke Howard]

> I guess that would be fine, except:
> - worth clarifying that you're talking about the Reserved2 field (rather
>  than Reserved1 or Reserved3)

Yes, well some Marvell chips can write at any offset, but Reserved2 is the expected offset.

> - any reason justifying the introduction of "edsa-arrts-trailed", or
>  would it be just for completeness? As a user, what would make me
>  decide between "edsa-arrts-trailer" and "edsa-arrts-ptp-reserved2"?

No good reason, and the newer Marvell chips only document support for Reserved2. Removing trailer support makes sense.

Luke