From: atull <atull@opensource.altera.com>
To: Rob Herring <robh@kernel.org>
Cc: Mark Rutland <mark.rutland@arm.com>,
Moritz Fischer <moritz.fischer@ettus.com>,
<delicious.quinoa@gmail.com>,
Ian Campbell <ijc+devicetree@hellion.org.uk>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Josh Cartwright <joshc@ni.com>,
Michal Simek <michal.simek@xilinx.com>,
<linux-kernel@vger.kernel.org>,
Cyril Chemparathy <cyril.chemparathy@xilinx.com>,
<devicetree@vger.kernel.org>,
Dinh Nguyen <dinguyen@opensource.altera.com>,
Jon Masters <jcm@redhat.com>,
"Matthew Gerlach" <mgerlach@opensource.altera.com>,
Frank Rowand <frowand.list@gmail.com>,
Walter Goossens <waltergoossens@home.nl>,
<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v19 01/12] fpga: add bindings document for fpga region
Date: Tue, 11 Oct 2016 14:31:58 -0500 [thread overview]
Message-ID: <alpine.DEB.2.02.1610111427370.2480@linuxheads99> (raw)
In-Reply-To: <20161008204928.GB11595@rob-hp-laptop>
On Sat, 8 Oct 2016, Rob Herring wrote:
> On Wed, Sep 28, 2016 at 01:21:49PM -0500, Alan Tull wrote:
> > New bindings document for FPGA Region to support programming
> > FPGA's under Device Tree control
> >
> > Signed-off-by: Alan Tull <atull@opensource.altera.com>
> > Signed-off-by: Moritz Fischer <moritz.fischer@ettus.com>
> > ---
> > v9: initial version added to this patchset
> > v10: s/fpga/FPGA/g
> > replace DT overlay example with slightly more complicated example
> > move to staging/simple-fpga-bus
> > v11: No change in this patch for v11 of the patch set
> > v12: Moved out of staging.
> > Changed to use FPGA bridges framework instead of resets
> > for bridges.
> > v13: bridge@0xff20000 -> bridge@ff200000, etc
> > Leave out directly talking about overlays
> > Remove regs and clocks directly under simple-fpga-bus in example
> > Use common "firmware-name" binding instead of "fpga-firmware"
> > v14: Use firmware-name in bindings description
> > Call it FPGA Area
> > Remove bindings that specify FPGA Manager and FPGA Bridges
> > v15: Cleanup as per Rob's comments
> > Combine usage doc with bindings document
> > Document as being Altera specific
> > Additions and changes to add FPGA Bus
> > v16: Reworked to document FPGA Regions
> > rename altera-fpga-bus-fpga-area.txt -> fpga-region.txt
> > Remove references that made it sound exclusive to Altera
> > Remove altr, prefix from fpga-bus and fpga-area compatible strings
> > Added Moritz' usage example with Xilinx
> > Cleaned up unit addresses
> > v17: Lots of rewrites to try to make things clearer
> > Clarify that overlay can be rejected if FPGA isn't programmed
> > Add external-fpga-config binding already used in u-boot
> > Change partial-reconfig binding to partial-fpga-config to align
> > with existing u-boot binding format *-fpga-config
> > Add a document from Xilinx' website
> > v18: Fix node names underscores to be hyphens
> > Fix copy/pasted duplicate nodes in diagram
> > v19: Fix more underscores
> > Make FPGA regions to be children of bridges
> > General cleanup and clarification
> > ---
> > .../devicetree/bindings/fpga/fpga-region.txt | 494 +++++++++++++++++++++
> > 1 file changed, 494 insertions(+)
> > create mode 100644 Documentation/devicetree/bindings/fpga/fpga-region.txt
>
> Reviewed-by: Rob Herring <robh@kernel.org>
>
> Nice job.
>
> Rob
>
Thanks, Rob! I'll add your reviewed-by in v20 just
to make it even.
Besides that, squashing some patches and some
minor changes for the Arria10 FPGA manager support.
Alan
next prev parent reply other threads:[~2016-10-11 20:07 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-28 18:21 [PATCH v19 00/12] Device Tree support for FPGA Programming Alan Tull
2016-09-28 18:21 ` [PATCH v19 01/12] fpga: add bindings document for fpga region Alan Tull
2016-10-08 20:49 ` Rob Herring
2016-10-11 19:31 ` atull [this message]
2016-09-28 18:21 ` [PATCH v19 02/12] doc: fpga-mgr: add fpga image info to api Alan Tull
2016-09-28 18:21 ` [PATCH v19 03/12] add bindings document for altera freeze bridge Alan Tull
2016-10-08 20:49 ` Rob Herring
2016-10-10 18:45 ` atull
2016-09-28 18:21 ` [PATCH v19 04/12] add sysfs document for fpga bridge class Alan Tull
2016-09-29 21:54 ` Moritz Fischer
2016-09-28 18:21 ` [PATCH v19 05/12] fpga-mgr: add fpga image information struct Alan Tull
2016-09-28 23:41 ` Moritz Fischer
2016-09-29 4:34 ` Alan Tull
2016-09-29 17:43 ` Michal Simek
2016-09-28 18:21 ` [PATCH v19 06/12] fpga: add fpga image information struct for socfpga support Alan Tull
2016-09-28 18:21 ` [PATCH v19 07/12] fpga: add fpga image information struct for zynq support Alan Tull
2016-09-28 18:21 ` [PATCH v19 08/12] fpga: add fpga bridge framework Alan Tull
2016-09-28 18:21 ` [PATCH v19 09/12] fpga: fpga-region: device tree control for FPGA Alan Tull
2016-09-28 18:21 ` [PATCH v19 10/12] ARM: socfpga: fpga bridge driver support Alan Tull
2016-09-28 18:21 ` [PATCH v19 11/12] fpga: add altera freeze bridge support Alan Tull
2016-09-28 18:22 ` [PATCH v19 12/12] fpga-manager: Add Socfpga Arria10 support Alan Tull
2016-09-29 16:49 ` Moritz Fischer
2016-09-29 21:35 ` atull
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=alpine.DEB.2.02.1610111427370.2480@linuxheads99 \
--to=atull@opensource.altera.com \
--cc=cyril.chemparathy@xilinx.com \
--cc=delicious.quinoa@gmail.com \
--cc=devicetree@vger.kernel.org \
--cc=dinguyen@opensource.altera.com \
--cc=frowand.list@gmail.com \
--cc=gregkh@linuxfoundation.org \
--cc=ijc+devicetree@hellion.org.uk \
--cc=jcm@redhat.com \
--cc=joshc@ni.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=mgerlach@opensource.altera.com \
--cc=michal.simek@xilinx.com \
--cc=moritz.fischer@ettus.com \
--cc=robh@kernel.org \
--cc=waltergoossens@home.nl \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).