Hello Rob, thanks much for review. On Saturday 04 of January 2020 00:53:59 Rob Herring wrote: > On Sat, Dec 21, 2019 at 03:07:31PM +0100, pisa@xxxxxxxxxxxxxxxx wrote: > > From: Pavel Pisa <pisa@xxxxxxxxxxxxxxxx> > > .../devicetree/bindings/net/can/ctu,ctucanfd.txt | 61 > > Bindings are moving DT schema format now. Not something I'd require on a > respin I've already reviewed, but OTOH it's been 10 months to respin > from v2. So: Please, can you send me pointer to some CAN or other bindings doc which is well formed according to future direction? I have not dig deeper but I have not found relevant discussion about introduction of DT schema format. > If you have a v4, then please convert to a schema. I expect that we need at least one more iteration. When recheck, I have found that I have forgot to update paths in RST documentation when moved from standalone automatic CI build into kernel tree [PATCH v3 6/6] docs: ctucanfd: CTU CAN FD open-source IP core documentation. https://lkml.org/lkml/2019/12/21/96 And the most important is review of the driver core to allow the project (http://canbus.pages.fel.cvut.cz/) to move forward. [PATCH v3 3/6] can: ctucanfd: add support for CTU CAN FD open-source IP core - bus independent part. https://lkml.org/lkml/2019/12/21/95 The code has no errors and a few questionable warnings reported by 4.19 patchcheck (we have run many iterations of it to cleanup code) but 5.4 kernel patchcheck is more strict as I noticed after submission and reports a few more warnings and some of them could be easily resolved. What makes me to feel good is that CTU CAN FD IP core development stabilized, there are only changes to better cover the core by test framework and more than one month there is no commit disturbing CI build process of IP core integration for Xilinx Zynq. CI builds complete driver and FPGA design and then deploys and runs tests between multiple CTU CAN FD cores and against OpenCores SJA1000 cores with FD tolerance https://gitlab.fel.cvut.cz/canbus/zynq/zynq-can-sja1000-top/pipelines https://gitlab.fel.cvut.cz/canbus/zynq/zynq-can-sja1000-top/-/jobs/51334 The second link points to one of many logs of test running on MZ_APO (Xilinx Zynq) education kits designed for Department of Control Engineering https://dce.fel.cvut.cz/en at PiKRON.com. MZ_APO kist are used in Computer Architectures and Real-time Systems Programming courses. Thanks for help, Pavel
