* [PATCH v2 0/3] board: imx8mn-rve-gateway: add support for RVE gateway board
@ 2023-11-01 18:35 Hugo Villeneuve
2023-11-01 18:35 ` [PATCH v2 1/3] dt-bindings: vendor-prefixes: add rve Hugo Villeneuve
` (2 more replies)
0 siblings, 3 replies; 8+ messages in thread
From: Hugo Villeneuve @ 2023-11-01 18:35 UTC (permalink / raw)
To: robh+dt, krzysztof.kozlowski+dt, conor+dt, hvilleneuve, shawnguo,
s.hauer, kernel, festevam, linux-imx, leoyang.li, robh
Cc: devicetree, linux-kernel, linux-arm-kernel, hugo
From: Hugo Villeneuve <hvilleneuve@dimonoff.com>
Hello,
this patch series add support for the RVE gateway board.
Thank you.
Link: [v1] https://lore.kernel.org/all/20231101144303.2653464-1-hugo@hugovil.com/raw
https://lore.kernel.org/all/20231101144057.2653271-1-hugo@hugovil.com/raw
Changes for V2:
- Combine separate patches as a series. Start series at 2 to avoid confusion.
Hugo Villeneuve (3):
dt-bindings: vendor-prefixes: add rve
dt-bindings: arm: fsl: add RVE gateway board
arm64: dts: freescale: introduce rve-gateway board
.../devicetree/bindings/arm/fsl.yaml | 1 +
.../devicetree/bindings/vendor-prefixes.yaml | 2 +
MAINTAINERS | 7 +
arch/arm64/boot/dts/freescale/Makefile | 1 +
.../boot/dts/freescale/imx8mn-rve-gateway.dts | 284 ++++++++++++++++++
5 files changed, 295 insertions(+)
create mode 100644 arch/arm64/boot/dts/freescale/imx8mn-rve-gateway.dts
base-commit: 8bc9e6515183935fa0cccaf67455c439afe4982b
--
2.39.2
^ permalink raw reply [flat|nested] 8+ messages in thread* [PATCH v2 1/3] dt-bindings: vendor-prefixes: add rve 2023-11-01 18:35 [PATCH v2 0/3] board: imx8mn-rve-gateway: add support for RVE gateway board Hugo Villeneuve @ 2023-11-01 18:35 ` Hugo Villeneuve 2023-11-02 7:30 ` Krzysztof Kozlowski 2023-11-01 18:35 ` [PATCH v2 2/3] dt-bindings: arm: fsl: add RVE gateway board Hugo Villeneuve 2023-11-01 18:35 ` [PATCH v2 3/3] arm64: dts: freescale: introduce rve-gateway board Hugo Villeneuve 2 siblings, 1 reply; 8+ messages in thread From: Hugo Villeneuve @ 2023-11-01 18:35 UTC (permalink / raw) To: robh+dt, krzysztof.kozlowski+dt, conor+dt, hvilleneuve, shawnguo, s.hauer, kernel, festevam, linux-imx, leoyang.li, robh Cc: devicetree, linux-kernel, linux-arm-kernel, hugo From: Hugo Villeneuve <hvilleneuve@dimonoff.com> Add vendor prefix for Recharge Véhicule Électrique (RVE), which manufactures electric vehicle chargers infrastructure components. Signed-off-by: Hugo Villeneuve <hvilleneuve@dimonoff.com> --- Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml index e48b0a02827a..8455d7f5cfae 100644 --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml @@ -1161,6 +1161,8 @@ patternProperties: description: Shenzhen Roofull Technology Co, Ltd "^roseapplepi,.*": description: RoseapplePi.org + "^rve,.*": + description: Recharge Véhicule Électrique (RVE) inc. "^saef,.*": description: Saef Technology Limited "^samsung,.*": -- 2.39.2 ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v2 1/3] dt-bindings: vendor-prefixes: add rve 2023-11-01 18:35 ` [PATCH v2 1/3] dt-bindings: vendor-prefixes: add rve Hugo Villeneuve @ 2023-11-02 7:30 ` Krzysztof Kozlowski 0 siblings, 0 replies; 8+ messages in thread From: Krzysztof Kozlowski @ 2023-11-02 7:30 UTC (permalink / raw) To: Hugo Villeneuve, robh+dt, krzysztof.kozlowski+dt, conor+dt, hvilleneuve, shawnguo, s.hauer, kernel, festevam, linux-imx, leoyang.li, robh Cc: devicetree, linux-kernel, linux-arm-kernel On 01/11/2023 19:35, Hugo Villeneuve wrote: > From: Hugo Villeneuve <hvilleneuve@dimonoff.com> > > Add vendor prefix for Recharge Véhicule Électrique (RVE), which > manufactures electric vehicle chargers infrastructure components. > > Signed-off-by: Hugo Villeneuve <hvilleneuve@dimonoff.com> > --- Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v2 2/3] dt-bindings: arm: fsl: add RVE gateway board 2023-11-01 18:35 [PATCH v2 0/3] board: imx8mn-rve-gateway: add support for RVE gateway board Hugo Villeneuve 2023-11-01 18:35 ` [PATCH v2 1/3] dt-bindings: vendor-prefixes: add rve Hugo Villeneuve @ 2023-11-01 18:35 ` Hugo Villeneuve 2023-11-02 7:30 ` Krzysztof Kozlowski 2023-11-01 18:35 ` [PATCH v2 3/3] arm64: dts: freescale: introduce rve-gateway board Hugo Villeneuve 2 siblings, 1 reply; 8+ messages in thread From: Hugo Villeneuve @ 2023-11-01 18:35 UTC (permalink / raw) To: robh+dt, krzysztof.kozlowski+dt, conor+dt, hvilleneuve, shawnguo, s.hauer, kernel, festevam, linux-imx, leoyang.li, robh Cc: devicetree, linux-kernel, linux-arm-kernel, hugo From: Hugo Villeneuve <hvilleneuve@dimonoff.com> Add DT compatible string for RVE gateway board based on a Variscite VAR-SOM-NANO with a NXP MX8MN nano CPU. Signed-off-by: Hugo Villeneuve <hvilleneuve@dimonoff.com> --- Documentation/devicetree/bindings/arm/fsl.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml index 9450b2c8a678..a1028fe8ed02 100644 --- a/Documentation/devicetree/bindings/arm/fsl.yaml +++ b/Documentation/devicetree/bindings/arm/fsl.yaml @@ -1007,6 +1007,7 @@ properties: - fsl,imx8mn-ddr4-evk # i.MX8MN DDR4 EVK Board - fsl,imx8mn-evk # i.MX8MN LPDDR4 EVK Board - gw,imx8mn-gw7902 # i.MX8MM Gateworks Board + - rve,rve-gateway # i.MX8MN RVE Gateway Board - const: fsl,imx8mn - description: Variscite VAR-SOM-MX8MN based boards -- 2.39.2 ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v2 2/3] dt-bindings: arm: fsl: add RVE gateway board 2023-11-01 18:35 ` [PATCH v2 2/3] dt-bindings: arm: fsl: add RVE gateway board Hugo Villeneuve @ 2023-11-02 7:30 ` Krzysztof Kozlowski 0 siblings, 0 replies; 8+ messages in thread From: Krzysztof Kozlowski @ 2023-11-02 7:30 UTC (permalink / raw) To: Hugo Villeneuve, robh+dt, krzysztof.kozlowski+dt, conor+dt, hvilleneuve, shawnguo, s.hauer, kernel, festevam, linux-imx, leoyang.li, robh Cc: devicetree, linux-kernel, linux-arm-kernel On 01/11/2023 19:35, Hugo Villeneuve wrote: > From: Hugo Villeneuve <hvilleneuve@dimonoff.com> > > Add DT compatible string for RVE gateway board based on a Variscite > VAR-SOM-NANO with a NXP MX8MN nano CPU. > > Signed-off-by: Hugo Villeneuve <hvilleneuve@dimonoff.com> > --- Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v2 3/3] arm64: dts: freescale: introduce rve-gateway board 2023-11-01 18:35 [PATCH v2 0/3] board: imx8mn-rve-gateway: add support for RVE gateway board Hugo Villeneuve 2023-11-01 18:35 ` [PATCH v2 1/3] dt-bindings: vendor-prefixes: add rve Hugo Villeneuve 2023-11-01 18:35 ` [PATCH v2 2/3] dt-bindings: arm: fsl: add RVE gateway board Hugo Villeneuve @ 2023-11-01 18:35 ` Hugo Villeneuve 2023-11-02 7:36 ` Krzysztof Kozlowski 2 siblings, 1 reply; 8+ messages in thread From: Hugo Villeneuve @ 2023-11-01 18:35 UTC (permalink / raw) To: robh+dt, krzysztof.kozlowski+dt, conor+dt, hvilleneuve, shawnguo, s.hauer, kernel, festevam, linux-imx, leoyang.li, robh Cc: devicetree, linux-kernel, linux-arm-kernel, hugo From: Hugo Villeneuve <hvilleneuve@dimonoff.com> The RVE gateway board is based on a Variscite VAR-SOM-NANO, with a NXP MX8MN nano CPU. Signed-off-by: Hugo Villeneuve <hvilleneuve@dimonoff.com> --- MAINTAINERS | 7 + arch/arm64/boot/dts/freescale/Makefile | 1 + .../boot/dts/freescale/imx8mn-rve-gateway.dts | 284 ++++++++++++++++++ 3 files changed, 292 insertions(+) create mode 100644 arch/arm64/boot/dts/freescale/imx8mn-rve-gateway.dts diff --git a/MAINTAINERS b/MAINTAINERS index 43c5ed3c4508..9423b1bf35fc 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -18778,6 +18778,13 @@ F: samples/rust/ F: scripts/*rust* K: \b(?i:rust)\b +RVE BOARD SUPPORT +M: Hugo Villeneuve <hvilleneuve@dimonoff.com> +L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers) +S: Supported +W: https://rve.ca +F: arch/arm64/boot/dts/freescale/imx*rve-*.dts* + RXRPC SOCKETS (AF_RXRPC) M: David Howells <dhowells@redhat.com> M: Marc Dionne <marc.dionne@auristor.com> diff --git a/arch/arm64/boot/dts/freescale/Makefile b/arch/arm64/boot/dts/freescale/Makefile index 89aee6c92576..760a0ca7adc3 100644 --- a/arch/arm64/boot/dts/freescale/Makefile +++ b/arch/arm64/boot/dts/freescale/Makefile @@ -89,6 +89,7 @@ dtb-$(CONFIG_ARCH_MXC) += imx8mn-bsh-smm-s2pro.dtb dtb-$(CONFIG_ARCH_MXC) += imx8mn-evk.dtb dtb-$(CONFIG_ARCH_MXC) += imx8mn-ddr3l-evk.dtb dtb-$(CONFIG_ARCH_MXC) += imx8mn-ddr4-evk.dtb +dtb-$(CONFIG_ARCH_MXC) += imx8mn-rve-gateway.dtb dtb-$(CONFIG_ARCH_MXC) += imx8mn-tqma8mqnl-mba8mx.dtb dtb-$(CONFIG_ARCH_MXC) += imx8mn-var-som-symphony.dtb dtb-$(CONFIG_ARCH_MXC) += imx8mn-venice-gw7902.dtb diff --git a/arch/arm64/boot/dts/freescale/imx8mn-rve-gateway.dts b/arch/arm64/boot/dts/freescale/imx8mn-rve-gateway.dts new file mode 100644 index 000000000000..312718982a52 --- /dev/null +++ b/arch/arm64/boot/dts/freescale/imx8mn-rve-gateway.dts @@ -0,0 +1,284 @@ +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) +/* + * Copyright 2023 DimOnOff + */ + +/dts-v1/; + +#include <dt-bindings/usb/pd.h> +#include "imx8mn-var-som.dtsi" + +/ { + model = "RVE gateway"; + compatible = "rve,rve-gateway", "fsl,imx8mn"; + + crystal_duart_24m: crystal-duart-24m { + compatible = "fixed-clock"; + #clock-cells = <0>; + clock-frequency = <24000000>; + }; + + gpio-keys { + compatible = "gpio-keys"; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_gpio_keys>; + status = "okay"; + + key-enter { + label = "enter"; + gpios = <&gpio1 1 GPIO_ACTIVE_LOW>; + linux,code = <KEY_ENTER>; + }; + + key-exit { + label = "exit"; + gpios = <&gpio3 23 GPIO_ACTIVE_LOW>; + linux,code = <KEY_ESC>; + }; + }; + + lcd { + compatible = "hit,hd44780"; + display-height-chars = <2>; + display-width-chars = <20>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_lcd>; + data-gpios = + <&gpio5 1 GPIO_ACTIVE_HIGH>, + <&gpio1 6 GPIO_ACTIVE_HIGH>, + <&gpio1 14 GPIO_ACTIVE_HIGH>, + <&gpio4 28 GPIO_ACTIVE_HIGH>, + <&gpio5 24 GPIO_ACTIVE_HIGH>, + <&gpio5 2 GPIO_ACTIVE_HIGH>, + <&gpio1 12 GPIO_ACTIVE_HIGH>, + <&gpio5 25 GPIO_ACTIVE_HIGH>; + enable-gpios = <&gpio5 23 GPIO_ACTIVE_HIGH>; + rs-gpios = <&gpio1 7 GPIO_ACTIVE_HIGH>; + rw-gpios = <&gpio4 27 GPIO_ACTIVE_HIGH>; + }; + + reg_3p3v: regulator-3p3v { + compatible = "regulator-fixed"; + regulator-name = "3P3V"; + regulator-min-microvolt = <3300000>; + regulator-max-microvolt = <3300000>; + regulator-always-on; + }; + + rotary: rotary-encoder { /* Bourns PEC12R rotary encoder, 24 steps. */ + compatible = "rotary-encoder"; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_rotary>; + gpios = + <&gpio1 5 GPIO_ACTIVE_LOW>, /* A */ + <&gpio3 21 GPIO_ACTIVE_LOW>; /* B */ + linux,axis = <0>; /* REL_X */ + rotary-encoder,encoding = "gray"; + rotary-encoder,relative-axis; + }; +}; + +&ecspi1 { + cs-gpios = <&gpio5 9 GPIO_ACTIVE_LOW>; + + duart1: serial@0 { + compatible = "nxp,sc16is752"; + reg = <0>; + spi-rx-bus-width = <1>; + spi-tx-bus-width = <1>; + spi-max-frequency = <4000000>; + clocks = <&crystal_duart_24m>; + interrupt-parent = <&gpio3>; + interrupts = <22 IRQ_TYPE_EDGE_FALLING>; + gpio-controller; + #gpio-cells = <2>; + gpio-line-names = "RADIO0", "RADIO1", "RADIO2", "RADIO3", + "RADIO4", "RADIO_RESET", "TP12", "TP11"; + linux,rs485-enabled-at-boot-time; + rs485-rts-active-low; + }; + + /delete-node/ touchscreen@0; +}; + +&ecspi2 { + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_ecspi2>; + cs-gpios = <&gpio5 13 GPIO_ACTIVE_LOW>; + /delete-property/ dmas; + /delete-property/ dma-names; + status = "okay"; + + duart2: serial@0 { + compatible = "nxp,sc16is752"; + reg = <0>; + spi-rx-bus-width = <1>; + spi-tx-bus-width = <1>; + spi-max-frequency = <4000000>; + clocks = <&crystal_duart_24m>; + interrupt-parent = <&gpio3>; + interrupts = <20 IRQ_TYPE_EDGE_FALLING>; + gpio-controller; + #gpio-cells = <2>; + gpio-line-names = "LED_B_USER", "LED_R_USER", "LED_G_USER", + "GPIO_EXT3", "GPIO_EXT2", "GPIO_EXT1", "GPIO_EXT0", "TP13"; + linux,rs485-enabled-at-boot-time; + rs485-rts-active-low; + }; +}; + +/* Configure PWM pins in GPIO mode: */ +&gpio5 { + gpio-line-names = "", "", "", "PWM3", "PWM2", "PWM1"; +}; + +&gpu { + status = "disabled"; +}; + +&i2c2 { + clock-frequency = <400000>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_i2c2>; + status = "okay"; + + eeprom_cb: eeprom@56 { /* Carrier board EEPROM */ + compatible = "atmel,24c04"; + reg = <0x56>; + pagesize = <16>; + vcc-supply = <®_3p3v>; + }; + + lm75: sensor@48 { + compatible = "st,stlm75"; + reg = <0x48>; + vs-supply = <®_3p3v>; + }; + + mcp7940: rtc@6f { + compatible = "microchip,mcp7940x"; + reg = <0x6f>; + }; +}; + +&i2c3 { + codec@1a { + status = "disabled"; + }; +}; + +&i2c4 { + clock-frequency = <400000>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_i2c4>; + status = "okay"; + + pcf8574_1: gpio@38 { + status = "okay"; + compatible = "nxp,pcf8574"; + reg = <0x38>; + gpio-controller; + #gpio-cells = <2>; + gpio-line-names = "LED_B_COMM3", "LED_R_COMM3", "LED_G_COMM3", + "TP14", "TP15", "LED_G_COMM4", "LED_R_COMM4", "LED_B_COMM4"; + }; + + pcf8574_2: gpio@39 { + status = "okay"; + compatible = "nxp,pcf8574"; + reg = <0x39>; + gpio-controller; + #gpio-cells = <2>; + gpio-line-names = "LED_B_COMM2", "LED_G_COMM2", "LED_B_COMM1", + "LED_R_COMM2", "LED_R_COMM1", "LED_G_COMM1", "TP16", "TP17"; + }; +}; + +&uart2 { + /delete-node/ bluetooth; +}; + +&usbotg1 { + dr_mode = "host"; + disable-over-current; + status = "okay"; +}; + +/* SD interface on expansion connector. */ +&usdhc2 { + /delete-property/vmmc-supply; + cd-gpios = <&gpio1 13 GPIO_ACTIVE_LOW>; +}; + +&iomuxc { + pinctrl_ecspi1: ecspi1grp { + fsl,pins = < + MX8MN_IOMUXC_ECSPI1_SCLK_ECSPI1_SCLK 0x13 + MX8MN_IOMUXC_ECSPI1_MOSI_ECSPI1_MOSI 0x13 + MX8MN_IOMUXC_ECSPI1_MISO_ECSPI1_MISO 0x13 + MX8MN_IOMUXC_ECSPI1_SS0_GPIO5_IO9 0x13 /* SS0 */ + MX8MN_IOMUXC_SAI5_RXD1_GPIO3_IO22 0x13 /* SC16 IRQ */ + >; + }; + + pinctrl_ecspi2: ecspi2grp { + fsl,pins = < + MX8MN_IOMUXC_ECSPI2_SCLK_ECSPI2_SCLK 0x13 + MX8MN_IOMUXC_ECSPI2_MOSI_ECSPI2_MOSI 0x13 + MX8MN_IOMUXC_ECSPI2_MISO_ECSPI2_MISO 0x13 + MX8MN_IOMUXC_ECSPI2_SS0_GPIO5_IO13 0x13 /* SS0 */ + MX8MN_IOMUXC_SAI5_RXC_GPIO3_IO20 0x13 /* SC16 IRQ */ + >; + }; + + pinctrl_gpio_keys: gpiokeysgrp { + fsl,pins = < + MX8MN_IOMUXC_GPIO1_IO01_GPIO1_IO1 0xc6 /* Enter */ + MX8MN_IOMUXC_SAI5_RXD2_GPIO3_IO23 0xc6 /* Exit */ + >; + }; + + pinctrl_i2c2: i2c2grp { + fsl,pins = < + MX8MN_IOMUXC_I2C2_SCL_I2C2_SCL 0x400001c3 + MX8MN_IOMUXC_I2C2_SDA_I2C2_SDA 0x400001c3 + >; + }; + + pinctrl_i2c4: i2c4grp { + fsl,pins = < + MX8MN_IOMUXC_I2C4_SCL_I2C4_SCL 0x400001c3 + MX8MN_IOMUXC_I2C4_SDA_I2C4_SDA 0x400001c3 + >; + }; + + pinctrl_lcd: lcdgrp { + fsl,pins = < + MX8MN_IOMUXC_SAI3_TXD_GPIO5_IO1 0x00000156 /* D0 */ + MX8MN_IOMUXC_GPIO1_IO06_GPIO1_IO6 0x00000156 /* D1 */ + MX8MN_IOMUXC_GPIO1_IO14_GPIO1_IO14 0x00000156 /* D2 */ + MX8MN_IOMUXC_SAI3_RXFS_GPIO4_IO28 0x00000156 /* D3 */ + MX8MN_IOMUXC_UART2_RXD_GPIO5_IO24 0x00000156 /* D4 */ + MX8MN_IOMUXC_SAI3_MCLK_GPIO5_IO2 0x00000156 /* D5 */ + MX8MN_IOMUXC_GPIO1_IO12_GPIO1_IO12 0x00000156 /* D6 */ + MX8MN_IOMUXC_UART2_TXD_GPIO5_IO25 0x00000156 /* D7 */ + MX8MN_IOMUXC_UART1_TXD_GPIO5_IO23 0x00000156 /* E */ + MX8MN_IOMUXC_GPIO1_IO07_GPIO1_IO7 0x00000156 /* RS */ + MX8MN_IOMUXC_SAI2_MCLK_GPIO4_IO27 0x00000156 /* R/W */ + >; + }; + + pinctrl_rotary: rotarygrp { + fsl,pins = < + MX8MN_IOMUXC_GPIO1_IO05_GPIO1_IO5 0x00000156 /* A */ + MX8MN_IOMUXC_SAI5_RXD0_GPIO3_IO21 0x00000156 /* B */ + >; + }; + + /* Override Card Detect function GPIO value (GPIO1_IO10) from SOM: */ + pinctrl_usdhc2_gpio: usdhc2gpiogrp { + fsl,pins = < + MX8MN_IOMUXC_GPIO1_IO13_GPIO1_IO13 0x41 + >; + }; +}; -- 2.39.2 ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v2 3/3] arm64: dts: freescale: introduce rve-gateway board 2023-11-01 18:35 ` [PATCH v2 3/3] arm64: dts: freescale: introduce rve-gateway board Hugo Villeneuve @ 2023-11-02 7:36 ` Krzysztof Kozlowski 2023-11-02 15:10 ` Hugo Villeneuve 0 siblings, 1 reply; 8+ messages in thread From: Krzysztof Kozlowski @ 2023-11-02 7:36 UTC (permalink / raw) To: Hugo Villeneuve, robh+dt, krzysztof.kozlowski+dt, conor+dt, hvilleneuve, shawnguo, s.hauer, kernel, festevam, linux-imx, leoyang.li, robh Cc: devicetree, linux-kernel, linux-arm-kernel On 01/11/2023 19:35, Hugo Villeneuve wrote: > From: Hugo Villeneuve <hvilleneuve@dimonoff.com> > > The RVE gateway board is based on a Variscite VAR-SOM-NANO, > with a NXP MX8MN nano CPU. > > Signed-off-by: Hugo Villeneuve <hvilleneuve@dimonoff.com> > --- > MAINTAINERS | 7 + > arch/arm64/boot/dts/freescale/Makefile | 1 + > .../boot/dts/freescale/imx8mn-rve-gateway.dts | 284 ++++++++++++++++++ > 3 files changed, 292 insertions(+) > create mode 100644 arch/arm64/boot/dts/freescale/imx8mn-rve-gateway.dts > > diff --git a/MAINTAINERS b/MAINTAINERS > index 43c5ed3c4508..9423b1bf35fc 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -18778,6 +18778,13 @@ F: samples/rust/ > F: scripts/*rust* > K: \b(?i:rust)\b > > +RVE BOARD SUPPORT > +M: Hugo Villeneuve <hvilleneuve@dimonoff.com> > +L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers) > +S: Supported > +W: https://rve.ca > +F: arch/arm64/boot/dts/freescale/imx*rve-*.dts* I still don't think individual boards should receive maintainer entry. https://lore.kernel.org/all/20210809080204.8381-1-shawnguo@kernel.org/ > + > RXRPC SOCKETS (AF_RXRPC) > M: David Howells <dhowells@redhat.com> > M: Marc Dionne <marc.dionne@auristor.com> > diff --git a/arch/arm64/boot/dts/freescale/Makefile b/arch/arm64/boot/dts/freescale/Makefile > index 89aee6c92576..760a0ca7adc3 100644 > --- a/arch/arm64/boot/dts/freescale/Makefile > +++ b/arch/arm64/boot/dts/freescale/Makefile > @@ -89,6 +89,7 @@ dtb-$(CONFIG_ARCH_MXC) += imx8mn-bsh-smm-s2pro.dtb > dtb-$(CONFIG_ARCH_MXC) += imx8mn-evk.dtb > dtb-$(CONFIG_ARCH_MXC) += imx8mn-ddr3l-evk.dtb > dtb-$(CONFIG_ARCH_MXC) += imx8mn-ddr4-evk.dtb > +dtb-$(CONFIG_ARCH_MXC) += imx8mn-rve-gateway.dtb > dtb-$(CONFIG_ARCH_MXC) += imx8mn-tqma8mqnl-mba8mx.dtb > dtb-$(CONFIG_ARCH_MXC) += imx8mn-var-som-symphony.dtb > dtb-$(CONFIG_ARCH_MXC) += imx8mn-venice-gw7902.dtb > diff --git a/arch/arm64/boot/dts/freescale/imx8mn-rve-gateway.dts b/arch/arm64/boot/dts/freescale/imx8mn-rve-gateway.dts > new file mode 100644 > index 000000000000..312718982a52 > --- /dev/null > +++ b/arch/arm64/boot/dts/freescale/imx8mn-rve-gateway.dts > @@ -0,0 +1,284 @@ > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > +/* > + * Copyright 2023 DimOnOff > + */ > + > +/dts-v1/; > + > +#include <dt-bindings/usb/pd.h> > +#include "imx8mn-var-som.dtsi" > + > +/ { > + model = "RVE gateway"; > + compatible = "rve,rve-gateway", "fsl,imx8mn"; Missing compatible of SoM. > + > + crystal_duart_24m: crystal-duart-24m { > + compatible = "fixed-clock"; > + #clock-cells = <0>; > + clock-frequency = <24000000>; > + }; > + > + gpio-keys { > + compatible = "gpio-keys"; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_gpio_keys>; > + status = "okay"; Drop > + > + key-enter { > + label = "enter"; > + gpios = <&gpio1 1 GPIO_ACTIVE_LOW>; > + linux,code = <KEY_ENTER>; > + }; > + > + key-exit { > + label = "exit"; > + gpios = <&gpio3 23 GPIO_ACTIVE_LOW>; > + linux,code = <KEY_ESC>; > + }; > + }; > + > + lcd { > + compatible = "hit,hd44780"; > + display-height-chars = <2>; > + display-width-chars = <20>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_lcd>; > + data-gpios = > + <&gpio5 1 GPIO_ACTIVE_HIGH>, This should be after =, with the reset of lines properly aligned. > + <&gpio1 6 GPIO_ACTIVE_HIGH>, > + <&gpio1 14 GPIO_ACTIVE_HIGH>, > + <&gpio4 28 GPIO_ACTIVE_HIGH>, > + <&gpio5 24 GPIO_ACTIVE_HIGH>, > + <&gpio5 2 GPIO_ACTIVE_HIGH>, > + <&gpio1 12 GPIO_ACTIVE_HIGH>, > + <&gpio5 25 GPIO_ACTIVE_HIGH>; > + enable-gpios = <&gpio5 23 GPIO_ACTIVE_HIGH>; > + rs-gpios = <&gpio1 7 GPIO_ACTIVE_HIGH>; > + rw-gpios = <&gpio4 27 GPIO_ACTIVE_HIGH>; > + }; > + > + reg_3p3v: regulator-3p3v { > + compatible = "regulator-fixed"; > + regulator-name = "3P3V"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-always-on; > + }; Fix indentation. > + > + rotary: rotary-encoder { /* Bourns PEC12R rotary encoder, 24 steps. */ No comments after {. Put them in line before or after. This applies to other places as well. > + compatible = "rotary-encoder"; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_rotary>; > + gpios = > + <&gpio1 5 GPIO_ACTIVE_LOW>, /* A */ This should be after =, with the reset of lines properly aligned. > + <&gpio3 21 GPIO_ACTIVE_LOW>; /* B */ > + linux,axis = <0>; /* REL_X */ > + rotary-encoder,encoding = "gray"; > + rotary-encoder,relative-axis; > + }; > +}; > + > +&ecspi1 { > + cs-gpios = <&gpio5 9 GPIO_ACTIVE_LOW>; > + > + duart1: serial@0 { > + compatible = "nxp,sc16is752"; > + reg = <0>; > + spi-rx-bus-width = <1>; > + spi-tx-bus-width = <1>; > + spi-max-frequency = <4000000>; > + clocks = <&crystal_duart_24m>; > + interrupt-parent = <&gpio3>; > + interrupts = <22 IRQ_TYPE_EDGE_FALLING>; > + gpio-controller; > + #gpio-cells = <2>; > + gpio-line-names = "RADIO0", "RADIO1", "RADIO2", "RADIO3", > + "RADIO4", "RADIO_RESET", "TP12", "TP11"; > + linux,rs485-enabled-at-boot-time; > + rs485-rts-active-low; > + }; > + > + /delete-node/ touchscreen@0; > +}; > + > +&ecspi2 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_ecspi2>; > + cs-gpios = <&gpio5 13 GPIO_ACTIVE_LOW>; > + /delete-property/ dmas; > + /delete-property/ dma-names; > + status = "okay"; > + > + duart2: serial@0 { > + compatible = "nxp,sc16is752"; > + reg = <0>; > + spi-rx-bus-width = <1>; > + spi-tx-bus-width = <1>; > + spi-max-frequency = <4000000>; > + clocks = <&crystal_duart_24m>; > + interrupt-parent = <&gpio3>; > + interrupts = <20 IRQ_TYPE_EDGE_FALLING>; > + gpio-controller; > + #gpio-cells = <2>; > + gpio-line-names = "LED_B_USER", "LED_R_USER", "LED_G_USER", > + "GPIO_EXT3", "GPIO_EXT2", "GPIO_EXT1", "GPIO_EXT0", "TP13"; > + linux,rs485-enabled-at-boot-time; > + rs485-rts-active-low; > + }; > +}; > + > +/* Configure PWM pins in GPIO mode: */ > +&gpio5 { > + gpio-line-names = "", "", "", "PWM3", "PWM2", "PWM1"; > +}; > + > +&gpu { > + status = "disabled"; > +}; > + > +&i2c2 { > + clock-frequency = <400000>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_i2c2>; > + status = "okay"; > + > + eeprom_cb: eeprom@56 { /* Carrier board EEPROM */ > + compatible = "atmel,24c04"; > + reg = <0x56>; > + pagesize = <16>; > + vcc-supply = <®_3p3v>; > + }; > + > + lm75: sensor@48 { > + compatible = "st,stlm75"; > + reg = <0x48>; > + vs-supply = <®_3p3v>; > + }; > + > + mcp7940: rtc@6f { > + compatible = "microchip,mcp7940x"; > + reg = <0x6f>; > + }; > +}; > + > +&i2c3 { > + codec@1a { > + status = "disabled"; > + }; > +}; > + > +&i2c4 { > + clock-frequency = <400000>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_i2c4>; > + status = "okay"; > + > + pcf8574_1: gpio@38 { > + status = "okay"; Drop > + compatible = "nxp,pcf8574"; > + reg = <0x38>; > + gpio-controller; > + #gpio-cells = <2>; > + gpio-line-names = "LED_B_COMM3", "LED_R_COMM3", "LED_G_COMM3", > + "TP14", "TP15", "LED_G_COMM4", "LED_R_COMM4", "LED_B_COMM4"; > + }; > + > + pcf8574_2: gpio@39 { > + status = "okay"; Drop > + compatible = "nxp,pcf8574"; > + reg = <0x39>; > + gpio-controller; > + #gpio-cells = <2>; > + gpio-line-names = "LED_B_COMM2", "LED_G_COMM2", "LED_B_COMM1", > + "LED_R_COMM2", "LED_R_COMM1", "LED_G_COMM1", "TP16", "TP17"; > + }; > +}; > + > +&uart2 { > + /delete-node/ bluetooth; Why? If the hardware does not exist, why it is even in the DTSI? It does exist, so removing is not correct. > +}; > + > +&usbotg1 { > + dr_mode = "host"; > + disable-over-current; > + status = "okay"; > +}; > + > +/* SD interface on expansion connector. */ > +&usdhc2 { > + /delete-property/vmmc-supply; Why? The supply is needed. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v2 3/3] arm64: dts: freescale: introduce rve-gateway board 2023-11-02 7:36 ` Krzysztof Kozlowski @ 2023-11-02 15:10 ` Hugo Villeneuve 0 siblings, 0 replies; 8+ messages in thread From: Hugo Villeneuve @ 2023-11-02 15:10 UTC (permalink / raw) To: Krzysztof Kozlowski Cc: robh+dt, krzysztof.kozlowski+dt, conor+dt, hvilleneuve, shawnguo, s.hauer, kernel, festevam, linux-imx, leoyang.li, robh, devicetree, linux-kernel, linux-arm-kernel On Thu, 2 Nov 2023 08:36:32 +0100 Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote: > On 01/11/2023 19:35, Hugo Villeneuve wrote: > > From: Hugo Villeneuve <hvilleneuve@dimonoff.com> > > > > The RVE gateway board is based on a Variscite VAR-SOM-NANO, > > with a NXP MX8MN nano CPU. > > > > Signed-off-by: Hugo Villeneuve <hvilleneuve@dimonoff.com> > > --- > > MAINTAINERS | 7 + > > arch/arm64/boot/dts/freescale/Makefile | 1 + > > .../boot/dts/freescale/imx8mn-rve-gateway.dts | 284 ++++++++++++++++++ > > 3 files changed, 292 insertions(+) > > create mode 100644 arch/arm64/boot/dts/freescale/imx8mn-rve-gateway.dts > > > > diff --git a/MAINTAINERS b/MAINTAINERS > > index 43c5ed3c4508..9423b1bf35fc 100644 > > --- a/MAINTAINERS > > +++ b/MAINTAINERS > > @@ -18778,6 +18778,13 @@ F: samples/rust/ > > F: scripts/*rust* > > K: \b(?i:rust)\b > > > > +RVE BOARD SUPPORT > > +M: Hugo Villeneuve <hvilleneuve@dimonoff.com> > > +L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers) > > +S: Supported > > +W: https://rve.ca > > +F: arch/arm64/boot/dts/freescale/imx*rve-*.dts* > > I still don't think individual boards should receive maintainer entry. > https://lore.kernel.org/all/20210809080204.8381-1-shawnguo@kernel.org/ Hi, ok, I was not aware of that. Will drop it for V3. > > + > > RXRPC SOCKETS (AF_RXRPC) > > M: David Howells <dhowells@redhat.com> > > M: Marc Dionne <marc.dionne@auristor.com> > > diff --git a/arch/arm64/boot/dts/freescale/Makefile b/arch/arm64/boot/dts/freescale/Makefile > > index 89aee6c92576..760a0ca7adc3 100644 > > --- a/arch/arm64/boot/dts/freescale/Makefile > > +++ b/arch/arm64/boot/dts/freescale/Makefile > > @@ -89,6 +89,7 @@ dtb-$(CONFIG_ARCH_MXC) += imx8mn-bsh-smm-s2pro.dtb > > dtb-$(CONFIG_ARCH_MXC) += imx8mn-evk.dtb > > dtb-$(CONFIG_ARCH_MXC) += imx8mn-ddr3l-evk.dtb > > dtb-$(CONFIG_ARCH_MXC) += imx8mn-ddr4-evk.dtb > > +dtb-$(CONFIG_ARCH_MXC) += imx8mn-rve-gateway.dtb > > dtb-$(CONFIG_ARCH_MXC) += imx8mn-tqma8mqnl-mba8mx.dtb > > dtb-$(CONFIG_ARCH_MXC) += imx8mn-var-som-symphony.dtb > > dtb-$(CONFIG_ARCH_MXC) += imx8mn-venice-gw7902.dtb > > diff --git a/arch/arm64/boot/dts/freescale/imx8mn-rve-gateway.dts b/arch/arm64/boot/dts/freescale/imx8mn-rve-gateway.dts > > new file mode 100644 > > index 000000000000..312718982a52 > > --- /dev/null > > +++ b/arch/arm64/boot/dts/freescale/imx8mn-rve-gateway.dts > > @@ -0,0 +1,284 @@ > > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > > +/* > > + * Copyright 2023 DimOnOff > > + */ > > + > > +/dts-v1/; > > + > > +#include <dt-bindings/usb/pd.h> > > +#include "imx8mn-var-som.dtsi" > > + > > +/ { > > + model = "RVE gateway"; > > + compatible = "rve,rve-gateway", "fsl,imx8mn"; > > Missing compatible of SoM. Fixed in V3. > > > + > > + crystal_duart_24m: crystal-duart-24m { > > + compatible = "fixed-clock"; > > + #clock-cells = <0>; > > + clock-frequency = <24000000>; > > + }; > > + > > + gpio-keys { > > + compatible = "gpio-keys"; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_gpio_keys>; > > + status = "okay"; > > Drop Fixed in V3. > > > + > > + key-enter { > > + label = "enter"; > > + gpios = <&gpio1 1 GPIO_ACTIVE_LOW>; > > + linux,code = <KEY_ENTER>; > > + }; > > + > > + key-exit { > > + label = "exit"; > > + gpios = <&gpio3 23 GPIO_ACTIVE_LOW>; > > + linux,code = <KEY_ESC>; > > + }; > > + }; > > + > > + lcd { > > + compatible = "hit,hd44780"; > > + display-height-chars = <2>; > > + display-width-chars = <20>; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_lcd>; > > + data-gpios = > > + <&gpio5 1 GPIO_ACTIVE_HIGH>, > > This should be after =, with the reset of lines properly aligned. Fixed in V3. > > > + <&gpio1 6 GPIO_ACTIVE_HIGH>, > > + <&gpio1 14 GPIO_ACTIVE_HIGH>, > > + <&gpio4 28 GPIO_ACTIVE_HIGH>, > > + <&gpio5 24 GPIO_ACTIVE_HIGH>, > > + <&gpio5 2 GPIO_ACTIVE_HIGH>, > > + <&gpio1 12 GPIO_ACTIVE_HIGH>, > > + <&gpio5 25 GPIO_ACTIVE_HIGH>; > > + enable-gpios = <&gpio5 23 GPIO_ACTIVE_HIGH>; > > + rs-gpios = <&gpio1 7 GPIO_ACTIVE_HIGH>; > > + rw-gpios = <&gpio4 27 GPIO_ACTIVE_HIGH>; > > + }; > > + > > + reg_3p3v: regulator-3p3v { > > + compatible = "regulator-fixed"; > > + regulator-name = "3P3V"; > > + regulator-min-microvolt = <3300000>; > > + regulator-max-microvolt = <3300000>; > > + regulator-always-on; > > + }; > > Fix indentation. Fixed in V3. > > > + > > + rotary: rotary-encoder { /* Bourns PEC12R rotary encoder, 24 steps. */ > > No comments after {. Put them in line before or after. This applies to > other places as well. Fixed all in V3. > > > + compatible = "rotary-encoder"; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_rotary>; > > + gpios = > > + <&gpio1 5 GPIO_ACTIVE_LOW>, /* A */ > > This should be after =, with the reset of lines properly aligned. Fixed in V3. > > > > + <&gpio3 21 GPIO_ACTIVE_LOW>; /* B */ > > + linux,axis = <0>; /* REL_X */ > > + rotary-encoder,encoding = "gray"; > > + rotary-encoder,relative-axis; > > + }; > > +}; > > + > > +&ecspi1 { > > + cs-gpios = <&gpio5 9 GPIO_ACTIVE_LOW>; > > + > > + duart1: serial@0 { > > + compatible = "nxp,sc16is752"; > > + reg = <0>; > > + spi-rx-bus-width = <1>; > > + spi-tx-bus-width = <1>; > > + spi-max-frequency = <4000000>; > > + clocks = <&crystal_duart_24m>; > > + interrupt-parent = <&gpio3>; > > + interrupts = <22 IRQ_TYPE_EDGE_FALLING>; > > + gpio-controller; > > + #gpio-cells = <2>; > > + gpio-line-names = "RADIO0", "RADIO1", "RADIO2", "RADIO3", > > + "RADIO4", "RADIO_RESET", "TP12", "TP11"; > > + linux,rs485-enabled-at-boot-time; > > + rs485-rts-active-low; > > + }; > > + > > + /delete-node/ touchscreen@0; > > +}; > > + > > +&ecspi2 { > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_ecspi2>; > > + cs-gpios = <&gpio5 13 GPIO_ACTIVE_LOW>; > > + /delete-property/ dmas; > > + /delete-property/ dma-names; > > + status = "okay"; > > + > > + duart2: serial@0 { > > + compatible = "nxp,sc16is752"; > > + reg = <0>; > > + spi-rx-bus-width = <1>; > > + spi-tx-bus-width = <1>; > > + spi-max-frequency = <4000000>; > > + clocks = <&crystal_duart_24m>; > > + interrupt-parent = <&gpio3>; > > + interrupts = <20 IRQ_TYPE_EDGE_FALLING>; > > + gpio-controller; > > + #gpio-cells = <2>; > > + gpio-line-names = "LED_B_USER", "LED_R_USER", "LED_G_USER", > > + "GPIO_EXT3", "GPIO_EXT2", "GPIO_EXT1", "GPIO_EXT0", "TP13"; > > + linux,rs485-enabled-at-boot-time; > > + rs485-rts-active-low; > > + }; > > +}; > > + > > +/* Configure PWM pins in GPIO mode: */ > > +&gpio5 { > > + gpio-line-names = "", "", "", "PWM3", "PWM2", "PWM1"; > > +}; > > + > > +&gpu { > > + status = "disabled"; > > +}; > > + > > +&i2c2 { > > + clock-frequency = <400000>; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_i2c2>; > > + status = "okay"; > > + > > + eeprom_cb: eeprom@56 { /* Carrier board EEPROM */ > > + compatible = "atmel,24c04"; > > + reg = <0x56>; > > + pagesize = <16>; > > + vcc-supply = <®_3p3v>; > > + }; > > + > > + lm75: sensor@48 { > > + compatible = "st,stlm75"; > > + reg = <0x48>; > > + vs-supply = <®_3p3v>; > > + }; > > + > > + mcp7940: rtc@6f { > > + compatible = "microchip,mcp7940x"; > > + reg = <0x6f>; > > + }; > > +}; > > + > > +&i2c3 { > > + codec@1a { > > + status = "disabled"; > > + }; > > +}; > > + > > +&i2c4 { > > + clock-frequency = <400000>; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_i2c4>; > > + status = "okay"; > > + > > + pcf8574_1: gpio@38 { > > + status = "okay"; > > Drop Fixed all in V3. > > > + compatible = "nxp,pcf8574"; > > + reg = <0x38>; > > + gpio-controller; > > + #gpio-cells = <2>; > > + gpio-line-names = "LED_B_COMM3", "LED_R_COMM3", "LED_G_COMM3", > > + "TP14", "TP15", "LED_G_COMM4", "LED_R_COMM4", "LED_B_COMM4"; > > + }; > > + > > + pcf8574_2: gpio@39 { > > + status = "okay"; > > Drop > > > > + compatible = "nxp,pcf8574"; > > + reg = <0x39>; > > + gpio-controller; > > + #gpio-cells = <2>; > > + gpio-line-names = "LED_B_COMM2", "LED_G_COMM2", "LED_B_COMM1", > > + "LED_R_COMM2", "LED_R_COMM1", "LED_G_COMM1", "TP16", "TP17"; > > + }; > > +}; > > + > > +&uart2 { > > + /delete-node/ bluetooth; > > Why? If the hardware does not exist, why it is even in the DTSI? It does > exist, so removing is not correct. The current SOM DTSI is a melting pot of all "potentially" available hardware, but there are a lot of SOM versions, with different combinations of hardware components that can be populated or not... The hardware do exist on some of our SOM versions, but not on some others, and we do not use it at the moment. In V3 I changed the line to: status = "disabled"; > > > > +}; > > + > > +&usbotg1 { > > + dr_mode = "host"; > > + disable-over-current; > > + status = "okay"; > > +}; > > + > > +/* SD interface on expansion connector. */ > > +&usdhc2 { > > + /delete-property/vmmc-supply; > > Why? The supply is needed. The SOM dtsi references a regulator named reg_usdhc2_vmmc, but it is not present in the SOM dtsi itself. Since we do not have such a dedicated usdhc2 regulator on our board, initially I did this to get rid of this error: Reference to non-existent node or label "reg_usdhc2_vmmc" But for V3, I will change the vmmc-supply to refer to our board fixed 3.3V regulator: &usdhc2 { + vmmc-supply = <®_3p3v>; Thank you for the review. Hugo. > > > Best regards, > Krzysztof > > ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2023-11-02 15:10 UTC | newest] Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2023-11-01 18:35 [PATCH v2 0/3] board: imx8mn-rve-gateway: add support for RVE gateway board Hugo Villeneuve 2023-11-01 18:35 ` [PATCH v2 1/3] dt-bindings: vendor-prefixes: add rve Hugo Villeneuve 2023-11-02 7:30 ` Krzysztof Kozlowski 2023-11-01 18:35 ` [PATCH v2 2/3] dt-bindings: arm: fsl: add RVE gateway board Hugo Villeneuve 2023-11-02 7:30 ` Krzysztof Kozlowski 2023-11-01 18:35 ` [PATCH v2 3/3] arm64: dts: freescale: introduce rve-gateway board Hugo Villeneuve 2023-11-02 7:36 ` Krzysztof Kozlowski 2023-11-02 15:10 ` Hugo Villeneuve
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®