Re: [PATCH 2/5] ARM: dts: stm32: Add Engicam MicroGEA STM32MP1 SoM
On Fri, Feb 26, 2021 at 8:39 PM Alexandre TORGUE wrote: > > Hi Jagan > > On 2/24/21 7:05 PM, Jagan Teki wrote: > > On Mon, Jan 25, 2021 at 8:35 PM Alexandre TORGUE > > wrote: > >> > >> Hi, > >> > >> On 12/23/20 8:13 PM, Jagan Teki wrote: > >>> MicroGEA STM32MP1 is an EDIMM SoM based on STM32MP157A from Engicam. > >>> > >>> General features: > >>> - STM32MP157AAC > >>> - Up to 1GB DDR3L-800 > >>> - 512MB Nand flash > >>> - I2S > >>> > >>> MicroGEA STM32MP1 needs to mount on top of Engicam MicroDev carrier > >>> boards for creating complete platform solutions. > >>> > >>> Add support for it. > >>> > >>> Signed-off-by: Matteo Lisi > >>> Signed-off-by: Francesco Utel > >>> Signed-off-by: Mirko Ardinghi > >>> Signed-off-by: Jagan Teki > >>> --- > >>>.../dts/stm32mp157a-microgea-stm32mp1.dtsi| 147 ++ > >>>1 file changed, 147 insertions(+) > >>>create mode 100644 arch/arm/boot/dts/stm32mp157a-microgea-stm32mp1.dtsi > >>> > >>> diff --git a/arch/arm/boot/dts/stm32mp157a-microgea-stm32mp1.dtsi > >>> b/arch/arm/boot/dts/stm32mp157a-microgea-stm32mp1.dtsi > >>> new file mode 100644 > >>> index ..97d569107bfe > >>> --- /dev/null > >>> +++ b/arch/arm/boot/dts/stm32mp157a-microgea-stm32mp1.dtsi > >>> @@ -0,0 +1,147 @@ > >>> +// SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause) > >>> +/* > >>> + * Copyright (c) STMicroelectronics 2019 - All Rights Reserved > >>> + * Copyright (c) 2020 Engicam srl > >>> + * Copyright (c) 2020 Amarula Solutons(India) > >>> + */ > >>> + > >> > >> If STM32MP157AAC is soldered onto this board, you should include SoC > >> dtsi here and no into MicroDev 2.0 board. No ? > > > > No, it's an SoM dtsi. and it can be associated with the respective > > carrier board and include files are attached in its dts. > > > > Please check the next patches on this series. > Ok, I thought it was a mistake as you mentioned "- STM32MP157AAC" in > your commit message. Please check, v2 for this series. thanks! Jagan.
Re: [PATCH 2/5] ARM: dts: stm32: Add Engicam MicroGEA STM32MP1 SoM
Hi Jagan On 2/24/21 7:05 PM, Jagan Teki wrote: On Mon, Jan 25, 2021 at 8:35 PM Alexandre TORGUE wrote: Hi, On 12/23/20 8:13 PM, Jagan Teki wrote: MicroGEA STM32MP1 is an EDIMM SoM based on STM32MP157A from Engicam. General features: - STM32MP157AAC - Up to 1GB DDR3L-800 - 512MB Nand flash - I2S MicroGEA STM32MP1 needs to mount on top of Engicam MicroDev carrier boards for creating complete platform solutions. Add support for it. Signed-off-by: Matteo Lisi Signed-off-by: Francesco Utel Signed-off-by: Mirko Ardinghi Signed-off-by: Jagan Teki --- .../dts/stm32mp157a-microgea-stm32mp1.dtsi| 147 ++ 1 file changed, 147 insertions(+) create mode 100644 arch/arm/boot/dts/stm32mp157a-microgea-stm32mp1.dtsi diff --git a/arch/arm/boot/dts/stm32mp157a-microgea-stm32mp1.dtsi b/arch/arm/boot/dts/stm32mp157a-microgea-stm32mp1.dtsi new file mode 100644 index ..97d569107bfe --- /dev/null +++ b/arch/arm/boot/dts/stm32mp157a-microgea-stm32mp1.dtsi @@ -0,0 +1,147 @@ +// SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause) +/* + * Copyright (c) STMicroelectronics 2019 - All Rights Reserved + * Copyright (c) 2020 Engicam srl + * Copyright (c) 2020 Amarula Solutons(India) + */ + If STM32MP157AAC is soldered onto this board, you should include SoC dtsi here and no into MicroDev 2.0 board. No ? No, it's an SoM dtsi. and it can be associated with the respective carrier board and include files are attached in its dts. Please check the next patches on this series. Ok, I thought it was a mistake as you mentioned "- STM32MP157AAC" in your commit message. regards alex Jagan.
Re: [PATCH 2/5] ARM: dts: stm32: Add Engicam MicroGEA STM32MP1 SoM
On Mon, Jan 25, 2021 at 8:35 PM Alexandre TORGUE wrote: > > Hi, > > On 12/23/20 8:13 PM, Jagan Teki wrote: > > MicroGEA STM32MP1 is an EDIMM SoM based on STM32MP157A from Engicam. > > > > General features: > > - STM32MP157AAC > > - Up to 1GB DDR3L-800 > > - 512MB Nand flash > > - I2S > > > > MicroGEA STM32MP1 needs to mount on top of Engicam MicroDev carrier > > boards for creating complete platform solutions. > > > > Add support for it. > > > > Signed-off-by: Matteo Lisi > > Signed-off-by: Francesco Utel > > Signed-off-by: Mirko Ardinghi > > Signed-off-by: Jagan Teki > > --- > > .../dts/stm32mp157a-microgea-stm32mp1.dtsi| 147 ++ > > 1 file changed, 147 insertions(+) > > create mode 100644 arch/arm/boot/dts/stm32mp157a-microgea-stm32mp1.dtsi > > > > diff --git a/arch/arm/boot/dts/stm32mp157a-microgea-stm32mp1.dtsi > > b/arch/arm/boot/dts/stm32mp157a-microgea-stm32mp1.dtsi > > new file mode 100644 > > index ..97d569107bfe > > --- /dev/null > > +++ b/arch/arm/boot/dts/stm32mp157a-microgea-stm32mp1.dtsi > > @@ -0,0 +1,147 @@ > > +// SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause) > > +/* > > + * Copyright (c) STMicroelectronics 2019 - All Rights Reserved > > + * Copyright (c) 2020 Engicam srl > > + * Copyright (c) 2020 Amarula Solutons(India) > > + */ > > + > > If STM32MP157AAC is soldered onto this board, you should include SoC > dtsi here and no into MicroDev 2.0 board. No ? No, it's an SoM dtsi. and it can be associated with the respective carrier board and include files are attached in its dts. Please check the next patches on this series. Jagan.
Re: [PATCH 2/5] ARM: dts: stm32: Add Engicam MicroGEA STM32MP1 SoM
On 1/25/21 4:05 PM, Alexandre TORGUE wrote:
Hi,
On 12/23/20 8:13 PM, Jagan Teki wrote:
MicroGEA STM32MP1 is an EDIMM SoM based on STM32MP157A from Engicam.
General features:
- STM32MP157AAC
- Up to 1GB DDR3L-800
- 512MB Nand flash
- I2S
MicroGEA STM32MP1 needs to mount on top of Engicam MicroDev carrier
boards for creating complete platform solutions.
Add support for it.
Signed-off-by: Matteo Lisi
Signed-off-by: Francesco Utel
Signed-off-by: Mirko Ardinghi
Signed-off-by: Jagan Teki
---
.../dts/stm32mp157a-microgea-stm32mp1.dtsi | 147 ++
1 file changed, 147 insertions(+)
create mode 100644 arch/arm/boot/dts/stm32mp157a-microgea-stm32mp1.dtsi
diff --git a/arch/arm/boot/dts/stm32mp157a-microgea-stm32mp1.dtsi
b/arch/arm/boot/dts/stm32mp157a-microgea-stm32mp1.dtsi
new file mode 100644
index ..97d569107bfe
--- /dev/null
+++ b/arch/arm/boot/dts/stm32mp157a-microgea-stm32mp1.dtsi
@@ -0,0 +1,147 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause)
+/*
+ * Copyright (c) STMicroelectronics 2019 - All Rights Reserved
+ * Copyright (c) 2020 Engicam srl
+ * Copyright (c) 2020 Amarula Solutons(India)
+ */
+
If STM32MP157AAC is soldered onto this board, you should include SoC
dtsi here and no into MicroDev 2.0 board. No ?
+#include "stm32mp15-pinctrl.dtsi"
+#include "stm32mp15xxaa-pinctrl.dtsi"
and this one:
+#include "stm32mp157.dtsi"
Regards
Alex
+/ {
+ compatible = "engicam,microgea-stm32mp1", "st,stm32mp157";
+
+ memory@c000 {
+ reg = <0xc000 0x1000>;
+ };
+
+ reserved-memory {
+ #address-cells = <1>;
+ #size-cells = <1>;
+ ranges;
+
+ mcuram2: mcuram2@1000 {
+ compatible = "shared-dma-pool";
+ reg = <0x1000 0x4>;
+ no-map;
+ };
+
+ vdev0vring0: vdev0vring0@1004 {
+ compatible = "shared-dma-pool";
+ reg = <0x1004 0x1000>;
+ no-map;
+ };
+
+ vdev0vring1: vdev0vring1@10041000 {
+ compatible = "shared-dma-pool";
+ reg = <0x10041000 0x1000>;
+ no-map;
+ };
+
+ vdev0buffer: vdev0buffer@10042000 {
+ compatible = "shared-dma-pool";
+ reg = <0x10042000 0x4000>;
+ no-map;
+ };
+
+ mcuram: mcuram@3000 {
+ compatible = "shared-dma-pool";
+ reg = <0x3000 0x4>;
+ no-map;
+ };
+
+ retram: retram@3800 {
+ compatible = "shared-dma-pool";
+ reg = <0x3800 0x1>;
+ no-map;
+ };
+ };
+
+ vin: regulator-vin {
+ compatible = "regulator-fixed";
+ regulator-name = "vin";
+ regulator-min-microvolt = <500>;
+ regulator-max-microvolt = <500>;
+ regulator-always-on;
+ };
+
+ vddcore: regulator-vddcore {
+ compatible = "regulator-fixed";
+ regulator-name = "vddcore";
+ regulator-min-microvolt = <120>;
+ regulator-max-microvolt = <120>;
+ regulator-always-on;
+ vin-supply = <>;
+ };
+
+ vdd: regulator-vdd {
+ compatible = "regulator-fixed";
+ regulator-name = "vdd";
+ regulator-min-microvolt = <330>;
+ regulator-max-microvolt = <330>;
+ regulator-always-on;
+ vin-supply = <>;
+ };
+
+ vddq_ddr: regulator-vddq-ddr {
+ compatible = "regulator-fixed";
+ regulator-name = "vddq_ddr";
+ regulator-min-microvolt = <135>;
+ regulator-max-microvolt = <135>;
+ regulator-always-on;
+ vin-supply = <>;
+ };
+};
+
+ {
+ status = "okay";
+};
+
+ {
+ pinctrl-names = "default", "sleep";
+ pinctrl-0 = <_pins_a>;
+ pinctrl-1 = <_sleep_pins_a>;
+ status = "okay";
+
+ nand-controller@4,0 {
+ status = "okay";
+
+ nand@0 {
+ reg = <0>;
+ nand-on-flash-bbt;
+ #address-cells = <1>;
+ #size-cells = <1>;
+ };
+ };
+};
+
+ {
+ status = "okay";
+};
+
+{
+ timeout-sec = <32>;
+ status = "okay";
+};
+
+_rproc{
+ memory-region = <>, <>, <>, <>,
+ <>, <>;
+ mboxes = < 0>, < 1>, < 2>;
+ mbox-names = "vq0", "vq1", "shutdown";
+ interrupt-parent = <>;
+ interrupts = <68 1>;
+ status = "okay";
+};
+
+ {
+ status = "okay";
+};
+
+{
+ status = "okay";
+};
+
+ {
+ regulator-min-microvolt = <250>;
+ regulator-max-microvolt = <250>;
+ vdda-supply = <>;
+ status = "okay";
+};
Re: [PATCH 2/5] ARM: dts: stm32: Add Engicam MicroGEA STM32MP1 SoM
Hi,
On 12/23/20 8:13 PM, Jagan Teki wrote:
MicroGEA STM32MP1 is an EDIMM SoM based on STM32MP157A from Engicam.
General features:
- STM32MP157AAC
- Up to 1GB DDR3L-800
- 512MB Nand flash
- I2S
MicroGEA STM32MP1 needs to mount on top of Engicam MicroDev carrier
boards for creating complete platform solutions.
Add support for it.
Signed-off-by: Matteo Lisi
Signed-off-by: Francesco Utel
Signed-off-by: Mirko Ardinghi
Signed-off-by: Jagan Teki
---
.../dts/stm32mp157a-microgea-stm32mp1.dtsi| 147 ++
1 file changed, 147 insertions(+)
create mode 100644 arch/arm/boot/dts/stm32mp157a-microgea-stm32mp1.dtsi
diff --git a/arch/arm/boot/dts/stm32mp157a-microgea-stm32mp1.dtsi
b/arch/arm/boot/dts/stm32mp157a-microgea-stm32mp1.dtsi
new file mode 100644
index ..97d569107bfe
--- /dev/null
+++ b/arch/arm/boot/dts/stm32mp157a-microgea-stm32mp1.dtsi
@@ -0,0 +1,147 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause)
+/*
+ * Copyright (c) STMicroelectronics 2019 - All Rights Reserved
+ * Copyright (c) 2020 Engicam srl
+ * Copyright (c) 2020 Amarula Solutons(India)
+ */
+
If STM32MP157AAC is soldered onto this board, you should include SoC
dtsi here and no into MicroDev 2.0 board. No ?
+#include "stm32mp15-pinctrl.dtsi"
+#include "stm32mp15xxaa-pinctrl.dtsi"
Regards
Alex
+/ {
+ compatible = "engicam,microgea-stm32mp1", "st,stm32mp157";
+
+ memory@c000 {
+ reg = <0xc000 0x1000>;
+ };
+
+ reserved-memory {
+ #address-cells = <1>;
+ #size-cells = <1>;
+ ranges;
+
+ mcuram2: mcuram2@1000 {
+ compatible = "shared-dma-pool";
+ reg = <0x1000 0x4>;
+ no-map;
+ };
+
+ vdev0vring0: vdev0vring0@1004 {
+ compatible = "shared-dma-pool";
+ reg = <0x1004 0x1000>;
+ no-map;
+ };
+
+ vdev0vring1: vdev0vring1@10041000 {
+ compatible = "shared-dma-pool";
+ reg = <0x10041000 0x1000>;
+ no-map;
+ };
+
+ vdev0buffer: vdev0buffer@10042000 {
+ compatible = "shared-dma-pool";
+ reg = <0x10042000 0x4000>;
+ no-map;
+ };
+
+ mcuram: mcuram@3000 {
+ compatible = "shared-dma-pool";
+ reg = <0x3000 0x4>;
+ no-map;
+ };
+
+ retram: retram@3800 {
+ compatible = "shared-dma-pool";
+ reg = <0x3800 0x1>;
+ no-map;
+ };
+ };
+
+ vin: regulator-vin {
+ compatible = "regulator-fixed";
+ regulator-name = "vin";
+ regulator-min-microvolt = <500>;
+ regulator-max-microvolt = <500>;
+ regulator-always-on;
+ };
+
+ vddcore: regulator-vddcore {
+ compatible = "regulator-fixed";
+ regulator-name = "vddcore";
+ regulator-min-microvolt = <120>;
+ regulator-max-microvolt = <120>;
+ regulator-always-on;
+ vin-supply = <>;
+ };
+
+ vdd: regulator-vdd {
+ compatible = "regulator-fixed";
+ regulator-name = "vdd";
+ regulator-min-microvolt = <330>;
+ regulator-max-microvolt = <330>;
+ regulator-always-on;
+ vin-supply = <>;
+ };
+
+ vddq_ddr: regulator-vddq-ddr {
+ compatible = "regulator-fixed";
+ regulator-name = "vddq_ddr";
+ regulator-min-microvolt = <135>;
+ regulator-max-microvolt = <135>;
+ regulator-always-on;
+ vin-supply = <>;
+ };
+};
+
+ {
+ status = "okay";
+};
+
+ {
+ pinctrl-names = "default", "sleep";
+ pinctrl-0 = <_pins_a>;
+ pinctrl-1 = <_sleep_pins_a>;
+ status = "okay";
+
+ nand-controller@4,0 {
+ status = "okay";
+
+ nand@0 {
+ reg = <0>;
+ nand-on-flash-bbt;
+ #address-cells = <1>;
+ #size-cells = <1>;
+ };
+ };
+};
+
+ {
+ status = "okay";
+};
+
+{
+ timeout-sec = <32>;
+ status = "okay";
+};
+
+_rproc{
+ memory-region = <>, <>, <>, <>,
+ <>, <>;
+ mboxes = < 0>, < 1>, < 2>;
+ mbox-names = "vq0", "vq1", "shutdown";
+ interrupt-parent = <>;
+ interrupts = <68 1>;
+ status = "okay";
+};
+
+ {
+ status = "okay";
+};
+
+{
+ status = "okay";
+};
+
+ {
+
[PATCH 2/5] ARM: dts: stm32: Add Engicam MicroGEA STM32MP1 SoM
MicroGEA STM32MP1 is an EDIMM SoM based on STM32MP157A from Engicam.
General features:
- STM32MP157AAC
- Up to 1GB DDR3L-800
- 512MB Nand flash
- I2S
MicroGEA STM32MP1 needs to mount on top of Engicam MicroDev carrier
boards for creating complete platform solutions.
Add support for it.
Signed-off-by: Matteo Lisi
Signed-off-by: Francesco Utel
Signed-off-by: Mirko Ardinghi
Signed-off-by: Jagan Teki
---
.../dts/stm32mp157a-microgea-stm32mp1.dtsi| 147 ++
1 file changed, 147 insertions(+)
create mode 100644 arch/arm/boot/dts/stm32mp157a-microgea-stm32mp1.dtsi
diff --git a/arch/arm/boot/dts/stm32mp157a-microgea-stm32mp1.dtsi
b/arch/arm/boot/dts/stm32mp157a-microgea-stm32mp1.dtsi
new file mode 100644
index ..97d569107bfe
--- /dev/null
+++ b/arch/arm/boot/dts/stm32mp157a-microgea-stm32mp1.dtsi
@@ -0,0 +1,147 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause)
+/*
+ * Copyright (c) STMicroelectronics 2019 - All Rights Reserved
+ * Copyright (c) 2020 Engicam srl
+ * Copyright (c) 2020 Amarula Solutons(India)
+ */
+
+/ {
+ compatible = "engicam,microgea-stm32mp1", "st,stm32mp157";
+
+ memory@c000 {
+ reg = <0xc000 0x1000>;
+ };
+
+ reserved-memory {
+ #address-cells = <1>;
+ #size-cells = <1>;
+ ranges;
+
+ mcuram2: mcuram2@1000 {
+ compatible = "shared-dma-pool";
+ reg = <0x1000 0x4>;
+ no-map;
+ };
+
+ vdev0vring0: vdev0vring0@1004 {
+ compatible = "shared-dma-pool";
+ reg = <0x1004 0x1000>;
+ no-map;
+ };
+
+ vdev0vring1: vdev0vring1@10041000 {
+ compatible = "shared-dma-pool";
+ reg = <0x10041000 0x1000>;
+ no-map;
+ };
+
+ vdev0buffer: vdev0buffer@10042000 {
+ compatible = "shared-dma-pool";
+ reg = <0x10042000 0x4000>;
+ no-map;
+ };
+
+ mcuram: mcuram@3000 {
+ compatible = "shared-dma-pool";
+ reg = <0x3000 0x4>;
+ no-map;
+ };
+
+ retram: retram@3800 {
+ compatible = "shared-dma-pool";
+ reg = <0x3800 0x1>;
+ no-map;
+ };
+ };
+
+ vin: regulator-vin {
+ compatible = "regulator-fixed";
+ regulator-name = "vin";
+ regulator-min-microvolt = <500>;
+ regulator-max-microvolt = <500>;
+ regulator-always-on;
+ };
+
+ vddcore: regulator-vddcore {
+ compatible = "regulator-fixed";
+ regulator-name = "vddcore";
+ regulator-min-microvolt = <120>;
+ regulator-max-microvolt = <120>;
+ regulator-always-on;
+ vin-supply = <>;
+ };
+
+ vdd: regulator-vdd {
+ compatible = "regulator-fixed";
+ regulator-name = "vdd";
+ regulator-min-microvolt = <330>;
+ regulator-max-microvolt = <330>;
+ regulator-always-on;
+ vin-supply = <>;
+ };
+
+ vddq_ddr: regulator-vddq-ddr {
+ compatible = "regulator-fixed";
+ regulator-name = "vddq_ddr";
+ regulator-min-microvolt = <135>;
+ regulator-max-microvolt = <135>;
+ regulator-always-on;
+ vin-supply = <>;
+ };
+};
+
+ {
+ status = "okay";
+};
+
+ {
+ pinctrl-names = "default", "sleep";
+ pinctrl-0 = <_pins_a>;
+ pinctrl-1 = <_sleep_pins_a>;
+ status = "okay";
+
+ nand-controller@4,0 {
+ status = "okay";
+
+ nand@0 {
+ reg = <0>;
+ nand-on-flash-bbt;
+ #address-cells = <1>;
+ #size-cells = <1>;
+ };
+ };
+};
+
+ {
+ status = "okay";
+};
+
+{
+ timeout-sec = <32>;
+ status = "okay";
+};
+
+_rproc{
+ memory-region = <>, <>, <>, <>,
+ <>, <>;
+ mboxes = < 0>, < 1>, < 2>;
+ mbox-names = "vq0", "vq1", "shutdown";
+ interrupt-parent = <>;
+ interrupts = <68 1>;
+ status = "okay";
+};
+
+ {
+ status = "okay";
+};
+
+{
+ status = "okay";
+};
+
+ {
+ regulator-min-microvolt = <250>;
+ regulator-max-microvolt = <250>;
+ vdda-supply = <>;
+ status = "okay";
+};
--
2.25.1
