On Fri, 3 Sep 2021 17:14:35 +0200 Maciej Machnikowski <maciej.machnikowski@xxxxxxxxx> wrote: > This patch series introduces basic interface for reading the Ethernet > Equipment Clock (EEC) state on a SyncE capable device. This state gives > information about the state of EEC. This interface is required to > implement Synchronization Status Messaging on upper layers. > > Initial implementation returns SyncE EEC state and flags attributes. > The only flag currently implemented is the EEC_SRC_PORT. When it's set > the EEC is synchronized to the recovered clock recovered from the > current port. > > SyncE EEC state read needs to be implemented as a ndo_get_eec_state > function. > > Signed-off-by: Maciej Machnikowski <maciej.machnikowski@xxxxxxxxx> Is there a simpler way to do this? Seems like you are adding a lot for a use case specific to a small class of devices. For example adding a new network device operation adds small amount of bloat to every other network device in the kernel.