Re: [PATCH v4 0/6] CTU CAN FD open-source IP core SocketCAN driver, PCI, platform integration and documentation
From: Jakub Kicinski
Date: Mon Aug 03 2020 - 16:29:54 EST
On Mon, 3 Aug 2020 20:34:48 +0200 pisa@xxxxxxxxxxxxxxxx wrote:
> From: Pavel Pisa <pisa@xxxxxxxxxxxxxxxx>
>
> This driver adds support for the CTU CAN FD open-source IP core.
> More documentation and core sources at project page
> (https://gitlab.fel.cvut.cz/canbus/ctucanfd_ip_core).
> The core integration to Xilinx Zynq system as platform driver
> is available (https://gitlab.fel.cvut.cz/canbus/zynq/zynq-can-sja1000-top).
> Implementation on Intel FPGA based PCI Express board is available
> from project (https://gitlab.fel.cvut.cz/canbus/pcie-ctu_can_fd).
> The CTU CAN FD core emulation send for review for QEMU mainline.
> Development repository for QEMU emulation - ctu-canfd branch of
> https://gitlab.fel.cvut.cz/canbus/qemu-canbus
>
> More about CAN related projects used and developed at the Faculty
> of the Electrical Engineering (http://www.fel.cvut.cz/en/)
> of Czech Technical University (https://www.cvut.cz/en)
> in Prague at http://canbus.pages.fel.cvut.cz/ .
Patches 3 and 4 have warnings when built with W=1 C=1 flags.
Please also remove the uses of static inline in C sources.
Those are rarely necessary and hide unused code warnings.