linux-kernel - Re: [PATCH 3/3] arm64: dts: Add phyBOARD-Pollux dts for rpmsg

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <da0363d65457c1f1de26b9bc03117d667d6a578e.camel@phytec.de>
Date: Wed, 7 Aug 2024 10:34:08 +0000
From: Yashwanth Varakala <Y.Varakala@...tec.de>
To: "kernel@...gutronix.de" <kernel@...gutronix.de>, "s.hauer@...gutronix.de"
	<s.hauer@...gutronix.de>, "peng.fan@....com" <peng.fan@....com>,
	"robh@...nel.org" <robh@...nel.org>, "shawnguo@...nel.org"
	<shawnguo@...nel.org>, "krzk+dt@...nel.org" <krzk+dt@...nel.org>,
	"festevam@...il.com" <festevam@...il.com>, "conor+dt@...nel.org"
	<conor+dt@...nel.org>
CC: "imx@...ts.linux.dev" <imx@...ts.linux.dev>,
	"linux-arm-kernel@...ts.infradead.org"
	<linux-arm-kernel@...ts.infradead.org>, "devicetree@...r.kernel.org"
	<devicetree@...r.kernel.org>, "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>, PHYTEC Upstream <upstream@...ts.phytec.de>
Subject: Re: [PATCH 3/3] arm64: dts: Add phyBOARD-Pollux dts for rpmsg

Hello Peng,

On Thu, 2024-08-01 at 02:27 +0000, Peng Fan wrote:
> > Subject: [PATCH 3/3] arm64: dts: Add phyBOARD-Pollux dts for rpmsg
> > 
> > Adds a devicetree containing reserved memory regions used for
> > intercore communication between A53 and M7 cores.
> > 
> > Signed-off-by: Yashwanth Varakala <y.varakala@...tec.de>
> > ---
> >  arch/arm64/boot/dts/freescale/Makefile        |  2 +
> >  .../dts/freescale/imx8mp-phycore-rpmsg.dtso   | 57
> > +++++++++++++++++++
> >  2 files changed, 59 insertions(+)
> >  create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-
> > phycore-rpmsg.dtso
> > 
> > diff --git a/arch/arm64/boot/dts/freescale/Makefile
> > b/arch/arm64/boot/dts/freescale/Makefile
> > index dedea4b5c319..80cc87d50301 100644
> > --- a/arch/arm64/boot/dts/freescale/Makefile
> > +++ b/arch/arm64/boot/dts/freescale/Makefile
> > @@ -177,9 +177,11 @@ dtb-$(CONFIG_ARCH_MXC) += imx8mp-
> > phyboard-pollux-rdk.dtb  imx8mp-phyboard-pollux-rdk-no-eth-dtbs +=
> > imx8mp-phyboard-pollux-rdk.dtb imx8mp-phycore-no-eth.dtbo
> > imx8mp-phyboard-pollux-rdk-no-rtc-dtbs += imx8mp-phyboard-pollux-
> > rdk.dtb imx8mp-phycore-no-rtc.dtbo  imx8mp-phyboard-pollux-rdk-no-
> > spiflash-dtbs += imx8mp-phyboard-pollux-rdk.dtb imx8mp-phycore-no-
> > spiflash.dtbo
> > +imx8mp-phyboard-pollux-rdk-rpmsg-dtbs += imx8mp-phyboard-
> > pollux-rdk.dtb
> > +imx8mp-phycore-rpmsg.dtbo
> >  dtb-$(CONFIG_ARCH_MXC) += imx8mp-phyboard-pollux-rdk-no-
> > eth.dtb
> >  dtb-$(CONFIG_ARCH_MXC) += imx8mp-phyboard-pollux-rdk-no-rtc.dtb
> >  dtb-$(CONFIG_ARCH_MXC) += imx8mp-phyboard-pollux-rdk-no-
> > spiflash.dtb
> > +dtb-$(CONFIG_ARCH_MXC) += imx8mp-phyboard-pollux-rdk-
> > rpmsg.dtb
> >  dtb-$(CONFIG_ARCH_MXC) += imx8mp-skov-revb-hdmi.dtb
> >  dtb-$(CONFIG_ARCH_MXC) += imx8mp-skov-revb-lt6.dtb
> >  dtb-$(CONFIG_ARCH_MXC) += imx8mp-skov-revb-mi1010ait-1cp1.dtb
> > diff --git a/arch/arm64/boot/dts/freescale/imx8mp-phycore-
> > rpmsg.dtso b/arch/arm64/boot/dts/freescale/imx8mp-phycore-
> > rpmsg.dtso
> > new file mode 100644
> > index 000000000000..a5694f3aecaa
> > --- /dev/null
> > +++ b/arch/arm64/boot/dts/freescale/imx8mp-phycore-rpmsg.dtso
> > @@ -0,0 +1,57 @@
> > +// SPDX-License-Identifier: GPL-2.0
> 
> Dual license is better device tree.
> 
> > +/*
> > + * Copyright (C) 2024 PHYTEC Messtechnik GmbH
> > + * Author: Dominik Haller <d.haller@...tec.de>
> > + *        Cem Tenruh <c.tenruh@...tec.de>
> > + */
> > +
> > +/dts-v1/;
> > +/plugin/;
> > +
> > +#include <dt-bindings/clock/imx8mp-clock.h>
> > +
> > +&{/} {
> > +       imx8mp-cm7 {
> > +               compatible = "fsl,imx8mn-cm7";
> > +               clocks = <&clk IMX8MP_CLK_M7_DIV>;
> > +               mboxes = <&mu 0 1
> > +                       &mu 1 1
> > +                       &mu 3 1>;
> 
>         mboxes = <&mu1 0
> 1>,                                                                  
>       
>                           <&mu1 1
> 1>,                                                                  
>       
>                           <&mu1 3 1>;
Thank you for the feedback. I checked and found only mu and mu2 labels
of mailboxes are present. mu1 is not used in imx8mp.dtsi. Can you
please tell me why I have to use mu1 here?

> 
> > +               mbox-names = "tx", "rx", "rxdb";
> > +               memory-region = <&vdevbuffer>, <&vdev0vring0>,
> > <&vdev0vring1>, <&rsc_table>;
> > +               rsc-da = <0x55000000>;
> 
> Drop this "rsc-da".
> 
> > +               status = "okay";
> 
> Drop it, default is "okay".
> 
> > +       };
> > +
> > +       reserved-memory {
> > +               ranges;
> > +               #address-cells = <2>;
> > +               #size-cells = <2>;
> > +
> > +               m7_reserved: m7@...0000000 {
> > +                       no-map;
> > +                       reg = <0 0x80000000 0 0x1000000>;
> > +               };
> > +
> > +               rsc_table: rsc_table@...ff000 {
> 
> "rsc-table@...ff000". Seems you not test this patch with linux
> remoteproc, otherwise you will see vring not set up.
> 
> Regards,
> Peng.

-- 
-
Regards,
Yashwanth