[PATCH v3 2/3] dt-bindings: fpga: Add binding doc for versal fpga manager

[Nava kishore Manne]

Hi Rob,

	Thanks for providing the review comments.
Please find my response inline.

> -----Original Message-----
> From: Rob Herring <robh at kernel.org>
> Sent: Tuesday, March 9, 2021 1:50 AM
> To: Nava kishore Manne <navam at xilinx.com>
> Cc: mdf at kernel.org; trix at redhat.com; Michal Simek <michals at xilinx.com>;
> gregkh at linuxfoundation.org; Jolly Shah <JOLLYS at xilinx.com>; Rajan Vaja
> <RAJANV at xilinx.com>; arnd at arndb.de; Manish Narani
> <MNARANI at xilinx.com>; Amit Sunil Dhamne <amitsuni at xilinx.com>; Tejas
> Patel <tejasp at xlnx.xilinx.com>; linux-fpga at vger.kernel.org;
> devicetree at vger.kernel.org; linux-kernel at vger.kernel.org; linux-arm-
> kernel at lists.infradead.org; chinnikishore369 at gmail.com; Appana Durga
> Kedareswara Rao <appanad at xilinx.com>
> Subject: Re: [PATCH v3 2/3] dt-bindings: fpga: Add binding doc for versal fpga
> manager
> 
> On Thu, Mar 04, 2021 at 12:32:23PM +0530, Nava kishore Manne wrote:
> > From: Appana Durga Kedareswara rao <appana.durga.rao at xilinx.com>
> >
> > This patch adds binding doc for versal fpga manager driver.
> 
> Why do you need a DT entry for this? Can't the Versal firmware driver
> instantiate the fpga-mgr device?
> 
This dt entry is need to handle the fpga regions properly for both full and Partial bitstream loading use cases and it cannot be done by Versal firmware driver instantiate.

Complete firmware DT node example is shown below for your reference.
Example for full FPGA configuration:
fpga-region0 {
	compatible = "fpga-region";
	fpga-mgr = <&versal_fpga>;
	#address-cells = <0x1>;
	#size-cells = <0x1>;
};

firmware {
	zynqmp_firmware: zynqmp-firmware {
		compatible = "xlnx,zynqmp-firmware";
		method = "smc";
		versal_fpga: fpga {
			compatible = "xlnx,versal-fpga";
		};
	};
};

Please correct me if my understanding is wrong...

Regards,
Navakishore.