From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: Florian Fainelli <f.fainelli@gmail.com>
Cc: Alan Tull <atull@kernel.org>, Moritz Fischer <mdf@kernel.org>,
linux-fpga@vger.kernel.org, Marek Vasut <marek.vasut@gmail.com>
Subject: Re: [PATCH] fpga: add simple userspace interface to trigger FPGA programming
Date: Sun, 10 Dec 2017 23:42:11 +0100 [thread overview]
Message-ID: <20171210234211.6c704dfa@windsurf.lan> (raw)
In-Reply-To: <e730ef80-df19-3caa-68b2-02d6ef56f04e@gmail.com>
Hello,
On Sat, 9 Dec 2017 10:05:55 -0800, Florian Fainelli wrote:
> > The problem I see with this is that it's allowing userspace to switch
> > out what's on the FPGA without controlling bridges and without
> > unloading drivers first.
>
> We can actually use the Linux device driver model to do that though.
> Assuming all peripherals behind the FPGA do have the FPGA manager's
> struct device as a parent (which they should),
Well in my setup, that isn't the case. The device implemented in the
FPGA is a PCIe endpoint, so its parent device from a device model point
of view if the PCIe root complex, which is a platform_device in the SoC.
So there is currently no connection between the PCIe endpoint
implemented in the FPGA, and the FPGA device itself.
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
next prev parent reply other threads:[~2017-12-10 22:42 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-04 15:43 [PATCH] fpga: add simple userspace interface to trigger FPGA programming Thomas Petazzoni
2017-12-04 15:50 ` Alan Tull
2017-12-04 15:58 ` Thomas Petazzoni
2017-12-04 16:25 ` Alan Tull
2017-12-04 16:49 ` Moritz Fischer
2017-12-04 17:30 ` Alan Tull
2017-12-09 18:05 ` Florian Fainelli
2017-12-10 4:03 ` Alan Tull
2017-12-10 22:44 ` Thomas Petazzoni
2017-12-10 22:59 ` Florian Fainelli
2017-12-11 22:05 ` Moritz Fischer
2017-12-11 23:32 ` Alan Tull
2017-12-13 6:30 ` yves.vandervennet
2017-12-13 5:23 ` Thomas Petazzoni
2017-12-13 15:59 ` Alan Tull
2017-12-14 5:27 ` Thomas Petazzoni
2017-12-14 20:10 ` Alan Tull
2017-12-10 22:42 ` Thomas Petazzoni [this message]
2018-08-11 15:01 ` Philippe De Muyter
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20171210234211.6c704dfa@windsurf.lan \
--to=thomas.petazzoni@free-electrons.com \
--cc=atull@kernel.org \
--cc=f.fainelli@gmail.com \
--cc=linux-fpga@vger.kernel.org \
--cc=marek.vasut@gmail.com \
--cc=mdf@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.