[PATCH v4 0/1] Introducing new CAN FD bittiming parameters: Transmission Delay Compensation (TDC)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Automotive Discussions]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]     [CAN Bus]

  Powered by Linux