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
