==============================================
+o Naming issues:
+
+ - baudrate, speed -> bitrate
+ - bit_time -> bittime
+ - names of CAN devices and modules
+ - prefix in variables and definitions of the SJA1000 driver
+
+
+o Coding style issues:
+
+ E.g. net/can/dev.[ch] needs reformatting and beautification.
+
+
o CAN bit rate setting:
Default CAN bit rate calculation and custom bit rate setting in a
linked into the kernel (no module parameters).
+o Local loopback device (echo).
+
+ The SJA1000 driver has a local loopback implemented:
+
+ /*
+ * CAN network devices *should* support a local echo functionality
+ * (see Documentation/networking/can.txt). To test the handling of CAN
+ * interfaces that do not support the local echo both driver types are
+ * implemented inside this sja1000 driver. In the case that the driver does
+ * not support the echo the IFF_ECHO remains clear in dev->flags.
+ * This causes the PF_CAN core to perform the echo as a fallback solution.
+ */
+
+
o Feature RX-Probe:
See https://lists.berlios.de/pipermail/socketcan-core/2007-November/000789.html
it work with static linking as well? Should we switch to SYSFS.
-o Naming issues:
-
- - baudrate, speed -> bitrate
- - bit_time -> bittime
- - names of CAN devices and modules
- - prefix in variables and definitions of the SJA1000 driver
-
-
o CAN device statistics:
I think it is to much biased by the SJA1000. It actually counts