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

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