On 11/28/20 4:00 PM, Sonal Santan wrote: > Hello, > > This patch series adds management physical function driver for Xilinx Alveo PCIe > accelerator cards, https://www.xilinx.com/products/boards-and-kits/alveo.html > This driver is part of Xilinx Runtime (XRT) open source stack. A few general things. Use scripts/get_maintainer.pl to find who a patch should go to, i should have been on the cc line. Each patch should at a minimum pass scripts/checkpatch.pl, none do. Looking broadly at the files, there are competing names xrt or alveo. It seems like xrt is the dfl equivalent, so maybe drivers/fpga/alveo should be drivers/fpga/xrt There are a lot of files with unnecessary prefixes ex/ fpga/alveo/include/xrt-ucs.h could just be fpga/alveo/include/ucs.h individual subdev's may not belong in the fpga subsystem. I think it would be better to submit these one at a time as is done for dfl. So this will not block getting the basics done, in the next revision, can you leave the subdev's out ? Because of the checkpatch.pl failures, I will wait for the next revision. Tom
