Date: Thu, 20 Apr 2017 19:47:15 +0200
From: Anatolij Gustschin <agust@denx.de>
To: matthew.gerlach@linux.intel.com
Cc: linux-fpga@vger.kernel.org, Alan Tull <atull@opensource.altera.com>,
        Yi Li <yi1.li@linux.intel.com>,
        Moritz Fischer <moritz.fischer@ettus.com>,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3] fpga manager: Add Altera CvP driver
Message-ID: <20170420194715.0ff0ccf1@crub>
In-Reply-To: <alpine.DEB.2.20.1704201021330.8224@mgerlach-VirtualBox>
References: <1492700308-10511-1-git-send-email-agust@denx.de>
        <alpine.DEB.2.20.1704201021330.8224@mgerlach-VirtualBox>
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1072
Lines: 29

Hi Matthew,

On Thu, 20 Apr 2017 10:29:34 -0700 (PDT)
matthew.gerlach@linux.intel.com matthew.gerlach@linux.intel.com wrote:
...
>Since you say the driver works with Arria-10, I thought I would give it 
>a try with the Altera Arria10 PCIe DevKit I am using.  I successfully
>compiled your patch as an out of tree module against a 3.10 kernel.  The 
>module successfully loaded and created instances for both boards in the 
>host.

thanks for testing!

>Now that I have the driver instances running, I'm not sure howto actually 
>perform CvP.  Do you use a debugfs interface or something else?  Do you 
>use the sof or an rbf file?

I used the debugfs interface for first testing, now I'm using custom
fpga config interface driver, but it depends on many other drivers
and won't work without additional special hardware.

I use an rbf file. The v2 patch [1] used a module parameter to specify
firmware file name for a given PCIe device, but it is bypassing the
framework and is not acceptable for mainline.

Thanks,
Anatolij

[1] https://patchwork.kernel.org/patch/9582731