Köry Maincent wrote: > On Wed, 08 Mar 2023 10:28:51 -0500 > Willem de Bruijn <willemdebruijn.kernel@xxxxxxxxx> wrote: > > > > > > > + enum timestamping_layer selected_timestamping_layer; > > > + > > > > can perhaps be a single bit rather than an enum > > I need at least two bits to be able to list the PTPs available. > Look at the ethtool_list_ptp function of the second patch. In the available bitmap, yes. Since there are only two options, in the selected case, a single bit would suffice.