Re: [PATCH v4 0/7] Introduce optional DLC element for Classic CAN

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

 





On 09.11.20 11:26, Oliver Hartkopp wrote:
Introduce improved DLC handling for Classic CAN with introduces a new
element 'len8_dlc' to the struct can_frame and additionally rename
the 'can_dlc' element to 'len' as it represents a plain payload length.

Before implementing the CAN_CTRLMODE_CC_LEN8_DLC handling on driver level
this patch set cleans up and renames the relevant code.

No functional changes.

This patch set is based on mkl/linux-can.git (testing branch).

In fact it is based on the latest net-next ...

https://git.kernel.org/pub/scm/linux/kernel/git/netdev/net-next.git

copy/paste error, sorry.

m(


Changes in v2:
   - rephrase commit message of patch 4 about can_dlc replacement

Changes in v3:
   - remove unnecessarily introduced u8 cast in flexcan.c

Changes in v4:
   - adopt phrasing suggestions from Vincent Mailhol
   - separate and extend CAN documentation (Documentation/networking/can.rst)
   - add new patches for len8_dlc handling for CAN drivers
       - add new helpers in include/linux/can/dev.h
       - add len8_dlc support for various CAN USB adapters as reference

Oliver Hartkopp (7):
   can: add optional DLC element to Classical CAN frame structure
   can: rename get_can_dlc() macro with can_get_cc_len()
   can: remove obsolete get_canfd_dlc() macro
   can: replace can_dlc as variable/element for payload length
   can: update documentation for DLC usage in Classical CAN
   can-dev: introduce helpers to access Classical CAN DLC values
   can-dev: add len8_dlc support for various CAN USB adapters

  Documentation/networking/can.rst              | 68 ++++++++++++++-----
  drivers/net/can/at91_can.c                    | 14 ++--
  drivers/net/can/c_can/c_can.c                 | 20 +++---
  drivers/net/can/cc770/cc770.c                 | 14 ++--
  drivers/net/can/dev.c                         | 10 +--
  drivers/net/can/flexcan.c                     |  4 +-
  drivers/net/can/grcan.c                       | 10 +--
  drivers/net/can/ifi_canfd/ifi_canfd.c         |  6 +-
  drivers/net/can/janz-ican3.c                  | 20 +++---
  drivers/net/can/kvaser_pciefd.c               |  4 +-
  drivers/net/can/m_can/m_can.c                 |  6 +-
  drivers/net/can/mscan/mscan.c                 | 20 +++---
  drivers/net/can/pch_can.c                     | 14 ++--
  drivers/net/can/peak_canfd/peak_canfd.c       | 16 ++---
  drivers/net/can/rcar/rcar_can.c               | 14 ++--
  drivers/net/can/rcar/rcar_canfd.c             |  8 +--
  drivers/net/can/rx-offload.c                  |  2 +-
  drivers/net/can/sja1000/sja1000.c             | 10 +--
  drivers/net/can/slcan.c                       | 32 ++++-----
  drivers/net/can/softing/softing_fw.c          |  2 +-
  drivers/net/can/softing/softing_main.c        | 14 ++--
  drivers/net/can/spi/hi311x.c                  | 20 +++---
  drivers/net/can/spi/mcp251x.c                 | 20 +++---
  .../net/can/spi/mcp251xfd/mcp251xfd-core.c    |  4 +-
  drivers/net/can/sun4i_can.c                   | 10 +--
  drivers/net/can/ti_hecc.c                     |  8 +--
  drivers/net/can/usb/ems_usb.c                 | 16 ++---
  drivers/net/can/usb/esd_usb2.c                | 16 ++---
  drivers/net/can/usb/gs_usb.c                  | 20 +++---
  .../net/can/usb/kvaser_usb/kvaser_usb_core.c  |  2 +-
  .../net/can/usb/kvaser_usb/kvaser_usb_hydra.c | 20 +++---
  .../net/can/usb/kvaser_usb/kvaser_usb_leaf.c  | 22 +++---
  drivers/net/can/usb/mcba_usb.c                | 10 +--
  drivers/net/can/usb/peak_usb/pcan_usb.c       | 20 +++---
  drivers/net/can/usb/peak_usb/pcan_usb_fd.c    | 29 +++++---
  drivers/net/can/usb/peak_usb/pcan_usb_pro.c   | 14 ++--
  drivers/net/can/usb/ucan.c                    | 20 +++---
  drivers/net/can/usb/usb_8dev.c                | 21 +++---
  drivers/net/can/xilinx_can.c                  | 12 ++--
  include/linux/can/dev.h                       | 30 ++++++--
  include/linux/can/dev/peak_canfd.h            |  2 +-
  include/uapi/linux/can.h                      | 38 +++++++----
  include/uapi/linux/can/netlink.h              |  1 +
  net/can/af_can.c                              |  2 +-
  net/can/gw.c                                  |  2 +-
  net/can/j1939/main.c                          |  4 +-
  46 files changed, 377 insertions(+), 294 deletions(-)




[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