Hi all, Gentle ping. Are we okay with this series? Will appreciate feedback as our next set of patches for Alveo is based on this feature. Thanks, -Sonal > -----Original Message----- > From: Lizhi Hou <lizhi.hou@xxxxxxxxxx> > Sent: Monday, January 31, 2022 4:50 PM > To: Lizhi Hou <lizhih@xxxxxxxxxx>; devicetree@xxxxxxxxxxxxxxx; > robh@xxxxxxxxxx > Cc: linux-kernel@xxxxxxxxxxxxxxx; yilun.xu@xxxxxxxxx; Max Zhen > <maxz@xxxxxxxxxx>; Sonal Santan <sonals@xxxxxxxxxx>; Yu Liu > <yliu@xxxxxxxxxx>; Michal Simek <michals@xxxxxxxxxx>; Stefano Stabellini > <stefanos@xxxxxxxxxx>; trix@xxxxxxxxxx; mdf@xxxxxxxxxx; > dwmw2@xxxxxxxxxxxxx > Subject: Re: [PATCH V1 Create empty OF root 0/1] XRT Alveo driver > infrastructure overview > > Hi Rob, > > > We are still waiting for your feedback and guidance on this. Could you take a > look? > > > Thanks, > > Lizhi > > On 1/25/22 9:48 PM, Lizhi Hou wrote: > > Hello, > > > > Xilinx Alveo PCIe accelerator cards use flattened device tree to describe > > HW subsystems or endpoints. Each device tree node represents a hardware > > endpoint and each endpoint is an hardware unit which requires a driver. > > The product detail: > > https://www.xilinx.com/products/boards-and-kits/alveo.html > > > > The feedback from the previous patches was to create a base tree if there > > is not one and apply the unflattened device nodes by existing Linux > > platform device and OF infrastructure. Please refer to previous discussion > > with device tree and fpga maintainers. > > > https://lore.kernel.org/lkml/CAL_JsqJfyRymB=VxLuQqLpep+Q1Eie48dobv9sC5Oi > zDz0d2DQ@xxxxxxxxxxxxxx/ > > https://lore.kernel.org/lkml/20220105225013.1567871-1- > lizhi.hou@xxxxxxxxxx/ > > > > This patch adds OF_EMPTY_ROOT config. When it is selected and there is not > > a device tree, create an empty device tree root node. > > > > Lizhi Hou (1): > > of: create empty of root > > > > drivers/of/Kconfig | 3 +++ > > drivers/of/Makefile | 1 + > > drivers/of/of_empty_root.c | 51 > ++++++++++++++++++++++++++++++++++++++ > > 3 files changed, 55 insertions(+) > > create mode 100644 drivers/of/of_empty_root.c > >
