Skip to content

Commit

Permalink
can: isotp: remove ISO 15675-2 specification version where possible
Browse files Browse the repository at this point in the history
With the new ISO 15765-2:2024 release the former documentation and comments
have to be reworked. This patch removes the ISO specification version/date
where possible.

Signed-off-by: Oliver Hartkopp <[email protected]>
Acked-by: Vincent Mailhol <[email protected]>
Acked-by: Francesco Valla <[email protected]>
Link: https://lore.kernel.org/all/[email protected]
Signed-off-by: Marc Kleine-Budde <[email protected]>
  • Loading branch information
hartkopp authored and marckleinebudde committed Jun 20, 2024
1 parent 58b34cd commit ba63a7e
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 12 deletions.
2 changes: 1 addition & 1 deletion include/uapi/linux/can/isotp.h
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
/*
* linux/can/isotp.h
*
* Definitions for isotp CAN sockets (ISO 15765-2:2016)
* Definitions for ISO 15765-2 CAN transport protocol sockets
*
* Copyright (c) 2020 Volkswagen Group Electronic Research
* All rights reserved.
Expand Down
11 changes: 5 additions & 6 deletions net/can/Kconfig
Original file line number Diff line number Diff line change
Expand Up @@ -56,18 +56,17 @@ config CAN_GW
source "net/can/j1939/Kconfig"

config CAN_ISOTP
tristate "ISO 15765-2:2016 CAN transport protocol"
tristate "ISO 15765-2 CAN transport protocol"
help
CAN Transport Protocols offer support for segmented Point-to-Point
communication between CAN nodes via two defined CAN Identifiers.
This protocol driver implements segmented data transfers for CAN CC
(aka Classical CAN, CAN 2.0B) and CAN FD frame types which were
introduced with ISO 15765-2:2016.
As CAN frames can only transport a small amount of data bytes
(max. 8 bytes for 'classic' CAN and max. 64 bytes for CAN FD) this
(max. 8 bytes for CAN CC and max. 64 bytes for CAN FD) this
segmentation is needed to transport longer Protocol Data Units (PDU)
as needed e.g. for vehicle diagnosis (UDS, ISO 14229) or IP-over-CAN
traffic.
This protocol driver implements data transfers according to
ISO 15765-2:2016 for 'classic' CAN and CAN FD frame types.
If you want to perform automotive vehicle diagnostic services (UDS),
say 'y'.

endif
11 changes: 6 additions & 5 deletions net/can/isotp.c
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@
#include <net/sock.h>
#include <net/net_namespace.h>

MODULE_DESCRIPTION("PF_CAN isotp 15765-2:2016 protocol");
MODULE_DESCRIPTION("PF_CAN ISO 15765-2 transport protocol");
MODULE_LICENSE("Dual BSD/GPL");
MODULE_AUTHOR("Oliver Hartkopp <[email protected]>");
MODULE_ALIAS("can-proto-6");
Expand All @@ -83,10 +83,11 @@ MODULE_ALIAS("can-proto-6");
(CAN_EFF_MASK | CAN_EFF_FLAG | CAN_RTR_FLAG) : \
(CAN_SFF_MASK | CAN_EFF_FLAG | CAN_RTR_FLAG))

/* ISO 15765-2:2016 supports more than 4095 byte per ISO PDU as the FF_DL can
* take full 32 bit values (4 Gbyte). We would need some good concept to handle
* this between user space and kernel space. For now set the static buffer to
* something about 8 kbyte to be able to test this new functionality.
/* Since ISO 15765-2:2016 the CAN isotp protocol supports more than 4095
* byte per ISO PDU as the FF_DL can take full 32 bit values (4 Gbyte).
* We would need some good concept to handle this between user space and
* kernel space. For now set the static buffer to something about 8 kbyte
* to be able to test this new functionality.
*/
#define DEFAULT_MAX_PDU_SIZE 8300

Expand Down

0 comments on commit ba63a7e

Please sign in to comment.