From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Cyrus-Session-Id: sloti22d1t05-3813382-1518514994-2-3556703381724263916 X-Sieve: CMU Sieve 3.0 X-Spam-known-sender: no X-Spam-score: 0.0 X-Spam-hits: BAYES_00 -1.9, HEADER_FROM_DIFFERENT_DOMAINS 0.001, ME_NOAUTH 0.01, RCVD_IN_DNSWL_HI -5, T_RP_MATCHES_RCVD -0.01, LANGUAGES roen, BAYES_USED global, SA_VERSION 3.4.0 X-Spam-source: IP='209.132.180.67', Host='vger.kernel.org', Country='US', FromHeader='com', MailFrom='org' X-Spam-charsets: X-Resolved-to: greg@kroah.com X-Delivered-to: greg@kroah.com X-Mail-from: linux-api-owner@vger.kernel.org ARC-Seal: i=1; a=rsa-sha256; cv=none; d=messagingengine.com; s=arctest; t=1518514993; b=eYzJrc1I6rSKT/QPuXhLfitJ6LMAGHwcvLvBzNXPbO5RPTh buyYZp7heUVNLVcrZsF+rsNhpkUgJ2G2MNDy+o6PnuuNdNr9uorMYcV4a16Qv6Et TBDkx9uQQ2Hy90uEt4xicHCTiCWXTDC8yXLM70k0DSnihAIeqtPKBA9Uhf8VQ3af 84uqloD07jixRughR2CR2RWzaXrfL+fMWZGOfZBCFb/NEgd6rvDYeet9CkWhnpgJ dmcIFIOuQQcELI7bpIlNSoVCpTI6pQfFUvGtA5XAUClNZ1dGl5fRv5MDHP/3x7hx xw211iRS6kFHM6iWmujb/OcbmT8M3vUumG+VnJA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=from:to:cc:subject:date:message-id :in-reply-to:references:sender:list-id; s=arctest; t=1518514993; bh=QF+kF60wOFadEoUNpAJ5qt4RHq943LxBtyQ/50gek8U=; b=bMGHkGxCSzlM ZVYhDN99Eu73q0mgv+N/+LxuVdFuDjem+ektoj6cMXCc+2XGyl35ncynLMjEIE6/ nFof9amDEgf8tXH7zMNuAoq0VjuO7Uys3C6kVkF+TjgBZ2tn3bB84Oyr6uBBr/U6 e+jtXxJzF/Fw4fSrLlYwMyjH2N22Rut3sfva9NLGE2EGkmbTAaf/eNALldlz7Vxp 5RtPIfWOkRLycYue0X39zgIZS97pqoXcYSUxsK6iAGNIMQVSJD4D5VXwx08uf7Lf IgAg1Vc0MICyrJko0IqN1oI8kXhyulTNMA7GCkr1TH5IY+RT+21+A0VNyU1YEE6+ LIpNjaYBrg== ARC-Authentication-Results: i=1; mx6.messagingengine.com; arc=none (no signatures found); dkim=none (no signatures found); dmarc=none (p=none,has-list-id=yes,d=none) header.from=intel.com; iprev=pass policy.iprev=209.132.180.67 (vger.kernel.org); spf=none smtp.mailfrom=linux-api-owner@vger.kernel.org smtp.helo=vger.kernel.org; x-aligned-from=fail; x-ptr=pass x-ptr-helo=vger.kernel.org x-ptr-lookup=vger.kernel.org; x-return-mx=pass smtp.domain=vger.kernel.org smtp.result=pass smtp_org.domain=kernel.org smtp_org.result=pass smtp_is_org_domain=no header.domain=intel.com header.result=pass header_is_org_domain=yes Authentication-Results: mx6.messagingengine.com; arc=none (no signatures found); dkim=none (no signatures found); dmarc=none (p=none,has-list-id=yes,d=none) header.from=intel.com; iprev=pass policy.iprev=209.132.180.67 (vger.kernel.org); spf=none smtp.mailfrom=linux-api-owner@vger.kernel.org smtp.helo=vger.kernel.org; x-aligned-from=fail; x-ptr=pass x-ptr-helo=vger.kernel.org x-ptr-lookup=vger.kernel.org; x-return-mx=pass smtp.domain=vger.kernel.org smtp.result=pass smtp_org.domain=kernel.org smtp_org.result=pass smtp_is_org_domain=no header.domain=intel.com header.result=pass header_is_org_domain=yes Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933912AbeBMJnK (ORCPT ); Tue, 13 Feb 2018 04:43:10 -0500 Received: from mga03.intel.com ([134.134.136.65]:27121 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934203AbeBMJfg (ORCPT ); Tue, 13 Feb 2018 04:35:36 -0500 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.46,507,1511856000"; d="scan'208";a="26889538" From: Wu Hao To: atull@kernel.org, mdf@kernel.org, linux-fpga@vger.kernel.org, linux-kernel@vger.kernel.org Cc: linux-api@vger.kernel.org, luwei.kang@intel.com, yi.z.zhang@intel.com, hao.wu@intel.com, Tim Whisonant , Enno Luebbers , Shiva Rao , Christopher Rauer , Xiao Guangrong Subject: [PATCH v4 09/24] fpga: dfl-pci: add enumeration for feature devices Date: Tue, 13 Feb 2018 17:24:38 +0800 Message-Id: <1518513893-4719-10-git-send-email-hao.wu@intel.com> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1518513893-4719-1-git-send-email-hao.wu@intel.com> References: <1518513893-4719-1-git-send-email-hao.wu@intel.com> Sender: linux-api-owner@vger.kernel.org X-Mailing-List: linux-api@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-Mailing-List: linux-kernel@vger.kernel.org List-ID: The Device Feature List (DFL) is implemented in MMIO, and features are linked via the DFLs. This patch enables pcie driver to prepare enumeration information (e.g locations of all device feature lists in MMIO) and use common APIs provided by the Device Feature List framework to enumerate each feature device linked. Signed-off-by: Tim Whisonant Signed-off-by: Enno Luebbers Signed-off-by: Shiva Rao Signed-off-by: Christopher Rauer Signed-off-by: Zhang Yi Signed-off-by: Xiao Guangrong Signed-off-by: Wu Hao --- v3: split from another patch use common functions from DFL framework for enumeration. v4: rebase --- drivers/fpga/dfl-pci.c | 199 ++++++++++++++++++++++++++++++++++++++++++++++++- 1 file changed, 197 insertions(+), 2 deletions(-) diff --git a/drivers/fpga/dfl-pci.c b/drivers/fpga/dfl-pci.c index d91ea42..8ce8a94 100644 --- a/drivers/fpga/dfl-pci.c +++ b/drivers/fpga/dfl-pci.c @@ -22,9 +22,52 @@ #include #include +#include "dfl.h" + #define DRV_VERSION "0.8" #define DRV_NAME "dfl-pci" +struct cci_drvdata { + struct fpga_cdev *cdev; /* container device */ + struct list_head regions; /* list of pci bar mapping region */ +}; + +/* pci bar mapping info */ +struct cci_region { + int bar; + void __iomem *ioaddr; /* pointer to mapped bar region */ + struct list_head node; +}; + +static void __iomem *cci_pci_ioremap_bar(struct pci_dev *pcidev, int bar) +{ + struct cci_drvdata *drvdata = pci_get_drvdata(pcidev); + struct cci_region *region; + + list_for_each_entry(region, &drvdata->regions, node) + if (region->bar == bar) { + dev_dbg(&pcidev->dev, "BAR %d region exists\n", bar); + return region->ioaddr; + } + + region = devm_kzalloc(&pcidev->dev, sizeof(*region), GFP_KERNEL); + if (!region) + return NULL; + + region->bar = bar; + region->ioaddr = pci_ioremap_bar(pcidev, bar); + if (!region->ioaddr) { + dev_err(&pcidev->dev, "can't ioremap memory from BAR %d.\n", + bar); + devm_kfree(&pcidev->dev, region); + return NULL; + } + + list_add(®ion->node, &drvdata->regions); + + return region->ioaddr; +} + /* PCI Device ID */ #define PCIE_DEVICE_ID_PF_INT_5_X 0xBCBD #define PCIE_DEVICE_ID_PF_INT_6_X 0xBCC0 @@ -45,6 +88,143 @@ static struct pci_device_id cci_pcie_id_tbl[] = { }; MODULE_DEVICE_TABLE(pci, cci_pcie_id_tbl); +static int cci_init_drvdata(struct pci_dev *pcidev) +{ + struct cci_drvdata *drvdata; + + drvdata = devm_kzalloc(&pcidev->dev, sizeof(*drvdata), GFP_KERNEL); + if (!drvdata) + return -ENOMEM; + + INIT_LIST_HEAD(&drvdata->regions); + + pci_set_drvdata(pcidev, drvdata); + + return 0; +} + +static void cci_pci_release_regions(struct pci_dev *pcidev) +{ + struct cci_drvdata *drvdata = pci_get_drvdata(pcidev); + struct cci_region *tmp, *region; + + list_for_each_entry_safe(region, tmp, &drvdata->regions, node) { + list_del(®ion->node); + if (region->ioaddr) + pci_iounmap(pcidev, region->ioaddr); + devm_kfree(&pcidev->dev, region); + } +} + +static void cci_remove_drvdata(struct pci_dev *pcidev) +{ + struct cci_drvdata *drvdata = pci_get_drvdata(pcidev); + + cci_pci_release_regions(pcidev); + pci_set_drvdata(pcidev, NULL); + devm_kfree(&pcidev->dev, drvdata); +} + +static void cci_remove_feature_devs(struct pci_dev *pcidev) +{ + struct cci_drvdata *drvdata = pci_get_drvdata(pcidev); + + /* remove all children feature devices */ + fpga_remove_feature_devs(drvdata->cdev); +} + +/* enumerate feature devices under pci device */ +static int cci_enumerate_feature_devs(struct pci_dev *pcidev) +{ + struct cci_drvdata *drvdata = pci_get_drvdata(pcidev); + struct fpga_cdev *cdev; + struct fpga_enum_info *info; + resource_size_t start, len; + void __iomem *base; + int port_num, bar, i, ret = 0; + u32 offset; + u64 v; + + /* allocate enumeration info via pci_dev */ + info = fpga_enum_info_alloc(&pcidev->dev); + if (!info) + return -ENOMEM; + + /* start to find Device Feature List from Bar 0 */ + base = cci_pci_ioremap_bar(pcidev, 0); + if (!base) { + ret = -ENOMEM; + goto enum_info_free_exit; + } + + /* + * PF device has FME and Ports/AFUs, and VF device only has 1 Port/AFU. + * check them and add related "Device Feature List" info for the next + * step enumeration. + */ + if (feature_is_fme(base)) { + start = pci_resource_start(pcidev, 0); + len = pci_resource_len(pcidev, 0); + + fpga_enum_info_add_dfl(info, start, len, base); + + /* + * find more Device Feature Lists (e.g Ports) per information + * indicated by FME module. + */ + v = readq(base + FME_HDR_CAP); + port_num = FIELD_GET(FME_CAP_NUM_PORTS, v); + + WARN_ON(port_num > MAX_FPGA_PORT_NUM); + + for (i = 0; i < port_num; i++) { + v = readq(base + FME_HDR_PORT_OFST(i)); + + /* skip ports which are not implemented. */ + if (!(v & FME_PORT_OFST_IMP)) + continue; + + /* + * add Port's Device Feature List information for next + * step enumeration. + */ + bar = FIELD_GET(FME_PORT_OFST_BAR_ID, v); + offset = FIELD_GET(FME_PORT_OFST_DFH_OFST, v); + base = cci_pci_ioremap_bar(pcidev, bar); + if (!base) + continue; + + start = pci_resource_start(pcidev, bar) + offset; + len = pci_resource_len(pcidev, bar) - offset; + + fpga_enum_info_add_dfl(info, start, len, base + offset); + } + } else if (feature_is_port(base)) { + start = pci_resource_start(pcidev, 0); + len = pci_resource_len(pcidev, 0); + + fpga_enum_info_add_dfl(info, start, len, base); + } else { + ret = -ENODEV; + goto enum_info_free_exit; + } + + /* start enumeration with prepared enumeration information */ + cdev = fpga_enumerate_feature_devs(info); + if (IS_ERR(cdev)) { + dev_err(&pcidev->dev, "Enumeration failure\n"); + ret = PTR_ERR(cdev); + goto enum_info_free_exit; + } + + drvdata->cdev = cdev; + +enum_info_free_exit: + fpga_enum_info_free(info); + + return ret; +} + static int cci_pci_probe(struct pci_dev *pcidev, const struct pci_device_id *pcidevid) { @@ -82,9 +262,22 @@ int cci_pci_probe(struct pci_dev *pcidev, const struct pci_device_id *pcidevid) goto release_region_exit; } - /* TODO: create and add the platform device per feature list */ - return 0; + ret = cci_init_drvdata(pcidev); + if (ret) { + dev_err(&pcidev->dev, "Fail to init drvdata %d.\n", ret); + goto release_region_exit; + } + + ret = cci_enumerate_feature_devs(pcidev); + if (ret) { + dev_err(&pcidev->dev, "enumeration failure %d.\n", ret); + goto remove_drvdata_exit; + } + + return ret; +remove_drvdata_exit: + cci_remove_drvdata(pcidev); release_region_exit: pci_release_regions(pcidev); disable_error_report_exit: @@ -95,6 +288,8 @@ int cci_pci_probe(struct pci_dev *pcidev, const struct pci_device_id *pcidevid) static void cci_pci_remove(struct pci_dev *pcidev) { + cci_remove_feature_devs(pcidev); + cci_remove_drvdata(pcidev); pci_release_regions(pcidev); pci_disable_pcie_error_reporting(pcidev); pci_disable_device(pcidev); -- 2.7.4