diff options
author | Vincent Mailhol <mailhol.vincent@wanadoo.fr> | 2022-07-19 23:35:48 +0900 |
---|---|---|
committer | Marc Kleine-Budde <mkl@pengutronix.de> | 2022-07-20 09:26:50 +0200 |
commit | e70a3263a7eed768d5f947b8f2aff8d2a79c9d97 (patch) | |
tree | 2af6096a88f235253013aca90e1f129e0af368cc /drivers/net/can/xilinx_can.c | |
parent | aebe8a2433cd090ccdc222861f44bddb75eb01de (diff) |
can: error: specify the values of data[5..7] of CAN error frames
Currently, data[5..7] of struct can_frame, when used as a CAN error
frame, are defined as being "controller specific". Device specific
behaviours are problematic because it prevents someone from writing
code which is portable between devices.
As a matter of fact, data[5] is never used, data[6] is always used to
report TX error counter and data[7] is always used to report RX error
counter. can-utils also relies on this.
This patch updates the comment in the uapi header to specify that
data[5] is reserved (and thus should not be used) and that data[6..7]
are used for error counters.
Fixes: 0d66548a10cb ("[CAN]: Add PF_CAN core module")
Link: https://lore.kernel.org/all/20220719143550.3681-11-mailhol.vincent@wanadoo.fr
Signed-off-by: Vincent Mailhol <mailhol.vincent@wanadoo.fr>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
Diffstat (limited to 'drivers/net/can/xilinx_can.c')
0 files changed, 0 insertions, 0 deletions