mbox series

[iproute2-next,5.16,v6,0/5] iplink_can: cleaning, fixes and adding TDC support.

Message ID 20211103164428.692722-1-mailhol.vincent@wanadoo.fr (mailing list archive)
Headers show
Series iplink_can: cleaning, fixes and adding TDC support. | expand

Message

Vincent Mailhol Nov. 3, 2021, 4:44 p.m. UTC
The main purpose is to add commandline support for Transmitter Delay
Compensation (TDC) in iproute. Other issues found during the
development of this feature also get addressed.

This patch series contains four patches which respectively:

  1. Correct the bittiming ranges in the print_usage function and add
  the units to give more clarity: some parameters are in milliseconds,
  some in nano seconds, some in time quantum and the newly TDC
  parameters introduced in this series would be in clock period.

  2. Do some code refactoring on function print_ctrlmode().

  3. factorize the many print_*(PRINT_JSON, ...) and fprintf
  occurrences in a single print_*(PRINT_ANY, ...) call and fix the
  signedness while doing that.

  4. report the value of the bitrate prescalers (brp and dbrp).

  5. adds command line support for the TDC in iproute and goes together
  with below series in the kernel:
  https://lore.kernel.org/linux-can/20210814091750.73931-1-mailhol.vincent@wanadoo.fr/T/#t


** Changelog **

From RFC v5 to v6:
  * Dropped the RFC tag because the related patch series on the kernel
    side were pulled into net-next.
  * Remove the changes in include/uapi/linux/can/netlink.h because
    these should be pulled separately.
  * Add another patch (the second of this series) to do some cleanup
    on function print_ctrlmode().
  * Minor fixes in the patch comments (grammar, rephrasing).

From RFC v4 to RFC v5:
  * Add the unit (bps, tq, ns or ms) in print_usage()
  * Rewrote void can_print_timing_min_max() to better factorize the
    code.
  * Rewrote the commit message of the two last patches (those related
    to TDC) to either add clarification of fix inacurracies.

From v3 to RFC v4:
  * Reflect the changes made on the kernel side.

From RFC v2 to v3:
  * Dropped the RFC tag. Now that the kernel patch reach the testing
    branch, I am finaly ready.
  * Regression fix: configuring a link with only nominal bittiming
    returned -EOPNOTSUPP
  * Added two more patches to the series:
      - iplink_can: fix configuration ranges in print_usage()
      - iplink_can: print brp and dbrp bittiming variables
  * Other small fixes on formatting.

From RFC v1 to RFC v2:
  * Add an additional patch to the series to fix the issues reported
    by Stephen Hemminger
    Ref: https://lore.kernel.org/linux-can/20210506112007.1666738-1-mailhol.vincent@wanadoo.fr/T/#t

Vincent Mailhol (5):
  iplink_can: fix configuration ranges in print_usage() and add unit
  iplink_can: code refactoring of print_ctrlmode()
  iplink_can: use PRINT_ANY to factorize code and fix signedness
  iplink_can: print brp and dbrp bittiming variables
  iplink_can: add new CAN FD bittiming parameters: Transmitter Delay
    Compensation (TDC)

 ip/iplink_can.c | 512 ++++++++++++++++++++++++++----------------------
 1 file changed, 282 insertions(+), 230 deletions(-)

Comments

patchwork-bot+netdevbpf@kernel.org Nov. 4, 2021, 3:50 p.m. UTC | #1
Hello:

This series was applied to iproute2/iproute2-next.git (main)
by David Ahern <dsahern@kernel.org>:

On Thu,  4 Nov 2021 01:44:23 +0900 you wrote:
> The main purpose is to add commandline support for Transmitter Delay
> Compensation (TDC) in iproute. Other issues found during the
> development of this feature also get addressed.
> 
> This patch series contains four patches which respectively:
> 
>   1. Correct the bittiming ranges in the print_usage function and add
>   the units to give more clarity: some parameters are in milliseconds,
>   some in nano seconds, some in time quantum and the newly TDC
>   parameters introduced in this series would be in clock period.
> 
> [...]

Here is the summary with links:
  - [iproute2-next,5.16,v6,1/5] iplink_can: fix configuration ranges in print_usage() and add unit
    https://git.kernel.org/pub/scm/network/iproute2/iproute2-next.git/commit/?id=8316df6e6db4
  - [iproute2-next,5.16,v6,2/5] iplink_can: code refactoring of print_ctrlmode()
    https://git.kernel.org/pub/scm/network/iproute2/iproute2-next.git/commit/?id=fd5e958c494c
  - [iproute2-next,5.16,v6,3/5] iplink_can: use PRINT_ANY to factorize code and fix signedness
    https://git.kernel.org/pub/scm/network/iproute2/iproute2-next.git/commit/?id=67f3c7a5cc0d
  - [iproute2-next,5.16,v6,4/5] iplink_can: print brp and dbrp bittiming variables
    https://git.kernel.org/pub/scm/network/iproute2/iproute2-next.git/commit/?id=0f7bb8d842b1
  - [iproute2-next,5.16,v6,5/5] iplink_can: add new CAN FD bittiming parameters: Transmitter Delay Compensation (TDC)
    https://git.kernel.org/pub/scm/network/iproute2/iproute2-next.git/commit/?id=0c263d7c36ff

You are awesome, thank you!