Re: [PATCH v8 5/7] can: ctucanfd: CTU CAN FD open-source IP core - platform/SoC support.

[Geert Uytterhoeven]

Hi Pavel,

On Tue, Mar 22, 2022 at 1:06 AM Pavel Pisa <pisa@xxxxxxxxxxxxxxxx> wrote:
> Platform bus adaptation for CTU CAN FD open-source IP core.
>
> The core has been tested together with OpenCores SJA1000
> modified to be CAN FD frames tolerant on MicroZed Zynq based
> MZ_APO education kits designed by Petr Porazil from PiKRON.com
> company. FPGA design
>
>   https://gitlab.fel.cvut.cz/canbus/zynq/zynq-can-sja1000-top.
>
> The kit description at the Computer Architectures course pages
>
>   https://cw.fel.cvut.cz/wiki/courses/b35apo/documentation/mz_apo/start .
>
> Kit carrier board and mechanics design source files
>
>   https://gitlab.com/pikron/projects/mz_apo/microzed_apo
>
> The work is documented in Martin Jeřábek's diploma theses
> Open-source and Open-hardware CAN FD Protocol Support
>
>   https://dspace.cvut.cz/handle/10467/80366
> .
>
> Signed-off-by: Pavel Pisa <pisa@xxxxxxxxxxxxxxxx>
> Signed-off-by: Martin Jerabek <martin.jerabek01@xxxxxxxxx>
> Signed-off-by: Ondrej Ille <ondrej.ille@xxxxxxxxx>

Thanks for your patch, which is now commit e8f0c23a2415fa8f ("can:
ctucanfd: CTU CAN FD open-source IP core - platform/SoC support.") in
linux-can-next/master.

> --- /dev/null
> +++ b/drivers/net/can/ctucanfd/ctucanfd_platform.c

> +/* Match table for OF platform binding */
> +static const struct of_device_id ctucan_of_match[] = {
> +       { .compatible = "ctu,ctucanfd-2", },

Do you need to match on the above compatible value?
The driver seems to treat the hardware the same, and the DT
bindings state the compatible value below should always be present.

> +       { .compatible = "ctu,ctucanfd", },
> +       { /* end of list */ },
> +};
> +MODULE_DEVICE_TABLE(of, ctucan_of_match);

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds