On 21/06/2022 12:15, Steffen Trumtrar wrote:
> Add the Phytec STM32MP1-3 Dev board. The devboard uses a Phytec stm32m15-som.
Use Linux coding style.
>
> Signed-off-by: Steffen Trumtrar <s.trumtrar@xxxxxxxxxxxxxx>
> ---
> .../dts/stm32mp157c-phycore-stm32mp1-3.dts | 56 +++++++++++++++++++
> 1 file changed, 56 insertions(+)
> create mode 100644 arch/arm/boot/dts/stm32mp157c-phycore-stm32mp1-3.dts
>
> diff --git a/arch/arm/boot/dts/stm32mp157c-phycore-stm32mp1-3.dts b/arch/arm/boot/dts/stm32mp157c-phycore-stm32mp1-3.dts
> new file mode 100644
> index 000000000000..e91c0ef499c9
> --- /dev/null
> +++ b/arch/arm/boot/dts/stm32mp157c-phycore-stm32mp1-3.dts
> @@ -0,0 +1,56 @@
> +// SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause)
> +/*
> + * Copyright (C) Phytec GmbH 2019-2020 - All Rights Reserved
> + * Author: Dom VOVARD <dom.vovard@xxxxxxxxx>.
> + */
> +
> +/dts-v1/;
> +
> +#include <dt-bindings/pinctrl/stm32-pinfunc.h>
> +#include "stm32mp157.dtsi"
> +#include "stm32mp15xc.dtsi"
> +#include "stm32mp15xxac-pinctrl.dtsi"
> +#include "stm32mp157c-phycore-stm32mp15-som.dtsi"
> +
> +/ {
> + model = "PHYTEC phyCORE-STM32MP1-3 Dev Board";
> + compatible = "phytec,phycore-stm32mp1-3", "st,stm32mp157";
Confusing approach. Either this is Som or dev board. If this is Som,
name it (so not "Dev Board"), although usually SoMs cannot boot by
themself, so this is a board.
If this is board, then phytec,phycore-stm32mp1-3 compatible is not
enough - you should have:
1. board compatible
2. SoM compatible
3. SoC compatible
> +};
> +
> +&cryp1 {
> + status = "okay";
> +};
> +
> +&dts {
> + status = "okay";
> +};
> +
> +&fmc {
> + status = "disabled";
> +};
> +
> +&gpu {
> + status = "okay";
> + contiguous-area = <&gpu_reserved>;
> +};
> +
> +&i2c4_eeprom {
> + status = "okay";
> +};
> +
> +&i2c4_rtc {
> + status = "okay";
> +};
> +
> +&qspi {
> + status = "okay";
> +};
> +
> +&sdmmc1 {
> + secure-status = "disabled";
> +};
> +
> +&sdmmc2 {
> + status = "okay";
> + secure-status = "disabled";
> +};
Which proves that aliases for MMC should be here.
Best regards,
Krzysztof
