I made a silly mistake which cause the automatic tdco calculation not to work and failed to catch it during my testing (I probably was to focussed to test the netlink interface, and forget to test the most obvious use case...) One incremental patch to be applied on the v3. The patch contains detailed instruction after the "---" scissors. Changes from v3: - do tdco calculation after data bittiming is copied to can_priv. Changes from v2: - Reorder the IFLA_CAN_TDC* entries in can_policy structure - Increase readability of the TDCO formula by introducing a temporary variable. Changes from v1: - Changed the commit message of the first patch of the serie to clarify the condition of when TDC is active. - Changed the alignment style from tabulations to single space in drivers/net/can/dev/netlink.c - Remove duplicated [IFLA_CAN_TERMINATION] entry in drivers/net/can/dev/netlink.c Changes from the RFC: - The RFC was a single e-mail with all comments in bulk, made this a proper patch series. - Added the netlink interface. - Rename the function can_set_tdc to can_calc_tdco (the formula is the same, everything around it was reworked). - Other small miscellaneous changes. Vincent Mailhol (1): can: netlink: do tdco calculation after data bittiming is copied to can_priv drivers/net/can/dev/netlink.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) -- 2.26.2