From: Rob Herring <robh@kernel.org> To: Pi-Hsun Shih <pihsun@chromium.org> Cc: Erin Lo <erin.lo@mediatek.com>, Ohad Ben-Cohen <ohad@wizery.com>, Bjorn Andersson <bjorn.andersson@linaro.org>, Mark Rutland <mark.rutland@arm.com>, Matthias Brugger <matthias.bgg@gmail.com>, "open list:REMOTE PROCESSOR (REMOTEPROC) SUBSYSTEM" <linux-remoteproc@vger.kernel.org>, "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" <devicetree@vger.kernel.org>, "moderated list:ARM/Mediatek SoC support" <linux-arm-kernel@lists.infradead.org>, "moderated list:ARM/Mediatek SoC support" <linux-mediatek@lists.infradead.org>, open list <linux-kernel@vger.kernel.org> Subject: Re: [PATCH v5 1/6] dt-bindings: Add a binding for Mediatek SCP Date: Fri, 22 Feb 2019 08:43:16 -0600 [thread overview] Message-ID: <20190222144316.GA19284@bogus> (raw) In-Reply-To: <20190221084729.101784-2-pihsun@chromium.org> On Thu, Feb 21, 2019 at 04:47:24PM +0800, Pi-Hsun Shih wrote: > From: Erin Lo <erin.lo@mediatek.com> > > Add a DT binding documentation of SCP for the > MT8183 SoC from Mediatek. > > Signed-off-by: Erin Lo <erin.lo@mediatek.com> > --- > Changes from v4: > - Add detail of more properties. > - Document the usage of mtk,rpmsg-name in subnode from the new design. > > Changes from v3: > - No change. > > Changes from v2: > - No change. I realized that for this patch series, there's no need to > add anything under the mt8183-scp node (neither the mt8183-rpmsg or > the cros-ec-rpmsg) for them to work, since mt8183-rpmsg is added > directly as a rproc_subdev by code, and cros-ec-rpmsg is dynamically > created by SCP name service. > > Changes from v1: > - No change. > --- > .../bindings/remoteproc/mtk,scp.txt | 37 +++++++++++++++++++ > 1 file changed, 37 insertions(+) > create mode 100644 Documentation/devicetree/bindings/remoteproc/mtk,scp.txt > > diff --git a/Documentation/devicetree/bindings/remoteproc/mtk,scp.txt b/Documentation/devicetree/bindings/remoteproc/mtk,scp.txt > new file mode 100644 > index 00000000000000..8cf8b0e0d98a4c > --- /dev/null > +++ b/Documentation/devicetree/bindings/remoteproc/mtk,scp.txt > @@ -0,0 +1,37 @@ > +Mediatek SCP Bindings > +---------------------------------------- > + > +This binding provides support for ARM Cortex M4 Co-processor found on some > +Mediatek SoCs. > + > +Required properties: > +- compatible Should be "mediatek,mt8183-scp" > +- reg Should contain the address ranges for the two memory > + regions, SRAM and CFG. > +- reg-names Contains the corresponding names for the two memory > + regions. These should be named "sram" & "cfg". > +- clocks Clock for co-processor (See: ../clock/clock-bindings.txt) > +- clock-names Contains the corresponding name for the clock. This > + should be named "main". > + > +Subnodes > +-------- > + > +When CONFIG_RPMSG_MTK_SCP is enabled, subnodes of the SCP represent rpmsg Bindings can't depend on kernel config options. > +devices. The names of the devices are not important. The properties of these > +nodes are defined by the individual bindings for the rpmsg devices - but must > +contain the following property: > + > +- mtk,rpmsg-name Contains the name for the rpmsg device. Used to match > + the subnode to rpmsg device announced by SCP. I don't think this belongs in DT, but without some examples I'm not really sure. > + > +Example: > + > + scp: scp@10500000 { > + compatible = "mediatek,mt8183-scp"; > + reg = <0 0x10500000 0 0x80000>, > + <0 0x105c0000 0 0x5000>; > + reg-names = "sram", "cfg"; > + clocks = <&infracfg CLK_INFRA_SCPSYS>; > + clock-names = "main"; > + }; > -- > 2.21.0.rc0.258.g878e2cd30e-goog >
WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh@kernel.org> To: Pi-Hsun Shih <pihsun@chromium.org> Cc: Ohad Ben-Cohen <ohad@wizery.com>, Mark Rutland <mark.rutland@arm.com>, "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" <devicetree@vger.kernel.org>, Erin Lo <erin.lo@mediatek.com>, "open list:REMOTE PROCESSOR \(REMOTEPROC\) SUBSYSTEM" <linux-remoteproc@vger.kernel.org>, open list <linux-kernel@vger.kernel.org>, Bjorn Andersson <bjorn.andersson@linaro.org>, "moderated list:ARM/Mediatek SoC support" <linux-mediatek@lists.infradead.org>, Matthias Brugger <matthias.bgg@gmail.com>, "moderated list:ARM/Mediatek SoC support" <linux-arm-kernel@lists.infradead.org> Subject: Re: [PATCH v5 1/6] dt-bindings: Add a binding for Mediatek SCP Date: Fri, 22 Feb 2019 08:43:16 -0600 [thread overview] Message-ID: <20190222144316.GA19284@bogus> (raw) In-Reply-To: <20190221084729.101784-2-pihsun@chromium.org> On Thu, Feb 21, 2019 at 04:47:24PM +0800, Pi-Hsun Shih wrote: > From: Erin Lo <erin.lo@mediatek.com> > > Add a DT binding documentation of SCP for the > MT8183 SoC from Mediatek. > > Signed-off-by: Erin Lo <erin.lo@mediatek.com> > --- > Changes from v4: > - Add detail of more properties. > - Document the usage of mtk,rpmsg-name in subnode from the new design. > > Changes from v3: > - No change. > > Changes from v2: > - No change. I realized that for this patch series, there's no need to > add anything under the mt8183-scp node (neither the mt8183-rpmsg or > the cros-ec-rpmsg) for them to work, since mt8183-rpmsg is added > directly as a rproc_subdev by code, and cros-ec-rpmsg is dynamically > created by SCP name service. > > Changes from v1: > - No change. > --- > .../bindings/remoteproc/mtk,scp.txt | 37 +++++++++++++++++++ > 1 file changed, 37 insertions(+) > create mode 100644 Documentation/devicetree/bindings/remoteproc/mtk,scp.txt > > diff --git a/Documentation/devicetree/bindings/remoteproc/mtk,scp.txt b/Documentation/devicetree/bindings/remoteproc/mtk,scp.txt > new file mode 100644 > index 00000000000000..8cf8b0e0d98a4c > --- /dev/null > +++ b/Documentation/devicetree/bindings/remoteproc/mtk,scp.txt > @@ -0,0 +1,37 @@ > +Mediatek SCP Bindings > +---------------------------------------- > + > +This binding provides support for ARM Cortex M4 Co-processor found on some > +Mediatek SoCs. > + > +Required properties: > +- compatible Should be "mediatek,mt8183-scp" > +- reg Should contain the address ranges for the two memory > + regions, SRAM and CFG. > +- reg-names Contains the corresponding names for the two memory > + regions. These should be named "sram" & "cfg". > +- clocks Clock for co-processor (See: ../clock/clock-bindings.txt) > +- clock-names Contains the corresponding name for the clock. This > + should be named "main". > + > +Subnodes > +-------- > + > +When CONFIG_RPMSG_MTK_SCP is enabled, subnodes of the SCP represent rpmsg Bindings can't depend on kernel config options. > +devices. The names of the devices are not important. The properties of these > +nodes are defined by the individual bindings for the rpmsg devices - but must > +contain the following property: > + > +- mtk,rpmsg-name Contains the name for the rpmsg device. Used to match > + the subnode to rpmsg device announced by SCP. I don't think this belongs in DT, but without some examples I'm not really sure. > + > +Example: > + > + scp: scp@10500000 { > + compatible = "mediatek,mt8183-scp"; > + reg = <0 0x10500000 0 0x80000>, > + <0 0x105c0000 0 0x5000>; > + reg-names = "sram", "cfg"; > + clocks = <&infracfg CLK_INFRA_SCPSYS>; > + clock-names = "main"; > + }; > -- > 2.21.0.rc0.258.g878e2cd30e-goog > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2019-02-22 14:43 UTC|newest] Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-02-21 8:47 [PATCH v5 0/6] Add support for mt8183 SCP Pi-Hsun Shih 2019-02-21 8:47 ` Pi-Hsun Shih 2019-02-21 8:47 ` [PATCH v5 1/6] dt-bindings: Add a binding for Mediatek SCP Pi-Hsun Shih 2019-02-21 8:47 ` Pi-Hsun Shih 2019-02-21 8:47 ` Pi-Hsun Shih 2019-02-22 14:43 ` Rob Herring [this message] 2019-02-22 14:43 ` Rob Herring 2019-02-22 14:43 ` Rob Herring 2019-02-25 6:15 ` Pi-Hsun Shih 2019-02-25 6:15 ` Pi-Hsun Shih 2019-02-25 6:15 ` Pi-Hsun Shih 2019-03-04 17:51 ` Rob Herring 2019-03-04 17:51 ` Rob Herring 2019-03-04 17:51 ` Rob Herring 2019-03-05 3:53 ` Pi-Hsun Shih 2019-03-05 3:53 ` Pi-Hsun Shih 2019-03-05 3:53 ` Pi-Hsun Shih 2019-02-21 8:47 ` [PATCH v5 2/6] remoteproc/mediatek: add SCP support for mt8183 Pi-Hsun Shih 2019-02-21 8:47 ` Pi-Hsun Shih 2019-02-21 8:47 ` Pi-Hsun Shih 2019-02-21 8:47 ` [PATCH v5 3/6] remoteproc: mt8183: add reserved memory manager API Pi-Hsun Shih 2019-02-21 8:47 ` Pi-Hsun Shih 2019-02-21 8:47 ` Pi-Hsun Shih 2019-02-21 8:47 ` [PATCH v5 4/6] rpmsg: add rpmsg support for mt8183 SCP Pi-Hsun Shih 2019-02-21 8:47 ` Pi-Hsun Shih 2019-02-21 8:47 ` Pi-Hsun Shih 2019-02-21 8:47 ` [PATCH v5 5/6] mfd: add EC host command support using rpmsg Pi-Hsun Shih 2019-02-22 12:05 ` Enric Balletbo i Serra 2019-02-21 8:47 ` [PATCH v5 6/6] cros_ec: differentiate SCP from EC by feature bit Pi-Hsun Shih
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=20190222144316.GA19284@bogus \ --to=robh@kernel.org \ --cc=bjorn.andersson@linaro.org \ --cc=devicetree@vger.kernel.org \ --cc=erin.lo@mediatek.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mediatek@lists.infradead.org \ --cc=linux-remoteproc@vger.kernel.org \ --cc=mark.rutland@arm.com \ --cc=matthias.bgg@gmail.com \ --cc=ohad@wizery.com \ --cc=pihsun@chromium.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.