On 01.07.2022 18:36:45, Jeroen Hofstee wrote: > Hello Dario, > > On 6/28/22 18:31, Dario Binacchi wrote: > > This series originated as a result of CAN communication tests for an > > application using the USBtin adapter (https://www.fischl.de/usbtin/). > > The tests showed some errors but for the driver everything was ok. > > Also, being the first time I used the slcan driver, I was amazed that > > it was not possible to configure the bitrate via the ip tool. > > For these two reasons, I started looking at the driver code and realized > > that it didn't use the CAN network device driver interface. > > > > Starting from these assumptions, I tried to: > > - Use the CAN network device driver interface. > > - Set the bitrate via the ip tool. > > - Send the open/close command to the adapter from the driver. > > - Add ethtool support to reset the adapter errors. > > - Extend the protocol to forward the adapter CAN communication > > errors and the CAN state changes to the netdev upper layers. > > > > Except for the protocol extension patches (i. e. forward the adapter CAN > > communication errors and the CAN state changes to the netdev upper > > layers), the whole series has been tested under QEMU with Linux 4.19.208 > > using the USBtin adapter. > > Testing the extension protocol patches requires updating the adapter > > firmware. Before modifying the firmware I think it makes sense to know if > > these extensions can be considered useful. > > > > Before applying the series I used these commands: > > > > slcan_attach -f -s6 -o /dev/ttyACM0 > > slcand ttyACM0 can0 > > ip link set can0 up > > > > After applying the series I am using these commands: > > > > slcan_attach /dev/ttyACM0 > > slcand ttyACM0 can0 > > ip link set dev can0 down > > ip link set can0 type can bitrate 500000 > > ethtool --set-priv-flags can0 err-rst-on-open on > > ip link set dev can0 up > > > > Now there is a clearer separation between serial line and CAN, > > but above all, it is possible to use the ip and ethtool commands > > as it happens for any CAN device driver. The changes are backward > > compatible, you can continue to use the slcand and slcan_attach > > command options. > > Many thanks, > > For what it is worth, up to "Send the open/close command to the adapter > from the driver.", I cherry-picked this back to 5.10. That seems to work. Thanks for testing. Can I add your Tested-by: for these patches? Marc -- Pengutronix e.K. | Marc Kleine-Budde | Embedded Linux | https://www.pengutronix.de | Vertretung West/Dortmund | Phone: +49-231-2826-924 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
Attachment:
signature.asc
Description: PGP signature
