From: Moritz Fischer <mdf@kernel.org>
To: Michal Simek <michal.simek@xilinx.com>
Cc: Moritz Fischer <mdf@kernel.org>,
linux-fpga@vger.kernel.org, linux-kernel@vger.kernel.org,
atull@kernel.org, robh+dt@kernel.org, mark.rutland@arm.com,
devicetree@vger.kernel.org
Subject: Re: [PATCH] dt-bindings: fpga: Consolidate bridge properties
Date: Thu, 22 Feb 2018 09:30:08 -0800 [thread overview]
Message-ID: <20180222173008.GA12391@tyrael.ni.corp.natinst.com> (raw)
In-Reply-To: <8751fd85-fc84-f422-94e7-dd5689cb6922@xilinx.com>
[-- Attachment #1: Type: text/plain, Size: 5802 bytes --]
On Thu, Feb 22, 2018 at 08:12:19AM +0100, Michal Simek wrote:
> On 21.2.2018 18:33, Moritz Fischer wrote:
> > Consolidate bridge properties in a single file, instead of duplicating
> > the same optional property over and over again.
> >
> > Signed-off-by: Moritz Fischer <mdf@kernel.org>
> > Cc: Alan Tull <atull@kernel.org>
> > Cc: Rob Herring <robh+dt@kernel.org>
> > ---
> > .../devicetree/bindings/fpga/altera-fpga2sdram-bridge.txt | 5 +----
> > .../devicetree/bindings/fpga/altera-freeze-bridge.txt | 5 +----
> > .../devicetree/bindings/fpga/altera-hps2fpga-bridge.txt | 5 +----
> > Documentation/devicetree/bindings/fpga/fpga-bridge.txt | 15 +++++++++++++++
> > .../devicetree/bindings/fpga/xilinx-pr-decoupler.txt | 8 ++------
> > 5 files changed, 20 insertions(+), 18 deletions(-)
> > create mode 100644 Documentation/devicetree/bindings/fpga/fpga-bridge.txt
> >
> > diff --git a/Documentation/devicetree/bindings/fpga/altera-fpga2sdram-bridge.txt b/Documentation/devicetree/bindings/fpga/altera-fpga2sdram-bridge.txt
> > index 817a8d4bf903..5dd0ff0f7b4e 100644
> > --- a/Documentation/devicetree/bindings/fpga/altera-fpga2sdram-bridge.txt
> > +++ b/Documentation/devicetree/bindings/fpga/altera-fpga2sdram-bridge.txt
> > @@ -3,10 +3,7 @@ Altera FPGA To SDRAM Bridge Driver
> > Required properties:
> > - compatible : Should contain "altr,socfpga-fpga2sdram-bridge"
> >
> > -Optional properties:
> > -- bridge-enable : 0 if driver should disable bridge at startup
> > - 1 if driver should enable bridge at startup
> > - Default is to leave bridge in current state.
> > +See Documentation/devicetree/bindings/fpga/fpga-bridge.txt for generic bindings.
> >
> > Example:
> > fpga_bridge3: fpga-bridge@ffc25080 {
> > diff --git a/Documentation/devicetree/bindings/fpga/altera-freeze-bridge.txt b/Documentation/devicetree/bindings/fpga/altera-freeze-bridge.txt
> > index f8e288c71b2d..8b26fbcff3c6 100644
> > --- a/Documentation/devicetree/bindings/fpga/altera-freeze-bridge.txt
> > +++ b/Documentation/devicetree/bindings/fpga/altera-freeze-bridge.txt
> > @@ -10,10 +10,7 @@ Required properties:
> > - compatible : Should contain "altr,freeze-bridge-controller"
> > - regs : base address and size for freeze bridge module
> >
> > -Optional properties:
> > -- bridge-enable : 0 if driver should disable bridge at startup
> > - 1 if driver should enable bridge at startup
> > - Default is to leave bridge in current state.
> > +See Documentation/devicetree/bindings/fpga/fpga-bridge.txt for generic bindings.
> >
> > Example:
> > freeze-controller@100000450 {
> > diff --git a/Documentation/devicetree/bindings/fpga/altera-hps2fpga-bridge.txt b/Documentation/devicetree/bindings/fpga/altera-hps2fpga-bridge.txt
> > index 6406f9337eeb..68cce3945b10 100644
> > --- a/Documentation/devicetree/bindings/fpga/altera-hps2fpga-bridge.txt
> > +++ b/Documentation/devicetree/bindings/fpga/altera-hps2fpga-bridge.txt
> > @@ -9,10 +9,7 @@ Required properties:
> > - resets : Phandle and reset specifier for this bridge's reset
> > - clocks : Clocks used by this module.
> >
> > -Optional properties:
> > -- bridge-enable : 0 if driver should disable bridge at startup.
> > - 1 if driver should enable bridge at startup.
> > - Default is to leave bridge in its current state.
> > +See Documentation/devicetree/bindings/fpga/fpga-bridge.txt for generic bindings.
> >
> > Example:
> > fpga_bridge0: fpga-bridge@ff400000 {
> > diff --git a/Documentation/devicetree/bindings/fpga/fpga-bridge.txt b/Documentation/devicetree/bindings/fpga/fpga-bridge.txt
> > new file mode 100644
> > index 000000000000..82607b23a287
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/fpga/fpga-bridge.txt
> > @@ -0,0 +1,15 @@
> > +FPGA Bridge Device Tree Binding
> > +
> > +Moritz Fischer 2018 (Consolidated from previous bindings)
>
> nit: I would remove your name and year - it is visible from git log.
Yeah can get rid of that, I copied from fpga-region.txt :)
>
> > +
> > +Optional properties:
> > +- bridge-enable : 0 if driver should disable bridge at startup
> > + 1 if driver should enable bridge at startup
> > + Default is to leave bridge in current state.
> > +
> > +Example:
> > + fpga_bridge3: fpga-bridge@ffc25080 {
> > + compatible = "altr,socfpga-fpga2sdram-bridge";
> > + reg = <0xffc25080 0x4>;
> > + bridge-enable = <0>;
> > + };
> > diff --git a/Documentation/devicetree/bindings/fpga/xilinx-pr-decoupler.txt b/Documentation/devicetree/bindings/fpga/xilinx-pr-decoupler.txt
> > index 8dcfba926bc7..4284d293fa61 100644
> > --- a/Documentation/devicetree/bindings/fpga/xilinx-pr-decoupler.txt
> > +++ b/Documentation/devicetree/bindings/fpga/xilinx-pr-decoupler.txt
> > @@ -18,12 +18,8 @@ Required properties:
> > - clocks : input clock to IP
> > - clock-names : should contain "aclk"
> >
> > -Optional properties:
> > -- bridge-enable : 0 if driver should disable bridge at startup
> > - 1 if driver should enable bridge at startup
> > - Default is to leave bridge in current state.
> > -
> > -See Documentation/devicetree/bindings/fpga/fpga-region.txt for generic bindings.
> > +See Documentation/devicetree/bindings/fpga/fpga-region.txt and
> > +Documentation/devicetree/bindings/fpga/fpga-bridge.txt for generic bindings.
> >
> > Example:
> > fpga-bridge@100000450 {
> >
>
> Acked-by: Michal Simek <michal.simek@xilinx.com>
>
> Thanks,
> Michal
> --
> To unsubscribe from this list: send the line "unsubscribe linux-fpga" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
- Moritz
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
next prev parent reply other threads:[~2018-02-22 17:34 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-02-21 17:33 [PATCH] dt-bindings: fpga: Consolidate bridge properties Moritz Fischer
2018-02-21 18:47 ` Alan Tull
2018-02-22 7:12 ` Michal Simek
2018-02-22 7:12 ` Michal Simek
2018-02-22 17:30 ` Moritz Fischer [this message]
2018-03-01 22:41 ` Rob Herring
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=20180222173008.GA12391@tyrael.ni.corp.natinst.com \
--to=mdf@kernel.org \
--cc=atull@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=linux-fpga@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=michal.simek@xilinx.com \
--cc=robh+dt@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.