* [PATCH 1/3] bindings: arm: fsl: Add PHYTEC i.MX8MM devicetree bindings
@ 2022-06-02 10:11 Teresa Remmet
2022-06-02 10:11 ` [PATCH 2/3] arm64: defconfig: Enable gpio-fan support Teresa Remmet
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Teresa Remmet @ 2022-06-02 10:11 UTC (permalink / raw)
To: linux-arm-kernel, devicetree
Cc: Rob Herring, Krzysztof Kozlowski, Shawn Guo, Fabio Estevam,
NXP Linux Team, Sascha Hauer, upstream
Add devicetree bindings for i.MX8MM based phyCORE-i.MX8MM
and phyBOARD-Polis RDK.
Signed-off-by: Teresa Remmet <t.remmet@phytec.de>
---
Documentation/devicetree/bindings/arm/fsl.yaml | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml
index ef524378d449..84134fdcf4f1 100644
--- a/Documentation/devicetree/bindings/arm/fsl.yaml
+++ b/Documentation/devicetree/bindings/arm/fsl.yaml
@@ -865,6 +865,12 @@ properties:
- const: toradex,verdin-imx8mm # Verdin iMX8M Mini Module
- const: fsl,imx8mm
+ - description: PHYTEC phyCORE-i.MX8MM SoM based boards
+ items:
+ - const: phytec,imx8mm-phyboard-polis-rdk # phyBOARD-Polis RDK
+ - const: phytec,imx8mm-phycore-som # phyCORE-i.MX8MM SoM
+ - const: fsl,imx8mm
+
- description: Variscite VAR-SOM-MX8MM based boards
items:
- const: variscite,var-som-mx8mm-symphony
--
2.25.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH 2/3] arm64: defconfig: Enable gpio-fan support 2022-06-02 10:11 [PATCH 1/3] bindings: arm: fsl: Add PHYTEC i.MX8MM devicetree bindings Teresa Remmet @ 2022-06-02 10:11 ` Teresa Remmet 2022-06-02 10:11 ` [PATCH 3/3] arm64: dts: freescale: Add phyBOARD-Polis-i.MX8MM support Teresa Remmet 2022-06-02 11:30 ` [PATCH 1/3] bindings: arm: fsl: Add PHYTEC i.MX8MM devicetree bindings Krzysztof Kozlowski 2 siblings, 0 replies; 6+ messages in thread From: Teresa Remmet @ 2022-06-02 10:11 UTC (permalink / raw) To: linux-arm-kernel, devicetree Cc: Rob Herring, Krzysztof Kozlowski, Shawn Guo, Fabio Estevam, NXP Linux Team, Sascha Hauer, upstream Enable gpio-fan support as this is needed by phyBOARD-Polis-i.MX8M Mini. Signed-off-by: Teresa Remmet <t.remmet@phytec.de> --- arch/arm64/configs/defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig index 7d1105343bc2..e04422b5e45a 100644 --- a/arch/arm64/configs/defconfig +++ b/arch/arm64/configs/defconfig @@ -577,6 +577,7 @@ CONFIG_CHARGER_BQ25890=m CONFIG_CHARGER_BQ25980=m CONFIG_SENSORS_ARM_SCMI=y CONFIG_SENSORS_ARM_SCPI=y +CONFIG_SENSORS_GPIO_FAN=m CONFIG_SENSORS_JC42=m CONFIG_SENSORS_LM75=m CONFIG_SENSORS_LM90=m -- 2.25.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 3/3] arm64: dts: freescale: Add phyBOARD-Polis-i.MX8MM support 2022-06-02 10:11 [PATCH 1/3] bindings: arm: fsl: Add PHYTEC i.MX8MM devicetree bindings Teresa Remmet 2022-06-02 10:11 ` [PATCH 2/3] arm64: defconfig: Enable gpio-fan support Teresa Remmet @ 2022-06-02 10:11 ` Teresa Remmet 2022-06-02 11:35 ` Krzysztof Kozlowski 2022-06-02 11:30 ` [PATCH 1/3] bindings: arm: fsl: Add PHYTEC i.MX8MM devicetree bindings Krzysztof Kozlowski 2 siblings, 1 reply; 6+ messages in thread From: Teresa Remmet @ 2022-06-02 10:11 UTC (permalink / raw) To: linux-arm-kernel, devicetree Cc: Rob Herring, Krzysztof Kozlowski, Shawn Guo, Fabio Estevam, NXP Linux Team, Sascha Hauer, upstream Add initial support for phyBOARD-Polis-i.MX8MM. Main features are: * Bluetooth and Wifi * CANFD * eMMC * i2c RTC * Ethernet * PCIe * RS232/RS485 * SD-Card * SPI-NOR flash * USB Signed-off-by: Teresa Remmet <t.remmet@phytec.de> --- arch/arm64/boot/dts/freescale/Makefile | 1 + .../freescale/imx8mm-phyboard-polis-rdk.dts | 450 ++++++++++++++++++ .../dts/freescale/imx8mm-phycore-som.dtsi | 433 +++++++++++++++++ 3 files changed, 884 insertions(+) create mode 100644 arch/arm64/boot/dts/freescale/imx8mm-phyboard-polis-rdk.dts create mode 100644 arch/arm64/boot/dts/freescale/imx8mm-phycore-som.dtsi diff --git a/arch/arm64/boot/dts/freescale/Makefile b/arch/arm64/boot/dts/freescale/Makefile index 238a83e5b8c6..1e25dec2109b 100644 --- a/arch/arm64/boot/dts/freescale/Makefile +++ b/arch/arm64/boot/dts/freescale/Makefile @@ -58,6 +58,7 @@ dtb-$(CONFIG_ARCH_MXC) += imx8mm-icore-mx8mm-edimm2.2.dtb dtb-$(CONFIG_ARCH_MXC) += imx8mm-kontron-n801x-s.dtb dtb-$(CONFIG_ARCH_MXC) += imx8mm-mx8menlo.dtb dtb-$(CONFIG_ARCH_MXC) += imx8mm-nitrogen-r2.dtb +dtb-$(CONFIG_ARCH_MXC) += imx8mm-phyboard-polis-rdk.dtb dtb-$(CONFIG_ARCH_MXC) += imx8mm-tqma8mqml-mba8mx.dtb dtb-$(CONFIG_ARCH_MXC) += imx8mm-var-som-symphony.dtb dtb-$(CONFIG_ARCH_MXC) += imx8mm-venice-gw71xx-0x.dtb diff --git a/arch/arm64/boot/dts/freescale/imx8mm-phyboard-polis-rdk.dts b/arch/arm64/boot/dts/freescale/imx8mm-phyboard-polis-rdk.dts new file mode 100644 index 000000000000..532b435b6871 --- /dev/null +++ b/arch/arm64/boot/dts/freescale/imx8mm-phyboard-polis-rdk.dts @@ -0,0 +1,450 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Copyright (C) 2022 PHYTEC Messtechnik GmbH + * Author: Teresa Remmet <t.remmet@phytec.de> + */ + +/dts-v1/; + +#include <dt-bindings/interrupt-controller/irq.h> +#include <dt-bindings/leds/common.h> +#include <dt-bindings/phy/phy-imx8-pcie.h> +#include "imx8mm-phycore-som.dtsi" + +/ { + model = "PHYTEC phyBOARD-Polis-i.MX8MM RDK"; + compatible = "phytec,imx8mm-phyboard-polis-rdk", + "phytec,imx8mm-phycore-som","fsl,imx8mm"; + + chosen { + stdout-path = &uart3; + }; + + bt_osc_32k: bt_lp-clock { + compatible = "fixed-clock"; + clock-frequency = <32768>; + clock-output-names = "bt_osc_32k"; + #clock-cells = <0>; + }; + + can_osc_40m: can-clock { + compatible = "fixed-clock"; + clock-frequency = <40000000>; + clock-output-names = "can_osc_40m"; + #clock-cells = <0>; + }; + + fan { + compatible = "gpio-fan"; + gpios = <&gpio4 8 GPIO_ACTIVE_HIGH>; + gpio-fan,speed-map = <0 0 + 13000 1>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_fan>; + #cooling-cells = <2>; + }; + + leds { + compatible = "gpio-leds"; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_leds>; + + led-0 { + color = <LED_COLOR_ID_RED>; + function = LED_FUNCTION_DISK; + gpios = <&gpio1 1 GPIO_ACTIVE_HIGH>; + linux,default-trigger = "mmc2"; + }; + + led-1 { + color = <LED_COLOR_ID_BLUE>; + function = LED_FUNCTION_DISK; + gpios = <&gpio1 15 GPIO_ACTIVE_HIGH>; + linux,default-trigger = "mmc1"; + }; + + led-2 { + color = <LED_COLOR_ID_GREEN>; + function = LED_FUNCTION_CPU; + gpios = <&gpio1 14 GPIO_ACTIVE_HIGH>; + linux,default-trigger = "heartbeat"; + }; + }; + + usdhc1_pwrseq: pwr-seq { + compatible = "mmc-pwrseq-simple"; + post-power-on-delay-ms = <100>; + power-off-delay-us = <60>; + reset-gpios = <&gpio2 7 GPIO_ACTIVE_LOW>; + }; + + reg_can_en: regulator-can-en { + compatible = "regulator-fixed"; + gpio = <&gpio1 9 GPIO_ACTIVE_LOW>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_can_en>; + regulator-max-microvolt = <3300000>; + regulator-min-microvolt = <3300000>; + regulator-name = "CAN_EN"; + startup-delay-us = <20>; + }; + + reg_usb_otg1_vbus: regulator-usb-otg1 { + compatible = "regulator-fixed"; + enable-active-high; + gpio = <&gpio1 12 GPIO_ACTIVE_HIGH>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_usbotg1pwrgrp>; + regulator-name = "usb_otg1_vbus"; + regulator-max-microvolt = <5000000>; + regulator-min-microvolt = <5000000>; + }; + + reg_usdhc2_vmmc: regulator-usdhc2 { + compatible = "regulator-fixed"; + enable-active-high; + gpio = <&gpio2 19 GPIO_ACTIVE_HIGH>; + off-on-delay-us = <20000>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_reg_usdhc2_vmmc>; + regulator-max-microvolt = <3300000>; + regulator-min-microvolt = <3300000>; + regulator-name = "VSD_3V3"; + }; + + reg_vcc_3v3: regulator-vcc_3v3 { + compatible = "regulator-fixed"; + regulator-max-microvolt = <3300000>; + regulator-min-microvolt = <3300000>; + regulator-name = "VCC_3V3"; + }; +}; + +/* SPI - CAN MCP251XFD */ +&ecspi1 { + cs-gpios = <&gpio5 9 GPIO_ACTIVE_LOW>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_ecspi1>; + status = "okay"; + + can0: can@0 { + compatible = "microchip,mcp251xfd"; + clocks = <&can_osc_40m>; + interrupt-parent = <&gpio1>; + interrupts = <8 IRQ_TYPE_LEVEL_LOW>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_can_int>; + reg = <0>; + spi-max-frequency = <20000000>; + xceiver-supply = <®_can_en>; + }; +}; + +&gpio1 { + gpio-line-names = "nINT_ETHPHY", "LED_RED", "WDOG_INT", "X_RTC_INT", + "", "", "", "RESET_ETHPHY", + "CAN_nINT", "CAN_EN", "nENABLE_FLATLINK", "", + "USB_OTG_VBUS_EN", "", "LED_GREEN", "LED_BLUE"; +}; + +&gpio2 { + gpio-line-names = "", "", "", "", + "", "", "BT_REG_ON", "WL_REG_ON", + "BT_DEV_WAKE", "BT_HOST_WAKE", "", "", + "X_SD2_CD_B", "", "", "", + "", "", "", "SD2_RESET_B"; +}; + +&gpio4 { + gpio-line-names = "", "", "", "", + "", "", "", "", + "FAN", "miniPCIe_nPERST", "", "", + "COEX1", "COEX2"; +}; + +&gpio5 { + gpio-line-names = "", "", "", "", + "", "", "", "", + "", "ECSPI1_SS0"; +}; + +/* PCIe */ +&pcie0 { + assigned-clocks = <&clk IMX8MM_CLK_PCIE1_AUX>, + <&clk IMX8MM_CLK_PCIE1_CTRL>; + assigned-clock-parents = <&clk IMX8MM_SYS_PLL2_50M>, + <&clk IMX8MM_SYS_PLL2_250M>; + assigned-clock-rates = <10000000>, <250000000>; + clocks = <&clk IMX8MM_CLK_PCIE1_ROOT>, <&clk IMX8MM_CLK_PCIE1_AUX>, + <&clk IMX8MM_CLK_PCIE1_PHY>; + clock-names = "pcie", "pcie_aux", "pcie_bus"; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_pcie>; + reset-gpio = <&gpio4 9 GPIO_ACTIVE_LOW>; + status = "okay"; +}; + +&pcie_phy { + clocks = <&clk IMX8MM_CLK_PCIE1_PHY>; + fsl,clkreq-unsupported; + fsl,refclk-pad-mode = <IMX8_PCIE_REFCLK_PAD_OUTPUT>; + fsl,tx-deemph-gen1 = <0x2d>; + fsl,tx-deemph-gen2 = <0xf>; + status = "okay"; +}; + +&rv3028 { + trickle-resistor-ohms = <3000>; +}; + +&snvs_pwrkey { + status = "okay"; +}; + +/* UART - RS232/RS485 */ +&uart1 { + assigned-clocks = <&clk IMX8MM_CLK_UART1>; + assigned-clock-parents = <&clk IMX8MM_SYS_PLL1_80M>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_uart1>; + status = "okay"; + uart-has-rtscts; +}; + +/* UART - Sterling-LWB Bluetooth */ +&uart2 { + assigned-clocks = <&clk IMX8MM_CLK_UART2>; + assigned-clock-parents = <&clk IMX8MM_SYS_PLL1_80M>; + fsl,dte-mode; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_uart2_bt>; + status = "okay"; + uart-has-rtscts; + + bluetooth { + compatible = "brcm,bcm43438-bt"; + clocks = <&bt_osc_32k>; + clock-names = "lpo"; + device-wakeup-gpios = <&gpio2 8 GPIO_ACTIVE_HIGH>; + interrupt-names = "host-wakeup"; + interrupt-parent = <&gpio2>; + interrupts = <9 IRQ_TYPE_EDGE_BOTH>; + max-speed = <2000000>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_bt>; + shutdown-gpios = <&gpio2 6 GPIO_ACTIVE_HIGH>; + vddio-supply = <®_vcc_3v3>; + }; +}; + +/* UART - console */ +&uart3 { + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_uart3>; + status = "okay"; +}; + +/* USB */ +&usbotg1 { + adp-disable; + dr_mode = "otg"; + over-current-active-low; + samsung,picophy-pre-emp-curr-control = <3>; + samsung,picophy-dc-vol-level-adjust = <7>; + srp-disable; + status = "okay"; + vbus-supply = <®_usb_otg1_vbus>; +}; + +&usbotg2 { + disable-over-current; + dr_mode = "host"; + samsung,picophy-pre-emp-curr-control = <3>; + samsung,picophy-dc-vol-level-adjust = <7>; + status = "okay"; +}; + +/* SDIO - Sterling-LWB Wifi */ +&usdhc1 { + assigned-clocks = <&clk IMX8MM_CLK_USDHC1>; + assigned-clock-rates = <200000000>; + bus-width = <4>; + mmc-pwrseq = <&usdhc1_pwrseq>; + non-removable; + no-1-8-v; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_usdhc1>, <&pinctrl_wlan>; + status = "okay"; + #address-cells = <1>; + #size-cells = <0>; + + brcmf: wifi@1 { + compatible = "brcm,bcm4329-fmac"; + reg = <1>; + }; +}; + +/* SD-Card */ +&usdhc2 { + assigned-clocks = <&clk IMX8MM_CLK_USDHC2>; + assigned-clock-rates = <200000000>; + bus-width = <4>; + cd-gpios = <&gpio2 12 GPIO_ACTIVE_LOW>; + disable-wp; + pinctrl-names = "default", "state_100mhz", "state_200mhz"; + pinctrl-0 = <&pinctrl_usdhc2>, <&pinctrl_usdhc2_gpio>; + pinctrl-1 = <&pinctrl_usdhc2_100mhz>, <&pinctrl_usdhc2_gpio>; + pinctrl-2 = <&pinctrl_usdhc2_200mhz>, <&pinctrl_usdhc2_gpio>; + status = "okay"; + vmmc-supply = <®_usdhc2_vmmc>; + vqmmc-supply = <®_nvcc_sd2>; +}; + +&iomuxc { + pinctrl_bt: btgrp { + fsl,pins = < + MX8MM_IOMUXC_SD1_DATA4_GPIO2_IO6 0x00 + MX8MM_IOMUXC_SD1_DATA6_GPIO2_IO8 0x00 + MX8MM_IOMUXC_SD1_DATA7_GPIO2_IO9 0x00 + >; + }; + + pinctrl_can_en: can-engrp { + fsl,pins = < + MX8MM_IOMUXC_GPIO1_IO09_GPIO1_IO9 0x00 + >; + }; + + pinctrl_can_int: can-intgrp { + fsl,pins = < + MX8MM_IOMUXC_GPIO1_IO08_GPIO1_IO8 0x00 + >; + }; + + pinctrl_ecspi1: ecspi1grp { + fsl,pins = < + MX8MM_IOMUXC_ECSPI1_MISO_ECSPI1_MISO 0x80 + MX8MM_IOMUXC_ECSPI1_MOSI_ECSPI1_MOSI 0x80 + MX8MM_IOMUXC_ECSPI1_SCLK_ECSPI1_SCLK 0x80 + MX8MM_IOMUXC_ECSPI1_SS0_GPIO5_IO9 0x00 + >; + }; + + pinctrl_fan: fan0grp { + fsl,pins = < + MX8MM_IOMUXC_SAI1_RXD6_GPIO4_IO8 0x16 + >; + }; + + pinctrl_leds: leds1grp { + fsl,pins = < + MX8MM_IOMUXC_GPIO1_IO01_GPIO1_IO1 0x16 + MX8MM_IOMUXC_GPIO1_IO14_GPIO1_IO14 0x16 + MX8MM_IOMUXC_GPIO1_IO15_GPIO1_IO15 0x16 + >; + }; + + pinctrl_pcie: pciegrp { + fsl,pins = < + MX8MM_IOMUXC_SAI1_RXD7_GPIO4_IO9 0x00 + MX8MM_IOMUXC_SAI1_TXD0_GPIO4_IO12 0x12 + MX8MM_IOMUXC_SAI1_TXD7_GPIO4_IO19 0x12 + >; + }; + + pinctrl_reg_usdhc2_vmmc: regusdhc2vmmcgrp { + fsl,pins = < + MX8MM_IOMUXC_SD2_RESET_B_GPIO2_IO19 0x41 + >; + }; + + pinctrl_uart1: uart1grp { + fsl,pins = < + MX8MM_IOMUXC_SAI2_RXC_UART1_DCE_RX 0x00 + MX8MM_IOMUXC_SAI2_RXD0_UART1_DCE_RTS_B 0x00 + MX8MM_IOMUXC_SAI2_RXFS_UART1_DCE_TX 0x00 + MX8MM_IOMUXC_SAI2_TXFS_UART1_DCE_CTS_B 0x00 + >; + }; + + pinctrl_uart2_bt: uart2btgrp { + fsl,pins = < + MX8MM_IOMUXC_SAI3_RXC_UART2_DTE_RTS_B 0x00 + MX8MM_IOMUXC_SAI3_RXD_UART2_DTE_CTS_B 0x00 + MX8MM_IOMUXC_SAI3_TXC_UART2_DTE_RX 0x00 + MX8MM_IOMUXC_SAI3_TXFS_UART2_DTE_TX 0x00 + >; + }; + + pinctrl_uart3: uart3grp { + fsl,pins = < + MX8MM_IOMUXC_UART3_RXD_UART3_DCE_RX 0x49 + MX8MM_IOMUXC_UART3_TXD_UART3_DCE_TX 0x49 + >; + }; + + pinctrl_usbotg1pwrgrp: usbotg1pwrgrp { + fsl,pins = < + MX8MM_IOMUXC_GPIO1_IO12_GPIO1_IO12 0x00 + >; + }; + + pinctrl_usdhc1: usdhc1grp { + fsl,pins = < + MX8MM_IOMUXC_SD1_CLK_USDHC1_CLK 0x182 + MX8MM_IOMUXC_SD1_CMD_USDHC1_CMD 0xc6 + MX8MM_IOMUXC_SD1_DATA0_USDHC1_DATA0 0xc6 + MX8MM_IOMUXC_SD1_DATA1_USDHC1_DATA1 0xc6 + MX8MM_IOMUXC_SD1_DATA2_USDHC1_DATA2 0xc6 + MX8MM_IOMUXC_SD1_DATA3_USDHC1_DATA3 0xc6 + >; + }; + + pinctrl_usdhc2_gpio: usdhc2gpiogrp { + fsl,pins = < + MX8MM_IOMUXC_SD2_CD_B_GPIO2_IO12 0x41 + >; + }; + + pinctrl_usdhc2: usdhc2grp { + fsl,pins = < + MX8MM_IOMUXC_GPIO1_IO04_USDHC2_VSELECT 0x1d0 + MX8MM_IOMUXC_SD2_CLK_USDHC2_CLK 0x192 + MX8MM_IOMUXC_SD2_CMD_USDHC2_CMD 0x1d2 + MX8MM_IOMUXC_SD2_DATA0_USDHC2_DATA0 0x1d2 + MX8MM_IOMUXC_SD2_DATA1_USDHC2_DATA1 0x1d2 + MX8MM_IOMUXC_SD2_DATA2_USDHC2_DATA2 0x1d2 + MX8MM_IOMUXC_SD2_DATA3_USDHC2_DATA3 0x1d2 + >; + }; + + pinctrl_usdhc2_100mhz: usdhc2-100mhzgrp { + fsl,pins = < + MX8MM_IOMUXC_GPIO1_IO04_USDHC2_VSELECT 0x1d0 + MX8MM_IOMUXC_SD2_CLK_USDHC2_CLK 0x194 + MX8MM_IOMUXC_SD2_CMD_USDHC2_CMD 0x1d4 + MX8MM_IOMUXC_SD2_DATA0_USDHC2_DATA0 0x1d4 + MX8MM_IOMUXC_SD2_DATA1_USDHC2_DATA1 0x1d4 + MX8MM_IOMUXC_SD2_DATA2_USDHC2_DATA2 0x1d4 + MX8MM_IOMUXC_SD2_DATA3_USDHC2_DATA3 0x1d4 + >; + }; + + pinctrl_usdhc2_200mhz: usdhc2-200mhzgrp { + fsl,pins = < + MX8MM_IOMUXC_GPIO1_IO04_USDHC2_VSELECT 0x1d0 + MX8MM_IOMUXC_SD2_CLK_USDHC2_CLK 0x196 + MX8MM_IOMUXC_SD2_CMD_USDHC2_CMD 0x1d6 + MX8MM_IOMUXC_SD2_DATA0_USDHC2_DATA0 0x1d6 + MX8MM_IOMUXC_SD2_DATA1_USDHC2_DATA1 0x1d6 + MX8MM_IOMUXC_SD2_DATA2_USDHC2_DATA2 0x1d6 + MX8MM_IOMUXC_SD2_DATA3_USDHC2_DATA3 0x1d6 + >; + }; + + pinctrl_wlan: wlangrp { + fsl,pins = < + MX8MM_IOMUXC_SD1_DATA5_GPIO2_IO7 0x00 + >; + }; +}; diff --git a/arch/arm64/boot/dts/freescale/imx8mm-phycore-som.dtsi b/arch/arm64/boot/dts/freescale/imx8mm-phycore-som.dtsi new file mode 100644 index 000000000000..1aa27291c7ff --- /dev/null +++ b/arch/arm64/boot/dts/freescale/imx8mm-phycore-som.dtsi @@ -0,0 +1,433 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Copyright (C) 2022 PHYTEC Messtechnik GmbH + * Author: Teresa Remmet <t.remmet@phytec.de> + */ + +#include "imx8mm.dtsi" +#include <dt-bindings/net/ti-dp83867.h> + +/ { + model = "PHYTEC phyCORE-i.MX8MM"; + compatible = "phytec,imx8mm-phycore-som", "fsl,imx8mm"; + + aliases { + rtc0 = &rv3028; + rtc1 = &snvs_rtc; + }; + + memory@40000000 { + device_type = "memory"; + reg = <0x0 0x40000000 0 0x80000000>; + }; + + reg_vdd_3v3_s: regulator-vdd_3v3_s { + compatible = "regulator-fixed"; + regulator-always-on; + regulator-boot-on; + regulator-max-microvolt = <3300000>; + regulator-min-microvolt = <3300000>; + regulator-name = "VDD_3V3_S"; + }; +}; + +&A53_0 { + cpu-supply = <®_vdd_arm>; +}; + +&A53_1 { + cpu-supply = <®_vdd_arm>; +}; + +&A53_2 { + cpu-supply = <®_vdd_arm>; +}; + +&A53_3 { + cpu-supply = <®_vdd_arm>; +}; + +&ddrc { + operating-points-v2 = <&ddrc_opp_table>; + + ddrc_opp_table: opp-table { + compatible = "operating-points-v2"; + + opp-25M { + opp-hz = /bits/ 64 <25000000>; + }; + + opp-100M { + opp-hz = /bits/ 64 <100000000>; + }; + + opp-750M { + opp-hz = /bits/ 64 <750000000>; + }; + }; +}; + +/* Ethernet */ +&fec1 { + fsl,magic-packet; + phy-mode = "rgmii-id"; + phy-handle = <ðphy0>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_fec1>; + status = "okay"; + + mdio { + #address-cells = <1>; + #size-cells = <0>; + + ethphy0: ethernet-phy@0 { + compatible = "ethernet-phy-ieee802.3-c22"; + enet-phy-lane-no-swap; + interrupts = <0 IRQ_TYPE_EDGE_FALLING>; + interrupt-parent = <&gpio1>; + ti,clk-output-sel = <DP83867_CLK_O_SEL_OFF>; + ti,fifo-depth = <DP83867_PHYCR_FIFO_DEPTH_4_B_NIB>; + ti,rx-internal-delay = <DP83867_RGMIIDCTL_2_00_NS>; + ti,tx-internal-delay = <DP83867_RGMIIDCTL_2_00_NS>; + reg = <0>; + reset-gpios = <&gpio1 7 GPIO_ACTIVE_HIGH>; + reset-assert-us = <1000>; + reset-deassert-us = <1000>; + }; + }; +}; + +/* SPI Flash */ +&flexspi { + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_flexspi0>; + status = "okay"; + + som_flash: flash@0 { + compatible = "jedec,spi-nor"; + reg = <0>; + spi-max-frequency = <80000000>; + spi-rx-bus-width = <4>; + spi-tx-bus-width = <1>; + }; +}; + +&gpio1 { + gpio-line-names = "nINT_ETHPHY", "", "WDOG_INT", "X_RTC_INT", + "", "", "", "RESET_ETHPHY", + "", "", "nENABLE_FLATLINK"; +}; + +/* I2C1 */ +&i2c1 { + clock-frequency = <400000>; + pinctrl-names = "default","gpio"; + pinctrl-0 = <&pinctrl_i2c1>; + pinctrl-1 = <&pinctrl_i2c1_gpio>; + scl-gpios = <&gpio5 14 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>; + sda-gpios = <&gpio5 15 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>; + status = "okay"; + + pmic@8 { + compatible = "nxp,pf8121a"; + reg = <0x08>; + + regulators { + reg_nvcc_sd1: ldo1 { + regulator-always-on; + regulator-boot-on; + regulator-max-microvolt = <3300000>; + regulator-min-microvolt = <3300000>; + regulator-name = "NVCC_SD1 (LDO1)"; + regulator-state-mem { + regulator-off-in-suspend; + }; + }; + + reg_nvcc_sd2: ldo2 { + regulator-always-on; + regulator-boot-on; + regulator-max-microvolt = <3300000>; + regulator-min-microvolt = <1800000>; + regulator-name = "NVCC_SD2 (LDO2)"; + vselect-en; + regulator-state-mem { + regulator-off-in-suspend; + }; + }; + + reg_vcc_enet: ldo3 { + regulator-always-on; + regulator-boot-on; + regulator-max-microvolt = <2500000>; + regulator-min-microvolt = <1500000>; + regulator-name = "VCC_ENET_2V5 (LDO3)"; + regulator-state-mem { + regulator-off-in-suspend; + }; + }; + + reg_vdda_1v8: ldo4 { + regulator-always-on; + regulator-boot-on; + regulator-max-microvolt = <1800000>; + regulator-min-microvolt = <1500000>; + regulator-name = "VDDA_1V8 (LDO4)"; + regulator-state-mem { + regulator-on-in-suspend; + regulator-suspend-min-microvolt = <1500000>; + regulator-suspend-max-microvolt = <1500000>; + }; + }; + + reg_soc_vdda_phy: buck1 { + regulator-always-on; + regulator-boot-on; + regulator-max-microvolt = <900000>; + regulator-min-microvolt = <400000>; + regulator-name = "VDD_SOC_VDDA_PHY_0P8 (BUCK1)"; + regulator-state-mem { + regulator-on-in-suspend; + regulator-suspend-min-microvolt = <400000>; + regulator-suspend-max-microvolt = <400000>; + }; + }; + + reg_vdd_gpu_dram: buck2 { + regulator-always-on; + regulator-boot-on; + regulator-max-microvolt = <1000000>; + regulator-min-microvolt = <1000000>; + regulator-name = "VDD_GPU_DRAM (BUCK2)"; + regulator-state-mem { + regulator-on-in-suspend; + regulator-suspend-max-microvolt = <1000000>; + regulator-suspend-min-microvolt = <1000000>; + }; + }; + + reg_vdd_gpu: buck3 { + regulator-always-on; + regulator-boot-on; + regulator-max-microvolt = <1000000>; + regulator-min-microvolt = <400000>; + regulator-name = "VDD_VPU (BUCK3)"; + regulator-state-mem { + regulator-off-in-suspend; + }; + }; + + reg_vdd_mipi: buck4 { + regulator-always-on; + regulator-boot-on; + regulator-max-microvolt = <1050000>; + regulator-min-microvolt = <900000>; + regulator-name = "VDD_MIPI_0P9 (BUCK4)"; + regulator-state-mem { + regulator-off-in-suspend; + }; + }; + + reg_vdd_arm: buck5 { + regulator-always-on; + regulator-boot-on; + regulator-max-microvolt = <1050000>; + regulator-min-microvolt = <400000>; + regulator-name = "VDD_ARM (BUCK5)"; + regulator-state-mem { + regulator-off-in-suspend; + }; + }; + + reg_vdd_1v8: buck6 { + regulator-always-on; + regulator-boot-on; + regulator-max-microvolt = <1800000>; + regulator-min-microvolt = <1800000>; + regulator-name = "VDD_1V8 (BUCK6)"; + regulator-state-mem { + regulator-on-in-suspend; + regulator-suspend-max-microvolt = <1800000>; + regulator-suspend-min-microvolt = <1800000>; + }; + }; + + reg_nvcc_dram: buck7 { + regulator-always-on; + regulator-boot-on; + regulator-max-microvolt = <1100000>; + regulator-min-microvolt = <1100000>; + regulator-name = "NVCC_DRAM_1P1V (BUCK7)"; + }; + + reg_vsnvs: vsnvs { + regulator-always-on; + regulator-boot-on; + regulator-max-microvolt = <1800000>; + regulator-min-microvolt = <1800000>; + regulator-name = "NVCC_SNVS_1P8 (VSNVS)"; + }; + }; + }; + + eeprom@51 { + compatible = "atmel,24c32"; + pagesize = <32>; + reg = <0x51>; + vcc-supply = <®_vdd_3v3_s>; + }; + + sn65dsi83: mipitolvds@2d { + compatible = "ti,sn65dsi83"; + enable-gpios = <&gpio1 10 GPIO_ACTIVE_LOW>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_sn65dsi83>; + reg = <0x2d>; + status = "disabled"; + }; + + rv3028: rtc@52 { + compatible = "microcrystal,rv3028"; + interrupts = <3 IRQ_TYPE_LEVEL_LOW>; + interrupt-parent = <&gpio1>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_rtc>; + reg = <0x52>; + }; +}; + +/* EMMC */ +&usdhc3 { + assigned-clocks = <&clk IMX8MM_CLK_USDHC3_ROOT>; + assigned-clock-rates = <400000000>; + bus-width = <8>; + keep-power-in-suspend; + pinctrl-names = "default", "state_100mhz", "state_200mhz"; + pinctrl-0 = <&pinctrl_usdhc3>; + pinctrl-1 = <&pinctrl_usdhc3_100mhz>; + pinctrl-2 = <&pinctrl_usdhc3_200mhz>; + non-removable; + status = "okay"; +}; + +/* Watchdog */ +&wdog1 { + fsl,ext-reset-output; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_wdog>; + status = "okay"; +}; + +&iomuxc { + pinctrl_fec1: fec1grp { + fsl,pins = < + MX8MM_IOMUXC_ENET_MDC_ENET1_MDC 0x3 + MX8MM_IOMUXC_ENET_MDIO_ENET1_MDIO 0x3 + MX8MM_IOMUXC_ENET_RD0_ENET1_RGMII_RD0 0x91 + MX8MM_IOMUXC_ENET_RD1_ENET1_RGMII_RD1 0x91 + MX8MM_IOMUXC_ENET_RD2_ENET1_RGMII_RD2 0x91 + MX8MM_IOMUXC_ENET_RD3_ENET1_RGMII_RD3 0x91 + MX8MM_IOMUXC_ENET_RXC_ENET1_RGMII_RXC 0x91 + MX8MM_IOMUXC_ENET_RX_CTL_ENET1_RGMII_RX_CTL 0x91 + MX8MM_IOMUXC_ENET_TD0_ENET1_RGMII_TD0 0x1f + MX8MM_IOMUXC_ENET_TD1_ENET1_RGMII_TD1 0x1f + MX8MM_IOMUXC_ENET_TD2_ENET1_RGMII_TD2 0x1f + MX8MM_IOMUXC_ENET_TD3_ENET1_RGMII_TD3 0x1f + MX8MM_IOMUXC_ENET_TXC_ENET1_RGMII_TXC 0x1f + MX8MM_IOMUXC_ENET_TX_CTL_ENET1_RGMII_TX_CTL 0x1f + MX8MM_IOMUXC_GPIO1_IO00_GPIO1_IO0 0x19 + MX8MM_IOMUXC_GPIO1_IO07_GPIO1_IO7 0x19 + >; + }; + + pinctrl_flexspi0: flexspi0grp { + fsl,pins = < + MX8MM_IOMUXC_NAND_ALE_QSPI_A_SCLK 0x1c2 + MX8MM_IOMUXC_NAND_CE0_B_QSPI_A_SS0_B 0x82 + MX8MM_IOMUXC_NAND_DATA00_QSPI_A_DATA0 0x82 + MX8MM_IOMUXC_NAND_DATA01_QSPI_A_DATA1 0x82 + MX8MM_IOMUXC_NAND_DATA02_QSPI_A_DATA2 0x82 + MX8MM_IOMUXC_NAND_DATA03_QSPI_A_DATA3 0x82 + >; + }; + + pinctrl_i2c1: i2c1grp { + fsl,pins = < + MX8MM_IOMUXC_I2C1_SDA_I2C1_SDA 0x400001c0 + MX8MM_IOMUXC_I2C1_SCL_I2C1_SCL 0x400001c0 + >; + }; + + pinctrl_i2c1_gpio: i2c1gpiogrp { + fsl,pins = < + MX8MM_IOMUXC_I2C1_SDA_GPIO5_IO15 0x1e0 + MX8MM_IOMUXC_I2C1_SCL_GPIO5_IO14 0x1e0 + >; + }; + + pinctrl_rtc: rtcgrp { + fsl,pins = < + MX8MM_IOMUXC_GPIO1_IO03_GPIO1_IO3 0x1c0 + >; + }; + + pinctrl_sn65dsi83: sn65dsi83grp { + fsl,pins = < + MX8MM_IOMUXC_GPIO1_IO10_GPIO1_IO10 0x0 + >; + }; + + pinctrl_usdhc3: usdhc3grp { + fsl,pins = < + MX8MM_IOMUXC_NAND_CLE_USDHC3_DATA7 0x1d0 + MX8MM_IOMUXC_NAND_CE1_B_USDHC3_STROBE 0x190 + MX8MM_IOMUXC_NAND_CE2_B_USDHC3_DATA5 0x1d0 + MX8MM_IOMUXC_NAND_CE3_B_USDHC3_DATA6 0x1d0 + MX8MM_IOMUXC_NAND_DATA04_USDHC3_DATA0 0x1d0 + MX8MM_IOMUXC_NAND_DATA05_USDHC3_DATA1 0x1d0 + MX8MM_IOMUXC_NAND_DATA06_USDHC3_DATA2 0x1d0 + MX8MM_IOMUXC_NAND_DATA07_USDHC3_DATA3 0x1d0 + MX8MM_IOMUXC_NAND_RE_B_USDHC3_DATA4 0x1d0 + MX8MM_IOMUXC_NAND_WE_B_USDHC3_CLK 0x190 + MX8MM_IOMUXC_NAND_WP_B_USDHC3_CMD 0x1d0 + >; + }; + + pinctrl_usdhc3_100mhz: usdhc3-100mhzgrp { + fsl,pins = < + MX8MM_IOMUXC_NAND_CLE_USDHC3_DATA7 0x1d4 + MX8MM_IOMUXC_NAND_CE1_B_USDHC3_STROBE 0x194 + MX8MM_IOMUXC_NAND_CE2_B_USDHC3_DATA5 0x1d4 + MX8MM_IOMUXC_NAND_CE3_B_USDHC3_DATA6 0x1d4 + MX8MM_IOMUXC_NAND_DATA04_USDHC3_DATA0 0x1d4 + MX8MM_IOMUXC_NAND_DATA05_USDHC3_DATA1 0x1d4 + MX8MM_IOMUXC_NAND_DATA06_USDHC3_DATA2 0x1d4 + MX8MM_IOMUXC_NAND_DATA07_USDHC3_DATA3 0x1d4 + MX8MM_IOMUXC_NAND_RE_B_USDHC3_DATA4 0x1d4 + MX8MM_IOMUXC_NAND_WE_B_USDHC3_CLK 0x194 + MX8MM_IOMUXC_NAND_WP_B_USDHC3_CMD 0x1d4 + >; + }; + + pinctrl_usdhc3_200mhz: usdhc3-200mhzgrp { + fsl,pins = < + MX8MM_IOMUXC_NAND_CLE_USDHC3_DATA7 0x1d6 + MX8MM_IOMUXC_NAND_CE1_B_USDHC3_STROBE 0x196 + MX8MM_IOMUXC_NAND_CE2_B_USDHC3_DATA5 0x1d6 + MX8MM_IOMUXC_NAND_CE3_B_USDHC3_DATA6 0x1d6 + MX8MM_IOMUXC_NAND_DATA04_USDHC3_DATA0 0x1d6 + MX8MM_IOMUXC_NAND_DATA05_USDHC3_DATA1 0x1d6 + MX8MM_IOMUXC_NAND_DATA06_USDHC3_DATA2 0x1d6 + MX8MM_IOMUXC_NAND_DATA07_USDHC3_DATA3 0x1d6 + MX8MM_IOMUXC_NAND_RE_B_USDHC3_DATA4 0x1d6 + MX8MM_IOMUXC_NAND_WE_B_USDHC3_CLK 0x196 + MX8MM_IOMUXC_NAND_WP_B_USDHC3_CMD 0x1d6 + >; + }; + + pinctrl_wdog: wdoggrp { + fsl,pins = < + MX8MM_IOMUXC_GPIO1_IO02_WDOG1_WDOG_B 0x26 + >; + }; +}; -- 2.25.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 3/3] arm64: dts: freescale: Add phyBOARD-Polis-i.MX8MM support 2022-06-02 10:11 ` [PATCH 3/3] arm64: dts: freescale: Add phyBOARD-Polis-i.MX8MM support Teresa Remmet @ 2022-06-02 11:35 ` Krzysztof Kozlowski 2022-06-02 13:17 ` Teresa Remmet 0 siblings, 1 reply; 6+ messages in thread From: Krzysztof Kozlowski @ 2022-06-02 11:35 UTC (permalink / raw) To: Teresa Remmet, linux-arm-kernel, devicetree Cc: Rob Herring, Krzysztof Kozlowski, Shawn Guo, Fabio Estevam, NXP Linux Team, Sascha Hauer, upstream On 02/06/2022 12:11, Teresa Remmet wrote: > Add initial support for phyBOARD-Polis-i.MX8MM. > Main features are: > * Bluetooth and Wifi > * CANFD > * eMMC > * i2c RTC > * Ethernet > * PCIe > * RS232/RS485 > * SD-Card > * SPI-NOR flash > * USB Thank you for your patch. There is something to discuss/improve. > > Signed-off-by: Teresa Remmet <t.remmet@phytec.de> > --- > arch/arm64/boot/dts/freescale/Makefile | 1 + > .../freescale/imx8mm-phyboard-polis-rdk.dts | 450 ++++++++++++++++++ > .../dts/freescale/imx8mm-phycore-som.dtsi | 433 +++++++++++++++++ > 3 files changed, 884 insertions(+) > create mode 100644 arch/arm64/boot/dts/freescale/imx8mm-phyboard-polis-rdk.dts > create mode 100644 arch/arm64/boot/dts/freescale/imx8mm-phycore-som.dtsi > > diff --git a/arch/arm64/boot/dts/freescale/Makefile b/arch/arm64/boot/dts/freescale/Makefile > index 238a83e5b8c6..1e25dec2109b 100644 > --- a/arch/arm64/boot/dts/freescale/Makefile > +++ b/arch/arm64/boot/dts/freescale/Makefile > @@ -58,6 +58,7 @@ dtb-$(CONFIG_ARCH_MXC) += imx8mm-icore-mx8mm-edimm2.2.dtb > dtb-$(CONFIG_ARCH_MXC) += imx8mm-kontron-n801x-s.dtb > dtb-$(CONFIG_ARCH_MXC) += imx8mm-mx8menlo.dtb > dtb-$(CONFIG_ARCH_MXC) += imx8mm-nitrogen-r2.dtb > +dtb-$(CONFIG_ARCH_MXC) += imx8mm-phyboard-polis-rdk.dtb > dtb-$(CONFIG_ARCH_MXC) += imx8mm-tqma8mqml-mba8mx.dtb > dtb-$(CONFIG_ARCH_MXC) += imx8mm-var-som-symphony.dtb > dtb-$(CONFIG_ARCH_MXC) += imx8mm-venice-gw71xx-0x.dtb > diff --git a/arch/arm64/boot/dts/freescale/imx8mm-phyboard-polis-rdk.dts b/arch/arm64/boot/dts/freescale/imx8mm-phyboard-polis-rdk.dts > new file mode 100644 > index 000000000000..532b435b6871 > --- /dev/null > +++ b/arch/arm64/boot/dts/freescale/imx8mm-phyboard-polis-rdk.dts > @@ -0,0 +1,450 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* > + * Copyright (C) 2022 PHYTEC Messtechnik GmbH > + * Author: Teresa Remmet <t.remmet@phytec.de> > + */ > + > +/dts-v1/; > + > +#include <dt-bindings/interrupt-controller/irq.h> > +#include <dt-bindings/leds/common.h> > +#include <dt-bindings/phy/phy-imx8-pcie.h> > +#include "imx8mm-phycore-som.dtsi" > + > +/ { > + model = "PHYTEC phyBOARD-Polis-i.MX8MM RDK"; > + compatible = "phytec,imx8mm-phyboard-polis-rdk", > + "phytec,imx8mm-phycore-som","fsl,imx8mm"; Missing space after comma ^^^ > + > + chosen { > + stdout-path = &uart3; > + }; > + > + bt_osc_32k: bt_lp-clock { No underscores in node names, please. > + compatible = "fixed-clock"; > + clock-frequency = <32768>; > + clock-output-names = "bt_osc_32k"; > + #clock-cells = <0>; > + }; > + > + can_osc_40m: can-clock { > + compatible = "fixed-clock"; > + clock-frequency = <40000000>; > + clock-output-names = "can_osc_40m"; > + #clock-cells = <0>; > + }; > + > + fan { > + compatible = "gpio-fan"; > + gpios = <&gpio4 8 GPIO_ACTIVE_HIGH>; > + gpio-fan,speed-map = <0 0 > + 13000 1>;> + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_fan>; > + #cooling-cells = <2>; > + }; > + > + leds { > + compatible = "gpio-leds"; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_leds>; > + > + led-0 { > + color = <LED_COLOR_ID_RED>; > + function = LED_FUNCTION_DISK; > + gpios = <&gpio1 1 GPIO_ACTIVE_HIGH>; > + linux,default-trigger = "mmc2"; > + }; > + > + led-1 { > + color = <LED_COLOR_ID_BLUE>; > + function = LED_FUNCTION_DISK; > + gpios = <&gpio1 15 GPIO_ACTIVE_HIGH>; > + linux,default-trigger = "mmc1"; > + }; > + > + led-2 { > + color = <LED_COLOR_ID_GREEN>; > + function = LED_FUNCTION_CPU; > + gpios = <&gpio1 14 GPIO_ACTIVE_HIGH>; > + linux,default-trigger = "heartbeat"; > + }; > + }; > + > + usdhc1_pwrseq: pwr-seq { > + compatible = "mmc-pwrseq-simple"; > + post-power-on-delay-ms = <100>; > + power-off-delay-us = <60>; > + reset-gpios = <&gpio2 7 GPIO_ACTIVE_LOW>; > + }; > + > + reg_can_en: regulator-can-en { > + compatible = "regulator-fixed"; > + gpio = <&gpio1 9 GPIO_ACTIVE_LOW>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_can_en>; > + regulator-max-microvolt = <3300000>; > + regulator-min-microvolt = <3300000>; > + regulator-name = "CAN_EN"; > + startup-delay-us = <20>; > + }; > + > + reg_usb_otg1_vbus: regulator-usb-otg1 { > + compatible = "regulator-fixed"; > + enable-active-high; > + gpio = <&gpio1 12 GPIO_ACTIVE_HIGH>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_usbotg1pwrgrp>; > + regulator-name = "usb_otg1_vbus"; > + regulator-max-microvolt = <5000000>; > + regulator-min-microvolt = <5000000>; > + }; > + > + reg_usdhc2_vmmc: regulator-usdhc2 { Indentation (here or previous nodes) look odd, probably wrong. > + compatible = "regulator-fixed"; > + enable-active-high; > + gpio = <&gpio2 19 GPIO_ACTIVE_HIGH>; > + off-on-delay-us = <20000>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_reg_usdhc2_vmmc>; > + regulator-max-microvolt = <3300000>; > + regulator-min-microvolt = <3300000>; > + regulator-name = "VSD_3V3"; > + }; > + > + reg_vcc_3v3: regulator-vcc_3v3 { No underscores in node names. > + compatible = "regulator-fixed"; > + regulator-max-microvolt = <3300000>; > + regulator-min-microvolt = <3300000>; > + regulator-name = "VCC_3V3"; > + }; > +}; > + > +/* SPI - CAN MCP251XFD */ > +&ecspi1 { > + cs-gpios = <&gpio5 9 GPIO_ACTIVE_LOW>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_ecspi1>; > + status = "okay"; > + > + can0: can@0 { > + compatible = "microchip,mcp251xfd"; > + clocks = <&can_osc_40m>; > + interrupt-parent = <&gpio1>; > + interrupts = <8 IRQ_TYPE_LEVEL_LOW>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_can_int>; > + reg = <0>; > + spi-max-frequency = <20000000>; > + xceiver-supply = <®_can_en>; > + }; > +}; > + > +&gpio1 { > + gpio-line-names = "nINT_ETHPHY", "LED_RED", "WDOG_INT", "X_RTC_INT", > + "", "", "", "RESET_ETHPHY", > + "CAN_nINT", "CAN_EN", "nENABLE_FLATLINK", "", > + "USB_OTG_VBUS_EN", "", "LED_GREEN", "LED_BLUE"; > +}; > + > +&gpio2 { > + gpio-line-names = "", "", "", "", > + "", "", "BT_REG_ON", "WL_REG_ON", > + "BT_DEV_WAKE", "BT_HOST_WAKE", "", "", > + "X_SD2_CD_B", "", "", "", > + "", "", "", "SD2_RESET_B"; > +}; > + > +&gpio4 { > + gpio-line-names = "", "", "", "", > + "", "", "", "", > + "FAN", "miniPCIe_nPERST", "", "", > + "COEX1", "COEX2"; > +}; > + > +&gpio5 { > + gpio-line-names = "", "", "", "", > + "", "", "", "", > + "", "ECSPI1_SS0"; > +}; > + > +/* PCIe */ > +&pcie0 { > + assigned-clocks = <&clk IMX8MM_CLK_PCIE1_AUX>, > + <&clk IMX8MM_CLK_PCIE1_CTRL>; > + assigned-clock-parents = <&clk IMX8MM_SYS_PLL2_50M>, > + <&clk IMX8MM_SYS_PLL2_250M>; > + assigned-clock-rates = <10000000>, <250000000>; > + clocks = <&clk IMX8MM_CLK_PCIE1_ROOT>, <&clk IMX8MM_CLK_PCIE1_AUX>, > + <&clk IMX8MM_CLK_PCIE1_PHY>; > + clock-names = "pcie", "pcie_aux", "pcie_bus"; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_pcie>; > + reset-gpio = <&gpio4 9 GPIO_ACTIVE_LOW>; > + status = "okay"; > +}; > + > +&pcie_phy { > + clocks = <&clk IMX8MM_CLK_PCIE1_PHY>; > + fsl,clkreq-unsupported; > + fsl,refclk-pad-mode = <IMX8_PCIE_REFCLK_PAD_OUTPUT>; > + fsl,tx-deemph-gen1 = <0x2d>; > + fsl,tx-deemph-gen2 = <0xf>; > + status = "okay"; > +}; > + > +&rv3028 { > + trickle-resistor-ohms = <3000>; > +}; > + > +&snvs_pwrkey { > + status = "okay"; > +}; > + > +/* UART - RS232/RS485 */ > +&uart1 { > + assigned-clocks = <&clk IMX8MM_CLK_UART1>; > + assigned-clock-parents = <&clk IMX8MM_SYS_PLL1_80M>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_uart1>; > + status = "okay"; > + uart-has-rtscts; > +}; > + > +/* UART - Sterling-LWB Bluetooth */ > +&uart2 { > + assigned-clocks = <&clk IMX8MM_CLK_UART2>; > + assigned-clock-parents = <&clk IMX8MM_SYS_PLL1_80M>; > + fsl,dte-mode; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_uart2_bt>; > + status = "okay"; Put status as last property, so after uart-has-rtscts. The same in other places. > + uart-has-rtscts; > + (...) > diff --git a/arch/arm64/boot/dts/freescale/imx8mm-phycore-som.dtsi b/arch/arm64/boot/dts/freescale/imx8mm-phycore-som.dtsi > new file mode 100644 > index 000000000000..1aa27291c7ff > --- /dev/null > +++ b/arch/arm64/boot/dts/freescale/imx8mm-phycore-som.dtsi > @@ -0,0 +1,433 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* > + * Copyright (C) 2022 PHYTEC Messtechnik GmbH > + * Author: Teresa Remmet <t.remmet@phytec.de> > + */ > + > +#include "imx8mm.dtsi" > +#include <dt-bindings/net/ti-dp83867.h> > + > +/ { > + model = "PHYTEC phyCORE-i.MX8MM"; > + compatible = "phytec,imx8mm-phycore-som", "fsl,imx8mm"; > + > + aliases { > + rtc0 = &rv3028; > + rtc1 = &snvs_rtc; > + }; > + > + memory@40000000 { > + device_type = "memory"; > + reg = <0x0 0x40000000 0 0x80000000>; > + }; > + > + reg_vdd_3v3_s: regulator-vdd_3v3_s { Do not use underscores in node names. > + compatible = "regulator-fixed"; > + regulator-always-on; > + regulator-boot-on; > + regulator-max-microvolt = <3300000>; > + regulator-min-microvolt = <3300000>; > + regulator-name = "VDD_3V3_S"; > + }; > +}; > + > +&A53_0 { > + cpu-supply = <®_vdd_arm>; > +}; > + > +&A53_1 { > + cpu-supply = <®_vdd_arm>; > +}; > + > +&A53_2 { > + cpu-supply = <®_vdd_arm>; > +}; > + > +&A53_3 { > + cpu-supply = <®_vdd_arm>; > +}; > + > +&ddrc { > + operating-points-v2 = <&ddrc_opp_table>; > + > + ddrc_opp_table: opp-table { > + compatible = "operating-points-v2"; > + > + opp-25M { > + opp-hz = /bits/ 64 <25000000>; > + }; > + > + opp-100M { > + opp-hz = /bits/ 64 <100000000>; > + }; > + > + opp-750M { > + opp-hz = /bits/ 64 <750000000>; > + }; > + }; > +}; > + > +/* Ethernet */ > +&fec1 { > + fsl,magic-packet; > + phy-mode = "rgmii-id"; > + phy-handle = <ðphy0>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_fec1>; > + status = "okay"; > + > + mdio { > + #address-cells = <1>; > + #size-cells = <0>; > + > + ethphy0: ethernet-phy@0 { > + compatible = "ethernet-phy-ieee802.3-c22"; > + enet-phy-lane-no-swap; > + interrupts = <0 IRQ_TYPE_EDGE_FALLING>; > + interrupt-parent = <&gpio1>; > + ti,clk-output-sel = <DP83867_CLK_O_SEL_OFF>; > + ti,fifo-depth = <DP83867_PHYCR_FIFO_DEPTH_4_B_NIB>; > + ti,rx-internal-delay = <DP83867_RGMIIDCTL_2_00_NS>; > + ti,tx-internal-delay = <DP83867_RGMIIDCTL_2_00_NS>; > + reg = <0>; > + reset-gpios = <&gpio1 7 GPIO_ACTIVE_HIGH>; > + reset-assert-us = <1000>; > + reset-deassert-us = <1000>; > + }; > + }; > +}; > + > +/* SPI Flash */ > +&flexspi { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_flexspi0>; > + status = "okay"; > + > + som_flash: flash@0 { > + compatible = "jedec,spi-nor"; > + reg = <0>; > + spi-max-frequency = <80000000>; > + spi-rx-bus-width = <4>; > + spi-tx-bus-width = <1>; > + }; > +}; > + > +&gpio1 { > + gpio-line-names = "nINT_ETHPHY", "", "WDOG_INT", "X_RTC_INT", > + "", "", "", "RESET_ETHPHY", > + "", "", "nENABLE_FLATLINK"; > +}; > + > +/* I2C1 */ > +&i2c1 { > + clock-frequency = <400000>; > + pinctrl-names = "default","gpio"; > + pinctrl-0 = <&pinctrl_i2c1>; > + pinctrl-1 = <&pinctrl_i2c1_gpio>; > + scl-gpios = <&gpio5 14 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>; > + sda-gpios = <&gpio5 15 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>; > + status = "okay"; > + > + pmic@8 { > + compatible = "nxp,pf8121a"; > + reg = <0x08>; > + > + regulators { > + reg_nvcc_sd1: ldo1 { > + regulator-always-on; > + regulator-boot-on; > + regulator-max-microvolt = <3300000>; > + regulator-min-microvolt = <3300000>; > + regulator-name = "NVCC_SD1 (LDO1)"; > + regulator-state-mem { > + regulator-off-in-suspend; > + }; > + }; > + > + reg_nvcc_sd2: ldo2 { > + regulator-always-on; > + regulator-boot-on; > + regulator-max-microvolt = <3300000>; > + regulator-min-microvolt = <1800000>; > + regulator-name = "NVCC_SD2 (LDO2)"; > + vselect-en; > + regulator-state-mem { > + regulator-off-in-suspend; > + }; > + }; > + > + reg_vcc_enet: ldo3 { > + regulator-always-on; > + regulator-boot-on; > + regulator-max-microvolt = <2500000>; > + regulator-min-microvolt = <1500000>; > + regulator-name = "VCC_ENET_2V5 (LDO3)"; > + regulator-state-mem { > + regulator-off-in-suspend; > + }; > + }; > + > + reg_vdda_1v8: ldo4 { > + regulator-always-on; > + regulator-boot-on; > + regulator-max-microvolt = <1800000>; > + regulator-min-microvolt = <1500000>; > + regulator-name = "VDDA_1V8 (LDO4)"; > + regulator-state-mem { > + regulator-on-in-suspend; > + regulator-suspend-min-microvolt = <1500000>; > + regulator-suspend-max-microvolt = <1500000>; > + }; > + }; > + > + reg_soc_vdda_phy: buck1 { > + regulator-always-on; > + regulator-boot-on; > + regulator-max-microvolt = <900000>; > + regulator-min-microvolt = <400000>; > + regulator-name = "VDD_SOC_VDDA_PHY_0P8 (BUCK1)"; > + regulator-state-mem { > + regulator-on-in-suspend; > + regulator-suspend-min-microvolt = <400000>; > + regulator-suspend-max-microvolt = <400000>; > + }; > + }; > + > + reg_vdd_gpu_dram: buck2 { > + regulator-always-on; > + regulator-boot-on; > + regulator-max-microvolt = <1000000>; > + regulator-min-microvolt = <1000000>; > + regulator-name = "VDD_GPU_DRAM (BUCK2)"; > + regulator-state-mem { > + regulator-on-in-suspend; > + regulator-suspend-max-microvolt = <1000000>; > + regulator-suspend-min-microvolt = <1000000>; > + }; > + }; > + > + reg_vdd_gpu: buck3 { > + regulator-always-on; > + regulator-boot-on; > + regulator-max-microvolt = <1000000>; > + regulator-min-microvolt = <400000>; > + regulator-name = "VDD_VPU (BUCK3)"; > + regulator-state-mem { > + regulator-off-in-suspend; > + }; > + }; > + > + reg_vdd_mipi: buck4 { > + regulator-always-on; > + regulator-boot-on; > + regulator-max-microvolt = <1050000>; > + regulator-min-microvolt = <900000>; > + regulator-name = "VDD_MIPI_0P9 (BUCK4)"; > + regulator-state-mem { > + regulator-off-in-suspend; > + }; > + }; > + > + reg_vdd_arm: buck5 { > + regulator-always-on; > + regulator-boot-on; > + regulator-max-microvolt = <1050000>; > + regulator-min-microvolt = <400000>; > + regulator-name = "VDD_ARM (BUCK5)"; > + regulator-state-mem { > + regulator-off-in-suspend; > + }; > + }; > + > + reg_vdd_1v8: buck6 { > + regulator-always-on; > + regulator-boot-on; > + regulator-max-microvolt = <1800000>; > + regulator-min-microvolt = <1800000>; > + regulator-name = "VDD_1V8 (BUCK6)"; > + regulator-state-mem { > + regulator-on-in-suspend; > + regulator-suspend-max-microvolt = <1800000>; > + regulator-suspend-min-microvolt = <1800000>; > + }; > + }; > + > + reg_nvcc_dram: buck7 { > + regulator-always-on; > + regulator-boot-on; > + regulator-max-microvolt = <1100000>; > + regulator-min-microvolt = <1100000>; > + regulator-name = "NVCC_DRAM_1P1V (BUCK7)"; > + }; > + > + reg_vsnvs: vsnvs { > + regulator-always-on; > + regulator-boot-on; > + regulator-max-microvolt = <1800000>; > + regulator-min-microvolt = <1800000>; > + regulator-name = "NVCC_SNVS_1P8 (VSNVS)"; > + }; > + }; > + }; > + > + eeprom@51 { > + compatible = "atmel,24c32"; > + pagesize = <32>; > + reg = <0x51>; > + vcc-supply = <®_vdd_3v3_s>; > + }; > + > + sn65dsi83: mipitolvds@2d { I think generic node name for this is "bridge"? > + compatible = "ti,sn65dsi83"; > + enable-gpios = <&gpio1 10 GPIO_ACTIVE_LOW>; > + pinctrl-names = "default"; Best regards, Krzysztof ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 3/3] arm64: dts: freescale: Add phyBOARD-Polis-i.MX8MM support 2022-06-02 11:35 ` Krzysztof Kozlowski @ 2022-06-02 13:17 ` Teresa Remmet 0 siblings, 0 replies; 6+ messages in thread From: Teresa Remmet @ 2022-06-02 13:17 UTC (permalink / raw) To: linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, krzysztof.kozlowski@linaro.org Cc: festevam@gmail.com, s.hauer@pengutronix.de, krzysztof.kozlowski+dt@linaro.org, upstream@phytec.de, shawnguo@kernel.org, robh+dt@kernel.org, linux-imx@nxp.com Hello Krzysztof, Am Donnerstag, dem 02.06.2022 um 13:35 +0200 schrieb Krzysztof Kozlowski: > On 02/06/2022 12:11, Teresa Remmet wrote: > > Add initial support for phyBOARD-Polis-i.MX8MM. > > Main features are: > > * Bluetooth and Wifi > > * CANFD > > * eMMC > > * i2c RTC > > * Ethernet > > * PCIe > > * RS232/RS485 > > * SD-Card > > * SPI-NOR flash > > * USB > > Thank you for your patch. There is something to discuss/improve. thank you very much for your review. I will address all of your comments in v2. Regards, Teresa > > > Signed-off-by: Teresa Remmet <t.remmet@phytec.de> > > --- > > arch/arm64/boot/dts/freescale/Makefile | 1 + > > .../freescale/imx8mm-phyboard-polis-rdk.dts | 450 > > ++++++++++++++++++ > > .../dts/freescale/imx8mm-phycore-som.dtsi | 433 > > +++++++++++++++++ > > 3 files changed, 884 insertions(+) > > create mode 100644 arch/arm64/boot/dts/freescale/imx8mm-phyboard- > > polis-rdk.dts > > create mode 100644 arch/arm64/boot/dts/freescale/imx8mm-phycore- > > som.dtsi > > > > diff --git a/arch/arm64/boot/dts/freescale/Makefile > > b/arch/arm64/boot/dts/freescale/Makefile > > index 238a83e5b8c6..1e25dec2109b 100644 > > --- a/arch/arm64/boot/dts/freescale/Makefile > > +++ b/arch/arm64/boot/dts/freescale/Makefile > > @@ -58,6 +58,7 @@ dtb-$(CONFIG_ARCH_MXC) += imx8mm-icore-mx8mm- > > edimm2.2.dtb > > dtb-$(CONFIG_ARCH_MXC) += imx8mm-kontron-n801x-s.dtb > > dtb-$(CONFIG_ARCH_MXC) += imx8mm-mx8menlo.dtb > > dtb-$(CONFIG_ARCH_MXC) += imx8mm-nitrogen-r2.dtb > > +dtb-$(CONFIG_ARCH_MXC) += imx8mm-phyboard-polis-rdk.dtb > > dtb-$(CONFIG_ARCH_MXC) += imx8mm-tqma8mqml-mba8mx.dtb > > dtb-$(CONFIG_ARCH_MXC) += imx8mm-var-som-symphony.dtb > > dtb-$(CONFIG_ARCH_MXC) += imx8mm-venice-gw71xx-0x.dtb > > diff --git a/arch/arm64/boot/dts/freescale/imx8mm-phyboard-polis- > > rdk.dts b/arch/arm64/boot/dts/freescale/imx8mm-phyboard-polis- > > rdk.dts > > new file mode 100644 > > index 000000000000..532b435b6871 > > --- /dev/null > > +++ b/arch/arm64/boot/dts/freescale/imx8mm-phyboard-polis-rdk.dts > > @@ -0,0 +1,450 @@ > > +// SPDX-License-Identifier: GPL-2.0 > > +/* > > + * Copyright (C) 2022 PHYTEC Messtechnik GmbH > > + * Author: Teresa Remmet <t.remmet@phytec.de> > > + */ > > + > > +/dts-v1/; > > + > > +#include <dt-bindings/interrupt-controller/irq.h> > > +#include <dt-bindings/leds/common.h> > > +#include <dt-bindings/phy/phy-imx8-pcie.h> > > +#include "imx8mm-phycore-som.dtsi" > > + > > +/ { > > + model = "PHYTEC phyBOARD-Polis-i.MX8MM RDK"; > > + compatible = "phytec,imx8mm-phyboard-polis-rdk", > > + "phytec,imx8mm-phycore-som","fsl,imx8mm"; > > Missing space after comma ^^^ > > > + > > + chosen { > > + stdout-path = &uart3; > > + }; > > + > > + bt_osc_32k: bt_lp-clock { > > No underscores in node names, please. > > > + compatible = "fixed-clock"; > > + clock-frequency = <32768>; > > + clock-output-names = "bt_osc_32k"; > > + #clock-cells = <0>; > > + }; > > + > > + can_osc_40m: can-clock { > > + compatible = "fixed-clock"; > > + clock-frequency = <40000000>; > > + clock-output-names = "can_osc_40m"; > > + #clock-cells = <0>; > > + }; > > + > > + fan { > > + compatible = "gpio-fan"; > > + gpios = <&gpio4 8 GPIO_ACTIVE_HIGH>; > > + gpio-fan,speed-map = <0 0 > > + 13000 1>;> + pinctrl- > > names = "default"; > > + pinctrl-0 = <&pinctrl_fan>; > > + #cooling-cells = <2>; > > + }; > > + > > + leds { > > + compatible = "gpio-leds"; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_leds>; > > + > > + led-0 { > > + color = <LED_COLOR_ID_RED>; > > + function = LED_FUNCTION_DISK; > > + gpios = <&gpio1 1 GPIO_ACTIVE_HIGH>; > > + linux,default-trigger = "mmc2"; > > + }; > > + > > + led-1 { > > + color = <LED_COLOR_ID_BLUE>; > > + function = LED_FUNCTION_DISK; > > + gpios = <&gpio1 15 GPIO_ACTIVE_HIGH>; > > + linux,default-trigger = "mmc1"; > > + }; > > + > > + led-2 { > > + color = <LED_COLOR_ID_GREEN>; > > + function = LED_FUNCTION_CPU; > > + gpios = <&gpio1 14 GPIO_ACTIVE_HIGH>; > > + linux,default-trigger = "heartbeat"; > > + }; > > + }; > > + > > + usdhc1_pwrseq: pwr-seq { > > + compatible = "mmc-pwrseq-simple"; > > + post-power-on-delay-ms = <100>; > > + power-off-delay-us = <60>; > > + reset-gpios = <&gpio2 7 GPIO_ACTIVE_LOW>; > > + }; > > + > > + reg_can_en: regulator-can-en { > > + compatible = "regulator-fixed"; > > + gpio = <&gpio1 9 GPIO_ACTIVE_LOW>; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_can_en>; > > + regulator-max-microvolt = <3300000>; > > + regulator-min-microvolt = <3300000>; > > + regulator-name = "CAN_EN"; > > + startup-delay-us = <20>; > > + }; > > + > > + reg_usb_otg1_vbus: regulator-usb-otg1 { > > + compatible = "regulator-fixed"; > > + enable-active-high; > > + gpio = <&gpio1 12 GPIO_ACTIVE_HIGH>; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_usbotg1pwrgrp>; > > + regulator-name = "usb_otg1_vbus"; > > + regulator-max-microvolt = <5000000>; > > + regulator-min-microvolt = <5000000>; > > + }; > > + > > + reg_usdhc2_vmmc: regulator-usdhc2 { > > Indentation (here or previous nodes) look odd, probably wrong. > > > + compatible = "regulator-fixed"; > > + enable-active-high; > > + gpio = <&gpio2 19 GPIO_ACTIVE_HIGH>; > > + off-on-delay-us = <20000>; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_reg_usdhc2_vmmc>; > > + regulator-max-microvolt = <3300000>; > > + regulator-min-microvolt = <3300000>; > > + regulator-name = "VSD_3V3"; > > + }; > > + > > + reg_vcc_3v3: regulator-vcc_3v3 { > > No underscores in node names. > > > + compatible = "regulator-fixed"; > > + regulator-max-microvolt = <3300000>; > > + regulator-min-microvolt = <3300000>; > > + regulator-name = "VCC_3V3"; > > + }; > > +}; > > + > > +/* SPI - CAN MCP251XFD */ > > +&ecspi1 { > > + cs-gpios = <&gpio5 9 GPIO_ACTIVE_LOW>; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_ecspi1>; > > + status = "okay"; > > + > > + can0: can@0 { > > + compatible = "microchip,mcp251xfd"; > > + clocks = <&can_osc_40m>; > > + interrupt-parent = <&gpio1>; > > + interrupts = <8 IRQ_TYPE_LEVEL_LOW>; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_can_int>; > > + reg = <0>; > > + spi-max-frequency = <20000000>; > > + xceiver-supply = <®_can_en>; > > + }; > > +}; > > + > > +&gpio1 { > > + gpio-line-names = "nINT_ETHPHY", "LED_RED", "WDOG_INT", > > "X_RTC_INT", > > + "", "", "", "RESET_ETHPHY", > > + "CAN_nINT", "CAN_EN", "nENABLE_FLATLINK", "", > > + "USB_OTG_VBUS_EN", "", "LED_GREEN", "LED_BLUE"; > > +}; > > + > > +&gpio2 { > > + gpio-line-names = "", "", "", "", > > + "", "", "BT_REG_ON", "WL_REG_ON", > > + "BT_DEV_WAKE", "BT_HOST_WAKE", "", "", > > + "X_SD2_CD_B", "", "", "", > > + "", "", "", "SD2_RESET_B"; > > +}; > > + > > +&gpio4 { > > + gpio-line-names = "", "", "", "", > > + "", "", "", "", > > + "FAN", "miniPCIe_nPERST", "", "", > > + "COEX1", "COEX2"; > > +}; > > + > > +&gpio5 { > > + gpio-line-names = "", "", "", "", > > + "", "", "", "", > > + "", "ECSPI1_SS0"; > > +}; > > + > > +/* PCIe */ > > +&pcie0 { > > + assigned-clocks = <&clk IMX8MM_CLK_PCIE1_AUX>, > > + <&clk IMX8MM_CLK_PCIE1_CTRL>; > > + assigned-clock-parents = <&clk IMX8MM_SYS_PLL2_50M>, > > + <&clk IMX8MM_SYS_PLL2_250M>; > > + assigned-clock-rates = <10000000>, <250000000>; > > + clocks = <&clk IMX8MM_CLK_PCIE1_ROOT>, <&clk > > IMX8MM_CLK_PCIE1_AUX>, > > + <&clk IMX8MM_CLK_PCIE1_PHY>; > > + clock-names = "pcie", "pcie_aux", "pcie_bus"; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_pcie>; > > + reset-gpio = <&gpio4 9 GPIO_ACTIVE_LOW>; > > + status = "okay"; > > +}; > > + > > +&pcie_phy { > > + clocks = <&clk IMX8MM_CLK_PCIE1_PHY>; > > + fsl,clkreq-unsupported; > > + fsl,refclk-pad-mode = <IMX8_PCIE_REFCLK_PAD_OUTPUT>; > > + fsl,tx-deemph-gen1 = <0x2d>; > > + fsl,tx-deemph-gen2 = <0xf>; > > + status = "okay"; > > +}; > > + > > +&rv3028 { > > + trickle-resistor-ohms = <3000>; > > +}; > > + > > +&snvs_pwrkey { > > + status = "okay"; > > +}; > > + > > +/* UART - RS232/RS485 */ > > +&uart1 { > > + assigned-clocks = <&clk IMX8MM_CLK_UART1>; > > + assigned-clock-parents = <&clk IMX8MM_SYS_PLL1_80M>; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_uart1>; > > + status = "okay"; > > + uart-has-rtscts; > > +}; > > + > > +/* UART - Sterling-LWB Bluetooth */ > > +&uart2 { > > + assigned-clocks = <&clk IMX8MM_CLK_UART2>; > > + assigned-clock-parents = <&clk IMX8MM_SYS_PLL1_80M>; > > + fsl,dte-mode; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_uart2_bt>; > > + status = "okay"; > > Put status as last property, so after uart-has-rtscts. The same in > other > places. > > > + uart-has-rtscts; > > + > > (...) > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mm-phycore-som.dtsi > > b/arch/arm64/boot/dts/freescale/imx8mm-phycore-som.dtsi > > new file mode 100644 > > index 000000000000..1aa27291c7ff > > --- /dev/null > > +++ b/arch/arm64/boot/dts/freescale/imx8mm-phycore-som.dtsi > > @@ -0,0 +1,433 @@ > > +// SPDX-License-Identifier: GPL-2.0 > > +/* > > + * Copyright (C) 2022 PHYTEC Messtechnik GmbH > > + * Author: Teresa Remmet <t.remmet@phytec.de> > > + */ > > + > > +#include "imx8mm.dtsi" > > +#include <dt-bindings/net/ti-dp83867.h> > > + > > +/ { > > + model = "PHYTEC phyCORE-i.MX8MM"; > > + compatible = "phytec,imx8mm-phycore-som", "fsl,imx8mm"; > > + > > + aliases { > > + rtc0 = &rv3028; > > + rtc1 = &snvs_rtc; > > + }; > > + > > + memory@40000000 { > > + device_type = "memory"; > > + reg = <0x0 0x40000000 0 0x80000000>; > > + }; > > + > > + reg_vdd_3v3_s: regulator-vdd_3v3_s { > > Do not use underscores in node names. > > + compatible = "regulator-fixed"; > > + regulator-always-on; > > + regulator-boot-on; > > + regulator-max-microvolt = <3300000>; > > + regulator-min-microvolt = <3300000>; > > + regulator-name = "VDD_3V3_S"; > > + }; > > +}; > > + > > +&A53_0 { > > + cpu-supply = <®_vdd_arm>; > > +}; > > + > > +&A53_1 { > > + cpu-supply = <®_vdd_arm>; > > +}; > > + > > +&A53_2 { > > + cpu-supply = <®_vdd_arm>; > > +}; > > + > > +&A53_3 { > > + cpu-supply = <®_vdd_arm>; > > +}; > > + > > +&ddrc { > > + operating-points-v2 = <&ddrc_opp_table>; > > + > > + ddrc_opp_table: opp-table { > > + compatible = "operating-points-v2"; > > + > > + opp-25M { > > + opp-hz = /bits/ 64 <25000000>; > > + }; > > + > > + opp-100M { > > + opp-hz = /bits/ 64 <100000000>; > > + }; > > + > > + opp-750M { > > + opp-hz = /bits/ 64 <750000000>; > > + }; > > + }; > > +}; > > + > > +/* Ethernet */ > > +&fec1 { > > + fsl,magic-packet; > > + phy-mode = "rgmii-id"; > > + phy-handle = <ðphy0>; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_fec1>; > > + status = "okay"; > > + > > + mdio { > > + #address-cells = <1>; > > + #size-cells = <0>; > > + > > + ethphy0: ethernet-phy@0 { > > + compatible = "ethernet-phy-ieee802.3-c22"; > > + enet-phy-lane-no-swap; > > + interrupts = <0 IRQ_TYPE_EDGE_FALLING>; > > + interrupt-parent = <&gpio1>; > > + ti,clk-output-sel = <DP83867_CLK_O_SEL_OFF>; > > + ti,fifo-depth = > > <DP83867_PHYCR_FIFO_DEPTH_4_B_NIB>; > > + ti,rx-internal-delay = > > <DP83867_RGMIIDCTL_2_00_NS>; > > + ti,tx-internal-delay = > > <DP83867_RGMIIDCTL_2_00_NS>; > > + reg = <0>; > > + reset-gpios = <&gpio1 7 GPIO_ACTIVE_HIGH>; > > + reset-assert-us = <1000>; > > + reset-deassert-us = <1000>; > > + }; > > + }; > > +}; > > + > > +/* SPI Flash */ > > +&flexspi { > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_flexspi0>; > > + status = "okay"; > > + > > + som_flash: flash@0 { > > + compatible = "jedec,spi-nor"; > > + reg = <0>; > > + spi-max-frequency = <80000000>; > > + spi-rx-bus-width = <4>; > > + spi-tx-bus-width = <1>; > > + }; > > +}; > > + > > +&gpio1 { > > + gpio-line-names = "nINT_ETHPHY", "", "WDOG_INT", "X_RTC_INT", > > + "", "", "", "RESET_ETHPHY", > > + "", "", "nENABLE_FLATLINK"; > > +}; > > + > > +/* I2C1 */ > > +&i2c1 { > > + clock-frequency = <400000>; > > + pinctrl-names = "default","gpio"; > > + pinctrl-0 = <&pinctrl_i2c1>; > > + pinctrl-1 = <&pinctrl_i2c1_gpio>; > > + scl-gpios = <&gpio5 14 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>; > > + sda-gpios = <&gpio5 15 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>; > > + status = "okay"; > > + > > + pmic@8 { > > + compatible = "nxp,pf8121a"; > > + reg = <0x08>; > > + > > + regulators { > > + reg_nvcc_sd1: ldo1 { > > + regulator-always-on; > > + regulator-boot-on; > > + regulator-max-microvolt = <3300000>; > > + regulator-min-microvolt = <3300000>; > > + regulator-name = "NVCC_SD1 (LDO1)"; > > + regulator-state-mem { > > + regulator-off-in-suspend; > > + }; > > + }; > > + > > + reg_nvcc_sd2: ldo2 { > > + regulator-always-on; > > + regulator-boot-on; > > + regulator-max-microvolt = <3300000>; > > + regulator-min-microvolt = <1800000>; > > + regulator-name = "NVCC_SD2 (LDO2)"; > > + vselect-en; > > + regulator-state-mem { > > + regulator-off-in-suspend; > > + }; > > + }; > > + > > + reg_vcc_enet: ldo3 { > > + regulator-always-on; > > + regulator-boot-on; > > + regulator-max-microvolt = <2500000>; > > + regulator-min-microvolt = <1500000>; > > + regulator-name = "VCC_ENET_2V5 (LDO3)"; > > + regulator-state-mem { > > + regulator-off-in-suspend; > > + }; > > + }; > > + > > + reg_vdda_1v8: ldo4 { > > + regulator-always-on; > > + regulator-boot-on; > > + regulator-max-microvolt = <1800000>; > > + regulator-min-microvolt = <1500000>; > > + regulator-name = "VDDA_1V8 (LDO4)"; > > + regulator-state-mem { > > + regulator-on-in-suspend; > > + regulator-suspend-min-microvolt > > = <1500000>; > > + regulator-suspend-max-microvolt > > = <1500000>; > > + }; > > + }; > > + > > + reg_soc_vdda_phy: buck1 { > > + regulator-always-on; > > + regulator-boot-on; > > + regulator-max-microvolt = <900000>; > > + regulator-min-microvolt = <400000>; > > + regulator-name = "VDD_SOC_VDDA_PHY_0P8 > > (BUCK1)"; > > + regulator-state-mem { > > + regulator-on-in-suspend; > > + regulator-suspend-min-microvolt > > = <400000>; > > + regulator-suspend-max-microvolt > > = <400000>; > > + }; > > + }; > > + > > + reg_vdd_gpu_dram: buck2 { > > + regulator-always-on; > > + regulator-boot-on; > > + regulator-max-microvolt = <1000000>; > > + regulator-min-microvolt = <1000000>; > > + regulator-name = "VDD_GPU_DRAM > > (BUCK2)"; > > + regulator-state-mem { > > + regulator-on-in-suspend; > > + regulator-suspend-max-microvolt > > = <1000000>; > > + regulator-suspend-min-microvolt > > = <1000000>; > > + }; > > + }; > > + > > + reg_vdd_gpu: buck3 { > > + regulator-always-on; > > + regulator-boot-on; > > + regulator-max-microvolt = <1000000>; > > + regulator-min-microvolt = <400000>; > > + regulator-name = "VDD_VPU (BUCK3)"; > > + regulator-state-mem { > > + regulator-off-in-suspend; > > + }; > > + }; > > + > > + reg_vdd_mipi: buck4 { > > + regulator-always-on; > > + regulator-boot-on; > > + regulator-max-microvolt = <1050000>; > > + regulator-min-microvolt = <900000>; > > + regulator-name = "VDD_MIPI_0P9 > > (BUCK4)"; > > + regulator-state-mem { > > + regulator-off-in-suspend; > > + }; > > + }; > > + > > + reg_vdd_arm: buck5 { > > + regulator-always-on; > > + regulator-boot-on; > > + regulator-max-microvolt = <1050000>; > > + regulator-min-microvolt = <400000>; > > + regulator-name = "VDD_ARM (BUCK5)"; > > + regulator-state-mem { > > + regulator-off-in-suspend; > > + }; > > + }; > > + > > + reg_vdd_1v8: buck6 { > > + regulator-always-on; > > + regulator-boot-on; > > + regulator-max-microvolt = <1800000>; > > + regulator-min-microvolt = <1800000>; > > + regulator-name = "VDD_1V8 (BUCK6)"; > > + regulator-state-mem { > > + regulator-on-in-suspend; > > + regulator-suspend-max-microvolt > > = <1800000>; > > + regulator-suspend-min-microvolt > > = <1800000>; > > + }; > > + }; > > + > > + reg_nvcc_dram: buck7 { > > + regulator-always-on; > > + regulator-boot-on; > > + regulator-max-microvolt = <1100000>; > > + regulator-min-microvolt = <1100000>; > > + regulator-name = "NVCC_DRAM_1P1V > > (BUCK7)"; > > + }; > > + > > + reg_vsnvs: vsnvs { > > + regulator-always-on; > > + regulator-boot-on; > > + regulator-max-microvolt = <1800000>; > > + regulator-min-microvolt = <1800000>; > > + regulator-name = "NVCC_SNVS_1P8 > > (VSNVS)"; > > + }; > > + }; > > + }; > > + > > + eeprom@51 { > > + compatible = "atmel,24c32"; > > + pagesize = <32>; > > + reg = <0x51>; > > + vcc-supply = <®_vdd_3v3_s>; > > + }; > > + > > + sn65dsi83: mipitolvds@2d { > > I think generic node name for this is "bridge"? > > > + compatible = "ti,sn65dsi83"; > > + enable-gpios = <&gpio1 10 GPIO_ACTIVE_LOW>; > > + pinctrl-names = "default"; > > Best regards, > Krzysztof -- PHYTEC Messtechnik GmbH | Robert-Koch-Str. 39 | 55129 Mainz, Germany Geschäftsführer: Dipl.-Ing. Michael Mitezki, Dipl.-Ing. Bodo Huber | Handelsregister Mainz HRB 4656 | Finanzamt Mainz | St.Nr. 266500608, DE 149059855 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/3] bindings: arm: fsl: Add PHYTEC i.MX8MM devicetree bindings 2022-06-02 10:11 [PATCH 1/3] bindings: arm: fsl: Add PHYTEC i.MX8MM devicetree bindings Teresa Remmet 2022-06-02 10:11 ` [PATCH 2/3] arm64: defconfig: Enable gpio-fan support Teresa Remmet 2022-06-02 10:11 ` [PATCH 3/3] arm64: dts: freescale: Add phyBOARD-Polis-i.MX8MM support Teresa Remmet @ 2022-06-02 11:30 ` Krzysztof Kozlowski 2 siblings, 0 replies; 6+ messages in thread From: Krzysztof Kozlowski @ 2022-06-02 11:30 UTC (permalink / raw) To: Teresa Remmet, linux-arm-kernel, devicetree Cc: Rob Herring, Krzysztof Kozlowski, Shawn Guo, Fabio Estevam, NXP Linux Team, Sascha Hauer, upstream On 02/06/2022 12:11, Teresa Remmet wrote: > Add devicetree bindings for i.MX8MM based phyCORE-i.MX8MM > and phyBOARD-Polis RDK. > > Signed-off-by: Teresa Remmet <t.remmet@phytec.de> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-06-02 13:17 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2022-06-02 10:11 [PATCH 1/3] bindings: arm: fsl: Add PHYTEC i.MX8MM devicetree bindings Teresa Remmet 2022-06-02 10:11 ` [PATCH 2/3] arm64: defconfig: Enable gpio-fan support Teresa Remmet 2022-06-02 10:11 ` [PATCH 3/3] arm64: dts: freescale: Add phyBOARD-Polis-i.MX8MM support Teresa Remmet 2022-06-02 11:35 ` Krzysztof Kozlowski 2022-06-02 13:17 ` Teresa Remmet 2022-06-02 11:30 ` [PATCH 1/3] bindings: arm: fsl: Add PHYTEC i.MX8MM devicetree bindings Krzysztof Kozlowski
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).