On Sun, Aug 29, 2021 at 04:42:55PM +0000, Machnikowski, Maciej wrote:
Please take a look at the 10.2 Operation modes of the G.8264 and at the Figure A.1 which depicts the EEC. This interface is to report the status of the EEC.
Well, I read it, and it is still fairly high level with no mention at all of "DPLL". I hope that the new RTNL states will cover other possible EEC implementations, too.
The "Reference source selection mechanism" is also quite vague. Your patch is more specific:
+enum if_eec_src { + IF_EEC_SRC_INVALID = 0, + IF_EEC_SRC_UNKNOWN, + IF_EEC_SRC_SYNCE, + IF_EEC_SRC_GNSS, + IF_EEC_SRC_PTP, + IF_EEC_SRC_EXT, + __IF_EEC_SRC_MAX, +};
But I guess your list is reasonable. It can always be expanded, right?
If you prefer EEC over DPLL I'm fine with the name change. I think it will be less confusing.
Yes, thanks for doing that.
Thanks, Richard