From: Pi-Hsun Shih <pihsun@chromium.org>
To: Rob Herring <robh@kernel.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: Mon, 25 Feb 2019 14:15:40 +0800 [thread overview]
Message-ID: <CANdKZ0d+2CqbVEwruW+pbqUeRMBwzXV7tFC7Cp7i+98CWiB_4w@mail.gmail.com> (raw)
In-Reply-To: <20190222144316.GA19284@bogus>
On Fri, Feb 22, 2019 at 10:43 PM Rob Herring <robh@kernel.org> wrote:
>
> 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.
>
What's the recommendation here if the subnode only has effect when the
config is enabled? Should I just skip the sentence "When ... is
enabled"?
> > +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.
>
This is similar to the qcom,smd-channels property in
Documentation/devicetree/bindings/soc/qcom/qcom,smd.txt, a example DT
for this:
scp {
compatible = "mediatek,mt8183-scp";
...
cros_ec {
compatible = "google,cros-ec-rpmsg";
mtk,rpmsg-name = "cros-ec-rpmsg";
cros_ec_codec {
compatible = "google,cros-ec-codec";
...
};
};
};
> > +
> > +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-25 6:16 UTC|newest]
Thread overview: 9+ 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 ` [PATCH v5 1/6] dt-bindings: Add a binding for Mediatek SCP Pi-Hsun Shih
2019-02-22 14:43 ` Rob Herring
2019-02-25 6:15 ` Pi-Hsun Shih [this message]
2019-03-04 17:51 ` Rob Herring
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 ` [PATCH v5 3/6] remoteproc: mt8183: add reserved memory manager API Pi-Hsun Shih
2019-02-21 8:47 ` [PATCH v5 4/6] rpmsg: add rpmsg support for mt8183 SCP 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=CANdKZ0d+2CqbVEwruW+pbqUeRMBwzXV7tFC7Cp7i+98CWiB_4w@mail.gmail.com \
--to=pihsun@chromium.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=robh@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 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).