From: Ahmad Fatoum <a.fatoum@pengutronix.de> To: Arnaud Pouliquen <arnaud.pouliquen@st.com>, Rob Herring <robh@kernel.org>, Alexandre Torgue <alexandre.torgue@st.com> Cc: Ohad Ben-Cohen <ohad@wizery.com>, devicetree@vger.kernel.org, Mathieu Poirier <mathieu.poirier@linaro.org>, linux-remoteproc@vger.kernel.org, linux-kernel@vger.kernel.org, Fabien Dessenne <fabien.dessenne@st.com>, Bjorn Andersson <bjorn.andersson@linaro.org>, linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org Subject: Re: [Linux-stm32] [PATCH 3/3] ARM: dts: stm32: update stm32mp151 for remote proc synchronisation support Date: Thu, 24 Sep 2020 07:45:11 +0200 [thread overview] Message-ID: <be888a4b-b931-521b-42c7-fd4e60afd945@pengutronix.de> (raw) In-Reply-To: <20200827072101.26588-4-arnaud.pouliquen@st.com> Hello Arnaud, On 8/27/20 9:21 AM, Arnaud Pouliquen wrote: > Two backup registers are used to store the Cortex-M4 state and the resource > table address. > Declare the tamp node and add associated properties in m4_rproc node > to allow Linux to attach to a firmware loaded by the first boot stages. > > Associated driver implementation is available in commit 9276536f455b3 > ("remoteproc: stm32: Parse syscon that will manage M4 synchronisation"). > > Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@st.com> > --- > arch/arm/boot/dts/stm32mp151.dtsi | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/arch/arm/boot/dts/stm32mp151.dtsi b/arch/arm/boot/dts/stm32mp151.dtsi > index bfe29023fbd5..842ecffae73a 100644 > --- a/arch/arm/boot/dts/stm32mp151.dtsi > +++ b/arch/arm/boot/dts/stm32mp151.dtsi > @@ -1541,6 +1541,11 @@ > status = "disabled"; > }; > > + tamp: tamp@5c00a000 { > + compatible = "st,stm32-tamp", "syscon"; > + reg = <0x5c00a000 0x400>; > + }; > + Just saw this now. I have a pending patch adding this node as well: https://lore.kernel.org/patchwork/patch/1306971/ For my use case, I need a "simple-mfd" compatible to allow child nodes to be probed. Could you CC me when you send out your v2, so I can rebase? (Or if you don't mind, just add the "simple-mfd" into the compatible list yourself :-) Cheers Ahmad > /* > * Break node order to solve dependency probe issue between > * pinctrl and exti. > @@ -1717,6 +1722,8 @@ > st,syscfg-holdboot = <&rcc 0x10C 0x1>; > st,syscfg-tz = <&rcc 0x000 0x1>; > st,syscfg-pdds = <&pwr_mcu 0x0 0x1>; > + st,syscfg-rsc-tbl = <&tamp 0x144 0xFFFFFFFF>; > + st,syscfg-m4-state = <&tamp 0x148 0xFFFFFFFF>; > status = "disabled"; > }; > }; > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
WARNING: multiple messages have this Message-ID (diff)
From: Ahmad Fatoum <a.fatoum@pengutronix.de> To: Arnaud Pouliquen <arnaud.pouliquen@st.com>, Rob Herring <robh@kernel.org>, Alexandre Torgue <alexandre.torgue@st.com> Cc: Ohad Ben-Cohen <ohad@wizery.com>, devicetree@vger.kernel.org, Mathieu Poirier <mathieu.poirier@linaro.org>, linux-remoteproc@vger.kernel.org, linux-kernel@vger.kernel.org, Fabien Dessenne <fabien.dessenne@st.com>, Bjorn Andersson <bjorn.andersson@linaro.org>, linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org Subject: Re: [Linux-stm32] [PATCH 3/3] ARM: dts: stm32: update stm32mp151 for remote proc synchronisation support Date: Thu, 24 Sep 2020 07:45:11 +0200 [thread overview] Message-ID: <be888a4b-b931-521b-42c7-fd4e60afd945@pengutronix.de> (raw) In-Reply-To: <20200827072101.26588-4-arnaud.pouliquen@st.com> Hello Arnaud, On 8/27/20 9:21 AM, Arnaud Pouliquen wrote: > Two backup registers are used to store the Cortex-M4 state and the resource > table address. > Declare the tamp node and add associated properties in m4_rproc node > to allow Linux to attach to a firmware loaded by the first boot stages. > > Associated driver implementation is available in commit 9276536f455b3 > ("remoteproc: stm32: Parse syscon that will manage M4 synchronisation"). > > Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@st.com> > --- > arch/arm/boot/dts/stm32mp151.dtsi | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/arch/arm/boot/dts/stm32mp151.dtsi b/arch/arm/boot/dts/stm32mp151.dtsi > index bfe29023fbd5..842ecffae73a 100644 > --- a/arch/arm/boot/dts/stm32mp151.dtsi > +++ b/arch/arm/boot/dts/stm32mp151.dtsi > @@ -1541,6 +1541,11 @@ > status = "disabled"; > }; > > + tamp: tamp@5c00a000 { > + compatible = "st,stm32-tamp", "syscon"; > + reg = <0x5c00a000 0x400>; > + }; > + Just saw this now. I have a pending patch adding this node as well: https://lore.kernel.org/patchwork/patch/1306971/ For my use case, I need a "simple-mfd" compatible to allow child nodes to be probed. Could you CC me when you send out your v2, so I can rebase? (Or if you don't mind, just add the "simple-mfd" into the compatible list yourself :-) Cheers Ahmad > /* > * Break node order to solve dependency probe issue between > * pinctrl and exti. > @@ -1717,6 +1722,8 @@ > st,syscfg-holdboot = <&rcc 0x10C 0x1>; > st,syscfg-tz = <&rcc 0x000 0x1>; > st,syscfg-pdds = <&pwr_mcu 0x0 0x1>; > + st,syscfg-rsc-tbl = <&tamp 0x144 0xFFFFFFFF>; > + st,syscfg-m4-state = <&tamp 0x148 0xFFFFFFFF>; > status = "disabled"; > }; > }; > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ 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:[~2020-09-24 5:45 UTC|newest] Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-08-27 7:20 [PATCH 0/3] ARM: stm32: add DT properties for remote proc synchronisation Arnaud Pouliquen 2020-08-27 7:20 ` Arnaud Pouliquen 2020-08-27 7:20 ` [PATCH 1/3] dt-bindings: arm: stm32: Add compatible for syscon tamp node Arnaud Pouliquen 2020-08-27 7:20 ` Arnaud Pouliquen 2020-09-09 20:19 ` Rob Herring 2020-09-09 20:19 ` Rob Herring 2020-08-27 7:21 ` [PATCH 2/3] dt-bindings: remoteproc: stm32_rproc: update for firmware synchronization Arnaud Pouliquen 2020-08-27 7:21 ` Arnaud Pouliquen 2020-09-09 20:22 ` Rob Herring 2020-09-09 20:22 ` Rob Herring 2020-09-11 13:49 ` Arnaud POULIQUEN 2020-09-11 13:49 ` Arnaud POULIQUEN 2020-09-23 14:30 ` Arnaud POULIQUEN 2020-09-23 14:30 ` Arnaud POULIQUEN 2020-08-27 7:21 ` [PATCH 3/3] ARM: dts: stm32: update stm32mp151 for remote proc synchronisation support Arnaud Pouliquen 2020-08-27 7:21 ` Arnaud Pouliquen 2020-09-24 5:45 ` Ahmad Fatoum [this message] 2020-09-24 5:45 ` [Linux-stm32] " Ahmad Fatoum 2020-09-24 8:02 ` Arnaud POULIQUEN 2020-09-24 8:02 ` Arnaud POULIQUEN
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=be888a4b-b931-521b-42c7-fd4e60afd945@pengutronix.de \ --to=a.fatoum@pengutronix.de \ --cc=alexandre.torgue@st.com \ --cc=arnaud.pouliquen@st.com \ --cc=bjorn.andersson@linaro.org \ --cc=devicetree@vger.kernel.org \ --cc=fabien.dessenne@st.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-remoteproc@vger.kernel.org \ --cc=linux-stm32@st-md-mailman.stormreply.com \ --cc=mathieu.poirier@linaro.org \ --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: 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.