diff options
| author | Vincent Mailhol <mailhol.vincent@wanadoo.fr> | 2021-09-18 11:56:33 +0200 |
|---|---|---|
| committer | Marc Kleine-Budde <mkl@pengutronix.de> | 2021-10-24 16:24:28 +0200 |
| commit | 39f66c9e229797a58a12ea78388cbbad1f81aec9 (patch) | |
| tree | a5e6dd1c3c0ec977737dff8c9e04aea72cae6ae4 /include/uapi | |
| parent | can: bittiming: allow TDC{V,O} to be zero and add can_tdc_const::tdc{v,o,f}_min (diff) | |
| download | linux-39f66c9e229797a58a12ea78388cbbad1f81aec9.tar.xz linux-39f66c9e229797a58a12ea78388cbbad1f81aec9.zip | |
can: bittiming: change unit of TDC parameters to clock periods
In the current implementation, all Transmission Delay Compensation
(TDC) parameters are expressed in time quantum. However, ISO 11898-1
actually specifies that these should be expressed in *minimum* time
quantum.
Furthermore, the minimum time quantum is specified to be "one node
clock period long" (c.f. paragraph 11.3.1.1 "Bit time"). For sake of
simplicity, we prefer to use the "clock period" term instead of
"minimum time quantum" because we believe that it is more broadly
understood.
This patch fixes that discrepancy by updating the documentation and
the formula for TDCO calculation.
N.B. In can_calc_tdco(), the sample point (in time quantum) was
calculated using a division, thus introducing a risk of rounding and
truncation errors. On top of changing the unit to clock period, we
also modified the formula to use only additions.
Link: https://lore.kernel.org/all/20210918095637.20108-3-mailhol.vincent@wanadoo.fr
Suggested-by: Stefan Mätje <Stefan.Maetje@esd.eu>
Signed-off-by: Vincent Mailhol <mailhol.vincent@wanadoo.fr>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
Diffstat (limited to '')
0 files changed, 0 insertions, 0 deletions