RE: [PATCH v2 2/3] fpga: region: Add fpga-region 'power-domains' property

Hi Yilun,

	Please find my response inline.

-----Original Message-----
From: Xu Yilun <yilun.xu@intel.com>
Sent: Friday, June 24, 2022 9:58 PM
To: Nava kishore Manne <redacted>
Cc: mdf@kernel.org; hao.wu@intel.com; trix@redhat.com;
robh+dt@kernel.org; krzysztof.kozlowski+dt@linaro.org;
michal.simek@xilinx.com; linux-fpga@vger.kernel.org;
devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; linux-arm-
kernel@lists.infradead.org; git@xilinx.com; Rob Herring [off-list ref]
Subject: Re: [PATCH v2 2/3] fpga: region: Add fpga-region 'power-domains'
property

CAUTION: This message has originated from an External Source. Please use
proper judgment and caution when opening attachments, clicking links, or
responding to this email.


On Mon, May 23, 2022 at 07:15:16PM +0530, Nava kishore Manne wrote:

quoted

Add fpga-region 'power-domains' property to allow to handle the
FPGA/PL power domains.

Signed-off-by: Nava kishore Manne <redacted>
Acked-by: Rob Herring <robh@kernel.org>
---
Changes for v2:
              - Updated power-domains description.

 .../devicetree/bindings/fpga/fpga-region.txt       | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/Documentation/devicetree/bindings/fpga/fpga-region.txt

b/Documentation/devicetree/bindings/fpga/fpga-region.txt
index 7d3515264838..f299c3749505 100644

--- a/Documentation/devicetree/bindings/fpga/fpga-region.txt
+++ b/Documentation/devicetree/bindings/fpga/fpga-region.txt

@@ -196,6 +196,20 @@ Optional properties:
 - config-complete-timeout-us : The maximum time in microseconds time

for the

quoted

      FPGA to go to operating mode after the region has been programmed.
 - child nodes : devices in the FPGA after programming.
+- power-domains : A phandle and power domain specifier pair to the

power domain

quoted

+     which is responsible for turning on/off the power to the FPGA/PL

region.

Could you help explain what is PL?

quoted

+Example:
+     fpga_full: fpga-full {
+                compatible = "fpga-region";
+                fpga-mgr = <&zynqmp_pcap>;
+                #address-cells = <2>;
+                #size-cells = <2>;
+                ranges;
+                power-domains = <&zynqmp_firmware PL_PD>;
+        };
+
+     The PL_PD power domain will be turned on before loading the
+bitstream and turned off while removing/unloading the bitstream using

overlays.

I think the single power-domain may not cover some use cases that of-fpga-
region driver supports.

I am not sure which use case you are talking about. Can you please point me the exact use case here?
  

It is possible there are already devices in fpga-region
for static OF tree, or an overlay with no 'firmware-name' but 'external-fpga-
config'. In these cases power domains may still be needed, is it?

It's an optional property user can decide whether he needs this support or not for 'external-fpga-config
Use case. 

Another case is the fpga-region may need multiple power domains?

In our use case full region and relevant partial regions have different power domains and
this patch is capable of handle different power domain regions (full and partial regions)
 

Since the of-fpga-region driver is a generic fpga-region driver, we may
investigate more for a compatible power-domain solution.

Please share your thoughts here we will try to align with it.

Regards,
Navakishore.

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel