fpga: Add support for Xilinx DFX AXI Shutdown manager
authorNava kishore Manne <nava.manne@xilinx.com>
Thu, 11 Feb 2021 05:11:48 +0000 (10:41 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 24 Mar 2021 07:26:26 +0000 (08:26 +0100)
This patch adds support for Xilinx Dynamic Function eXchange(DFX) AXI
shutdown manager IP. It can be used to safely handling the AXI traffic
on a Reconfigurable Partition when it is undergoing dynamic reconfiguration
and there by preventing system deadlock that may occur if AXI transactions
are interrupted during reconfiguration.

PR-Decoupler and AXI shutdown manager are completely different IPs.
But both the IP registers are compatible and also both belong to the
same sub-system (fpga-bridge).So using same driver for both IP's.

Signed-off-by: Nava kishore Manne <nava.manne@xilinx.com>
Reviewed-by: Tom Rix <trix@redhat.com>
Signed-off-by: Moritz Fischer <mdf@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/fpga/Kconfig
drivers/fpga/xilinx-pr-decoupler.c

index 5ff9438..d591dd9 100644 (file)
@@ -118,10 +118,17 @@ config XILINX_PR_DECOUPLER
        depends on FPGA_BRIDGE
        depends on HAS_IOMEM
        help
-         Say Y to enable drivers for Xilinx LogiCORE PR Decoupler.
+         Say Y to enable drivers for Xilinx LogiCORE PR Decoupler
+         or Xilinx Dynamic Function eXchnage AIX Shutdown Manager.
          The PR Decoupler exists in the FPGA fabric to isolate one
          region of the FPGA from the busses while that region is
          being reprogrammed during partial reconfig.
+         The Dynamic Function eXchange AXI shutdown manager prevents
+         AXI traffic from passing through the bridge. The controller
+         safely handles AXI4MM and AXI4-Lite interfaces on a
+         Reconfigurable Partition when it is undergoing dynamic
+         reconfiguration, preventing the system deadlock that can
+         occur if AXI transactions are interrupted by DFX.
 
 config FPGA_REGION
        tristate "FPGA Region"
index b0eaf26..ea2bde6 100644 (file)
@@ -1,7 +1,7 @@
 // SPDX-License-Identifier: GPL-2.0-only
 /*
  * Copyright (c) 2017, National Instruments Corp.
- * Copyright (c) 2017, Xilix Inc
+ * Copyright (c) 2017, Xilinx Inc
  *
  * FPGA Bridge Driver for the Xilinx LogiCORE Partial Reconfiguration
  * Decoupler IP Core.
 #define CTRL_CMD_COUPLE                0
 #define CTRL_OFFSET            0
 
+struct xlnx_config_data {
+       const char *name;
+};
+
 struct xlnx_pr_decoupler_data {
+       const struct xlnx_config_data *ipconfig;
        void __iomem *io_base;
        struct clk *clk;
 };
@@ -76,15 +81,28 @@ static const struct fpga_bridge_ops xlnx_pr_decoupler_br_ops = {
        .enable_show = xlnx_pr_decoupler_enable_show,
 };
 
+static const struct xlnx_config_data decoupler_config = {
+       .name = "Xilinx PR Decoupler",
+};
+
+static const struct xlnx_config_data shutdown_config = {
+       .name = "Xilinx DFX AXI Shutdown Manager",
+};
+
 static const struct of_device_id xlnx_pr_decoupler_of_match[] = {
-       { .compatible = "xlnx,pr-decoupler-1.00", },
-       { .compatible = "xlnx,pr-decoupler", },
+       { .compatible = "xlnx,pr-decoupler-1.00", .data = &decoupler_config },
+       { .compatible = "xlnx,pr-decoupler", .data = &decoupler_config },
+       { .compatible = "xlnx,dfx-axi-shutdown-manager-1.00",
+                                       .data = &shutdown_config },
+       { .compatible = "xlnx,dfx-axi-shutdown-manager",
+                                       .data = &shutdown_config },
        {},
 };
 MODULE_DEVICE_TABLE(of, xlnx_pr_decoupler_of_match);
 
 static int xlnx_pr_decoupler_probe(struct platform_device *pdev)
 {
+       struct device_node *np = pdev->dev.of_node;
        struct xlnx_pr_decoupler_data *priv;
        struct fpga_bridge *br;
        int err;
@@ -94,6 +112,14 @@ static int xlnx_pr_decoupler_probe(struct platform_device *pdev)
        if (!priv)
                return -ENOMEM;
 
+       if (np) {
+               const struct of_device_id *match;
+
+               match = of_match_node(xlnx_pr_decoupler_of_match, np);
+               if (match && match->data)
+                       priv->ipconfig = match->data;
+       }
+
        res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
        priv->io_base = devm_ioremap_resource(&pdev->dev, res);
        if (IS_ERR(priv->io_base))
@@ -112,7 +138,7 @@ static int xlnx_pr_decoupler_probe(struct platform_device *pdev)
 
        clk_disable(priv->clk);
 
-       br = devm_fpga_bridge_create(&pdev->dev, "Xilinx PR Decoupler",
+       br = devm_fpga_bridge_create(&pdev->dev, priv->ipconfig->name,
                                     &xlnx_pr_decoupler_br_ops, priv);
        if (!br) {
                err = -ENOMEM;
@@ -123,7 +149,8 @@ static int xlnx_pr_decoupler_probe(struct platform_device *pdev)
 
        err = fpga_bridge_register(br);
        if (err) {
-               dev_err(&pdev->dev, "unable to register Xilinx PR Decoupler");
+               dev_err(&pdev->dev, "unable to register %s",
+                       priv->ipconfig->name);
                goto err_clk;
        }