From: Kumar Gala <galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org> To: Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org> Cc: Bhupesh Sharma <bhupesh.sharma-KZfg59tc24xl57MIdRCFDg@public.gmane.org>, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, "arnd-r2nGTMty4D4@public.gmane.org" <arnd-r2nGTMty4D4@public.gmane.org>, "J. German Rivera" <German.Rivera-KZfg59tc24xl57MIdRCFDg@public.gmane.org>, Will Deacon <Will.Deacon-5wv7dgnIgG8@public.gmane.org>, "stuart.yoder-KZfg59tc24xl57MIdRCFDg@public.gmane.org" <stuart.yoder-KZfg59tc24xl57MIdRCFDg@public.gmane.org>, "grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org" <grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org>, Catalin Marinas <Catalin.Marinas-5wv7dgnIgG8@public.gmane.org>, "arnab.basu-KZfg59tc24xl57MIdRCFDg@public.gmane.org" <arnab.basu-KZfg59tc24xl57MIdRCFDg@public.gmane.org>, "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" <linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org> Subject: Re: [PATCH 3/6] Documentation: DT: Add entry for FSL Management Complex Date: Fri, 15 Aug 2014 08:35:34 -0500 [thread overview] Message-ID: <21CA0EC7-3711-4B13-991B-940EF576D2EF@codeaurora.org> (raw) In-Reply-To: <20140815110020.GC15621@leverpostej> On Aug 15, 2014, at 6:00 AM, Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org> wrote: > [devicetree-discuss is no more, fixing up to devicetree-u79uwXL29TbrhsbdSgBK9A@public.gmane.orgrg] > > Hi, > > On Fri, Aug 15, 2014 at 10:49:12AM +0100, Bhupesh Sharma wrote: >> This patch adds a devicetree binding documentation for FSL's >> Management Complex. >> >> Management Complex is a hardware resource manager that manages >> specialized hardware objects used in network-oriented packet >> processing applications >> >> Signed-off-by: Bhupesh Sharma <bhupesh.sharma-KZfg59tc24xl57MIdRCFDg@public.gmane.org> >> Signed-off-by: Stuart Yoder <stuart.yoder-KZfg59tc24xl57MIdRCFDg@public.gmane.org> >> Signed-off-by: J. German Rivera <German.Rivera-KZfg59tc24xl57MIdRCFDg@public.gmane.org> >> --- >> .../devicetree/bindings/misc/fsl,qoriq-mc.txt | 40 ++++++++++++++++++++ I’d probably start this off in bindings/soc/fsl/ >> 1 file changed, 40 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/misc/fsl,qoriq-mc.txt >> >> diff --git a/Documentation/devicetree/bindings/misc/fsl,qoriq-mc.txt b/Documentation/devicetree/bindings/misc/fsl,qoriq-mc.txt >> new file mode 100644 >> index 0000000..608529e >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/misc/fsl,qoriq-mc.txt >> @@ -0,0 +1,40 @@ >> +* Freescale Management Complex >> + >> +The Freescale Management Complex (fsl-mc) is a hardware resource >> +manager that manages specialized hardware objects used in >> +network-oriented packet processing applications. After the fsl-mc >> +block is enabled, pools of hardware resources are available, such as >> +queues, buffer pools, I/O interfaces. These resources are building >> +blocks that can be used to create functional hardware objects/devices >> +such as network interfaces, crypto accelerator instances, L2 switches, >> +etc. >> + >> +Required properties: >> + >> + - compatible >> + Value type: <string> >> + Definition: Must be "fsl,qoriq-mc". A Freescale Management Complex >> + compatible with this binding must have Block Revision >> + Registers BRR1 and BRR2 at offset 0x0BF8 and 0x0BFC in >> + the MC control register region. Hmm, this BRR1/BRR2 comments are a little odd, if we keep those, we should probably also state what value is BRR1 (I think that was the one that contented the IP ID). >> + >> + - reg >> + Value type: <prop-encoded-array> >> + Definition: A standard property. Specifies one or two regions >> + defining the MC's registers: >> + >> + -the first region is the command portal for the >> + this machine and must always be present >> + >> + -the second region is the MC control registers. This >> + region may not be present in some scenarios, such >> + as in the device tree presented to a virtual machine. Should we distinguish the second case w/a different compat? It seems like a major driver change if the second region isn’t there, or a different driver? > > This looks extremely simple. Is this unit self-contained or does it > relate to other blocks which will be described separately? > >> + >> +Example: >> + >> + fsl_mc: fsl-mc@80c000000 { >> + compatible = "fsl,qoriq-mc"; >> + reg = <0x00000008 0x0c000000 0 0x40 // MC portal base >> + 0x00000000 0x08340000 0 0x40000 >; // MC control reg > > Nit: could we bracket list entries individually please? > > Thanks, > Mark. > > _______________________________________________ - k -- Employee of Qualcomm Innovation Center, Inc. Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: galak@codeaurora.org (Kumar Gala) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 3/6] Documentation: DT: Add entry for FSL Management Complex Date: Fri, 15 Aug 2014 08:35:34 -0500 [thread overview] Message-ID: <21CA0EC7-3711-4B13-991B-940EF576D2EF@codeaurora.org> (raw) In-Reply-To: <20140815110020.GC15621@leverpostej> On Aug 15, 2014, at 6:00 AM, Mark Rutland <mark.rutland@arm.com> wrote: > [devicetree-discuss is no more, fixing up to devicetree at vger.kernel.org] > > Hi, > > On Fri, Aug 15, 2014 at 10:49:12AM +0100, Bhupesh Sharma wrote: >> This patch adds a devicetree binding documentation for FSL's >> Management Complex. >> >> Management Complex is a hardware resource manager that manages >> specialized hardware objects used in network-oriented packet >> processing applications >> >> Signed-off-by: Bhupesh Sharma <bhupesh.sharma@freescale.com> >> Signed-off-by: Stuart Yoder <stuart.yoder@freescale.com> >> Signed-off-by: J. German Rivera <German.Rivera@freescale.com> >> --- >> .../devicetree/bindings/misc/fsl,qoriq-mc.txt | 40 ++++++++++++++++++++ I?d probably start this off in bindings/soc/fsl/ >> 1 file changed, 40 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/misc/fsl,qoriq-mc.txt >> >> diff --git a/Documentation/devicetree/bindings/misc/fsl,qoriq-mc.txt b/Documentation/devicetree/bindings/misc/fsl,qoriq-mc.txt >> new file mode 100644 >> index 0000000..608529e >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/misc/fsl,qoriq-mc.txt >> @@ -0,0 +1,40 @@ >> +* Freescale Management Complex >> + >> +The Freescale Management Complex (fsl-mc) is a hardware resource >> +manager that manages specialized hardware objects used in >> +network-oriented packet processing applications. After the fsl-mc >> +block is enabled, pools of hardware resources are available, such as >> +queues, buffer pools, I/O interfaces. These resources are building >> +blocks that can be used to create functional hardware objects/devices >> +such as network interfaces, crypto accelerator instances, L2 switches, >> +etc. >> + >> +Required properties: >> + >> + - compatible >> + Value type: <string> >> + Definition: Must be "fsl,qoriq-mc". A Freescale Management Complex >> + compatible with this binding must have Block Revision >> + Registers BRR1 and BRR2 at offset 0x0BF8 and 0x0BFC in >> + the MC control register region. Hmm, this BRR1/BRR2 comments are a little odd, if we keep those, we should probably also state what value is BRR1 (I think that was the one that contented the IP ID). >> + >> + - reg >> + Value type: <prop-encoded-array> >> + Definition: A standard property. Specifies one or two regions >> + defining the MC's registers: >> + >> + -the first region is the command portal for the >> + this machine and must always be present >> + >> + -the second region is the MC control registers. This >> + region may not be present in some scenarios, such >> + as in the device tree presented to a virtual machine. Should we distinguish the second case w/a different compat? It seems like a major driver change if the second region isn?t there, or a different driver? > > This looks extremely simple. Is this unit self-contained or does it > relate to other blocks which will be described separately? > >> + >> +Example: >> + >> + fsl_mc: fsl-mc at 80c000000 { >> + compatible = "fsl,qoriq-mc"; >> + reg = <0x00000008 0x0c000000 0 0x40 // MC portal base >> + 0x00000000 0x08340000 0 0x40000 >; // MC control reg > > Nit: could we bracket list entries individually please? > > Thanks, > Mark. > > _______________________________________________ - k -- Employee of Qualcomm Innovation Center, Inc. Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation
next prev parent reply other threads:[~2014-08-15 13:35 UTC|newest] Thread overview: 46+ messages / expand[flat|nested] mbox.gz Atom feed top 2014-08-15 9:49 [PATCH 0/6] ARM64: Add support for FSL's LS2085A SoC Bhupesh Sharma 2014-08-15 9:49 ` [PATCH 1/6] Documentation: DT: Add bindings for FSL NS16550A UART Bhupesh Sharma 2014-08-15 10:46 ` Mark Rutland 2014-08-15 14:42 ` [PATCH 1/6] Documentation: DT: Add bindings for FSL NS16550A The UART bhupesh.sharma at freescale.com 2014-08-15 15:03 ` Mark Rutland 2014-08-20 9:08 ` bhupesh.sharma at freescale.com 2014-08-20 11:31 ` Mark Rutland 2014-08-20 12:20 ` bhupesh.sharma at freescale.com 2014-08-15 9:49 ` [PATCH 2/6] Documentation: DT: Add entry for FSL LS2085A SoC and Simulator model Bhupesh Sharma 2014-08-15 9:49 ` [PATCH 3/6] Documentation: DT: Add entry for FSL Management Complex Bhupesh Sharma [not found] ` <1408096156-29772-4-git-send-email-bhupesh.sharma-KZfg59tc24xl57MIdRCFDg@public.gmane.org> 2014-08-15 11:00 ` Mark Rutland 2014-08-15 11:00 ` Mark Rutland 2014-08-15 13:12 ` Stuart Yoder 2014-08-15 13:12 ` Stuart Yoder 2014-08-15 13:35 ` Kumar Gala [this message] 2014-08-15 13:35 ` Kumar Gala 2014-08-15 9:49 ` [PATCH 4/6] arm64: Add DTS support for FSL's LS2085A SoC Bhupesh Sharma 2014-08-15 10:12 ` Catalin Marinas 2014-08-15 12:53 ` Stuart Yoder 2014-08-15 13:10 ` Catalin Marinas 2014-08-15 14:31 ` bhupesh.sharma at freescale.com 2014-08-15 15:28 ` Catalin Marinas 2014-08-15 15:57 ` Stuart Yoder 2014-08-15 16:22 ` Mark Rutland 2014-08-15 16:25 ` Catalin Marinas 2014-08-15 16:44 ` Stuart Yoder 2014-08-15 10:23 ` Mark Rutland 2014-08-15 15:21 ` arnab.basu at freescale.com 2014-08-15 15:26 ` Kumar Gala [not found] ` <0DC5667C-E45A-4781-9AE6-0C5E1C70F268-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org> 2014-08-15 15:41 ` Stuart Yoder 2014-08-15 15:41 ` Stuart Yoder [not found] ` <9c8daab7efa446c884409ae3aaaaa151-swgC6WJTr6EFQ9CGjnlQrZwN6zqB+hSMnBOFsp37pqbUKgpGm//BTAC/G2K4zDHf@public.gmane.org> 2014-08-15 15:43 ` Kumar Gala 2014-08-15 15:43 ` Kumar Gala [not found] ` <3537CF6F-9BF9-4905-B051-EB94034FC1C2-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org> 2014-08-15 15:49 ` Stuart Yoder 2014-08-15 15:49 ` Stuart Yoder [not found] ` <77e80a2d69ee470a8458a85450a1510c-swgC6WJTr6EFQ9CGjnlQrZwN6zqB+hSMnBOFsp37pqbUKgpGm//BTAC/G2K4zDHf@public.gmane.org> 2014-08-15 16:02 ` Catalin Marinas 2014-08-15 16:02 ` Catalin Marinas 2014-08-15 15:37 ` Stuart Yoder 2014-08-15 16:12 ` Mark Rutland 2014-08-15 12:13 ` Marc Zyngier 2014-08-15 13:29 ` Kumar Gala 2014-08-15 14:26 ` bhupesh.sharma at freescale.com 2014-08-15 14:40 ` Kumar Gala 2014-08-15 16:19 ` Stuart Yoder 2014-08-15 9:49 ` [PATCH 5/6] arm64: dts/Makefile: Add support for FSL's LS2085A simulator model Bhupesh Sharma 2014-08-15 9:49 ` [PATCH 6/6] arm64: Add support for FSL's LS2085A SoC in Kconfig and defconfig Bhupesh Sharma
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=21CA0EC7-3711-4B13-991B-940EF576D2EF@codeaurora.org \ --to=galak-sgv2jx0feol9jmxxk+q4oq@public.gmane.org \ --cc=Catalin.Marinas-5wv7dgnIgG8@public.gmane.org \ --cc=German.Rivera-KZfg59tc24xl57MIdRCFDg@public.gmane.org \ --cc=Will.Deacon-5wv7dgnIgG8@public.gmane.org \ --cc=arnab.basu-KZfg59tc24xl57MIdRCFDg@public.gmane.org \ --cc=arnd-r2nGTMty4D4@public.gmane.org \ --cc=bhupesh.sharma-KZfg59tc24xl57MIdRCFDg@public.gmane.org \ --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org \ --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \ --cc=mark.rutland-5wv7dgnIgG8@public.gmane.org \ --cc=stuart.yoder-KZfg59tc24xl57MIdRCFDg@public.gmane.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: linkBe 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.