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:
 <PAXPR04MB84593B109878D86B72DEB4AD88B22@PAXPR04MB8459.eurprd04.prod.outlook.com>
Date: Thu, 1 Aug 2024 02:27:38 +0000
From: Peng Fan <peng.fan@....com>
To: Yashwanth Varakala <y.varakala@...tec.de>, "shawnguo@...nel.org"
	<shawnguo@...nel.org>, "s.hauer@...gutronix.de" <s.hauer@...gutronix.de>,
	"kernel@...gutronix.de" <kernel@...gutronix.de>, "festevam@...il.com"
	<festevam@...il.com>, "robh@...nel.org" <robh@...nel.org>,
	"krzk+dt@...nel.org" <krzk+dt@...nel.org>, "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>, "upstream@...ts.phytec.de"
	<upstream@...ts.phytec.de>
Subject: RE: [PATCH 3/3] arm64: dts: Add phyBOARD-Pollux dts for rpmsg

> 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>;
> +		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.