Re: [PATCH v5 2/4] fpga pr ip: Core driver support for Altera Partial Reconfiguration IP.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 






On Tue, 21 Mar 2017, Anatolij Gustschin wrote:

Hi Matthew,

Hi Anatolij,


On Fri, 10 Mar 2017 11:40:25 -0800
matthew.gerlach@xxxxxxxxxxxxxxx matthew.gerlach@xxxxxxxxxxxxxxx wrote:
...
+int alt_pr_unregister(struct device *dev)
+{
+	dev_dbg(dev, "%s\n", __func__);
+
+	fpga_mgr_unregister(dev);
+
+	return 0;
+}
+EXPORT_SYMBOL_GPL(alt_pr_unregister);

Can we also add a function for registering a PCIe device with
PR IP here? Something like:

If we have an alt_pr_pcie_register function, we will need the corresponding alt_pr_pcie_unregister function. Both of these functions should go into their own file like alt_pr_platform_probe() and alt_pr_platform_remove().


/**
* alt_pr_pcie_register - register PCIe device with PR-IP core
* @pci_dev:    PCI device with PR-IP
* @bar:        PR-IP BAR number
* @pr_offset:  offset of the PR-IP core registers
*
* Return: 0 on success, negative error code otherwise.
*
* To unregister the PCIe device, use alt_pr_unregister(&pdev->dev).
*/
int alt_pr_pcie_register(struct pci_dev *pdev, int bar, int pr_offset)
{
       void __iomem *base;
       int ret;

       if (!pci_is_enabled(pdev)) {
               ret = pci_enable_device(pdev);
               if (ret < 0) {
                       dev_err(&pdev->dev, "can't enable device: %d\n", ret);
                       return ret;
               }
       }

       base = devm_ioremap_resource(&pdev->dev, &pdev->resource[bar]);

Does this remap the whole bar? If it does, what happens if other components are also connected to the bar? How do those corresponding drivers get access to the mapped memory?


       if (IS_ERR(base)) {
               dev_warn(&pdev->dev, "mapping PR-IP BAR failed\n");
               return -ENOMEM;
       }

       return alt_pr_register(&pdev->dev, base + pr_offset);
}
EXPORT_SYMBOL_GPL(alt_pr_pcie_register);

Thanks,
Anatolij
--
To unsubscribe from this list: send the line "unsubscribe linux-fpga" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux