Re: [PATCH v2 3/3] mtd: spi-nor: add flag for reading dummy cycles from nv cfg reg

2017-10-16 Thread matthew . gerlach
Hi Cyrille, Thanks for the feedback. See my comments in line below. Matthew Gerlach On Tue, 10 Oct 2017, Cyrille Pitchen wrote: Hi Matthew NAK for this patch Le 20/09/2017 à 20:28, matthew.gerl...@linux.intel.com a écrit : From: Matthew Gerlach This patch is a work around for some

[PATCH v3 0/2] Altera ASMI Parallel II IP Core

2017-10-19 Thread matthew . gerlach
From: Matthew Gerlach This patch set adds a spi-nor flash driver for the Altera ASMI Parallel II IP Core. This driver was created based on feedback from Marek Vasut, Cyrill Pitchen, and Michal Suchanek regarding Version 2 of the Altera Quadspi Controller: https://lkml.org/lkml/2017/6/26/518

[PATCH v3 1/2] dt-bindings: mtd: Altera ASMI Parallel II IP Core

2017-10-19 Thread matthew . gerlach
From: Matthew Gerlach Device Tree bindings for Altera ASMI Parallel II IP Core connected to a flash chip. Signed-off-by: Matthew Gerlach Acked-by: Rob Herring --- v2: Made substitutions suggested by Rob Herring. Emphasize driver expects controller is connected to spi-nor flash. v3

[PATCH v3 2/2] mtd: spi-nor: Altera ASMI Parallel II IP Core

2017-10-19 Thread matthew . gerlach
From: Matthew Gerlach This patch adds support for a spi-nor, platform driver for the Altera ASMI Parallel II IP Core. The intended use case is to be able to update the flash used to load a FPGA at power up with mtd-utils. Signed-off-by: Matthew Gerlach --- v2: minor checkpatch fixing by

Re: [PATCH] fpga: fpga-region: comment on fpga_region_program_fpga locking

2018-01-25 Thread matthew . gerlach
On Thu, 25 Jan 2018, Alan Tull wrote: Hi Alan, I seem to remember issue coming up a couple of times. I think this comment will be very helpful. Matthew Gerlach Add a comment to the header of fpga_region_program_fpga() regarding locking of the bridges. Signed-off-by: Alan Tull --- drivers

Re: [PATCH v4 14/24] fpga: dfl: fme: add partial reconfiguration sub feature support

2018-03-11 Thread matthew . gerlach
consider new hardware implementations as well. Full disclosure, I am particularly interested in porting to a new hw implementation for partial reconfiguration. Please see some comments below. Matthew Gerlach On Tue, Feb 13, 2018 at 3:24 AM, Wu Hao wrote: Hi Hao, We are going to want to be able use

Re: [PATCH v4 14/24] fpga: dfl: fme: add partial reconfiguration sub feature support

2018-03-12 Thread matthew . gerlach
On Mon, 12 Mar 2018, Wu Hao wrote: Hi Hao, Please see my two comments inline. Thanks, Matthew Gerlach On Sun, Mar 11, 2018 at 01:09:31PM -0700, matthew.gerl...@linux.intel.com wrote: On Mon, 5 Mar 2018, Alan Tull wrote: Hi Hao, I do think we should consider different hw

Re: [PATCH] fpga: altera_cvp: restrict registration to CvP enabled devices

2018-10-24 Thread matthew . gerlach
On Wed, 24 Oct 2018, Moritz Fischer wrote: Hi Anatolij, Andreas, On Tue, Oct 23, 2018 at 06:46:47PM +, Andreas Puhm wrote: Hi Anatolij, The CvP docs says that on some FPGAs (e.g. Arria 10) the assertion of CVP status can take up to 500ms. However it is not clear whether this delay mig

[PATCH] fpga: dfl: pci: gracefully handle misconfigured port entries

2021-04-20 Thread matthew . gerlach
From: Matthew Gerlach Gracefully ignore misconfigured port entries encountered in incorrect FPGA images. Signed-off-by: Matthew Gerlach --- drivers/fpga/dfl-pci.c | 16 +++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/drivers/fpga/dfl-pci.c b/drivers/fpga/dfl

Re: [PATCH] fpga: dfl: pci: gracefully handle misconfigured port entries

2021-04-20 Thread matthew . gerlach
On Tue, 20 Apr 2021, Tom Rix wrote: On 4/20/21 10:27 AM, matthew.gerl...@linux.intel.com wrote: From: Matthew Gerlach Gracefully ignore misconfigured port entries encountered in incorrect FPGA images. Signed-off-by: Matthew Gerlach --- drivers/fpga/dfl-pci.c | 16 +++- 1

Re: [PATCH v2 1/1] fpga: dfl: afu: harden port enable logic

2021-02-03 Thread matthew . gerlach
ing I also confirmed with HW engineers. The original specification was not precise. The code should have been doing this all along. Matthew Gerlach hasn't been an issue for the current PAC products, as proven by our testing. However, with OFS we cannot anticipate what the timing will be

Re: [PATCH v4 1/1] fpga: dfl: afu: harden port enable logic

2021-02-05 Thread matthew . gerlach
Reviewed-by: Matthew Gerlach On Thu, 4 Feb 2021, Russ Weight wrote: Port enable is not complete until ACK = 0. Change __afu_port_enable() to guarantee that the enable process is complete by polling for ACK == 0. Reviewed-by: Tom Rix Signed-off-by: Russ Weight --- v4: - Added a dev_warn

Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting

2021-01-11 Thread matthew . gerlach
quote ends without a blank line; unexpected unindent. Rectify ReST formatting in ./Documentation/fpga/dfl.rst. Signed-off-by: Lukas Bulwahn Acked-by: Moritz Fischer Acked-by: Matthew Gerlach --- applies cleanly on next-20210111 Moritz, Matthew, please ack. Greg, please pick this doc fixup to

[PATCH v3 0/2] spi: altera: Add DFL bus support for Altera SPI

2021-04-16 Thread matthew . gerlach
From: Matthew Gerlach This patch set adds Device Feature List (DFL) bus support for the Altera SPI Master controller. Patch 1 separates spi-altera.c into spi-altera-core.c and spi-altera-platform.c. Patch 2 adds spi-altera-dfl.c. Matthew Gerlach (2): spi: altera: separate core code from

[PATCH v3 1/2] spi: altera: separate core code from platform code

2021-04-16 Thread matthew . gerlach
From: Matthew Gerlach In preparation of adding support for a new bus type, separate the core spi-altera code from the platform driver code. Signed-off-by: Matthew Gerlach --- drivers/spi/Kconfig | 9 +- drivers/spi/Makefile | 3 +- drivers/spi/spi-altera-core.c

[PATCH v3 2/2] spi: altera: Add DFL bus driver for Altera API Controller

2021-04-16 Thread matthew . gerlach
From: Matthew Gerlach This patch adds a Device Feature List (DFL) bus driver for the Altera SPI Master controller. The SPI master is connected to an Intel SPI Slave to Avalon Bridge inside an Intel MAX10 BMC Chip. Signed-off-by: Matthew Gerlach --- v3: Instantiate SPI device instead of

RE: [PATCH 2/3] fpga: dfl: Add DFL bus driver for Altera SPI Master

2021-04-12 Thread matthew . gerlach
On Fri, 9 Apr 2021, Wu, Hao wrote: On Fri, Apr 09, 2021 at 12:02:47PM +0800, Wu, Hao wrote: + +static void dfl_spi_altera_remove(struct dfl_device *dfl_dev) +{ +struct dfl_altera_spi *aspi = dev_get_drvdata(&dfl_dev->dev); + +platform_device_unregister(aspi->altr_spi); +} + +#define FME_FEAT

[PATCH v2 2/2] hwmon: intel-m10-bmc-hwmon: add sensor support of Intel D5005 card

2021-04-13 Thread matthew . gerlach
From: Matthew Gerlach Like the Intel N3000 card, the Intel D5005 has a MAX10 based BMC. This commit adds support for the D5005 sensors that are monitored by the MAX10 BMC. Signed-off-by: Matthew Gerlach Signed-off-by: Russ Weight Acked-by: Lee Jones --- v2: change variable name from

[PATCH v2 0/2] spi: add BMC support for Intel D5005 card

2021-04-13 Thread matthew . gerlach
From: Matthew Gerlach This patch set adds support for the Board Management Controller (BMC) of an Intel D5005 card. The BMC support requires a Device Feature List (DFL) bus driver for the Altera SPI Master controller and hwmon support for the sensors monitored by the BMC. Patch 1 adds a dfl

[PATCH v2 1/2] spi: Add DFL bus driver for Altera SPI Master

2021-04-13 Thread matthew . gerlach
From: Matthew Gerlach This patch adds a Device Feature List (DFL) bus driver for the Altera SPI Master controller. The SPI master is connected to an Intel SPI Slave to Avalon Master Bridge inside an Intel MAX10 BMC Chip. Signed-off-by: Matthew Gerlach --- v2: moved drivers/fpga/dfl-spi

Re: [PATCH v2 1/2] spi: Add DFL bus driver for Altera SPI Master

2021-04-14 Thread matthew . gerlach
On Wed, 14 Apr 2021, Mark Brown wrote: On Tue, Apr 13, 2021 at 03:58:34PM -0700, matthew.gerl...@linux.intel.com wrote: +++ b/drivers/spi/spi-altera-dfl.c @@ -0,0 +1,222 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * DFL bus driver for Altera SPI Master + * Please make the entire comment

Re: [PATCH 1/3] fpga: dfl: pci: add DID for D5005 PAC cards

2021-04-06 Thread matthew . gerlach
: Matthew Gerlach --- drivers/fpga/dfl-pci.c | 18 +++--- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/drivers/fpga/dfl-pci.c b/drivers/fpga/dfl-pci.c index 04e47e2..b44523e 100644 --- a/drivers/fpga/dfl-pci.c +++ b/drivers/fpga/dfl-pci.c @@ -69,14 +69,16 @@ static void

Re: [PATCH 2/3] fpga: dfl: Add DFL bus driver for Altera SPI Master

2021-04-06 Thread matthew . gerlach
Hi Moritz, On Mon, 5 Apr 2021, Moritz Fischer wrote: Hi Matthew, On Mon, Apr 05, 2021 at 04:53:00PM -0700, matthew.gerl...@linux.intel.com wrote: From: Matthew Gerlach This patch adds DFL bus driver for the Altera SPI Master controller. The SPI master is connected to an Intel SPI Slave

Re: [PATCH 3/3] hwmon: intel-m10-bmc-hwmon: add sensor support of Intel D5005 card

2021-04-06 Thread matthew . gerlach
Hi Yilun, On Tue, 6 Apr 2021, Xu Yilun wrote: On Mon, Apr 05, 2021 at 04:53:01PM -0700, matthew.gerl...@linux.intel.com wrote: From: Matthew Gerlach Like the Intel N3000 card, the Intel D5005 has a MAX10 based BMC. This commit adds support for the D5005 sensors that are monitored by the

[PATCH 1/3] fpga: dfl: pci: add DID for D5005 PAC cards

2021-04-05 Thread matthew . gerlach
From: Russ Weight This patch adds the approved PCI Express Device IDs for the PF and VF for the card for D5005 PAC cards. Signed-off-by: Russ Weight Signed-off-by: Matthew Gerlach --- drivers/fpga/dfl-pci.c | 18 +++--- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git

[PATCH 0/3] fpga: dfl: add support for Intel D5005 card

2021-04-05 Thread matthew . gerlach
From: Matthew Gerlach This patch set adds basic support for the Intel D5005 card to the dfl driver framework. Patch 1 adds the PCIe device id of the D5005 to the dfl-pci id table. Patch 2 adds a dfl driver for the Altera SPI Master which is connected to the board's BMC. Patch 3 adds

[PATCH 2/3] fpga: dfl: Add DFL bus driver for Altera SPI Master

2021-04-05 Thread matthew . gerlach
From: Matthew Gerlach This patch adds DFL bus driver for the Altera SPI Master controller. The SPI master is connected to an Intel SPI Slave to Avalon Master Bridge, inside an Intel MAX10 BMC Chip. Signed-off-by: Matthew Gerlach --- drivers/fpga/Kconfig | 9 ++ drivers/fpga

[PATCH 3/3] hwmon: intel-m10-bmc-hwmon: add sensor support of Intel D5005 card

2021-04-05 Thread matthew . gerlach
From: Matthew Gerlach Like the Intel N3000 card, the Intel D5005 has a MAX10 based BMC. This commit adds support for the D5005 sensors that are monitored by the MAX10 BMC. Signed-off-by: Matthew Gerlach Signed-off-by: Russ Weight --- drivers/hwmon/intel-m10-bmc-hwmon.c | 122

RE: [PATCH 1/2] fpga: dfl: refactor cci_enumerate_feature_devs()

2020-11-17 Thread matthew . gerlach
On Tue, 17 Nov 2020, Wu, Hao wrote: Subject: [PATCH 1/2] fpga: dfl: refactor cci_enumerate_feature_devs() From: Matthew Gerlach In preparation of looking for dfls based on a vendor specific pcie capability, move code that assumes Bar0/offset0 as start of DFL to its own function. Signed

Re: [PATCH 2/2] fpga: dfl: look for vendor specific capability

2020-11-17 Thread matthew . gerlach
On Tue, 17 Nov 2020, Xu Yilun wrote: On Mon, Nov 16, 2020 at 05:25:52PM -0800, matthew.gerl...@linux.intel.com wrote: From: Matthew Gerlach A DFL may not begin at offset 0 of BAR 0. A PCIe vendor specific capability can be used to specify the start of a number of DFLs. Signed-off-by

RE: [PATCH 2/2] fpga: dfl: look for vendor specific capability

2020-11-17 Thread matthew . gerlach
On Tue, 17 Nov 2020, Wu, Hao wrote: Subject: [PATCH 2/2] fpga: dfl: look for vendor specific capability From: Matthew Gerlach A DFL may not begin at offset 0 of BAR 0. A PCIe vendor specific capability can be used to specify the start of a number of DFLs. Signed-off-by: Matthew Gerlach

Re: [PATCH 2/2] fpga: dfl: look for vendor specific capability

2020-11-17 Thread matthew . gerlach
On Tue, 17 Nov 2020, Tom Rix wrote: On 11/16/20 5:25 PM, matthew.gerl...@linux.intel.com wrote: From: Matthew Gerlach A DFL may not begin at offset 0 of BAR 0. A PCIe vendor specific capability can be used to specify the start of a number of DFLs. Signed-off-by: Matthew Gerlach

RE: [PATCH 2/2] fpga: dfl: look for vendor specific capability

2020-11-18 Thread matthew . gerlach
On Wed, 18 Nov 2020, Wu, Hao wrote: On Tue, 17 Nov 2020, Wu, Hao wrote: [...] Open discussion === diff --git a/drivers/fpga/dfl-pci.c b/drivers/fpga/dfl-pci.c index b1b157b41942..5418e8bf2496 100644 --- a/drivers/fpga/dfl-pci.c +++ b/drivers/fpga/dfl-pci.c @@ -27,6 +27,13 @@

[PATCH v2 0/2] fpga: dfl: optional VSEC for start of dfl

2020-11-18 Thread matthew . gerlach
From: Matthew Gerlach The start of a Device Feature List (DFL) is currently assumed to be at Bar0/Offset 0 on the PCIe bus by drivers/fpga/dfl-pci.c. This patchset adds support for the start one or more DFLs to be specified in a Vendor-Specific Capability (VSEC) structure in PCIe config space

[PATCH v2 1/2] fpga: dfl: refactor cci_enumerate_feature_devs()

2020-11-18 Thread matthew . gerlach
From: Matthew Gerlach In preparation of looking for dfls based on a vendor specific pcie capability, move code that assumes Bar0/offset0 as start of DFL to its own function. Signed-off-by: Matthew Gerlach --- v2: remove spurious blank lines rename find_dfl_in_bar0 to find_dfls_by_default

[PATCH v2 2/2] fpga: dfl: look for vendor specific capability

2020-11-18 Thread matthew . gerlach
From: Matthew Gerlach A DFL may not begin at offset 0 of BAR 0. A PCIe vendor specific capability can be used to specify the start of a number of DFLs. Signed-off-by: Matthew Gerlach --- v2: Update documentation for clarity. Clean up macro names. Use GENMASK. Removed spurious

RE: [PATCH v3 1/2] fpga: dfl: refactor cci_enumerate_feature_devs()

2020-11-30 Thread matthew . gerlach
On Sat, 28 Nov 2020, Wu, Hao wrote: Subject: [PATCH v3 1/2] fpga: dfl: refactor cci_enumerate_feature_devs() From: Matthew Gerlach In preparation of looking for dfls based on a vendor specific pcie capability, move code that assumes Bar0/offset0 as start of DFL to its own function. as

RE: [PATCH v3 2/2] fpga: dfl: look for vendor specific capability

2020-11-30 Thread matthew . gerlach
On Sat, 28 Nov 2020, Wu, Hao wrote: Subject: [PATCH v3 2/2] fpga: dfl: look for vendor specific capability Maybe we can change the title a little bit, what about fpga: dfl-pci: locate DFLs by PCIe vendor specific capability From: Matthew Gerlach A DFL may not begin at offset 0 of BAR

Re: [PATCH v2 2/2] fpga: dfl: look for vendor specific capability

2020-11-23 Thread matthew . gerlach
On Sat, 21 Nov 2020, Moritz Fischer wrote: Hi Matthew, On Wed, Nov 18, 2020 at 11:01:51AM -0800, matthew.gerl...@linux.intel.com wrote: From: Matthew Gerlach A DFL may not begin at offset 0 of BAR 0. A PCIe vendor specific capability can be used to specify the start of a number of DFLs

[PATCH v3 0/2] fpga: dfl: optional VSEC for start of dfl

2020-11-24 Thread matthew . gerlach
From: Matthew Gerlach The start of a Device Feature List (DFL) is currently assumed to be at Bar0/Offset 0 on the PCIe bus by drivers/fpga/dfl-pci.c. This patchset adds support for the start one or more DFLs to be specified in a Vendor-Specific Capability (VSEC) structure in PCIe config space

[PATCH v3 2/2] fpga: dfl: look for vendor specific capability

2020-11-24 Thread matthew . gerlach
From: Matthew Gerlach A DFL may not begin at offset 0 of BAR 0. A PCIe vendor specific capability can be used to specify the start of a number of DFLs. Signed-off-by: Matthew Gerlach --- v3: Add text and ascii art to documentation. Ensure not to exceed PCIe config space in loop. v2

[PATCH v3 1/2] fpga: dfl: refactor cci_enumerate_feature_devs()

2020-11-24 Thread matthew . gerlach
From: Matthew Gerlach In preparation of looking for dfls based on a vendor specific pcie capability, move code that assumes Bar0/offset0 as start of DFL to its own function. Signed-off-by: Matthew Gerlach --- v3: no change v2: remove spurious blank lines rename find_dfl_in_bar0 to

RE: [PATCH v3 2/2] fpga: dfl: look for vendor specific capability

2020-12-02 Thread matthew . gerlach
vendor specific capability Maybe we can change the title a little bit, what about fpga: dfl-pci: locate DFLs by PCIe vendor specific capability From: Matthew Gerlach A DFL may not begin at offset 0 of BAR 0. A PCIe vendor specific capability can be used to specify the start of a number of DFLs

RE: [PATCH v3 2/2] fpga: dfl: look for vendor specific capability

2020-12-02 Thread matthew . gerlach
On Wed, 2 Dec 2020, Wu, Hao wrote: + } + + offset = dfl_res & PCI_VNDR_DFLS_RES_OFF_MASK; + if (offset >= len) { + dev_err(&pcidev->dev, "%s bad offset %u >= %pa\n", + __func__, offset, &len); +

[PATCH v4 0/2] fpga: dfl: optional VSEC for start of dfl

2020-12-03 Thread matthew . gerlach
From: Matthew Gerlach The start of a Device Feature List (DFL) is currently assumed to be at Bar0/Offset 0 on the PCIe bus by drivers/fpga/dfl-pci.c. This patchset adds support for the start one or more DFLs to be specified in a Vendor-Specific Capability (VSEC) structure in PCIe config space

[PATCH v4 2/2] fpga: dfl-pci: locate DFLs by PCIe vendor specific capability

2020-12-03 Thread matthew . gerlach
From: Matthew Gerlach A PCIe vendor specific extended capability is introduced by Intel to specify the start of a number of DFLs. Signed-off-by: Matthew Gerlach --- v4: Clarify PCI vs. PCIe in documentation Various cleanup suggested by hao...@intel.com Document and enforce specifying a

[PATCH v4 1/2] fpga: dfl: refactor cci_enumerate_feature_devs()

2020-12-03 Thread matthew . gerlach
From: Matthew Gerlach In preparation of looking for dfls based on a vendor specific pci capability, move the code for the default method of finding the first dfl at offset 0 of Bar 0 to its own function. Signed-off-by: Matthew Gerlach Acked-by: Wu Hao --- v4: add comment squash local

Re: [PATCH] spi: altera: Fix memory leak on error path

2021-01-21 Thread matthew . gerlach
On Wed, 20 Jan 2021, Pan Bian wrote: Release master that have been previously allocated if the number of chipselect is invalid. Fixes: 8e04187c1bc7 ("spi: altera: add SPI core parameters support via platform data.") Signed-off-by: Pan Bian Acked-by: Matthew Gerlach --- drive

Re: How to update a piece of flash for FPGA firmware?

2020-04-28 Thread matthew . gerlach
Hi Yilun, You raise some very interesting questions. Please see my comments below. Matthew On Tue, 28 Apr 2020, Xu Yilun wrote: Hi, I wonder if an updating of FPGA Flash (but cannot reload) could be implemented as fpga-mgr? I have the pcie based FPGA card. The bitstream for FPGA static regi

[PATCH v6 0/4] Altera Partial Reconfiguration IP

2017-03-21 Thread matthew . gerlach
From: Matthew Gerlach This set of patches implements a fpga-mgr driver for the Altera Partial Reconfiguration IP. The driver depends on a patch from Alan Tull that adds a config complete timeout. The driver code itself is divided into core functions and functions to implement a platform driver

[PATCH v6 1/4] fpga: add config complete timeout

2017-03-21 Thread matthew . gerlach
From: Alan Tull Adding timeout for maximum allowed time for FPGA to go to operating mode after a FPGA region has been programmed. Signed-off-by: Alan Tull --- drivers/fpga/fpga-region.c| 3 +++ include/linux/fpga/fpga-mgr.h | 3 +++ 2 files changed, 6 insertions(+) diff --git a/drivers/fp

[PATCH v6 3/4] fpga dt: bindings for Altera Partial Reconfiguration IP.

2017-03-21 Thread matthew . gerlach
From: Matthew Gerlach Device Tree bindings for Altera Partial Reconfiguration IP. Signed-off-by: Matthew Gerlach Acked-by: Rob Herring --- v5: fix comment as suggested by Rob Herring added Acked-by: Rob Herring v4: v3 patch set mistakenly sent out labeled as v4 v3: s/altr,pr-ip/altr

[PATCH v6 4/4] fpga pr ip: Platform driver for Altera Partial Reconfiguration IP.

2017-03-21 Thread matthew . gerlach
From: Matthew Gerlach This adds a platform bus driver for a fpga-mgr driver that uses the Altera Partial Reconfiguration IP component. Signed-off-by: Matthew Gerlach --- v6: add MODULE_LICENSE/DESCRIPTION/AUTHOR as suggested by Anatolij Gustschin v5: fix comment as suggested by Rob

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

2017-03-21 Thread matthew . gerlach
From: Matthew Gerlach Adding the core functions necessary for a fpga-mgr driver for the Altera Partial IP component. It is intended for these functions to be used by the various bus implementations like the platform bus or the PCIe bus. Signed-off-by: Matthew Gerlach --- v6: Suggestions

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

2017-03-22 Thread matthew . gerlach
On Tue, 21 Mar 2017, Anatolij Gustschin wrote: Hi Matthew, Hi Anatolij, On Fri, 10 Mar 2017 11:40:25 -0800 matthew.gerl...@linux.intel.com matthew.gerl...@linux.intel.com wrote: ... +int alt_pr_unregister(struct device *dev) +{ + dev_dbg(dev, "%s\n", __func__); + + fpga_mgr_u

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

2017-03-20 Thread matthew . gerlach
On Sat, 18 Mar 2017, Anatolij Gustschin wrote: Hi Matthew, Hi Anatolij, Thanks for all the feedback. I will create another patch set incorporating your suggestions. On Fri, 10 Mar 2017 11:40:27 -0800 matthew.gerl...@linux.intel.com matthew.gerl...@linux.intel.com wrote: ... +#inclu

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

2017-03-20 Thread matthew . gerlach
On Sat, 18 Mar 2017, Anatolij Gustschin wrote: Hi Matthew, Hi Anatolij, More good feedback. See below. thanks for the patches. Please see some comments below. On Fri, 10 Mar 2017 11:40:25 -0800 matthew.gerl...@linux.intel.com matthew.gerl...@linux.intel.com wrote: ... + if (!(

Re: [RFC 0/2] Add streaming API for firmware and FPGA manager

2017-03-10 Thread matthew . gerlach
On Thu, 9 Mar 2017, yi1...@linux.intel.com wrote: From: Yi Li Hi Yi, This functionality is extremely helpful. I am working with a firmware image of about 90 MBs, and even using scatter-gather instead of a continguous piece of memory is a lot of memory. Matthew Gerlach As the FPGA

Re: [RFC 1/2] firmware class: Add stream_firmware API.

2017-03-10 Thread matthew . gerlach
On Thu, 9 Mar 2017, yi1...@linux.intel.com wrote: From: Yi Li Hi Yi, Just one question below. Matthew Gerlach Add function to load firmware in multiple chucks instead of loading the whole big firmware file at once. Signed-off-by: Yi Li --- drivers/base/firmware_class.c | 128

[PATCH v5 3/4] fpga dt: bindings for Altera Partial Reconfiguration IP.

2017-03-10 Thread matthew . gerlach
From: Matthew Gerlach Device Tree bindings for Altera Partial Reconfiguration IP. Signed-off-by: Matthew Gerlach Acked-by: Rob Herring --- v5: fix comment as suggested by Rob Herring added Acked-by: Rob Herring v4: v3 patch set mistakenly sent out labeled as v4 v3: s/altr,pr-ip/altr

[PATCH v5 4/4] fpga pr ip: Platform driver for Altera Partial Reconfiguration IP.

2017-03-10 Thread matthew . gerlach
From: Matthew Gerlach This adds a platform bus driver for a fpga-mgr driver that uses the Altera Partial Reconfiguration IP component. Signed-off-by: Matthew Gerlach --- v5: fix comment as suggested by Rob Herring v4: v3 patch set mistakenly sent out labeled as v4 v3: s/altr,pr-ip/altr

[PATCH v5 1/4] fpga: add config complete timeout

2017-03-10 Thread matthew . gerlach
From: Alan Tull Adding timeout for maximum allowed time for FPGA to go to operating mode after a FPGA region has been programmed. Signed-off-by: Alan Tull --- drivers/fpga/fpga-region.c| 3 +++ include/linux/fpga/fpga-mgr.h | 3 +++ 2 files changed, 6 insertions(+) diff --git a/drivers/fp

[PATCH v5 0/4] Altera Partial Reconfiguration IP

2017-03-10 Thread matthew . gerlach
From: Matthew Gerlach This set of patches implements a fpga-mgr driver for the Altera Partial Reconfiguration IP. The driver depends on a patch from Alan Tull that adds a config complete timeout. The driver code itself is divided into core functions and functions to implement a platform driver

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

2017-03-10 Thread matthew . gerlach
From: Matthew Gerlach Adding the core functions necessary for a fpga-mgr driver for the Altera Partial IP component. It is intended for these functions to be used by the various bus implementations like the platform bus or the PCIe bus. Signed-off-by: Matthew Gerlach --- v5: Fix comment

Re: [RFC 1/2] firmware class: Add stream_firmware API.

2017-03-13 Thread matthew . gerlach
On Fri, 10 Mar 2017, Li, Yi wrote: Hi Matthew Hi Yi, On 3/10/2017 11:44 AM, matthew.gerl...@linux.intel.com wrote: On Thu, 9 Mar 2017, yi1...@linux.intel.com wrote: From: Yi Li Hi Yi, Just one question below. Matthew Gerlach Add function to load firmware in multiple chucks

Re: [RFC 1/2] firmware class: Add stream_firmware API.

2017-03-14 Thread matthew . gerlach
wrote: From: Yi Li Hi Yi, Just one question below. Matthew Gerlach Add function to load firmware in multiple chucks instead of loading the whole big firmware file at once. Signed-off-by: Yi Li --- drivers/base/firmware_class.c | 128 ++ include

Re: [PATCH] fpga fr br: separate freeze bridge driver code

2017-02-28 Thread matthew . gerlach
On Mon, 27 Feb 2017, Moritz Fischer wrote: Hi Matthew, small nit inline. On Mon, Feb 27, 2017 at 12:03 PM, wrote: From: Matthew Gerlach This patch separates the core Freeze Bridge driver code from the platform driver code. The intent is to allow the core driver code to be used without

[PATCH v4 3/4] fpga dt: bindings for Altera Partial Reconfiguration IP.

2017-03-02 Thread matthew . gerlach
From: Matthew Gerlach Device Tree bindings for Altera Partial Reconfiguration IP. v3: s/altr,pr-ip/altr,a10-pr-ip/ v2: s/Reconfiguraion/Reconfiguration/ Signed-off-by: Matthew Gerlach --- Documentation/devicetree/bindings/fpga/altera-pr-ip.txt | 12 1 file

[PATCH v4 1/4] fpga: add config complete timeout

2017-03-02 Thread matthew . gerlach
From: Alan Tull Adding timeout for maximum allowed time for FPGA to go to operating mode after a FPGA region has been programmed. Signed-off-by: Alan Tull --- drivers/fpga/fpga-region.c| 3 +++ include/linux/fpga/fpga-mgr.h | 3 +++ 2 files changed, 6 insertions(+) diff --git a/drivers/fp

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

2017-03-02 Thread matthew . gerlach
From: Matthew Gerlach Adding the core functions necessary for a fpga-mgr driver for the Altera Partial IP component. It is intended for these functions to be used by the various bus implementations like the platform bus or the PCIe bus. v3: s/alt_pr_probe/alt_pr_register/ s

[PATCH v4 0/4] Altera Partial Reconfiguration IP

2017-03-02 Thread matthew . gerlach
From: Matthew Gerlach This set of patches implements a fpga-mgr driver for the Altera Partial Reconfiguration IP. The driver depends on a patch from Alan Tull that adds a config complete timeout. The driver code itself is divided into core functions and functions to implement a platform driver

[PATCH v4 4/4] fpga pr ip: Platform driver for Altera Partial Reconfiguration IP.

2017-03-02 Thread matthew . gerlach
From: Matthew Gerlach This adds a platform bus driver for a fpga-mgr driver that uses the Altera Partial Reconfiguration IP component. v3: s/altr,pr-ip/altr,a10-pr-ip/ s/alt_pr_probe/alt_pr_register/ s/alt_pr_remove/alt_pr_unregister/ fix error found by kbuild robot with more

Re: [RFC 7/8] fpga-region: add sysfs interface

2017-02-15 Thread matthew . gerlach
e can parse a device tree blob. I also think someone mentioned the FIT format which is closely related to device tree format. Matthew Gerlach [raw bitfile follows, start byte in the file is aligned for DMA] I can publish a version of my python script which produces these files from typical

[PATCH 0/4] Altera Partial Reconfiguration IP

2017-02-15 Thread matthew . gerlach
From: Matthew Gerlach This set of patches implements a fpga-mgr driver for the Altera Partial Reconfiguration IP. The driver depends on a patch from Alan Tull that adds a config complete timeout. The driver code itself is divided into core functions and functions to implement a platform

[PATCH 3/4] fpga dt: bindings for Altera Partial Reconfiguraion IP.

2017-02-15 Thread matthew . gerlach
From: Matthew Gerlach Device Tree bindings for Altera Partial Reconfiguraion IP? Signed-off-by: Matthew Gerlach --- Documentation/devicetree/bindings/fpga/altera-pr-ip.txt | 12 1 file changed, 12 insertions(+) create mode 100644 Documentation/devicetree/bindings/fpga/altera-pr

[PATCH 4/4] fpga pr ip: Platform driver for Altera Partial Reconfiguration IP.

2017-02-15 Thread matthew . gerlach
From: Matthew Gerlach This adds a platform bus driver for a fpga-mgr driver that uses the Altera Partial Reconfiguration IP component. Signed-off-by: Matthew Gerlach --- drivers/fpga/Kconfig | 7 drivers/fpga/Makefile | 1 + drivers/fpga/altera-pr-ip

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

2017-02-15 Thread matthew . gerlach
From: Matthew Gerlach Adding the core functions necessary for a fpga-mgr driver for the Altera Partial IP component. It is intended for these functions to be used by the various bus implementations like the platform bus or the PCIe bus. Signed-off-by: Matthew Gerlach --- drivers/fpga/Kconfig

[PATCH 1/4] fpga: add config complete timeout

2017-02-15 Thread matthew . gerlach
From: Alan Tull Adding timeout for maximum allowed time for FPGA to go to operating mode after a FPGA region has been programmed. Signed-off-by: Alan Tull --- drivers/fpga/fpga-region.c| 3 +++ include/linux/fpga/fpga-mgr.h | 3 +++ 2 files changed, 6 insertions(+) diff --git a/drivers/fp

Re: [RFC 1/8] fpga-mgr: add a single function for fpga loading methods

2017-02-15 Thread matthew . gerlach
Hi Alan, On Wed, 15 Feb 2017, Alan Tull wrote: Currently fpga-mgr.c has three methods for loading FPGA's depending on how the FPGA image is presented: in a sg table, as a single buffer, or as a firmware file. This commit adds these parameters to the fpga_image_info stuct and adds a single fun

Re: [RFC 2/8] fpga-region: support more than one overlay per FPGA region

2017-02-16 Thread matthew . gerlach
Hi Alan, On Wed, 15 Feb 2017, Alan Tull wrote: Currently if a user applies > 1 overlays to a region and removes them, we get a slow warning from devm_kfree. because the pointer to the FPGA image info was overwritten. This commit adds a list to keep track of overlays applied to each FPGA regi

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

2017-02-16 Thread matthew . gerlach
Hi Moritz, Thanks for the feedback. On Thu, 16 Feb 2017, Moritz Fischer wrote: Hi Matthew, On Wed, Feb 15, 2017 at 1:10 PM, wrote: +static int alt_pr_fpga_write_complete(struct fpga_manager *mgr, + struct fpga_image_info *info) +{ + u32 i; + +

[PATCH v2 3/4] fpga dt: bindings for Altera Partial Reconfiguration IP.

2017-02-26 Thread matthew . gerlach
From: Matthew Gerlach Device Tree bindings for Altera Partial Reconfiguration IP. v2: s/Reconfiguraion/Reconfiguration/ Signed-off-by: Matthew Gerlach --- Documentation/devicetree/bindings/fpga/altera-pr-ip.txt | 12 1 file changed, 12 insertions(+) create

[PATCH v2 4/4] fpga pr ip: Platform driver for Altera Partial Reconfiguration IP.

2017-02-26 Thread matthew . gerlach
From: Matthew Gerlach This adds a platform bus driver for a fpga-mgr driver that uses the Altera Partial Reconfiguration IP component. v2: s/altr,pr-ip-core/altr,pr-ip/ Signed-off-by: Matthew Gerlach --- drivers/fpga/Kconfig | 7 drivers/fpga

[PATCH v2 1/4] fpga: add config complete timeout

2017-02-26 Thread matthew . gerlach
From: Alan Tull Adding timeout for maximum allowed time for FPGA to go to operating mode after a FPGA region has been programmed. Signed-off-by: Alan Tull --- drivers/fpga/fpga-region.c| 3 +++ include/linux/fpga/fpga-mgr.h | 3 +++ 2 files changed, 6 insertions(+) diff --git a/drivers/fp

[PATCH v2 0/4] Altera Partial Reconfiguration IP

2017-02-26 Thread matthew . gerlach
From: Matthew Gerlach This set of patches implements a fpga-mgr driver for the Altera Partial Reconfiguration IP. The driver depends on a patch from Alan Tull that adds a config complete timeout. The driver code itself is divided into core functions and functions to implement a platform

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

2017-02-26 Thread matthew . gerlach
From: Matthew Gerlach Adding the core functions necessary for a fpga-mgr driver for the Altera Partial IP component. It is intended for these functions to be used by the various bus implementations like the platform bus or the PCIe bus. Signed-off-by: Matthew Gerlach --- drivers/fpga/Kconfig

Re: [PATCH 3/4] fpga dt: bindings for Altera Partial Reconfiguraion IP.

2017-02-27 Thread matthew . gerlach
On Mon, 27 Feb 2017, Rob Herring wrote: Hi Rob, On Wed, Feb 15, 2017 at 01:10:37PM -0800, matthew.gerl...@linux.intel.com wrote: From: Matthew Gerlach Device Tree bindings for Altera Partial Reconfiguraion IP? Signed-off-by: Matthew Gerlach --- Documentation/devicetree/bindings/fpga

[PATCH] fpga fr br: separate freeze bridge driver code

2017-02-27 Thread matthew . gerlach
From: Matthew Gerlach This patch separates the core Freeze Bridge driver code from the platform driver code. The intent is to allow the core driver code to be used without requiring platform driver support. Signed-off-by: Matthew Gerlach --- drivers/fpga/Kconfig | 7

Re: [RFC 7/8] fpga-region: add sysfs interface

2017-02-27 Thread matthew . gerlach
work, which could be related to using a fairly old 3.10 kernel. Matthew Gerlach I really need to take another look at how non-dt systems enumerate to give better feedback on this. Cheers, Moritz -- To unsubscribe from this list: send the line "unsubscribe linux-fpga" in the body of

Re: [PATCH] fpga: dfl: Replace zero-length array with flexible-array

2020-05-08 Thread matthew . gerlach
This looks like a a good change to me. Tested-by: Matthew Gerlach On Thu, 7 May 2020, Gustavo A. R. Silva wrote: The current codebase makes use of the zero-length array language extension to the C90 standard, but the preferred mechanism to declare variable-length types such as these ones

[PATCH 2/2] fpga: dfl: look for vendor specific capability

2020-11-16 Thread matthew . gerlach
From: Matthew Gerlach A DFL may not begin at offset 0 of BAR 0. A PCIe vendor specific capability can be used to specify the start of a number of DFLs. Signed-off-by: Matthew Gerlach --- Documentation/fpga/dfl.rst | 10 + drivers/fpga/dfl-pci.c | 88

[PATCH 0/2] fpga: dfl: optional VSEC for start of dfl

2020-11-16 Thread matthew . gerlach
From: Matthew Gerlach The start of a Device Feature List (DFL) is currently assumed to be at Bar0/Offset 0 on the PCIe bus by drivers/fpga/dfl-pci.c. This patchset adds support for the start of one or more DFLs to be specified in a Vendor-Specific Capability (VSEC) structure in PCIe config

[PATCH 1/2] fpga: dfl: refactor cci_enumerate_feature_devs()

2020-11-16 Thread matthew . gerlach
From: Matthew Gerlach In preparation of looking for dfls based on a vendor specific pcie capability, move code that assumes Bar0/offset0 as start of DFL to its own function. Signed-off-by: Matthew Gerlach --- drivers/fpga/dfl-pci.c | 86 -- 1 file

Re: [PATCH] fpga: allow to compile-test Altera FPGA bridge drivers

2017-04-18 Thread matthew . gerlach
On Wed, 12 Apr 2017, kbuild test robot wrote: Hi Tobias, Hi Tobias, This is very interesting issue brought up by your patch that turns on COMPILE_TEST in drivers/fpga/Kconfig. See my comment below. Matthew Gerlach [auto build test WARNING on linus/master] [also build test WARNING on

Re: [PATCH v3] fpga manager: Add Altera CvP driver

2017-04-20 Thread matthew . gerlach
rface or something else? Do you use the sof or an rbf file? Thanks, Matthew Gerlach Changes in v3: - removed V-series from description (since the driver works also with Arria-10). Also renamed functions, config option and driver file name. Changed module description in Kconfig - dr

Re: [PATCH 11/16] fpga: intel: fme: add partial reconfiguration sub feature support

2017-04-03 Thread matthew . gerlach
On Mon, 3 Apr 2017, Alan Tull wrote: On Thu, Mar 30, 2017 at 7:08 AM, Wu Hao wrote: From: Kang Luwei Partial Reconfiguration (PR) is the most important function for FME. It allows reconfiguration for given Port/Accelerated Function Unit (AFU). This patch adds support for PR sub feature. I

[PATCH] fpga fr br: fix warning for unexpected version number

2017-04-05 Thread matthew . gerlach
From: Matthew Gerlach The value in the version register of the altera freeze bridge controller changed from the beta value of 2 to the value of 0xad03 in the official release of the IP. This patch supports the old and new version numbers without printing an warning. Signed-off-by: Matthew

Re: [PATCH] fpga fr br: fix warning for unexpected version number

2017-04-05 Thread matthew . gerlach
On Wed, 5 Apr 2017, Moritz Fischer wrote: Hi Matthew, Hi Moritz, On Wed, Apr 5, 2017 at 12:05 PM, wrote: From: Matthew Gerlach The value in the version register of the altera freeze bridge controller changed from the beta value of 2 to the value of 0xad03 in the official

[PATCH v2] fpga fr br: update supported version numbers

2017-04-07 Thread matthew . gerlach
From: Matthew Gerlach The value in the version register of the altera freeze bridge controller changed from the beta value of 2 to the value of 0xad03 in the official release of the IP. This patch supports the old and new version numbers, and the driver's probe function will fail if ne

Re: [PATCH 01/16] docs: fpga: add a document for Intel FPGA driver overview

2017-03-31 Thread matthew . gerlach
On Thu, 30 Mar 2017, Wu Hao wrote: Hi Wu Hao, Great documentation. I'm looking forward to diving into the rest of the patches. Please see my comments inline. Matthew Gerlach Add a document for Intel FPGA driver overview. Signed-off-by: Enno Luebbers Signed-off-by: Xiao Guan

  1   2   >