* [PATCH 1/2] dt-bindings: ARM: at91: Document Microchip SAMA5D29 Curiosity @ 2023-08-01 11:11 Mihai Sain 2023-08-01 11:11 ` [PATCH 2/2] ARM: dts: at91: sama5d29_curiosity: Add device tree for sama5d29_curiosity board Mihai Sain 2023-08-01 15:35 ` [PATCH 1/2] dt-bindings: ARM: at91: Document Microchip SAMA5D29 Curiosity Conor Dooley 0 siblings, 2 replies; 14+ messages in thread From: Mihai Sain @ 2023-08-01 11:11 UTC (permalink / raw) To: robh+dt, krzysztof.kozlowski+dt, conor+dt, nicolas.ferre, alexandre.belloni, claudiu.beznea, andre.przywara, andrei.simion, jerry.ray, devicetree, linux-arm-kernel, linux-kernel Cc: cristian.birsan, Mihai Sain From: Andrei Simion <andrei.simion@microchip.com> Document device tree binding of SAMA5D29 Curiosity, from Microchip. Reviewed-by: Mihai Sain <mihai.sain@microchip.com> Signed-off-by: Andrei Simion <andrei.simion@microchip.com> --- Documentation/devicetree/bindings/arm/atmel-at91.yaml | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/Documentation/devicetree/bindings/arm/atmel-at91.yaml b/Documentation/devicetree/bindings/arm/atmel-at91.yaml index dfb8fd089197..89d75fbb1de4 100644 --- a/Documentation/devicetree/bindings/arm/atmel-at91.yaml +++ b/Documentation/devicetree/bindings/arm/atmel-at91.yaml @@ -79,6 +79,13 @@ properties: - const: atmel,sama5d2 - const: atmel,sama5 + - description: Microchip SAMA5D29 Curiosity + items: + - const: microchip,sama5d29-curiosity + - const: atmel,sama5d29 + - const: atmel,sama5d2 + - const: atmel,sama5 + - items: - const: atmel,sama5d27 - const: atmel,sama5d2 -- 2.41.0 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 2/2] ARM: dts: at91: sama5d29_curiosity: Add device tree for sama5d29_curiosity board 2023-08-01 11:11 [PATCH 1/2] dt-bindings: ARM: at91: Document Microchip SAMA5D29 Curiosity Mihai Sain @ 2023-08-01 11:11 ` Mihai Sain 2023-08-02 4:40 ` claudiu beznea 2023-08-02 5:02 ` claudiu beznea 2023-08-01 15:35 ` [PATCH 1/2] dt-bindings: ARM: at91: Document Microchip SAMA5D29 Curiosity Conor Dooley 1 sibling, 2 replies; 14+ messages in thread From: Mihai Sain @ 2023-08-01 11:11 UTC (permalink / raw) To: robh+dt, krzysztof.kozlowski+dt, conor+dt, nicolas.ferre, alexandre.belloni, claudiu.beznea, andre.przywara, andrei.simion, jerry.ray, devicetree, linux-arm-kernel, linux-kernel Cc: cristian.birsan, Mihai Sain Add initial device tree file for sama5d29_curiosity board. Signed-off-by: Mihai Sain <mihai.sain@microchip.com> --- arch/arm/boot/dts/microchip/Makefile | 2 + .../dts/microchip/at91-sama5d29_curiosity.dts | 706 ++++++++++++++++++ 2 files changed, 708 insertions(+) create mode 100644 arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts diff --git a/arch/arm/boot/dts/microchip/Makefile b/arch/arm/boot/dts/microchip/Makefile index 31e03747cdf4..efde9546c8f4 100644 --- a/arch/arm/boot/dts/microchip/Makefile +++ b/arch/arm/boot/dts/microchip/Makefile @@ -4,6 +4,7 @@ DTC_FLAGS_at91-sam9x60_curiosity := -@ DTC_FLAGS_at91-sam9x60ek := -@ DTC_FLAGS_at91-sama5d27_som1_ek := -@ DTC_FLAGS_at91-sama5d27_wlsom1_ek := -@ +DTC_FLAGS_at91-sama5d29_curiosity := -@ DTC_FLAGS_at91-sama5d2_icp := -@ DTC_FLAGS_at91-sama5d2_ptc_ek := -@ DTC_FLAGS_at91-sama5d2_xplained := -@ @@ -64,6 +65,7 @@ dtb-$(CONFIG_SOC_SAM_V7) += \ at91-nattis-2-natte-2.dtb \ at91-sama5d27_som1_ek.dtb \ at91-sama5d27_wlsom1_ek.dtb \ + at91-sama5d29_curiosity.dtb \ at91-sama5d2_icp.dtb \ at91-sama5d2_ptc_ek.dtb \ at91-sama5d2_xplained.dtb \ diff --git a/arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts b/arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts new file mode 100644 index 000000000000..17af10446dac --- /dev/null +++ b/arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts @@ -0,0 +1,706 @@ +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) +/* + * at91-sama5d29_curiosity.dts - Device Tree file for SAMA5D29 Curiosity board + * + * Copyright (C) 2023 Microchip Technology Inc. and its subsidiaries + * + * Author: Mihai Sain <mihai.sain@microchip.com> + * + */ +/dts-v1/; +#include "sama5d29.dtsi" +#include "sama5d2-pinfunc.h" +#include <dt-bindings/gpio/gpio.h> +#include <dt-bindings/input/input.h> +#include <dt-bindings/mfd/atmel-flexcom.h> + +/ { + model = "Microchip SAMA5D29 Curiosity"; + compatible = "microchip,sama5d29-curiosity", "atmel,sama5d29", "atmel,sama5d2", "atmel,sama5"; + + aliases { + serial0 = &uart0; // debug + serial1 = &uart1; // RPi + serial2 = &uart3; // mikro BUS 2 + serial3 = &uart4; // mikro BUS 1 + serial4 = &uart6; // flx1 Bluetooth + i2c0 = &i2c0; + i2c1 = &i2c1; + }; + + chosen { + bootargs = "console=ttyS0,115200 root=/dev/mmcblk0p2 rw rootwait"; + stdout-path = "serial0:115200n8"; + }; + + clocks { + slow_xtal { + clock-frequency = <32768>; + }; + + main_xtal { + clock-frequency = <24000000>; + }; + }; + + gpio-keys { + compatible = "gpio-keys"; + + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_key_gpio_default>; + + button-1 { + label = "USER_PB"; + gpios = <&pioA PIN_PA17 GPIO_ACTIVE_LOW>; + linux,code = <KEY_PROG1>; + wakeup-source; + }; + }; + + leds { + compatible = "gpio-leds"; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_led_gpio_default>; + status = "okay"; + + red { + label = "red"; + gpios = <&pioA PIN_PA7 GPIO_ACTIVE_HIGH>; + }; + + green { + label = "green"; + gpios = <&pioA PIN_PA8 GPIO_ACTIVE_HIGH>; + }; + + blue { + label = "blue"; + gpios = <&pioA PIN_PA9 GPIO_ACTIVE_HIGH>; + linux,default-trigger = "heartbeat"; + }; + }; + + memory@20000000 { + device_type = "memory"; + reg = <0x20000000 0x20000000>; + }; + + vdd_1v8: regulator-vqmmc { + compatible = "regulator-fixed"; + regulator-name = "VDD_1V8"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1800000>; + status = "okay"; + }; +}; + +&adc { + vddana-supply = <&vdd_3v3>; + vref-supply = <&vdd_3v3>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_adc_default &pinctrl_adtrg_default>; + status = "okay"; +}; + +&can0 { + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_can0_default>; + status = "okay"; +}; + +&can1 { + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_can1_default>; + status = "okay"; +}; + +&flx1 { + atmel,flexcom-mode = <ATMEL_FLEXCOM_MODE_USART>; + status = "okay"; + + uart6: serial@200 { + pinctrl-0 = <&pinctrl_flx1_default>; + pinctrl-names = "default"; + atmel,use-dma-rx; + atmel,use-dma-tx; + status = "okay"; + }; +}; + +&flx4 { /* RPi spi */ + atmel,flexcom-mode = <ATMEL_FLEXCOM_MODE_SPI>; + status = "okay"; + + spi6: spi@400 { + dmas = <0>, <0>; + cs-gpios = <&pioA PIN_PD15 GPIO_ACTIVE_LOW>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_rpi_spi &pinctrl_rpi_spi_cs0>; + status = "okay"; + }; + + spi7: spi@400 { + dmas = <0>, <0>; + cs-gpios = <&pioA PIN_PD16 GPIO_ACTIVE_LOW>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_rpi_spi &pinctrl_rpi_spi_cs1>; + status = "okay"; + }; +}; + +&i2c0 { + dmas = <0>, <0>; + pinctrl-names = "default", "gpio"; + pinctrl-0 = <&pinctrl_i2c0_default>; + pinctrl-1 = <&pinctrl_i2c0_gpio>; + sda-gpios = <&pioA PIN_PB31 GPIO_ACTIVE_HIGH>; + scl-gpios = <&pioA PIN_PC0 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>; + i2c-sda-hold-time-ns = <350>; + status = "okay"; + + mcp16502@5b { + compatible = "microchip,mcp16502"; + reg = <0x5b>; + status = "okay"; + lpm-gpios = <&pioBU 0 GPIO_ACTIVE_LOW>; + + regulators { + vdd_3v3: VDD_IO { + regulator-name = "VDD_IO"; + regulator-min-microvolt = <3300000>; + regulator-max-microvolt = <3300000>; + regulator-initial-mode = <2>; + regulator-allowed-modes = <2>, <4>; + regulator-always-on; + + regulator-state-standby { + regulator-on-in-suspend; + regulator-mode = <4>; + }; + + regulator-state-mem { + regulator-off-in-suspend; + regulator-mode = <4>; + }; + }; + + vddio_ddr: VDD_DDR { + regulator-name = "VDD_DDR"; + regulator-min-microvolt = <1200000>; + regulator-max-microvolt = <1200000>; + regulator-initial-mode = <2>; + regulator-allowed-modes = <2>, <4>; + regulator-always-on; + + regulator-state-standby { + regulator-on-in-suspend; + regulator-suspend-microvolt = <1200000>; + regulator-changeable-in-suspend; + regulator-mode = <4>; + }; + + regulator-state-mem { + regulator-on-in-suspend; + regulator-suspend-microvolt = <1200000>; + regulator-changeable-in-suspend; + regulator-mode = <4>; + }; + }; + + vdd_core: VDD_CORE { + regulator-name = "VDD_CORE"; + regulator-min-microvolt = <1250000>; + regulator-max-microvolt = <1250000>; + regulator-initial-mode = <2>; + regulator-allowed-modes = <2>, <4>; + regulator-always-on; + + regulator-state-standby { + regulator-on-in-suspend; + regulator-mode = <4>; + }; + + regulator-state-mem { + regulator-off-in-suspend; + regulator-mode = <4>; + }; + }; + + vdd_ddr: VDD_OTHER { + regulator-name = "VDD_OTHER"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1800000>; + regulator-initial-mode = <2>; + regulator-allowed-modes = <2>, <4>; + regulator-always-on; + + regulator-state-standby { + regulator-on-in-suspend; + regulator-suspend-microvolt = <1800000>; + regulator-changeable-in-suspend; + regulator-mode = <4>; + }; + + regulator-state-mem { + regulator-on-in-suspend; + regulator-suspend-microvolt = <1800000>; + regulator-changeable-in-suspend; + regulator-mode = <4>; + }; + }; + + LDO1 { + regulator-name = "LDO1"; + regulator-min-microvolt = <2500000>; + regulator-max-microvolt = <2500000>; + regulator-always-on; + + regulator-state-standby { + regulator-on-in-suspend; + }; + + regulator-state-mem { + regulator-off-in-suspend; + }; + }; + + LDO2 { + regulator-name = "LDO2"; + regulator-min-microvolt = <3300000>; + regulator-max-microvolt = <3300000>; + regulator-always-on; + + regulator-state-standby { + regulator-on-in-suspend; + }; + + regulator-state-mem { + regulator-off-in-suspend; + }; + }; + }; + }; +}; + +&i2c1 { + dmas = <0>, <0>; + pinctrl-names = "default", "gpio"; + pinctrl-0 = <&pinctrl_i2c1_default>; + i2c-analog-filter; + i2c-digital-filter; + i2c-digital-filter-width-ns = <35>; + pinctrl-1 = <&pinctrl_i2c1_gpio>; + sda-gpios = <&pioA PIN_PD4 GPIO_ACTIVE_HIGH>; + scl-gpios = <&pioA PIN_PD5 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>; + status = "okay"; +}; + +&macb0 { + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_macb0_default &pinctrl_macb0_phy_irq>; + #address-cells = <1>; + #size-cells = <0>; + phy-mode = "rmii"; + status = "disabled"; +}; + +&pioA { + pinctrl_adc_default: adc_default { + pinmux = <PIN_PD25__GPIO>, + <PIN_PD26__GPIO>; + bias-disable; + }; + + pinctrl_adtrg_default: adtrg_default { + pinmux = <PIN_PD31__ADTRG>; + bias-pull-up; + }; + + pinctrl_rpi_spi: rpi_spi { + pinmux = <PIN_PD12__FLEXCOM4_IO0>, + <PIN_PD13__FLEXCOM4_IO1>, + <PIN_PD14__FLEXCOM4_IO2>; + bias-disable; + }; + + pinctrl_rpi_spi_cs0: rpi_spi_cs0 { + pinmux = <PIN_PD15__FLEXCOM4_IO3>; + bias-disable; + }; + + pinctrl_rpi_spi_cs1: rpi_spi_cs1 { + pinmux = <PIN_PD16__FLEXCOM4_IO4>; + bias-disable; + }; + + pinctrl_spi1_default: spi1_default { + pinmux = <PIN_PC1__SPI1_SPCK>, + <PIN_PC2__SPI1_MOSI>, + <PIN_PC3__SPI1_MISO>; + bias-disable; + }; + + pinctrl_spi1_cs: spi1_cs { + pinmux = <PIN_PC4__SPI1_NPCS0>, + <PIN_PC5__SPI1_NPCS1>, + <PIN_PC6__SPI1_NPCS2>, + <PIN_PC7__SPI1_NPCS3>; + bias-disable; + }; + + pinctrl_rpi_uart: rpi_uart { + pinmux = <PIN_PD2__URXD1>, + <PIN_PD3__UTXD1>; + bias-disable; + }; + + pinctrl_debug_uart: debug_uart { + pinmux = <PIN_PB26__URXD0>, + <PIN_PB27__UTXD0>; + bias-disable; + }; + + pinctrl_can0_default: can0_default { + pinmux = <PIN_PC10__CANTX0>, + <PIN_PC11__CANRX0>; + bias-disable; + }; + + pinctrl_can1_default: can1_default { + pinmux = <PIN_PC26__CANTX1>, + <PIN_PC27__CANRX1>; + bias-disable; + }; + + pinctrl_i2c0_default: i2c0_default { + pinmux = <PIN_PB31__TWD0>, + <PIN_PC0__TWCK0>; + bias-disable; + }; + + pinctrl_i2c0_gpio: i2c0_gpio { + pinmux = <PIN_PB31__GPIO>, + <PIN_PC0__GPIO>; + bias-disable; + }; + + pinctrl_i2c1_default: i2c1_default { + pinmux = <PIN_PD4__TWD1>, + <PIN_PD5__TWCK1>; + bias-disable; + }; + + pinctrl_i2c1_gpio: i2c1_gpio { + pinmux = <PIN_PD4__GPIO>, + <PIN_PD5__GPIO>; + bias-disable; + }; + + pinctrl_key_gpio_default: key_gpio_default { + pinmux = <PIN_PA17__GPIO>; + bias-pull-up; + }; + + pinctrl_led_gpio_default: led_gpio_default { + pinmux = <PIN_PA7__GPIO>, + <PIN_PA8__GPIO>, + <PIN_PA9__GPIO>; + bias-pull-up; + }; + + pinctrl_qspi1_default: qspi1_default { + pinmux = <PIN_PB5__QSPI1_SCK>, + <PIN_PB6__QSPI1_CS>, + <PIN_PB7__QSPI1_IO0>, + <PIN_PB8__QSPI1_IO1>, + <PIN_PB9__QSPI1_IO2>, + <PIN_PB10__QSPI1_IO3>; + bias-disable; + }; + + pinctrl_sdmmc0_default: sdmmc0_default { + pinmux = <PIN_PA0__SDMMC0_CK>, + <PIN_PA1__SDMMC0_CMD>, + <PIN_PA2__SDMMC0_DAT0>, + <PIN_PA3__SDMMC0_DAT1>, + <PIN_PA4__SDMMC0_DAT2>, + <PIN_PA5__SDMMC0_DAT3>, + <PIN_PA11__SDMMC0_VDDSEL>, + <PIN_PA13__SDMMC0_CD>; + bias-disable; + }; + + pinctrl_sdmmc1_default: sdmmc1_default { + pinmux = <PIN_PA18__SDMMC1_DAT0>, + <PIN_PA19__SDMMC1_DAT1>, + <PIN_PA20__SDMMC1_DAT2>, + <PIN_PA21__SDMMC1_DAT3>, + <PIN_PA22__SDMMC1_CK>, + <PIN_PA28__SDMMC1_CMD>, + <PIN_PA30__SDMMC1_CD>; + bias-disable; + }; + + pinctrl_mikrobus1_an: mikrobus1_an { + pinmux = <PIN_PD25__GPIO>; + bias-disable; + }; + + pinctrl_mikrobus1_rst: mikrobus1_rst { + pinmux = <PIN_PA16__GPIO>; + bias-disable; + }; + + pinctrl_mikrobus1_spi_cs: mikrobus1_spi_cs { + pinmux = <PIN_PC4__SPI1_NPCS0>; + bias-disable; + }; + + pinctrl_mikrobus1_spi: mikrobus1_spi { + pinmux = <PIN_PC1__SPI1_SPCK>, + <PIN_PC2__SPI1_MOSI>, + <PIN_PC3__SPI1_MISO>; + bias-disable; + }; + + pinctrl_mikrobus1_pwm: mikrobus1_pwm { + pinmux = <PIN_PA31__PWML0>; + bias-disable; + }; + + pinctrl_mikrobus1_int: mikrobus1_int { + pinmux = <PIN_PB2__GPIO>; + bias-disable; + }; + + pinctrl_mikrobus1_uart: mikrobus1_uart { + pinmux = <PIN_PB3__URXD4>, + <PIN_PB4__UTXD4>; + bias-disable; + }; + + pinctrl_mikrobus2_an: mikrobus2_an { + pinmux = <PIN_PD26__GPIO>; + bias-disable; + }; + + pinctrl_mikrobus2_rst: mikrobus2_rst { + pinmux = <PIN_PA16__GPIO>; + bias-disable; + }; + + pinctrl_mikrobus2_spi_cs: mikrobus2_spi_cs { + pinmux = <PIN_PC5__SPI1_NPCS1>; + bias-disable; + }; + + pinctrl_mikrobus2_spi: mikrobus2_spi { + pinmux = <PIN_PC1__SPI1_SPCK>, + <PIN_PC2__SPI1_MOSI>, + <PIN_PC3__SPI1_MISO>; + bias-disable; + }; + + pinctrl_mikrobus2_pwm: mikrobus2_pwm { + pinmux = <PIN_PB0__PWMH1>; + bias-disable; + }; + + pinctrl_mikrobus2_int: mikrobus2_int { + pinmux = <PIN_PB1__GPIO>; + bias-disable; + }; + + pinctrl_mikrobus2_uart: mikrobus2_uart { + pinmux = <PIN_PB11__URXD3>, + <PIN_PB12__UTXD3>; + bias-disable; + }; + + pinctrl_usb_default: usb_default { + pinmux = <PIN_PA6__GPIO>; + bias-disable; + }; + + pinctrl_usba_vbus: usba_vbus { + pinmux = <PIN_PB13__GPIO>; + bias-disable; + }; + + pinctrl_macb0_default: macb0_default { + pinmux = <PIN_PB14__GTXCK>, + <PIN_PB15__GTXEN>, + <PIN_PB16__GRXDV>, + <PIN_PB17__GRXER>, + <PIN_PB18__GRX0>, + <PIN_PB19__GRX1>, + <PIN_PB20__GTX0>, + <PIN_PB21__GTX1>, + <PIN_PB22__GMDC>, + <PIN_PB23__GMDIO>; + bias-disable; + }; + + pinctrl_macb0_phy_irq: macb0_phy_irq { + pinmux = <PIN_PB24__GPIO>; + bias-disable; + }; + + pinctrl_flx1_default: flx1_default { + pinmux = <PIN_PA24__FLEXCOM1_IO0>, + <PIN_PA23__FLEXCOM1_IO1>, + <PIN_PA25__FLEXCOM1_IO3>, + <PIN_PA26__FLEXCOM1_IO4>; + bias-disable; + }; +}; + +&pwm0 { + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_mikrobus1_pwm &pinctrl_mikrobus2_pwm>; + status = "okay"; +}; + +&qspi1 { + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_qspi1_default>; + status = "okay"; + + flash@0 { + #address-cells = <1>; + #size-cells = <1>; + compatible = "jedec,spi-nor"; + reg = <0>; + spi-max-frequency = <80000000>; + spi-tx-bus-width = <4>; + spi-rx-bus-width = <4>; + m25p,fast-read; + label = "atmel_qspi1"; + status = "okay"; + + at91bootstrap@0 { + label = "at91bootstrap"; + reg = <0x0 0x40000>; + }; + + bootloader@40000 { + label = "bootloader"; + reg = <0x40000 0xc0000>; + }; + + bootloaderenvred@100000 { + label = "bootloader env redundant"; + reg = <0x100000 0x40000>; + }; + + bootloaderenv@140000 { + label = "bootloader env"; + reg = <0x140000 0x40000>; + }; + + dtb@180000 { + label = "device tree"; + reg = <0x180000 0x80000>; + }; + + kernel@200000 { + label = "kernel"; + reg = <0x200000 0x600000>; + }; + }; +}; + +&sdmmc0 { + bus-width = <4>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_sdmmc0_default>; + vmmc-supply = <&vdd_3v3>; + vqmmc-supply = <&vdd_1v8>; + status = "okay"; +}; + +&sdmmc1 { + bus-width = <4>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_sdmmc1_default>; + vqmmc-supply = <&vdd_3v3>; + vmmc-supply = <&vdd_3v3>; + status = "okay"; +}; + +&shutdown_controller { + debounce-delay-us = <976>; + atmel,wakeup-rtc-timer; + + input@0 { + reg = <0>; + }; +}; + +&spi1 { + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_spi1_default &pinctrl_spi1_cs>; + status = "okay"; +}; + +&tcb0 { + timer0: timer@0 { + compatible = "atmel,tcb-timer"; + reg = <0>; + }; + + timer1: timer@1 { + compatible = "atmel,tcb-timer"; + reg = <1>; + }; +}; + +&uart0 { + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_debug_uart>; + atmel,use-dma-rx; + atmel,use-dma-tx; + status = "okay"; +}; + +&uart1 { + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_rpi_uart>; + atmel,use-dma-rx; + atmel,use-dma-tx; + status = "okay"; +}; + +&uart3 { + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_mikrobus2_uart>; + atmel,use-dma-rx; + atmel,use-dma-tx; + status = "okay"; +}; + +&uart4 { + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_mikrobus1_uart>; + atmel,use-dma-rx; + atmel,use-dma-tx; + status = "okay"; +}; + +&usb0 { + atmel,vbus-gpio = <&pioA PIN_PB13 GPIO_ACTIVE_HIGH>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_usba_vbus>; + status = "okay"; +}; + +&usb1 { + num-ports = <3>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_usb_default>; + status = "okay"; +}; + +&watchdog { + status = "okay"; +}; -- 2.41.0 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH 2/2] ARM: dts: at91: sama5d29_curiosity: Add device tree for sama5d29_curiosity board 2023-08-01 11:11 ` [PATCH 2/2] ARM: dts: at91: sama5d29_curiosity: Add device tree for sama5d29_curiosity board Mihai Sain @ 2023-08-02 4:40 ` claudiu beznea 2023-08-02 12:19 ` Mihai.Sain 2023-08-02 5:02 ` claudiu beznea 1 sibling, 1 reply; 14+ messages in thread From: claudiu beznea @ 2023-08-02 4:40 UTC (permalink / raw) To: Mihai Sain, robh+dt, krzysztof.kozlowski+dt, conor+dt, nicolas.ferre, alexandre.belloni, claudiu.beznea, andre.przywara, andrei.simion, jerry.ray, devicetree, linux-arm-kernel, linux-kernel Cc: cristian.birsan On 01.08.2023 14:11, Mihai Sain wrote: > Add initial device tree file for sama5d29_curiosity board. > > Signed-off-by: Mihai Sain <mihai.sain@microchip.com> > --- > arch/arm/boot/dts/microchip/Makefile | 2 + > .../dts/microchip/at91-sama5d29_curiosity.dts | 706 ++++++++++++++++++ > 2 files changed, 708 insertions(+) > create mode 100644 arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts > > diff --git a/arch/arm/boot/dts/microchip/Makefile b/arch/arm/boot/dts/microchip/Makefile > index 31e03747cdf4..efde9546c8f4 100644 > --- a/arch/arm/boot/dts/microchip/Makefile > +++ b/arch/arm/boot/dts/microchip/Makefile > @@ -4,6 +4,7 @@ DTC_FLAGS_at91-sam9x60_curiosity := -@ > DTC_FLAGS_at91-sam9x60ek := -@ > DTC_FLAGS_at91-sama5d27_som1_ek := -@ > DTC_FLAGS_at91-sama5d27_wlsom1_ek := -@ > +DTC_FLAGS_at91-sama5d29_curiosity := -@ > DTC_FLAGS_at91-sama5d2_icp := -@ > DTC_FLAGS_at91-sama5d2_ptc_ek := -@ > DTC_FLAGS_at91-sama5d2_xplained := -@ > @@ -64,6 +65,7 @@ dtb-$(CONFIG_SOC_SAM_V7) += \ > at91-nattis-2-natte-2.dtb \ > at91-sama5d27_som1_ek.dtb \ > at91-sama5d27_wlsom1_ek.dtb \ > + at91-sama5d29_curiosity.dtb \ > at91-sama5d2_icp.dtb \ > at91-sama5d2_ptc_ek.dtb \ > at91-sama5d2_xplained.dtb \ > diff --git a/arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts b/arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts > new file mode 100644 > index 000000000000..17af10446dac > --- /dev/null > +++ b/arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts > @@ -0,0 +1,706 @@ > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > +/* > + * at91-sama5d29_curiosity.dts - Device Tree file for SAMA5D29 Curiosity board > + * > + * Copyright (C) 2023 Microchip Technology Inc. and its subsidiaries > + * > + * Author: Mihai Sain <mihai.sain@microchip.com> > + * > + */ > +/dts-v1/; > +#include "sama5d29.dtsi" > +#include "sama5d2-pinfunc.h" > +#include <dt-bindings/gpio/gpio.h> > +#include <dt-bindings/input/input.h> > +#include <dt-bindings/mfd/atmel-flexcom.h> > + > +/ { > + model = "Microchip SAMA5D29 Curiosity"; > + compatible = "microchip,sama5d29-curiosity", "atmel,sama5d29", "atmel,sama5d2", "atmel,sama5"; > + > + aliases { > + serial0 = &uart0; // debug > + serial1 = &uart1; // RPi > + serial2 = &uart3; // mikro BUS 2 > + serial3 = &uart4; // mikro BUS 1 > + serial4 = &uart6; // flx1 Bluetooth > + i2c0 = &i2c0; > + i2c1 = &i2c1; > + }; > + > + chosen { > + bootargs = "console=ttyS0,115200 root=/dev/mmcblk0p2 rw rootwait"; > + stdout-path = "serial0:115200n8"; > + }; > + > + clocks { > + slow_xtal { > + clock-frequency = <32768>; > + }; > + > + main_xtal { > + clock-frequency = <24000000>; > + }; > + }; > + > + gpio-keys { > + compatible = "gpio-keys"; > + > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_key_gpio_default>; > + > + button-1 { > + label = "USER_PB"; > + gpios = <&pioA PIN_PA17 GPIO_ACTIVE_LOW>; > + linux,code = <KEY_PROG1>; > + wakeup-source; > + }; > + }; > + > + leds { > + compatible = "gpio-leds"; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_led_gpio_default>; > + status = "okay"; > + > + red { > + label = "red"; > + gpios = <&pioA PIN_PA7 GPIO_ACTIVE_HIGH>; > + }; > + > + green { > + label = "green"; > + gpios = <&pioA PIN_PA8 GPIO_ACTIVE_HIGH>; > + }; > + > + blue { > + label = "blue"; > + gpios = <&pioA PIN_PA9 GPIO_ACTIVE_HIGH>; > + linux,default-trigger = "heartbeat"; > + }; > + }; > + > + memory@20000000 { > + device_type = "memory"; > + reg = <0x20000000 0x20000000>; > + }; > + > + vdd_1v8: regulator-vqmmc { > + compatible = "regulator-fixed"; > + regulator-name = "VDD_1V8"; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1800000>; > + status = "okay"; No need for status = "okay", it is the default one. > + }; > +}; > + > +&adc { > + vddana-supply = <&vdd_3v3>; > + vref-supply = <&vdd_3v3>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_adc_default &pinctrl_adtrg_default>; > + status = "okay"; > +}; > + > +&can0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_can0_default>; > + status = "okay"; > +}; > + > +&can1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_can1_default>; > + status = "okay"; > +}; > + > +&flx1 { > + atmel,flexcom-mode = <ATMEL_FLEXCOM_MODE_USART>; > + status = "okay"; > + > + uart6: serial@200 { > + pinctrl-0 = <&pinctrl_flx1_default>; > + pinctrl-names = "default"; > + atmel,use-dma-rx; > + atmel,use-dma-tx; > + status = "okay"; > + }; > +}; > + > +&flx4 { /* RPi spi */ > + atmel,flexcom-mode = <ATMEL_FLEXCOM_MODE_SPI>; > + status = "okay"; > + > + spi6: spi@400 { > + dmas = <0>, <0>; > + cs-gpios = <&pioA PIN_PD15 GPIO_ACTIVE_LOW>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_rpi_spi &pinctrl_rpi_spi_cs0>; > + status = "okay"; > + }; > + > + spi7: spi@400 { > + dmas = <0>, <0>; > + cs-gpios = <&pioA PIN_PD16 GPIO_ACTIVE_LOW>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_rpi_spi &pinctrl_rpi_spi_cs1>; > + status = "okay"; > + }; Two SPIs on the same flexcom? It looks to me something is wrong here. > +}; > + > +&i2c0 { > + dmas = <0>, <0>; > + pinctrl-names = "default", "gpio"; > + pinctrl-0 = <&pinctrl_i2c0_default>; > + pinctrl-1 = <&pinctrl_i2c0_gpio>; > + sda-gpios = <&pioA PIN_PB31 GPIO_ACTIVE_HIGH>; > + scl-gpios = <&pioA PIN_PC0 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>; > + i2c-sda-hold-time-ns = <350>; > + status = "okay"; > + > + mcp16502@5b { > + compatible = "microchip,mcp16502"; > + reg = <0x5b>; > + status = "okay"; > + lpm-gpios = <&pioBU 0 GPIO_ACTIVE_LOW>; > + > + regulators { > + vdd_3v3: VDD_IO { > + regulator-name = "VDD_IO"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-initial-mode = <2>; > + regulator-allowed-modes = <2>, <4>; > + regulator-always-on; > + > + regulator-state-standby { > + regulator-on-in-suspend; > + regulator-mode = <4>; > + }; > + > + regulator-state-mem { > + regulator-off-in-suspend; > + regulator-mode = <4>; > + }; > + }; > + > + vddio_ddr: VDD_DDR { > + regulator-name = "VDD_DDR"; > + regulator-min-microvolt = <1200000>; > + regulator-max-microvolt = <1200000>; > + regulator-initial-mode = <2>; > + regulator-allowed-modes = <2>, <4>; > + regulator-always-on; > + > + regulator-state-standby { > + regulator-on-in-suspend; > + regulator-suspend-microvolt = <1200000>; > + regulator-changeable-in-suspend; > + regulator-mode = <4>; > + }; > + > + regulator-state-mem { > + regulator-on-in-suspend; > + regulator-suspend-microvolt = <1200000>; > + regulator-changeable-in-suspend; > + regulator-mode = <4>; > + }; > + }; > + > + vdd_core: VDD_CORE { > + regulator-name = "VDD_CORE"; > + regulator-min-microvolt = <1250000>; > + regulator-max-microvolt = <1250000>; > + regulator-initial-mode = <2>; > + regulator-allowed-modes = <2>, <4>; > + regulator-always-on; > + > + regulator-state-standby { > + regulator-on-in-suspend; > + regulator-mode = <4>; > + }; > + > + regulator-state-mem { > + regulator-off-in-suspend; > + regulator-mode = <4>; > + }; > + }; > + > + vdd_ddr: VDD_OTHER { > + regulator-name = "VDD_OTHER"; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1800000>; > + regulator-initial-mode = <2>; > + regulator-allowed-modes = <2>, <4>; > + regulator-always-on; > + > + regulator-state-standby { > + regulator-on-in-suspend; > + regulator-suspend-microvolt = <1800000>; > + regulator-changeable-in-suspend; > + regulator-mode = <4>; > + }; > + > + regulator-state-mem { > + regulator-on-in-suspend; > + regulator-suspend-microvolt = <1800000>; > + regulator-changeable-in-suspend; > + regulator-mode = <4>; > + }; > + }; > + > + LDO1 { > + regulator-name = "LDO1"; > + regulator-min-microvolt = <2500000>; > + regulator-max-microvolt = <2500000>; > + regulator-always-on; > + > + regulator-state-standby { > + regulator-on-in-suspend; > + }; > + > + regulator-state-mem { > + regulator-off-in-suspend; > + }; > + }; > + > + LDO2 { > + regulator-name = "LDO2"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-always-on; > + > + regulator-state-standby { > + regulator-on-in-suspend; > + }; > + > + regulator-state-mem { > + regulator-off-in-suspend; > + }; > + }; > + }; > + }; > +}; > + > +&i2c1 { > + dmas = <0>, <0>; > + pinctrl-names = "default", "gpio"; > + pinctrl-0 = <&pinctrl_i2c1_default>; > + i2c-analog-filter; > + i2c-digital-filter; > + i2c-digital-filter-width-ns = <35>; > + pinctrl-1 = <&pinctrl_i2c1_gpio>; Keep pinctrl-X grouped together. > + sda-gpios = <&pioA PIN_PD4 GPIO_ACTIVE_HIGH>; > + scl-gpios = <&pioA PIN_PD5 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>; > + status = "okay"; > +}; > + > +&macb0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_macb0_default &pinctrl_macb0_phy_irq>; > + #address-cells = <1>; > + #size-cells = <0>; > + phy-mode = "rmii"; > + status = "disabled"; Should any phy/mdio container be placed here? Also, any reason this node is disabled? > +}; > + > +&pioA { > + pinctrl_adc_default: adc_default { > + pinmux = <PIN_PD25__GPIO>, > + <PIN_PD26__GPIO>; > + bias-disable; > + }; > + > + pinctrl_adtrg_default: adtrg_default { > + pinmux = <PIN_PD31__ADTRG>; > + bias-pull-up; > + }; > + > + pinctrl_rpi_spi: rpi_spi { > + pinmux = <PIN_PD12__FLEXCOM4_IO0>, > + <PIN_PD13__FLEXCOM4_IO1>, > + <PIN_PD14__FLEXCOM4_IO2>; > + bias-disable; > + }; > + > + pinctrl_rpi_spi_cs0: rpi_spi_cs0 { > + pinmux = <PIN_PD15__FLEXCOM4_IO3>; > + bias-disable; > + }; > + > + pinctrl_rpi_spi_cs1: rpi_spi_cs1 { > + pinmux = <PIN_PD16__FLEXCOM4_IO4>; > + bias-disable; > + }; > + > + pinctrl_spi1_default: spi1_default { > + pinmux = <PIN_PC1__SPI1_SPCK>, > + <PIN_PC2__SPI1_MOSI>, > + <PIN_PC3__SPI1_MISO>; > + bias-disable; > + }; > + > + pinctrl_spi1_cs: spi1_cs { > + pinmux = <PIN_PC4__SPI1_NPCS0>, > + <PIN_PC5__SPI1_NPCS1>, > + <PIN_PC6__SPI1_NPCS2>, > + <PIN_PC7__SPI1_NPCS3>; > + bias-disable; > + }; > + > + pinctrl_rpi_uart: rpi_uart { > + pinmux = <PIN_PD2__URXD1>, > + <PIN_PD3__UTXD1>; > + bias-disable; > + }; > + > + pinctrl_debug_uart: debug_uart { > + pinmux = <PIN_PB26__URXD0>, > + <PIN_PB27__UTXD0>; > + bias-disable; > + }; > + > + pinctrl_can0_default: can0_default { > + pinmux = <PIN_PC10__CANTX0>, > + <PIN_PC11__CANRX0>; > + bias-disable; > + }; > + > + pinctrl_can1_default: can1_default { > + pinmux = <PIN_PC26__CANTX1>, > + <PIN_PC27__CANRX1>; > + bias-disable; > + }; > + > + pinctrl_i2c0_default: i2c0_default { > + pinmux = <PIN_PB31__TWD0>, > + <PIN_PC0__TWCK0>; > + bias-disable; > + }; > + > + pinctrl_i2c0_gpio: i2c0_gpio { > + pinmux = <PIN_PB31__GPIO>, > + <PIN_PC0__GPIO>; > + bias-disable; > + }; > + > + pinctrl_i2c1_default: i2c1_default { > + pinmux = <PIN_PD4__TWD1>, > + <PIN_PD5__TWCK1>; > + bias-disable; > + }; > + > + pinctrl_i2c1_gpio: i2c1_gpio { > + pinmux = <PIN_PD4__GPIO>, > + <PIN_PD5__GPIO>; > + bias-disable; > + }; > + > + pinctrl_key_gpio_default: key_gpio_default { > + pinmux = <PIN_PA17__GPIO>; > + bias-pull-up; > + }; > + > + pinctrl_led_gpio_default: led_gpio_default { > + pinmux = <PIN_PA7__GPIO>, > + <PIN_PA8__GPIO>, > + <PIN_PA9__GPIO>; > + bias-pull-up; > + }; > + > + pinctrl_qspi1_default: qspi1_default { > + pinmux = <PIN_PB5__QSPI1_SCK>, > + <PIN_PB6__QSPI1_CS>, > + <PIN_PB7__QSPI1_IO0>, > + <PIN_PB8__QSPI1_IO1>, > + <PIN_PB9__QSPI1_IO2>, > + <PIN_PB10__QSPI1_IO3>; > + bias-disable; > + }; > + > + pinctrl_sdmmc0_default: sdmmc0_default { > + pinmux = <PIN_PA0__SDMMC0_CK>, > + <PIN_PA1__SDMMC0_CMD>, > + <PIN_PA2__SDMMC0_DAT0>, > + <PIN_PA3__SDMMC0_DAT1>, > + <PIN_PA4__SDMMC0_DAT2>, > + <PIN_PA5__SDMMC0_DAT3>, > + <PIN_PA11__SDMMC0_VDDSEL>, > + <PIN_PA13__SDMMC0_CD>; > + bias-disable; > + }; > + > + pinctrl_sdmmc1_default: sdmmc1_default { > + pinmux = <PIN_PA18__SDMMC1_DAT0>, > + <PIN_PA19__SDMMC1_DAT1>, > + <PIN_PA20__SDMMC1_DAT2>, > + <PIN_PA21__SDMMC1_DAT3>, > + <PIN_PA22__SDMMC1_CK>, > + <PIN_PA28__SDMMC1_CMD>, > + <PIN_PA30__SDMMC1_CD>; > + bias-disable; > + }; > + These starting here: > + pinctrl_mikrobus1_an: mikrobus1_an { > + pinmux = <PIN_PD25__GPIO>; > + bias-disable; > + }; > + > + pinctrl_mikrobus1_rst: mikrobus1_rst { > + pinmux = <PIN_PA16__GPIO>; > + bias-disable; > + }; > + > + pinctrl_mikrobus1_spi_cs: mikrobus1_spi_cs { > + pinmux = <PIN_PC4__SPI1_NPCS0>; > + bias-disable; > + }; > + > + pinctrl_mikrobus1_spi: mikrobus1_spi { > + pinmux = <PIN_PC1__SPI1_SPCK>, > + <PIN_PC2__SPI1_MOSI>, > + <PIN_PC3__SPI1_MISO>; > + bias-disable; > + }; > + ending here are not used. Same on mikrobus2 > + pinctrl_mikrobus1_pwm: mikrobus1_pwm { > + pinmux = <PIN_PA31__PWML0>; > + bias-disable; > + }; > + > + pinctrl_mikrobus1_int: mikrobus1_int { > + pinmux = <PIN_PB2__GPIO>; > + bias-disable; > + }; > + > + pinctrl_mikrobus1_uart: mikrobus1_uart { > + pinmux = <PIN_PB3__URXD4>, > + <PIN_PB4__UTXD4>; > + bias-disable; > + }; Not used > + > + pinctrl_mikrobus2_an: mikrobus2_an { > + pinmux = <PIN_PD26__GPIO>; > + bias-disable; > + }; > + > + pinctrl_mikrobus2_rst: mikrobus2_rst { > + pinmux = <PIN_PA16__GPIO>; > + bias-disable; > + }; > + > + pinctrl_mikrobus2_spi_cs: mikrobus2_spi_cs { > + pinmux = <PIN_PC5__SPI1_NPCS1>; > + bias-disable; > + }; > + > + pinctrl_mikrobus2_spi: mikrobus2_spi { > + pinmux = <PIN_PC1__SPI1_SPCK>, > + <PIN_PC2__SPI1_MOSI>, > + <PIN_PC3__SPI1_MISO>; > + bias-disable; > + }; > + > + pinctrl_mikrobus2_pwm: mikrobus2_pwm { > + pinmux = <PIN_PB0__PWMH1>; > + bias-disable; > + }; > + > + pinctrl_mikrobus2_int: mikrobus2_int { > + pinmux = <PIN_PB1__GPIO>; > + bias-disable; > + }; > + > + pinctrl_mikrobus2_uart: mikrobus2_uart { > + pinmux = <PIN_PB11__URXD3>, > + <PIN_PB12__UTXD3>; > + bias-disable; > + }; > + > + pinctrl_usb_default: usb_default { > + pinmux = <PIN_PA6__GPIO>; > + bias-disable; > + }; > + > + pinctrl_usba_vbus: usba_vbus { > + pinmux = <PIN_PB13__GPIO>; > + bias-disable; > + }; > + > + pinctrl_macb0_default: macb0_default { > + pinmux = <PIN_PB14__GTXCK>, > + <PIN_PB15__GTXEN>, > + <PIN_PB16__GRXDV>, > + <PIN_PB17__GRXER>, > + <PIN_PB18__GRX0>, > + <PIN_PB19__GRX1>, > + <PIN_PB20__GTX0>, > + <PIN_PB21__GTX1>, > + <PIN_PB22__GMDC>, > + <PIN_PB23__GMDIO>; > + bias-disable; > + }; > + > + pinctrl_macb0_phy_irq: macb0_phy_irq { > + pinmux = <PIN_PB24__GPIO>; > + bias-disable; > + }; > + > + pinctrl_flx1_default: flx1_default { > + pinmux = <PIN_PA24__FLEXCOM1_IO0>, > + <PIN_PA23__FLEXCOM1_IO1>, > + <PIN_PA25__FLEXCOM1_IO3>, > + <PIN_PA26__FLEXCOM1_IO4>; > + bias-disable; > + }; > +}; > + > +&pwm0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_mikrobus1_pwm &pinctrl_mikrobus2_pwm>; > + status = "okay"; > +}; > + > +&qspi1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_qspi1_default>; > + status = "okay"; > + > + flash@0 { > + #address-cells = <1>; > + #size-cells = <1>; > + compatible = "jedec,spi-nor"; > + reg = <0>; > + spi-max-frequency = <80000000>; > + spi-tx-bus-width = <4>; > + spi-rx-bus-width = <4>; > + m25p,fast-read; > + label = "atmel_qspi1"; > + status = "okay"; > + > + at91bootstrap@0 { > + label = "at91bootstrap"; > + reg = <0x0 0x40000>; > + }; > + > + bootloader@40000 { > + label = "bootloader"; > + reg = <0x40000 0xc0000>; > + }; > + > + bootloaderenvred@100000 { > + label = "bootloader env redundant"; > + reg = <0x100000 0x40000>; > + }; > + > + bootloaderenv@140000 { > + label = "bootloader env"; > + reg = <0x140000 0x40000>; > + }; > + > + dtb@180000 { > + label = "device tree"; > + reg = <0x180000 0x80000>; > + }; > + > + kernel@200000 { > + label = "kernel"; > + reg = <0x200000 0x600000>; > + }; > + }; > +}; > + > +&sdmmc0 { > + bus-width = <4>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_sdmmc0_default>; > + vmmc-supply = <&vdd_3v3>; > + vqmmc-supply = <&vdd_1v8>; > + status = "okay"; > +}; > + > +&sdmmc1 { > + bus-width = <4>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_sdmmc1_default>; > + vqmmc-supply = <&vdd_3v3>; > + vmmc-supply = <&vdd_3v3>; > + status = "okay"; > +}; > + > +&shutdown_controller { > + debounce-delay-us = <976>; > + atmel,wakeup-rtc-timer; > + > + input@0 { > + reg = <0>; > + }; > +}; > + > +&spi1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_spi1_default &pinctrl_spi1_cs>; > + status = "okay"; > +}; > + > +&tcb0 { > + timer0: timer@0 { > + compatible = "atmel,tcb-timer"; > + reg = <0>; > + }; > + > + timer1: timer@1 { > + compatible = "atmel,tcb-timer"; > + reg = <1>; > + }; > +}; > + > +&uart0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_debug_uart>; > + atmel,use-dma-rx; > + atmel,use-dma-tx; > + status = "okay"; > +}; > + > +&uart1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_rpi_uart>; > + atmel,use-dma-rx; > + atmel,use-dma-tx; > + status = "okay"; > +}; > + > +&uart3 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_mikrobus2_uart>; > + atmel,use-dma-rx; > + atmel,use-dma-tx; > + status = "okay"; > +}; > + > +&uart4 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_mikrobus1_uart>; > + atmel,use-dma-rx; > + atmel,use-dma-tx; > + status = "okay"; > +}; > + > +&usb0 { > + atmel,vbus-gpio = <&pioA PIN_PB13 GPIO_ACTIVE_HIGH>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_usba_vbus>; > + status = "okay"; > +}; > + > +&usb1 { > + num-ports = <3>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_usb_default>; > + status = "okay"; > +}; > + > +&watchdog { > + status = "okay"; > +}; ^ permalink raw reply [flat|nested] 14+ messages in thread
* RE: [PATCH 2/2] ARM: dts: at91: sama5d29_curiosity: Add device tree for sama5d29_curiosity board 2023-08-02 4:40 ` claudiu beznea @ 2023-08-02 12:19 ` Mihai.Sain 2023-08-03 9:09 ` claudiu beznea 0 siblings, 1 reply; 14+ messages in thread From: Mihai.Sain @ 2023-08-02 12:19 UTC (permalink / raw) To: claudiu.beznea, robh+dt, krzysztof.kozlowski+dt, conor+dt, Nicolas.Ferre, alexandre.belloni, Claudiu.Beznea, andre.przywara, Andrei.Simion, Jerry.Ray, devicetree, linux-arm-kernel, linux-kernel Cc: Cristian.Birsan Hi Claudiu, Please find bellow my answers. -----Original Message----- From: claudiu beznea <claudiu.beznea@tuxon.dev> Sent: Wednesday, August 2, 2023 7:40 AM To: Mihai Sain - M19926 <Mihai.Sain@microchip.com>; robh+dt@kernel.org; krzysztof.kozlowski+dt@linaro.org; conor+dt@kernel.org; Nicolas Ferre - M43238 <Nicolas.Ferre@microchip.com>; alexandre.belloni@bootlin.com; Claudiu Beznea - M18063 <Claudiu.Beznea@microchip.com>; andre.przywara@arm.com; Andrei Simion - M76677 <Andrei.Simion@microchip.com>; Jerry Ray - C33025 <Jerry.Ray@microchip.com>; devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux-kernel@vger.kernel.org Cc: Cristian Birsan - M91496 <Cristian.Birsan@microchip.com> Subject: Re: [PATCH 2/2] ARM: dts: at91: sama5d29_curiosity: Add device tree for sama5d29_curiosity board [You don't often get email from claudiu.beznea@tuxon.dev. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ] EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe On 01.08.2023 14:11, Mihai Sain wrote: > Add initial device tree file for sama5d29_curiosity board. > > Signed-off-by: Mihai Sain <mihai.sain@microchip.com> > --- > arch/arm/boot/dts/microchip/Makefile | 2 + > .../dts/microchip/at91-sama5d29_curiosity.dts | 706 > ++++++++++++++++++ > 2 files changed, 708 insertions(+) > create mode 100644 > arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts > > diff --git a/arch/arm/boot/dts/microchip/Makefile > b/arch/arm/boot/dts/microchip/Makefile > index 31e03747cdf4..efde9546c8f4 100644 > --- a/arch/arm/boot/dts/microchip/Makefile > +++ b/arch/arm/boot/dts/microchip/Makefile > @@ -4,6 +4,7 @@ DTC_FLAGS_at91-sam9x60_curiosity := -@ > DTC_FLAGS_at91-sam9x60ek := -@ DTC_FLAGS_at91-sama5d27_som1_ek := -@ > DTC_FLAGS_at91-sama5d27_wlsom1_ek := -@ > +DTC_FLAGS_at91-sama5d29_curiosity := -@ > DTC_FLAGS_at91-sama5d2_icp := -@ > DTC_FLAGS_at91-sama5d2_ptc_ek := -@ > DTC_FLAGS_at91-sama5d2_xplained := -@ @@ -64,6 +65,7 @@ > dtb-$(CONFIG_SOC_SAM_V7) += \ > at91-nattis-2-natte-2.dtb \ > at91-sama5d27_som1_ek.dtb \ > at91-sama5d27_wlsom1_ek.dtb \ > + at91-sama5d29_curiosity.dtb \ > at91-sama5d2_icp.dtb \ > at91-sama5d2_ptc_ek.dtb \ > at91-sama5d2_xplained.dtb \ > diff --git a/arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts > b/arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts > new file mode 100644 > index 000000000000..17af10446dac > --- /dev/null > +++ b/arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts > @@ -0,0 +1,706 @@ > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > +/* > + * at91-sama5d29_curiosity.dts - Device Tree file for SAMA5D29 > +Curiosity board > + * > + * Copyright (C) 2023 Microchip Technology Inc. and its subsidiaries > + * > + * Author: Mihai Sain <mihai.sain@microchip.com> > + * > + */ > +/dts-v1/; > +#include "sama5d29.dtsi" > +#include "sama5d2-pinfunc.h" > +#include <dt-bindings/gpio/gpio.h> > +#include <dt-bindings/input/input.h> > +#include <dt-bindings/mfd/atmel-flexcom.h> > + > +/ { > + model = "Microchip SAMA5D29 Curiosity"; > + compatible = "microchip,sama5d29-curiosity", "atmel,sama5d29", > +"atmel,sama5d2", "atmel,sama5"; > + > + aliases { > + serial0 = &uart0; // debug > + serial1 = &uart1; // RPi > + serial2 = &uart3; // mikro BUS 2 > + serial3 = &uart4; // mikro BUS 1 > + serial4 = &uart6; // flx1 Bluetooth > + i2c0 = &i2c0; > + i2c1 = &i2c1; > + }; > + > + chosen { > + bootargs = "console=ttyS0,115200 root=/dev/mmcblk0p2 rw rootwait"; > + stdout-path = "serial0:115200n8"; > + }; > + > + clocks { > + slow_xtal { > + clock-frequency = <32768>; > + }; > + > + main_xtal { > + clock-frequency = <24000000>; > + }; > + }; > + > + gpio-keys { > + compatible = "gpio-keys"; > + > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_key_gpio_default>; > + > + button-1 { > + label = "USER_PB"; > + gpios = <&pioA PIN_PA17 GPIO_ACTIVE_LOW>; > + linux,code = <KEY_PROG1>; > + wakeup-source; > + }; > + }; > + > + leds { > + compatible = "gpio-leds"; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_led_gpio_default>; > + status = "okay"; > + > + red { > + label = "red"; > + gpios = <&pioA PIN_PA7 GPIO_ACTIVE_HIGH>; > + }; > + > + green { > + label = "green"; > + gpios = <&pioA PIN_PA8 GPIO_ACTIVE_HIGH>; > + }; > + > + blue { > + label = "blue"; > + gpios = <&pioA PIN_PA9 GPIO_ACTIVE_HIGH>; > + linux,default-trigger = "heartbeat"; > + }; > + }; > + > + memory@20000000 { > + device_type = "memory"; > + reg = <0x20000000 0x20000000>; > + }; > + > + vdd_1v8: regulator-vqmmc { > + compatible = "regulator-fixed"; > + regulator-name = "VDD_1V8"; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1800000>; > + status = "okay"; No need for status = "okay", it is the default one. I will remove the regulator in patch v2. > + }; > +}; > + > +&adc { > + vddana-supply = <&vdd_3v3>; > + vref-supply = <&vdd_3v3>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_adc_default &pinctrl_adtrg_default>; > + status = "okay"; > +}; > + > +&can0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_can0_default>; > + status = "okay"; > +}; > + > +&can1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_can1_default>; > + status = "okay"; > +}; > + > +&flx1 { > + atmel,flexcom-mode = <ATMEL_FLEXCOM_MODE_USART>; > + status = "okay"; > + > + uart6: serial@200 { > + pinctrl-0 = <&pinctrl_flx1_default>; > + pinctrl-names = "default"; > + atmel,use-dma-rx; > + atmel,use-dma-tx; > + status = "okay"; > + }; > +}; > + > +&flx4 { /* RPi spi */ > + atmel,flexcom-mode = <ATMEL_FLEXCOM_MODE_SPI>; > + status = "okay"; > + > + spi6: spi@400 { > + dmas = <0>, <0>; > + cs-gpios = <&pioA PIN_PD15 GPIO_ACTIVE_LOW>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_rpi_spi &pinctrl_rpi_spi_cs0>; > + status = "okay"; > + }; > + > + spi7: spi@400 { > + dmas = <0>, <0>; > + cs-gpios = <&pioA PIN_PD16 GPIO_ACTIVE_LOW>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_rpi_spi &pinctrl_rpi_spi_cs1>; > + status = "okay"; > + }; Two SPIs on the same flexcom? It looks to me something is wrong here. I will use only 1 spi node and pinctrl in patch v2. > +}; > + > +&i2c0 { > + dmas = <0>, <0>; > + pinctrl-names = "default", "gpio"; > + pinctrl-0 = <&pinctrl_i2c0_default>; > + pinctrl-1 = <&pinctrl_i2c0_gpio>; > + sda-gpios = <&pioA PIN_PB31 GPIO_ACTIVE_HIGH>; > + scl-gpios = <&pioA PIN_PC0 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>; > + i2c-sda-hold-time-ns = <350>; > + status = "okay"; > + > + mcp16502@5b { > + compatible = "microchip,mcp16502"; > + reg = <0x5b>; > + status = "okay"; > + lpm-gpios = <&pioBU 0 GPIO_ACTIVE_LOW>; > + > + regulators { > + vdd_3v3: VDD_IO { > + regulator-name = "VDD_IO"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-initial-mode = <2>; > + regulator-allowed-modes = <2>, <4>; > + regulator-always-on; > + > + regulator-state-standby { > + regulator-on-in-suspend; > + regulator-mode = <4>; > + }; > + > + regulator-state-mem { > + regulator-off-in-suspend; > + regulator-mode = <4>; > + }; > + }; > + > + vddio_ddr: VDD_DDR { > + regulator-name = "VDD_DDR"; > + regulator-min-microvolt = <1200000>; > + regulator-max-microvolt = <1200000>; > + regulator-initial-mode = <2>; > + regulator-allowed-modes = <2>, <4>; > + regulator-always-on; > + > + regulator-state-standby { > + regulator-on-in-suspend; > + regulator-suspend-microvolt = <1200000>; > + regulator-changeable-in-suspend; > + regulator-mode = <4>; > + }; > + > + regulator-state-mem { > + regulator-on-in-suspend; > + regulator-suspend-microvolt = <1200000>; > + regulator-changeable-in-suspend; > + regulator-mode = <4>; > + }; > + }; > + > + vdd_core: VDD_CORE { > + regulator-name = "VDD_CORE"; > + regulator-min-microvolt = <1250000>; > + regulator-max-microvolt = <1250000>; > + regulator-initial-mode = <2>; > + regulator-allowed-modes = <2>, <4>; > + regulator-always-on; > + > + regulator-state-standby { > + regulator-on-in-suspend; > + regulator-mode = <4>; > + }; > + > + regulator-state-mem { > + regulator-off-in-suspend; > + regulator-mode = <4>; > + }; > + }; > + > + vdd_ddr: VDD_OTHER { > + regulator-name = "VDD_OTHER"; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1800000>; > + regulator-initial-mode = <2>; > + regulator-allowed-modes = <2>, <4>; > + regulator-always-on; > + > + regulator-state-standby { > + regulator-on-in-suspend; > + regulator-suspend-microvolt = <1800000>; > + regulator-changeable-in-suspend; > + regulator-mode = <4>; > + }; > + > + regulator-state-mem { > + regulator-on-in-suspend; > + regulator-suspend-microvolt = <1800000>; > + regulator-changeable-in-suspend; > + regulator-mode = <4>; > + }; > + }; > + > + LDO1 { > + regulator-name = "LDO1"; > + regulator-min-microvolt = <2500000>; > + regulator-max-microvolt = <2500000>; > + regulator-always-on; > + > + regulator-state-standby { > + regulator-on-in-suspend; > + }; > + > + regulator-state-mem { > + regulator-off-in-suspend; > + }; > + }; > + > + LDO2 { > + regulator-name = "LDO2"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-always-on; > + > + regulator-state-standby { > + regulator-on-in-suspend; > + }; > + > + regulator-state-mem { > + regulator-off-in-suspend; > + }; > + }; > + }; > + }; > +}; > + > +&i2c1 { > + dmas = <0>, <0>; > + pinctrl-names = "default", "gpio"; > + pinctrl-0 = <&pinctrl_i2c1_default>; > + i2c-analog-filter; > + i2c-digital-filter; > + i2c-digital-filter-width-ns = <35>; > + pinctrl-1 = <&pinctrl_i2c1_gpio>; Keep pinctrl-X grouped together. Sure, I will update. > + sda-gpios = <&pioA PIN_PD4 GPIO_ACTIVE_HIGH>; > + scl-gpios = <&pioA PIN_PD5 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>; > + status = "okay"; > +}; > + > +&macb0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_macb0_default &pinctrl_macb0_phy_irq>; > + #address-cells = <1>; > + #size-cells = <0>; > + phy-mode = "rmii"; > + status = "disabled"; Should any phy/mdio container be placed here? Also, any reason this node is disabled? The board has no phy. We will add external phy boards to macb interface and they will be present in dt-overlay. > +}; > + > +&pioA { > + pinctrl_adc_default: adc_default { > + pinmux = <PIN_PD25__GPIO>, > + <PIN_PD26__GPIO>; > + bias-disable; > + }; > + > + pinctrl_adtrg_default: adtrg_default { > + pinmux = <PIN_PD31__ADTRG>; > + bias-pull-up; > + }; > + > + pinctrl_rpi_spi: rpi_spi { > + pinmux = <PIN_PD12__FLEXCOM4_IO0>, > + <PIN_PD13__FLEXCOM4_IO1>, > + <PIN_PD14__FLEXCOM4_IO2>; > + bias-disable; > + }; > + > + pinctrl_rpi_spi_cs0: rpi_spi_cs0 { > + pinmux = <PIN_PD15__FLEXCOM4_IO3>; > + bias-disable; > + }; > + > + pinctrl_rpi_spi_cs1: rpi_spi_cs1 { > + pinmux = <PIN_PD16__FLEXCOM4_IO4>; > + bias-disable; > + }; > + > + pinctrl_spi1_default: spi1_default { > + pinmux = <PIN_PC1__SPI1_SPCK>, > + <PIN_PC2__SPI1_MOSI>, > + <PIN_PC3__SPI1_MISO>; > + bias-disable; > + }; > + > + pinctrl_spi1_cs: spi1_cs { > + pinmux = <PIN_PC4__SPI1_NPCS0>, > + <PIN_PC5__SPI1_NPCS1>, > + <PIN_PC6__SPI1_NPCS2>, > + <PIN_PC7__SPI1_NPCS3>; > + bias-disable; > + }; > + > + pinctrl_rpi_uart: rpi_uart { > + pinmux = <PIN_PD2__URXD1>, > + <PIN_PD3__UTXD1>; > + bias-disable; > + }; > + > + pinctrl_debug_uart: debug_uart { > + pinmux = <PIN_PB26__URXD0>, > + <PIN_PB27__UTXD0>; > + bias-disable; > + }; > + > + pinctrl_can0_default: can0_default { > + pinmux = <PIN_PC10__CANTX0>, > + <PIN_PC11__CANRX0>; > + bias-disable; > + }; > + > + pinctrl_can1_default: can1_default { > + pinmux = <PIN_PC26__CANTX1>, > + <PIN_PC27__CANRX1>; > + bias-disable; > + }; > + > + pinctrl_i2c0_default: i2c0_default { > + pinmux = <PIN_PB31__TWD0>, > + <PIN_PC0__TWCK0>; > + bias-disable; > + }; > + > + pinctrl_i2c0_gpio: i2c0_gpio { > + pinmux = <PIN_PB31__GPIO>, > + <PIN_PC0__GPIO>; > + bias-disable; > + }; > + > + pinctrl_i2c1_default: i2c1_default { > + pinmux = <PIN_PD4__TWD1>, > + <PIN_PD5__TWCK1>; > + bias-disable; > + }; > + > + pinctrl_i2c1_gpio: i2c1_gpio { > + pinmux = <PIN_PD4__GPIO>, > + <PIN_PD5__GPIO>; > + bias-disable; > + }; > + > + pinctrl_key_gpio_default: key_gpio_default { > + pinmux = <PIN_PA17__GPIO>; > + bias-pull-up; > + }; > + > + pinctrl_led_gpio_default: led_gpio_default { > + pinmux = <PIN_PA7__GPIO>, > + <PIN_PA8__GPIO>, > + <PIN_PA9__GPIO>; > + bias-pull-up; > + }; > + > + pinctrl_qspi1_default: qspi1_default { > + pinmux = <PIN_PB5__QSPI1_SCK>, > + <PIN_PB6__QSPI1_CS>, > + <PIN_PB7__QSPI1_IO0>, > + <PIN_PB8__QSPI1_IO1>, > + <PIN_PB9__QSPI1_IO2>, > + <PIN_PB10__QSPI1_IO3>; > + bias-disable; > + }; > + > + pinctrl_sdmmc0_default: sdmmc0_default { > + pinmux = <PIN_PA0__SDMMC0_CK>, > + <PIN_PA1__SDMMC0_CMD>, > + <PIN_PA2__SDMMC0_DAT0>, > + <PIN_PA3__SDMMC0_DAT1>, > + <PIN_PA4__SDMMC0_DAT2>, > + <PIN_PA5__SDMMC0_DAT3>, > + <PIN_PA11__SDMMC0_VDDSEL>, > + <PIN_PA13__SDMMC0_CD>; > + bias-disable; > + }; > + > + pinctrl_sdmmc1_default: sdmmc1_default { > + pinmux = <PIN_PA18__SDMMC1_DAT0>, > + <PIN_PA19__SDMMC1_DAT1>, > + <PIN_PA20__SDMMC1_DAT2>, > + <PIN_PA21__SDMMC1_DAT3>, > + <PIN_PA22__SDMMC1_CK>, > + <PIN_PA28__SDMMC1_CMD>, > + <PIN_PA30__SDMMC1_CD>; > + bias-disable; > + }; > + These starting here: > + pinctrl_mikrobus1_an: mikrobus1_an { > + pinmux = <PIN_PD25__GPIO>; > + bias-disable; > + }; > + > + pinctrl_mikrobus1_rst: mikrobus1_rst { > + pinmux = <PIN_PA16__GPIO>; > + bias-disable; > + }; > + > + pinctrl_mikrobus1_spi_cs: mikrobus1_spi_cs { > + pinmux = <PIN_PC4__SPI1_NPCS0>; > + bias-disable; > + }; > + > + pinctrl_mikrobus1_spi: mikrobus1_spi { > + pinmux = <PIN_PC1__SPI1_SPCK>, > + <PIN_PC2__SPI1_MOSI>, > + <PIN_PC3__SPI1_MISO>; > + bias-disable; > + }; > + ending here are not used. Same on mikrobus2 I will remove them in patch v2. > + pinctrl_mikrobus1_pwm: mikrobus1_pwm { > + pinmux = <PIN_PA31__PWML0>; > + bias-disable; > + }; pinctrl_mikrobus1_pwm is used on pwm0 node; > + > + pinctrl_mikrobus1_int: mikrobus1_int { > + pinmux = <PIN_PB2__GPIO>; > + bias-disable; > + }; I will remove pinctrl_mikrobus1_int; > + > + pinctrl_mikrobus1_uart: mikrobus1_uart { > + pinmux = <PIN_PB3__URXD4>, > + <PIN_PB4__UTXD4>; > + bias-disable; > + }; Not used > + > + pinctrl_mikrobus2_an: mikrobus2_an { > + pinmux = <PIN_PD26__GPIO>; > + bias-disable; > + }; I will remove pinctrl_mikrobus2_an; > + > + pinctrl_mikrobus2_rst: mikrobus2_rst { > + pinmux = <PIN_PA16__GPIO>; > + bias-disable; > + }; I will remove pinctrl_mikrobus2_rst; > + > + pinctrl_mikrobus2_spi_cs: mikrobus2_spi_cs { > + pinmux = <PIN_PC5__SPI1_NPCS1>; > + bias-disable; > + }; I will remove pinctrl_mikrobus2_spi_cs; > + > + pinctrl_mikrobus2_spi: mikrobus2_spi { > + pinmux = <PIN_PC1__SPI1_SPCK>, > + <PIN_PC2__SPI1_MOSI>, > + <PIN_PC3__SPI1_MISO>; > + bias-disable; > + }; I will remove pinctrl_mikrobus2_spi; > + > + pinctrl_mikrobus2_pwm: mikrobus2_pwm { > + pinmux = <PIN_PB0__PWMH1>; > + bias-disable; > + }; pinctrl_mikrobus2_pwm is used on pwm0 node; > + > + pinctrl_mikrobus2_int: mikrobus2_int { > + pinmux = <PIN_PB1__GPIO>; > + bias-disable; > + }; I will remove pinctrl_mikrobus2_int; > + > + pinctrl_mikrobus2_uart: mikrobus2_uart { > + pinmux = <PIN_PB11__URXD3>, > + <PIN_PB12__UTXD3>; > + bias-disable; > + }; > + > + pinctrl_usb_default: usb_default { > + pinmux = <PIN_PA6__GPIO>; > + bias-disable; > + }; > + > + pinctrl_usba_vbus: usba_vbus { > + pinmux = <PIN_PB13__GPIO>; > + bias-disable; > + }; > + > + pinctrl_macb0_default: macb0_default { > + pinmux = <PIN_PB14__GTXCK>, > + <PIN_PB15__GTXEN>, > + <PIN_PB16__GRXDV>, > + <PIN_PB17__GRXER>, > + <PIN_PB18__GRX0>, > + <PIN_PB19__GRX1>, > + <PIN_PB20__GTX0>, > + <PIN_PB21__GTX1>, > + <PIN_PB22__GMDC>, > + <PIN_PB23__GMDIO>; > + bias-disable; > + }; > + > + pinctrl_macb0_phy_irq: macb0_phy_irq { > + pinmux = <PIN_PB24__GPIO>; > + bias-disable; > + }; > + > + pinctrl_flx1_default: flx1_default { > + pinmux = <PIN_PA24__FLEXCOM1_IO0>, > + <PIN_PA23__FLEXCOM1_IO1>, > + <PIN_PA25__FLEXCOM1_IO3>, > + <PIN_PA26__FLEXCOM1_IO4>; > + bias-disable; > + }; > +}; > + > +&pwm0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_mikrobus1_pwm &pinctrl_mikrobus2_pwm>; > + status = "okay"; > +}; > + > +&qspi1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_qspi1_default>; > + status = "okay"; > + > + flash@0 { > + #address-cells = <1>; > + #size-cells = <1>; > + compatible = "jedec,spi-nor"; > + reg = <0>; > + spi-max-frequency = <80000000>; > + spi-tx-bus-width = <4>; > + spi-rx-bus-width = <4>; > + m25p,fast-read; > + label = "atmel_qspi1"; > + status = "okay"; > + > + at91bootstrap@0 { > + label = "at91bootstrap"; > + reg = <0x0 0x40000>; > + }; > + > + bootloader@40000 { > + label = "bootloader"; > + reg = <0x40000 0xc0000>; > + }; > + > + bootloaderenvred@100000 { > + label = "bootloader env redundant"; > + reg = <0x100000 0x40000>; > + }; > + > + bootloaderenv@140000 { > + label = "bootloader env"; > + reg = <0x140000 0x40000>; > + }; > + > + dtb@180000 { > + label = "device tree"; > + reg = <0x180000 0x80000>; > + }; > + > + kernel@200000 { > + label = "kernel"; > + reg = <0x200000 0x600000>; > + }; > + }; > +}; > + > +&sdmmc0 { > + bus-width = <4>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_sdmmc0_default>; > + vmmc-supply = <&vdd_3v3>; > + vqmmc-supply = <&vdd_1v8>; > + status = "okay"; > +}; > + > +&sdmmc1 { > + bus-width = <4>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_sdmmc1_default>; > + vqmmc-supply = <&vdd_3v3>; > + vmmc-supply = <&vdd_3v3>; > + status = "okay"; > +}; > + > +&shutdown_controller { > + debounce-delay-us = <976>; > + atmel,wakeup-rtc-timer; > + > + input@0 { > + reg = <0>; > + }; > +}; > + > +&spi1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_spi1_default &pinctrl_spi1_cs>; > + status = "okay"; > +}; > + > +&tcb0 { > + timer0: timer@0 { > + compatible = "atmel,tcb-timer"; > + reg = <0>; > + }; > + > + timer1: timer@1 { > + compatible = "atmel,tcb-timer"; > + reg = <1>; > + }; > +}; > + > +&uart0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_debug_uart>; > + atmel,use-dma-rx; > + atmel,use-dma-tx; > + status = "okay"; > +}; > + > +&uart1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_rpi_uart>; > + atmel,use-dma-rx; > + atmel,use-dma-tx; > + status = "okay"; > +}; > + > +&uart3 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_mikrobus2_uart>; > + atmel,use-dma-rx; > + atmel,use-dma-tx; > + status = "okay"; > +}; > + > +&uart4 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_mikrobus1_uart>; > + atmel,use-dma-rx; > + atmel,use-dma-tx; > + status = "okay"; > +}; > + > +&usb0 { > + atmel,vbus-gpio = <&pioA PIN_PB13 GPIO_ACTIVE_HIGH>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_usba_vbus>; > + status = "okay"; > +}; > + > +&usb1 { > + num-ports = <3>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_usb_default>; > + status = "okay"; > +}; > + > +&watchdog { > + status = "okay"; > +}; ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 2/2] ARM: dts: at91: sama5d29_curiosity: Add device tree for sama5d29_curiosity board 2023-08-02 12:19 ` Mihai.Sain @ 2023-08-03 9:09 ` claudiu beznea 2023-09-11 16:01 ` Nicolas Ferre 0 siblings, 1 reply; 14+ messages in thread From: claudiu beznea @ 2023-08-03 9:09 UTC (permalink / raw) To: Mihai.Sain, robh+dt, krzysztof.kozlowski+dt, conor+dt, Nicolas.Ferre, alexandre.belloni, andre.przywara, Andrei.Simion, Jerry.Ray, devicetree, linux-arm-kernel, linux-kernel Cc: Cristian.Birsan Hi, Mihai, On 02.08.2023 15:19, Mihai.Sain@microchip.com wrote: >> +&macb0 { >> + pinctrl-names = "default"; >> + pinctrl-0 = <&pinctrl_macb0_default &pinctrl_macb0_phy_irq>; >> + #address-cells = <1>; >> + #size-cells = <0>; >> + phy-mode = "rmii"; >> + status = "disabled"; > Should any phy/mdio container be placed here? Also, any reason this node is disabled? > The board has no phy. > We will add external phy boards to macb interface and they will be present in dt-overlay. > Then remove the macb0 node from here and keep it in overlay. Thank you, Claudiu Beznea ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 2/2] ARM: dts: at91: sama5d29_curiosity: Add device tree for sama5d29_curiosity board 2023-08-03 9:09 ` claudiu beznea @ 2023-09-11 16:01 ` Nicolas Ferre 2023-09-12 11:01 ` claudiu beznea 0 siblings, 1 reply; 14+ messages in thread From: Nicolas Ferre @ 2023-09-11 16:01 UTC (permalink / raw) To: claudiu beznea, Mihai.Sain, robh+dt, krzysztof.kozlowski+dt, conor+dt, alexandre.belloni, andre.przywara, Andrei.Simion, Jerry.Ray, devicetree, linux-arm-kernel, linux-kernel Cc: Cristian.Birsan Hi Mihai, Claudiu, First of all, thanks a lot for your help on this board DT, that's great! On 03/08/2023 at 11:09, claudiu beznea wrote: > Hi, Mihai, > > On 02.08.2023 15:19,Mihai.Sain@microchip.com wrote: >>> +&macb0 { >>> + pinctrl-names = "default"; >>> + pinctrl-0 = <&pinctrl_macb0_default &pinctrl_macb0_phy_irq>; >>> + #address-cells = <1>; >>> + #size-cells = <0>; >>> + phy-mode = "rmii"; >>> + status = "disabled"; >> Should any phy/mdio container be placed here? Also, any reason this node is disabled? >> The board has no phy. >> We will add external phy boards to macb interface and they will be present in dt-overlay. >> > Then remove the macb0 node from here and keep it in overlay. What about choosing a phy and making it enabled by default. Because I doubt people will use this board without a phy under Linux, honestly. Overlays would remove this "by default" node and add the replacing phy if hardware plugged-in is different. By doing this we avoid having difficult situations where the most used phy with this board is disabled or non existant and that we absolutely need an overlay for such a basic thing as having Ethernet working out-of-the-box... Just my $0.02, but it might simplify our life in the long run... Best regards, Nicolas ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 2/2] ARM: dts: at91: sama5d29_curiosity: Add device tree for sama5d29_curiosity board 2023-09-11 16:01 ` Nicolas Ferre @ 2023-09-12 11:01 ` claudiu beznea 0 siblings, 0 replies; 14+ messages in thread From: claudiu beznea @ 2023-09-12 11:01 UTC (permalink / raw) To: Nicolas Ferre, Mihai.Sain, robh+dt, krzysztof.kozlowski+dt, conor+dt, alexandre.belloni, andre.przywara, Andrei.Simion, Jerry.Ray, devicetree, linux-arm-kernel, linux-kernel Cc: Cristian.Birsan Hi, Nicolas, Mihai, On 11.09.2023 19:01, Nicolas Ferre wrote: > Hi Mihai, Claudiu, > > First of all, thanks a lot for your help on this board DT, that's great! > > On 03/08/2023 at 11:09, claudiu beznea wrote: >> Hi, Mihai, >> >> On 02.08.2023 15:19,Mihai.Sain@microchip.com wrote: >>>> +&macb0 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&pinctrl_macb0_default &pinctrl_macb0_phy_irq>; >>>> + #address-cells = <1>; >>>> + #size-cells = <0>; >>>> + phy-mode = "rmii"; >>>> + status = "disabled"; >>> Should any phy/mdio container be placed here? Also, any reason this node >>> is disabled? >>> The board has no phy. >>> We will add external phy boards to macb interface and they will be >>> present in dt-overlay. >>> >> Then remove the macb0 node from here and keep it in overlay. > > What about choosing a phy and making it enabled by default. Because I doubt > people will use this board without a phy under Linux, honestly. > Overlays would remove this "by default" node and add the replacing phy if > hardware plugged-in is different. The only thing about it (FMPOV) is that there might be people expecting the default bindings to work with every PHY. > > By doing this we avoid having difficult situations where the most used phy > with this board is disabled or non existant and that we absolutely need an > overlay for such a basic thing as having Ethernet working out-of-the-box... I undestand your POV but DTS should describe the hardware. At least, is there any PHY shipped with the board to be considered the default one? SAMA5D3 EDS uses the same approach: it defines individual overlays for each supported ethernet PHYs. Thank you, Claudiu Beznea > > Just my $0.02, but it might simplify our life in the long run... > > Best regards, > Nicolas ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 2/2] ARM: dts: at91: sama5d29_curiosity: Add device tree for sama5d29_curiosity board 2023-08-01 11:11 ` [PATCH 2/2] ARM: dts: at91: sama5d29_curiosity: Add device tree for sama5d29_curiosity board Mihai Sain 2023-08-02 4:40 ` claudiu beznea @ 2023-08-02 5:02 ` claudiu beznea 2023-08-02 11:59 ` Mihai.Sain 1 sibling, 1 reply; 14+ messages in thread From: claudiu beznea @ 2023-08-02 5:02 UTC (permalink / raw) To: Mihai Sain, robh+dt, krzysztof.kozlowski+dt, conor+dt, nicolas.ferre, alexandre.belloni, claudiu.beznea, andre.przywara, andrei.simion, jerry.ray, devicetree, linux-arm-kernel, linux-kernel Cc: cristian.birsan Hi, Mihai, Few other comments. See below. On 01.08.2023 14:11, Mihai Sain wrote: > Add initial device tree file for sama5d29_curiosity board. > > Signed-off-by: Mihai Sain <mihai.sain@microchip.com> > --- > arch/arm/boot/dts/microchip/Makefile | 2 + > .../dts/microchip/at91-sama5d29_curiosity.dts | 706 ++++++++++++++++++ > 2 files changed, 708 insertions(+) > create mode 100644 arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts > > diff --git a/arch/arm/boot/dts/microchip/Makefile b/arch/arm/boot/dts/microchip/Makefile > index 31e03747cdf4..efde9546c8f4 100644 > --- a/arch/arm/boot/dts/microchip/Makefile > +++ b/arch/arm/boot/dts/microchip/Makefile > @@ -4,6 +4,7 @@ DTC_FLAGS_at91-sam9x60_curiosity := -@ > DTC_FLAGS_at91-sam9x60ek := -@ > DTC_FLAGS_at91-sama5d27_som1_ek := -@ > DTC_FLAGS_at91-sama5d27_wlsom1_ek := -@ > +DTC_FLAGS_at91-sama5d29_curiosity := -@ > DTC_FLAGS_at91-sama5d2_icp := -@ > DTC_FLAGS_at91-sama5d2_ptc_ek := -@ > DTC_FLAGS_at91-sama5d2_xplained := -@ > @@ -64,6 +65,7 @@ dtb-$(CONFIG_SOC_SAM_V7) += \ > at91-nattis-2-natte-2.dtb \ > at91-sama5d27_som1_ek.dtb \ > at91-sama5d27_wlsom1_ek.dtb \ > + at91-sama5d29_curiosity.dtb \ > at91-sama5d2_icp.dtb \ > at91-sama5d2_ptc_ek.dtb \ > at91-sama5d2_xplained.dtb \ > diff --git a/arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts b/arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts > new file mode 100644 > index 000000000000..17af10446dac > --- /dev/null > +++ b/arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts > @@ -0,0 +1,706 @@ > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > +/* > + * at91-sama5d29_curiosity.dts - Device Tree file for SAMA5D29 Curiosity board > + * > + * Copyright (C) 2023 Microchip Technology Inc. and its subsidiaries > + * > + * Author: Mihai Sain <mihai.sain@microchip.com> > + * > + */ > +/dts-v1/; > +#include "sama5d29.dtsi" > +#include "sama5d2-pinfunc.h" > +#include <dt-bindings/gpio/gpio.h> > +#include <dt-bindings/input/input.h> > +#include <dt-bindings/mfd/atmel-flexcom.h> > + > +/ { > + model = "Microchip SAMA5D29 Curiosity"; > + compatible = "microchip,sama5d29-curiosity", "atmel,sama5d29", "atmel,sama5d2", "atmel,sama5"; > + > + aliases { > + serial0 = &uart0; // debug > + serial1 = &uart1; // RPi > + serial2 = &uart3; // mikro BUS 2 > + serial3 = &uart4; // mikro BUS 1 > + serial4 = &uart6; // flx1 Bluetooth > + i2c0 = &i2c0; > + i2c1 = &i2c1; > + }; > + > + chosen { > + bootargs = "console=ttyS0,115200 root=/dev/mmcblk0p2 rw rootwait"; > + stdout-path = "serial0:115200n8"; > + }; > + > + clocks { > + slow_xtal { > + clock-frequency = <32768>; > + }; > + > + main_xtal { > + clock-frequency = <24000000>; > + }; > + }; > + > + gpio-keys { > + compatible = "gpio-keys"; > + > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_key_gpio_default>; > + > + button-1 { > + label = "USER_PB"; > + gpios = <&pioA PIN_PA17 GPIO_ACTIVE_LOW>; > + linux,code = <KEY_PROG1>; > + wakeup-source; > + }; > + }; > + > + leds { > + compatible = "gpio-leds"; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_led_gpio_default>; > + status = "okay"; > + > + red { > + label = "red"; > + gpios = <&pioA PIN_PA7 GPIO_ACTIVE_HIGH>; > + }; > + > + green { > + label = "green"; > + gpios = <&pioA PIN_PA8 GPIO_ACTIVE_HIGH>; > + }; > + > + blue { > + label = "blue"; > + gpios = <&pioA PIN_PA9 GPIO_ACTIVE_HIGH>; > + linux,default-trigger = "heartbeat"; > + }; > + }; > + > + memory@20000000 { > + device_type = "memory"; > + reg = <0x20000000 0x20000000>; > + }; > + > + vdd_1v8: regulator-vqmmc { > + compatible = "regulator-fixed"; > + regulator-name = "VDD_1V8"; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1800000>; > + status = "okay"; > + }; > +}; > + > +&adc { > + vddana-supply = <&vdd_3v3>; > + vref-supply = <&vdd_3v3>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_adc_default &pinctrl_adtrg_default>; > + status = "okay"; > +}; > + > +&can0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_can0_default>; > + status = "okay"; > +}; > + > +&can1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_can1_default>; > + status = "okay"; > +}; > + > +&flx1 { > + atmel,flexcom-mode = <ATMEL_FLEXCOM_MODE_USART>; > + status = "okay"; > + > + uart6: serial@200 { > + pinctrl-0 = <&pinctrl_flx1_default>; > + pinctrl-names = "default"; > + atmel,use-dma-rx; > + atmel,use-dma-tx; > + status = "okay"; > + }; > +}; > + > +&flx4 { /* RPi spi */ > + atmel,flexcom-mode = <ATMEL_FLEXCOM_MODE_SPI>; > + status = "okay"; > + > + spi6: spi@400 { > + dmas = <0>, <0>; > + cs-gpios = <&pioA PIN_PD15 GPIO_ACTIVE_LOW>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_rpi_spi &pinctrl_rpi_spi_cs0>; > + status = "okay"; > + }; > + > + spi7: spi@400 { > + dmas = <0>, <0>; > + cs-gpios = <&pioA PIN_PD16 GPIO_ACTIVE_LOW>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_rpi_spi &pinctrl_rpi_spi_cs1>; > + status = "okay"; > + }; > +}; > + > +&i2c0 { > + dmas = <0>, <0>; > + pinctrl-names = "default", "gpio"; > + pinctrl-0 = <&pinctrl_i2c0_default>; > + pinctrl-1 = <&pinctrl_i2c0_gpio>; > + sda-gpios = <&pioA PIN_PB31 GPIO_ACTIVE_HIGH>; > + scl-gpios = <&pioA PIN_PC0 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>; > + i2c-sda-hold-time-ns = <350>; > + status = "okay"; > + > + mcp16502@5b { > + compatible = "microchip,mcp16502"; > + reg = <0x5b>; > + status = "okay"; > + lpm-gpios = <&pioBU 0 GPIO_ACTIVE_LOW>; > + > + regulators { > + vdd_3v3: VDD_IO { Dashes in node name, thus VDD-IO (same for the other entires). > + regulator-name = "VDD_IO"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-initial-mode = <2>; > + regulator-allowed-modes = <2>, <4>; > + regulator-always-on; > + > + regulator-state-standby { > + regulator-on-in-suspend; > + regulator-mode = <4>; > + }; > + > + regulator-state-mem { > + regulator-off-in-suspend; > + regulator-mode = <4>; > + }; > + }; > + > + vddio_ddr: VDD_DDR { > + regulator-name = "VDD_DDR"; > + regulator-min-microvolt = <1200000>; > + regulator-max-microvolt = <1200000>; > + regulator-initial-mode = <2>; > + regulator-allowed-modes = <2>, <4>; > + regulator-always-on; > + > + regulator-state-standby { > + regulator-on-in-suspend; > + regulator-suspend-microvolt = <1200000>; > + regulator-changeable-in-suspend; > + regulator-mode = <4>; > + }; > + > + regulator-state-mem { > + regulator-on-in-suspend; > + regulator-suspend-microvolt = <1200000>; > + regulator-changeable-in-suspend; > + regulator-mode = <4>; > + }; > + }; > + > + vdd_core: VDD_CORE { > + regulator-name = "VDD_CORE"; > + regulator-min-microvolt = <1250000>; > + regulator-max-microvolt = <1250000>; > + regulator-initial-mode = <2>; > + regulator-allowed-modes = <2>, <4>; > + regulator-always-on; > + > + regulator-state-standby { > + regulator-on-in-suspend; > + regulator-mode = <4>; > + }; > + > + regulator-state-mem { > + regulator-off-in-suspend; > + regulator-mode = <4>; > + }; > + }; > + > + vdd_ddr: VDD_OTHER { > + regulator-name = "VDD_OTHER"; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1800000>; > + regulator-initial-mode = <2>; > + regulator-allowed-modes = <2>, <4>; > + regulator-always-on; > + > + regulator-state-standby { > + regulator-on-in-suspend; > + regulator-suspend-microvolt = <1800000>; > + regulator-changeable-in-suspend; > + regulator-mode = <4>; > + }; > + > + regulator-state-mem { > + regulator-on-in-suspend; > + regulator-suspend-microvolt = <1800000>; > + regulator-changeable-in-suspend; > + regulator-mode = <4>; > + }; > + }; > + > + LDO1 { > + regulator-name = "LDO1"; > + regulator-min-microvolt = <2500000>; > + regulator-max-microvolt = <2500000>; > + regulator-always-on; > + > + regulator-state-standby { > + regulator-on-in-suspend; > + }; > + > + regulator-state-mem { > + regulator-off-in-suspend; > + }; > + }; > + > + LDO2 { > + regulator-name = "LDO2"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-always-on; > + > + regulator-state-standby { > + regulator-on-in-suspend; > + }; > + > + regulator-state-mem { > + regulator-off-in-suspend; > + }; > + }; > + }; > + }; > +}; > + > +&i2c1 { > + dmas = <0>, <0>; > + pinctrl-names = "default", "gpio"; > + pinctrl-0 = <&pinctrl_i2c1_default>; > + i2c-analog-filter; > + i2c-digital-filter; > + i2c-digital-filter-width-ns = <35>; > + pinctrl-1 = <&pinctrl_i2c1_gpio>; > + sda-gpios = <&pioA PIN_PD4 GPIO_ACTIVE_HIGH>; > + scl-gpios = <&pioA PIN_PD5 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>; > + status = "okay"; > +}; > + > +&macb0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_macb0_default &pinctrl_macb0_phy_irq>; > + #address-cells = <1>; > + #size-cells = <0>; > + phy-mode = "rmii"; > + status = "disabled"; > +}; > + > +&pioA { > + pinctrl_adc_default: adc_default { Dashes in node name, thus adc-default (same for the other entries). > + pinmux = <PIN_PD25__GPIO>, > + <PIN_PD26__GPIO>; > + bias-disable; > + }; > + > + pinctrl_adtrg_default: adtrg_default { > + pinmux = <PIN_PD31__ADTRG>; > + bias-pull-up; > + }; > + > + pinctrl_rpi_spi: rpi_spi { > + pinmux = <PIN_PD12__FLEXCOM4_IO0>, > + <PIN_PD13__FLEXCOM4_IO1>, > + <PIN_PD14__FLEXCOM4_IO2>; > + bias-disable; > + }; > + > + pinctrl_rpi_spi_cs0: rpi_spi_cs0 { > + pinmux = <PIN_PD15__FLEXCOM4_IO3>; > + bias-disable; > + }; > + > + pinctrl_rpi_spi_cs1: rpi_spi_cs1 { > + pinmux = <PIN_PD16__FLEXCOM4_IO4>; > + bias-disable; > + }; > + > + pinctrl_spi1_default: spi1_default { > + pinmux = <PIN_PC1__SPI1_SPCK>, > + <PIN_PC2__SPI1_MOSI>, > + <PIN_PC3__SPI1_MISO>; > + bias-disable; > + }; > + > + pinctrl_spi1_cs: spi1_cs { > + pinmux = <PIN_PC4__SPI1_NPCS0>, > + <PIN_PC5__SPI1_NPCS1>, > + <PIN_PC6__SPI1_NPCS2>, > + <PIN_PC7__SPI1_NPCS3>; > + bias-disable; > + }; > + > + pinctrl_rpi_uart: rpi_uart { > + pinmux = <PIN_PD2__URXD1>, > + <PIN_PD3__UTXD1>; > + bias-disable; > + }; > + > + pinctrl_debug_uart: debug_uart { > + pinmux = <PIN_PB26__URXD0>, > + <PIN_PB27__UTXD0>; > + bias-disable; > + }; > + > + pinctrl_can0_default: can0_default { > + pinmux = <PIN_PC10__CANTX0>, > + <PIN_PC11__CANRX0>; > + bias-disable; > + }; > + > + pinctrl_can1_default: can1_default { > + pinmux = <PIN_PC26__CANTX1>, > + <PIN_PC27__CANRX1>; > + bias-disable; > + }; > + > + pinctrl_i2c0_default: i2c0_default { > + pinmux = <PIN_PB31__TWD0>, > + <PIN_PC0__TWCK0>; > + bias-disable; > + }; > + > + pinctrl_i2c0_gpio: i2c0_gpio { > + pinmux = <PIN_PB31__GPIO>, > + <PIN_PC0__GPIO>; > + bias-disable; > + }; > + > + pinctrl_i2c1_default: i2c1_default { > + pinmux = <PIN_PD4__TWD1>, > + <PIN_PD5__TWCK1>; > + bias-disable; > + }; > + > + pinctrl_i2c1_gpio: i2c1_gpio { > + pinmux = <PIN_PD4__GPIO>, > + <PIN_PD5__GPIO>; > + bias-disable; > + }; > + > + pinctrl_key_gpio_default: key_gpio_default { > + pinmux = <PIN_PA17__GPIO>; > + bias-pull-up; > + }; > + > + pinctrl_led_gpio_default: led_gpio_default { > + pinmux = <PIN_PA7__GPIO>, > + <PIN_PA8__GPIO>, > + <PIN_PA9__GPIO>; > + bias-pull-up; > + }; > + > + pinctrl_qspi1_default: qspi1_default { > + pinmux = <PIN_PB5__QSPI1_SCK>, > + <PIN_PB6__QSPI1_CS>, > + <PIN_PB7__QSPI1_IO0>, > + <PIN_PB8__QSPI1_IO1>, > + <PIN_PB9__QSPI1_IO2>, > + <PIN_PB10__QSPI1_IO3>; > + bias-disable; > + }; > + > + pinctrl_sdmmc0_default: sdmmc0_default { > + pinmux = <PIN_PA0__SDMMC0_CK>, > + <PIN_PA1__SDMMC0_CMD>, > + <PIN_PA2__SDMMC0_DAT0>, > + <PIN_PA3__SDMMC0_DAT1>, > + <PIN_PA4__SDMMC0_DAT2>, > + <PIN_PA5__SDMMC0_DAT3>, > + <PIN_PA11__SDMMC0_VDDSEL>, > + <PIN_PA13__SDMMC0_CD>; > + bias-disable; > + }; > + > + pinctrl_sdmmc1_default: sdmmc1_default { > + pinmux = <PIN_PA18__SDMMC1_DAT0>, > + <PIN_PA19__SDMMC1_DAT1>, > + <PIN_PA20__SDMMC1_DAT2>, > + <PIN_PA21__SDMMC1_DAT3>, > + <PIN_PA22__SDMMC1_CK>, > + <PIN_PA28__SDMMC1_CMD>, > + <PIN_PA30__SDMMC1_CD>; > + bias-disable; > + }; > + > + pinctrl_mikrobus1_an: mikrobus1_an { > + pinmux = <PIN_PD25__GPIO>; > + bias-disable; > + }; > + > + pinctrl_mikrobus1_rst: mikrobus1_rst { > + pinmux = <PIN_PA16__GPIO>; > + bias-disable; > + }; > + > + pinctrl_mikrobus1_spi_cs: mikrobus1_spi_cs { > + pinmux = <PIN_PC4__SPI1_NPCS0>; > + bias-disable; > + }; > + > + pinctrl_mikrobus1_spi: mikrobus1_spi { > + pinmux = <PIN_PC1__SPI1_SPCK>, > + <PIN_PC2__SPI1_MOSI>, > + <PIN_PC3__SPI1_MISO>; > + bias-disable; > + }; > + > + pinctrl_mikrobus1_pwm: mikrobus1_pwm { > + pinmux = <PIN_PA31__PWML0>; > + bias-disable; > + }; > + > + pinctrl_mikrobus1_int: mikrobus1_int { > + pinmux = <PIN_PB2__GPIO>; > + bias-disable; > + }; > + > + pinctrl_mikrobus1_uart: mikrobus1_uart { > + pinmux = <PIN_PB3__URXD4>, > + <PIN_PB4__UTXD4>; > + bias-disable; > + }; > + > + pinctrl_mikrobus2_an: mikrobus2_an { > + pinmux = <PIN_PD26__GPIO>; > + bias-disable; > + }; > + > + pinctrl_mikrobus2_rst: mikrobus2_rst { > + pinmux = <PIN_PA16__GPIO>; > + bias-disable; > + }; > + > + pinctrl_mikrobus2_spi_cs: mikrobus2_spi_cs { > + pinmux = <PIN_PC5__SPI1_NPCS1>; > + bias-disable; > + }; > + > + pinctrl_mikrobus2_spi: mikrobus2_spi { > + pinmux = <PIN_PC1__SPI1_SPCK>, > + <PIN_PC2__SPI1_MOSI>, > + <PIN_PC3__SPI1_MISO>; > + bias-disable; > + }; > + > + pinctrl_mikrobus2_pwm: mikrobus2_pwm { > + pinmux = <PIN_PB0__PWMH1>; > + bias-disable; > + }; > + > + pinctrl_mikrobus2_int: mikrobus2_int { > + pinmux = <PIN_PB1__GPIO>; > + bias-disable; > + }; > + > + pinctrl_mikrobus2_uart: mikrobus2_uart { > + pinmux = <PIN_PB11__URXD3>, > + <PIN_PB12__UTXD3>; > + bias-disable; > + }; > + > + pinctrl_usb_default: usb_default { > + pinmux = <PIN_PA6__GPIO>; > + bias-disable; > + }; > + > + pinctrl_usba_vbus: usba_vbus { > + pinmux = <PIN_PB13__GPIO>; > + bias-disable; > + }; > + > + pinctrl_macb0_default: macb0_default { > + pinmux = <PIN_PB14__GTXCK>, > + <PIN_PB15__GTXEN>, > + <PIN_PB16__GRXDV>, > + <PIN_PB17__GRXER>, > + <PIN_PB18__GRX0>, > + <PIN_PB19__GRX1>, > + <PIN_PB20__GTX0>, > + <PIN_PB21__GTX1>, > + <PIN_PB22__GMDC>, > + <PIN_PB23__GMDIO>; > + bias-disable; > + }; > + > + pinctrl_macb0_phy_irq: macb0_phy_irq { > + pinmux = <PIN_PB24__GPIO>; > + bias-disable; > + }; > + > + pinctrl_flx1_default: flx1_default { > + pinmux = <PIN_PA24__FLEXCOM1_IO0>, > + <PIN_PA23__FLEXCOM1_IO1>, > + <PIN_PA25__FLEXCOM1_IO3>, > + <PIN_PA26__FLEXCOM1_IO4>; > + bias-disable; > + }; > +}; > + > +&pwm0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_mikrobus1_pwm &pinctrl_mikrobus2_pwm>; > + status = "okay"; > +}; > + > +&qspi1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_qspi1_default>; > + status = "okay"; > + > + flash@0 { > + #address-cells = <1>; > + #size-cells = <1>; > + compatible = "jedec,spi-nor"; > + reg = <0>; > + spi-max-frequency = <80000000>; > + spi-tx-bus-width = <4>; > + spi-rx-bus-width = <4>; > + m25p,fast-read; > + label = "atmel_qspi1"; > + status = "okay"; > + > + at91bootstrap@0 { > + label = "at91bootstrap"; > + reg = <0x0 0x40000>; > + }; > + > + bootloader@40000 { > + label = "bootloader"; > + reg = <0x40000 0xc0000>; > + }; > + > + bootloaderenvred@100000 { > + label = "bootloader env redundant"; > + reg = <0x100000 0x40000>; > + }; > + > + bootloaderenv@140000 { > + label = "bootloader env"; > + reg = <0x140000 0x40000>; > + }; > + > + dtb@180000 { > + label = "device tree"; > + reg = <0x180000 0x80000>; > + }; > + > + kernel@200000 { > + label = "kernel"; > + reg = <0x200000 0x600000>; > + }; > + }; > +}; > + > +&sdmmc0 { > + bus-width = <4>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_sdmmc0_default>; > + vmmc-supply = <&vdd_3v3>; > + vqmmc-supply = <&vdd_1v8>; > + status = "okay"; > +}; > + > +&sdmmc1 { > + bus-width = <4>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_sdmmc1_default>; > + vqmmc-supply = <&vdd_3v3>; > + vmmc-supply = <&vdd_3v3>; > + status = "okay"; > +}; > + > +&shutdown_controller { > + debounce-delay-us = <976>; > + atmel,wakeup-rtc-timer; > + > + input@0 { > + reg = <0>; > + }; > +}; > + > +&spi1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_spi1_default &pinctrl_spi1_cs>; > + status = "okay"; > +}; > + > +&tcb0 { > + timer0: timer@0 { > + compatible = "atmel,tcb-timer"; > + reg = <0>; > + }; > + > + timer1: timer@1 { > + compatible = "atmel,tcb-timer"; > + reg = <1>; > + }; > +}; > + > +&uart0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_debug_uart>; > + atmel,use-dma-rx; > + atmel,use-dma-tx; > + status = "okay"; > +}; > + > +&uart1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_rpi_uart>; > + atmel,use-dma-rx; > + atmel,use-dma-tx; > + status = "okay"; > +}; > + > +&uart3 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_mikrobus2_uart>; > + atmel,use-dma-rx; > + atmel,use-dma-tx; > + status = "okay"; > +}; > + > +&uart4 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_mikrobus1_uart>; > + atmel,use-dma-rx; > + atmel,use-dma-tx; > + status = "okay"; > +}; > + > +&usb0 { > + atmel,vbus-gpio = <&pioA PIN_PB13 GPIO_ACTIVE_HIGH>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_usba_vbus>; > + status = "okay"; > +}; > + > +&usb1 { > + num-ports = <3>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_usb_default>; > + status = "okay"; > +}; > + > +&watchdog { > + status = "okay"; > +}; ^ permalink raw reply [flat|nested] 14+ messages in thread
* RE: [PATCH 2/2] ARM: dts: at91: sama5d29_curiosity: Add device tree for sama5d29_curiosity board 2023-08-02 5:02 ` claudiu beznea @ 2023-08-02 11:59 ` Mihai.Sain 0 siblings, 0 replies; 14+ messages in thread From: Mihai.Sain @ 2023-08-02 11:59 UTC (permalink / raw) To: claudiu.beznea, robh+dt, krzysztof.kozlowski+dt, conor+dt, Nicolas.Ferre, alexandre.belloni, Claudiu.Beznea, andre.przywara, Andrei.Simion, Jerry.Ray, devicetree, linux-arm-kernel, linux-kernel Cc: Cristian.Birsan Hi Claudiu, Thank you for reviewing the dts. See bellow my answers. -----Original Message----- From: claudiu beznea <claudiu.beznea@tuxon.dev> Sent: Wednesday, August 2, 2023 8:02 AM To: Mihai Sain - M19926 <Mihai.Sain@microchip.com>; robh+dt@kernel.org; krzysztof.kozlowski+dt@linaro.org; conor+dt@kernel.org; Nicolas Ferre - M43238 <Nicolas.Ferre@microchip.com>; alexandre.belloni@bootlin.com; Claudiu Beznea - M18063 <Claudiu.Beznea@microchip.com>; andre.przywara@arm.com; Andrei Simion - M76677 <Andrei.Simion@microchip.com>; Jerry Ray - C33025 <Jerry.Ray@microchip.com>; devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux-kernel@vger.kernel.org Cc: Cristian Birsan - M91496 <Cristian.Birsan@microchip.com> Subject: Re: [PATCH 2/2] ARM: dts: at91: sama5d29_curiosity: Add device tree for sama5d29_curiosity board [You don't often get email from claudiu.beznea@tuxon.dev. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ] EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe Hi, Mihai, Few other comments. See below. On 01.08.2023 14:11, Mihai Sain wrote: > Add initial device tree file for sama5d29_curiosity board. > > Signed-off-by: Mihai Sain <mihai.sain@microchip.com> > --- > arch/arm/boot/dts/microchip/Makefile | 2 + > .../dts/microchip/at91-sama5d29_curiosity.dts | 706 > ++++++++++++++++++ > 2 files changed, 708 insertions(+) > create mode 100644 > arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts > > diff --git a/arch/arm/boot/dts/microchip/Makefile > b/arch/arm/boot/dts/microchip/Makefile > index 31e03747cdf4..efde9546c8f4 100644 > --- a/arch/arm/boot/dts/microchip/Makefile > +++ b/arch/arm/boot/dts/microchip/Makefile > @@ -4,6 +4,7 @@ DTC_FLAGS_at91-sam9x60_curiosity := -@ > DTC_FLAGS_at91-sam9x60ek := -@ DTC_FLAGS_at91-sama5d27_som1_ek := -@ > DTC_FLAGS_at91-sama5d27_wlsom1_ek := -@ > +DTC_FLAGS_at91-sama5d29_curiosity := -@ > DTC_FLAGS_at91-sama5d2_icp := -@ > DTC_FLAGS_at91-sama5d2_ptc_ek := -@ > DTC_FLAGS_at91-sama5d2_xplained := -@ @@ -64,6 +65,7 @@ > dtb-$(CONFIG_SOC_SAM_V7) += \ > at91-nattis-2-natte-2.dtb \ > at91-sama5d27_som1_ek.dtb \ > at91-sama5d27_wlsom1_ek.dtb \ > + at91-sama5d29_curiosity.dtb \ > at91-sama5d2_icp.dtb \ > at91-sama5d2_ptc_ek.dtb \ > at91-sama5d2_xplained.dtb \ > diff --git a/arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts > b/arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts > new file mode 100644 > index 000000000000..17af10446dac > --- /dev/null > +++ b/arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts > @@ -0,0 +1,706 @@ > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > +/* > + * at91-sama5d29_curiosity.dts - Device Tree file for SAMA5D29 > +Curiosity board > + * > + * Copyright (C) 2023 Microchip Technology Inc. and its subsidiaries > + * > + * Author: Mihai Sain <mihai.sain@microchip.com> > + * > + */ > +/dts-v1/; > +#include "sama5d29.dtsi" > +#include "sama5d2-pinfunc.h" > +#include <dt-bindings/gpio/gpio.h> > +#include <dt-bindings/input/input.h> > +#include <dt-bindings/mfd/atmel-flexcom.h> > + > +/ { > + model = "Microchip SAMA5D29 Curiosity"; > + compatible = "microchip,sama5d29-curiosity", "atmel,sama5d29", > +"atmel,sama5d2", "atmel,sama5"; > + > + aliases { > + serial0 = &uart0; // debug > + serial1 = &uart1; // RPi > + serial2 = &uart3; // mikro BUS 2 > + serial3 = &uart4; // mikro BUS 1 > + serial4 = &uart6; // flx1 Bluetooth > + i2c0 = &i2c0; > + i2c1 = &i2c1; > + }; > + > + chosen { > + bootargs = "console=ttyS0,115200 root=/dev/mmcblk0p2 rw rootwait"; > + stdout-path = "serial0:115200n8"; > + }; > + > + clocks { > + slow_xtal { > + clock-frequency = <32768>; > + }; > + > + main_xtal { > + clock-frequency = <24000000>; > + }; > + }; > + > + gpio-keys { > + compatible = "gpio-keys"; > + > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_key_gpio_default>; > + > + button-1 { > + label = "USER_PB"; > + gpios = <&pioA PIN_PA17 GPIO_ACTIVE_LOW>; > + linux,code = <KEY_PROG1>; > + wakeup-source; > + }; > + }; > + > + leds { > + compatible = "gpio-leds"; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_led_gpio_default>; > + status = "okay"; > + > + red { > + label = "red"; > + gpios = <&pioA PIN_PA7 GPIO_ACTIVE_HIGH>; > + }; > + > + green { > + label = "green"; > + gpios = <&pioA PIN_PA8 GPIO_ACTIVE_HIGH>; > + }; > + > + blue { > + label = "blue"; > + gpios = <&pioA PIN_PA9 GPIO_ACTIVE_HIGH>; > + linux,default-trigger = "heartbeat"; > + }; > + }; > + > + memory@20000000 { > + device_type = "memory"; > + reg = <0x20000000 0x20000000>; > + }; > + > + vdd_1v8: regulator-vqmmc { > + compatible = "regulator-fixed"; > + regulator-name = "VDD_1V8"; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1800000>; > + status = "okay"; > + }; > +}; > + > +&adc { > + vddana-supply = <&vdd_3v3>; > + vref-supply = <&vdd_3v3>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_adc_default &pinctrl_adtrg_default>; > + status = "okay"; > +}; > + > +&can0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_can0_default>; > + status = "okay"; > +}; > + > +&can1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_can1_default>; > + status = "okay"; > +}; > + > +&flx1 { > + atmel,flexcom-mode = <ATMEL_FLEXCOM_MODE_USART>; > + status = "okay"; > + > + uart6: serial@200 { > + pinctrl-0 = <&pinctrl_flx1_default>; > + pinctrl-names = "default"; > + atmel,use-dma-rx; > + atmel,use-dma-tx; > + status = "okay"; > + }; > +}; > + > +&flx4 { /* RPi spi */ > + atmel,flexcom-mode = <ATMEL_FLEXCOM_MODE_SPI>; > + status = "okay"; > + > + spi6: spi@400 { > + dmas = <0>, <0>; > + cs-gpios = <&pioA PIN_PD15 GPIO_ACTIVE_LOW>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_rpi_spi &pinctrl_rpi_spi_cs0>; > + status = "okay"; > + }; > + > + spi7: spi@400 { > + dmas = <0>, <0>; > + cs-gpios = <&pioA PIN_PD16 GPIO_ACTIVE_LOW>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_rpi_spi &pinctrl_rpi_spi_cs1>; > + status = "okay"; > + }; > +}; > + > +&i2c0 { > + dmas = <0>, <0>; > + pinctrl-names = "default", "gpio"; > + pinctrl-0 = <&pinctrl_i2c0_default>; > + pinctrl-1 = <&pinctrl_i2c0_gpio>; > + sda-gpios = <&pioA PIN_PB31 GPIO_ACTIVE_HIGH>; > + scl-gpios = <&pioA PIN_PC0 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>; > + i2c-sda-hold-time-ns = <350>; > + status = "okay"; > + > + mcp16502@5b { > + compatible = "microchip,mcp16502"; > + reg = <0x5b>; > + status = "okay"; > + lpm-gpios = <&pioBU 0 GPIO_ACTIVE_LOW>; > + > + regulators { > + vdd_3v3: VDD_IO { Dashes in node name, thus VDD-IO (same for the other entires). I will modify all nodes to use "-". > + regulator-name = "VDD_IO"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-initial-mode = <2>; > + regulator-allowed-modes = <2>, <4>; > + regulator-always-on; > + > + regulator-state-standby { > + regulator-on-in-suspend; > + regulator-mode = <4>; > + }; > + > + regulator-state-mem { > + regulator-off-in-suspend; > + regulator-mode = <4>; > + }; > + }; > + > + vddio_ddr: VDD_DDR { > + regulator-name = "VDD_DDR"; > + regulator-min-microvolt = <1200000>; > + regulator-max-microvolt = <1200000>; > + regulator-initial-mode = <2>; > + regulator-allowed-modes = <2>, <4>; > + regulator-always-on; > + > + regulator-state-standby { > + regulator-on-in-suspend; > + regulator-suspend-microvolt = <1200000>; > + regulator-changeable-in-suspend; > + regulator-mode = <4>; > + }; > + > + regulator-state-mem { > + regulator-on-in-suspend; > + regulator-suspend-microvolt = <1200000>; > + regulator-changeable-in-suspend; > + regulator-mode = <4>; > + }; > + }; > + > + vdd_core: VDD_CORE { > + regulator-name = "VDD_CORE"; > + regulator-min-microvolt = <1250000>; > + regulator-max-microvolt = <1250000>; > + regulator-initial-mode = <2>; > + regulator-allowed-modes = <2>, <4>; > + regulator-always-on; > + > + regulator-state-standby { > + regulator-on-in-suspend; > + regulator-mode = <4>; > + }; > + > + regulator-state-mem { > + regulator-off-in-suspend; > + regulator-mode = <4>; > + }; > + }; > + > + vdd_ddr: VDD_OTHER { > + regulator-name = "VDD_OTHER"; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1800000>; > + regulator-initial-mode = <2>; > + regulator-allowed-modes = <2>, <4>; > + regulator-always-on; > + > + regulator-state-standby { > + regulator-on-in-suspend; > + regulator-suspend-microvolt = <1800000>; > + regulator-changeable-in-suspend; > + regulator-mode = <4>; > + }; > + > + regulator-state-mem { > + regulator-on-in-suspend; > + regulator-suspend-microvolt = <1800000>; > + regulator-changeable-in-suspend; > + regulator-mode = <4>; > + }; > + }; > + > + LDO1 { > + regulator-name = "LDO1"; > + regulator-min-microvolt = <2500000>; > + regulator-max-microvolt = <2500000>; > + regulator-always-on; > + > + regulator-state-standby { > + regulator-on-in-suspend; > + }; > + > + regulator-state-mem { > + regulator-off-in-suspend; > + }; > + }; > + > + LDO2 { > + regulator-name = "LDO2"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-always-on; > + > + regulator-state-standby { > + regulator-on-in-suspend; > + }; > + > + regulator-state-mem { > + regulator-off-in-suspend; > + }; > + }; > + }; > + }; > +}; > + > +&i2c1 { > + dmas = <0>, <0>; > + pinctrl-names = "default", "gpio"; > + pinctrl-0 = <&pinctrl_i2c1_default>; > + i2c-analog-filter; > + i2c-digital-filter; > + i2c-digital-filter-width-ns = <35>; > + pinctrl-1 = <&pinctrl_i2c1_gpio>; > + sda-gpios = <&pioA PIN_PD4 GPIO_ACTIVE_HIGH>; > + scl-gpios = <&pioA PIN_PD5 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>; > + status = "okay"; > +}; > + > +&macb0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_macb0_default &pinctrl_macb0_phy_irq>; > + #address-cells = <1>; > + #size-cells = <0>; > + phy-mode = "rmii"; > + status = "disabled"; > +}; > + > +&pioA { > + pinctrl_adc_default: adc_default { Dashes in node name, thus adc-default (same for the other entries). Sure, I will modify. > + pinmux = <PIN_PD25__GPIO>, > + <PIN_PD26__GPIO>; > + bias-disable; > + }; > + > + pinctrl_adtrg_default: adtrg_default { > + pinmux = <PIN_PD31__ADTRG>; > + bias-pull-up; > + }; > + > + pinctrl_rpi_spi: rpi_spi { > + pinmux = <PIN_PD12__FLEXCOM4_IO0>, > + <PIN_PD13__FLEXCOM4_IO1>, > + <PIN_PD14__FLEXCOM4_IO2>; > + bias-disable; > + }; > + > + pinctrl_rpi_spi_cs0: rpi_spi_cs0 { > + pinmux = <PIN_PD15__FLEXCOM4_IO3>; > + bias-disable; > + }; > + > + pinctrl_rpi_spi_cs1: rpi_spi_cs1 { > + pinmux = <PIN_PD16__FLEXCOM4_IO4>; > + bias-disable; > + }; > + > + pinctrl_spi1_default: spi1_default { > + pinmux = <PIN_PC1__SPI1_SPCK>, > + <PIN_PC2__SPI1_MOSI>, > + <PIN_PC3__SPI1_MISO>; > + bias-disable; > + }; > + > + pinctrl_spi1_cs: spi1_cs { > + pinmux = <PIN_PC4__SPI1_NPCS0>, > + <PIN_PC5__SPI1_NPCS1>, > + <PIN_PC6__SPI1_NPCS2>, > + <PIN_PC7__SPI1_NPCS3>; > + bias-disable; > + }; > + > + pinctrl_rpi_uart: rpi_uart { > + pinmux = <PIN_PD2__URXD1>, > + <PIN_PD3__UTXD1>; > + bias-disable; > + }; > + > + pinctrl_debug_uart: debug_uart { > + pinmux = <PIN_PB26__URXD0>, > + <PIN_PB27__UTXD0>; > + bias-disable; > + }; > + > + pinctrl_can0_default: can0_default { > + pinmux = <PIN_PC10__CANTX0>, > + <PIN_PC11__CANRX0>; > + bias-disable; > + }; > + > + pinctrl_can1_default: can1_default { > + pinmux = <PIN_PC26__CANTX1>, > + <PIN_PC27__CANRX1>; > + bias-disable; > + }; > + > + pinctrl_i2c0_default: i2c0_default { > + pinmux = <PIN_PB31__TWD0>, > + <PIN_PC0__TWCK0>; > + bias-disable; > + }; > + > + pinctrl_i2c0_gpio: i2c0_gpio { > + pinmux = <PIN_PB31__GPIO>, > + <PIN_PC0__GPIO>; > + bias-disable; > + }; > + > + pinctrl_i2c1_default: i2c1_default { > + pinmux = <PIN_PD4__TWD1>, > + <PIN_PD5__TWCK1>; > + bias-disable; > + }; > + > + pinctrl_i2c1_gpio: i2c1_gpio { > + pinmux = <PIN_PD4__GPIO>, > + <PIN_PD5__GPIO>; > + bias-disable; > + }; > + > + pinctrl_key_gpio_default: key_gpio_default { > + pinmux = <PIN_PA17__GPIO>; > + bias-pull-up; > + }; > + > + pinctrl_led_gpio_default: led_gpio_default { > + pinmux = <PIN_PA7__GPIO>, > + <PIN_PA8__GPIO>, > + <PIN_PA9__GPIO>; > + bias-pull-up; > + }; > + > + pinctrl_qspi1_default: qspi1_default { > + pinmux = <PIN_PB5__QSPI1_SCK>, > + <PIN_PB6__QSPI1_CS>, > + <PIN_PB7__QSPI1_IO0>, > + <PIN_PB8__QSPI1_IO1>, > + <PIN_PB9__QSPI1_IO2>, > + <PIN_PB10__QSPI1_IO3>; > + bias-disable; > + }; > + > + pinctrl_sdmmc0_default: sdmmc0_default { > + pinmux = <PIN_PA0__SDMMC0_CK>, > + <PIN_PA1__SDMMC0_CMD>, > + <PIN_PA2__SDMMC0_DAT0>, > + <PIN_PA3__SDMMC0_DAT1>, > + <PIN_PA4__SDMMC0_DAT2>, > + <PIN_PA5__SDMMC0_DAT3>, > + <PIN_PA11__SDMMC0_VDDSEL>, > + <PIN_PA13__SDMMC0_CD>; > + bias-disable; > + }; > + > + pinctrl_sdmmc1_default: sdmmc1_default { > + pinmux = <PIN_PA18__SDMMC1_DAT0>, > + <PIN_PA19__SDMMC1_DAT1>, > + <PIN_PA20__SDMMC1_DAT2>, > + <PIN_PA21__SDMMC1_DAT3>, > + <PIN_PA22__SDMMC1_CK>, > + <PIN_PA28__SDMMC1_CMD>, > + <PIN_PA30__SDMMC1_CD>; > + bias-disable; > + }; > + > + pinctrl_mikrobus1_an: mikrobus1_an { > + pinmux = <PIN_PD25__GPIO>; > + bias-disable; > + }; > + > + pinctrl_mikrobus1_rst: mikrobus1_rst { > + pinmux = <PIN_PA16__GPIO>; > + bias-disable; > + }; > + > + pinctrl_mikrobus1_spi_cs: mikrobus1_spi_cs { > + pinmux = <PIN_PC4__SPI1_NPCS0>; > + bias-disable; > + }; > + > + pinctrl_mikrobus1_spi: mikrobus1_spi { > + pinmux = <PIN_PC1__SPI1_SPCK>, > + <PIN_PC2__SPI1_MOSI>, > + <PIN_PC3__SPI1_MISO>; > + bias-disable; > + }; > + > + pinctrl_mikrobus1_pwm: mikrobus1_pwm { > + pinmux = <PIN_PA31__PWML0>; > + bias-disable; > + }; > + > + pinctrl_mikrobus1_int: mikrobus1_int { > + pinmux = <PIN_PB2__GPIO>; > + bias-disable; > + }; > + > + pinctrl_mikrobus1_uart: mikrobus1_uart { > + pinmux = <PIN_PB3__URXD4>, > + <PIN_PB4__UTXD4>; > + bias-disable; > + }; > + > + pinctrl_mikrobus2_an: mikrobus2_an { > + pinmux = <PIN_PD26__GPIO>; > + bias-disable; > + }; > + > + pinctrl_mikrobus2_rst: mikrobus2_rst { > + pinmux = <PIN_PA16__GPIO>; > + bias-disable; > + }; > + > + pinctrl_mikrobus2_spi_cs: mikrobus2_spi_cs { > + pinmux = <PIN_PC5__SPI1_NPCS1>; > + bias-disable; > + }; > + > + pinctrl_mikrobus2_spi: mikrobus2_spi { > + pinmux = <PIN_PC1__SPI1_SPCK>, > + <PIN_PC2__SPI1_MOSI>, > + <PIN_PC3__SPI1_MISO>; > + bias-disable; > + }; > + > + pinctrl_mikrobus2_pwm: mikrobus2_pwm { > + pinmux = <PIN_PB0__PWMH1>; > + bias-disable; > + }; > + > + pinctrl_mikrobus2_int: mikrobus2_int { > + pinmux = <PIN_PB1__GPIO>; > + bias-disable; > + }; > + > + pinctrl_mikrobus2_uart: mikrobus2_uart { > + pinmux = <PIN_PB11__URXD3>, > + <PIN_PB12__UTXD3>; > + bias-disable; > + }; > + > + pinctrl_usb_default: usb_default { > + pinmux = <PIN_PA6__GPIO>; > + bias-disable; > + }; > + > + pinctrl_usba_vbus: usba_vbus { > + pinmux = <PIN_PB13__GPIO>; > + bias-disable; > + }; > + > + pinctrl_macb0_default: macb0_default { > + pinmux = <PIN_PB14__GTXCK>, > + <PIN_PB15__GTXEN>, > + <PIN_PB16__GRXDV>, > + <PIN_PB17__GRXER>, > + <PIN_PB18__GRX0>, > + <PIN_PB19__GRX1>, > + <PIN_PB20__GTX0>, > + <PIN_PB21__GTX1>, > + <PIN_PB22__GMDC>, > + <PIN_PB23__GMDIO>; > + bias-disable; > + }; > + > + pinctrl_macb0_phy_irq: macb0_phy_irq { > + pinmux = <PIN_PB24__GPIO>; > + bias-disable; > + }; > + > + pinctrl_flx1_default: flx1_default { > + pinmux = <PIN_PA24__FLEXCOM1_IO0>, > + <PIN_PA23__FLEXCOM1_IO1>, > + <PIN_PA25__FLEXCOM1_IO3>, > + <PIN_PA26__FLEXCOM1_IO4>; > + bias-disable; > + }; > +}; > + > +&pwm0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_mikrobus1_pwm &pinctrl_mikrobus2_pwm>; > + status = "okay"; > +}; > + > +&qspi1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_qspi1_default>; > + status = "okay"; > + > + flash@0 { > + #address-cells = <1>; > + #size-cells = <1>; > + compatible = "jedec,spi-nor"; > + reg = <0>; > + spi-max-frequency = <80000000>; > + spi-tx-bus-width = <4>; > + spi-rx-bus-width = <4>; > + m25p,fast-read; > + label = "atmel_qspi1"; > + status = "okay"; > + > + at91bootstrap@0 { > + label = "at91bootstrap"; > + reg = <0x0 0x40000>; > + }; > + > + bootloader@40000 { > + label = "bootloader"; > + reg = <0x40000 0xc0000>; > + }; > + > + bootloaderenvred@100000 { > + label = "bootloader env redundant"; > + reg = <0x100000 0x40000>; > + }; > + > + bootloaderenv@140000 { > + label = "bootloader env"; > + reg = <0x140000 0x40000>; > + }; > + > + dtb@180000 { > + label = "device tree"; > + reg = <0x180000 0x80000>; > + }; > + > + kernel@200000 { > + label = "kernel"; > + reg = <0x200000 0x600000>; > + }; > + }; > +}; > + > +&sdmmc0 { > + bus-width = <4>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_sdmmc0_default>; > + vmmc-supply = <&vdd_3v3>; > + vqmmc-supply = <&vdd_1v8>; > + status = "okay"; > +}; > + > +&sdmmc1 { > + bus-width = <4>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_sdmmc1_default>; > + vqmmc-supply = <&vdd_3v3>; > + vmmc-supply = <&vdd_3v3>; > + status = "okay"; > +}; > + > +&shutdown_controller { > + debounce-delay-us = <976>; > + atmel,wakeup-rtc-timer; > + > + input@0 { > + reg = <0>; > + }; > +}; > + > +&spi1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_spi1_default &pinctrl_spi1_cs>; > + status = "okay"; > +}; > + > +&tcb0 { > + timer0: timer@0 { > + compatible = "atmel,tcb-timer"; > + reg = <0>; > + }; > + > + timer1: timer@1 { > + compatible = "atmel,tcb-timer"; > + reg = <1>; > + }; > +}; > + > +&uart0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_debug_uart>; > + atmel,use-dma-rx; > + atmel,use-dma-tx; > + status = "okay"; > +}; > + > +&uart1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_rpi_uart>; > + atmel,use-dma-rx; > + atmel,use-dma-tx; > + status = "okay"; > +}; > + > +&uart3 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_mikrobus2_uart>; > + atmel,use-dma-rx; > + atmel,use-dma-tx; > + status = "okay"; > +}; > + > +&uart4 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_mikrobus1_uart>; > + atmel,use-dma-rx; > + atmel,use-dma-tx; > + status = "okay"; > +}; > + > +&usb0 { > + atmel,vbus-gpio = <&pioA PIN_PB13 GPIO_ACTIVE_HIGH>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_usba_vbus>; > + status = "okay"; > +}; > + > +&usb1 { > + num-ports = <3>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_usb_default>; > + status = "okay"; > +}; > + > +&watchdog { > + status = "okay"; > +}; ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/2] dt-bindings: ARM: at91: Document Microchip SAMA5D29 Curiosity 2023-08-01 11:11 [PATCH 1/2] dt-bindings: ARM: at91: Document Microchip SAMA5D29 Curiosity Mihai Sain 2023-08-01 11:11 ` [PATCH 2/2] ARM: dts: at91: sama5d29_curiosity: Add device tree for sama5d29_curiosity board Mihai Sain @ 2023-08-01 15:35 ` Conor Dooley 2023-08-02 12:42 ` Mihai.Sain 2023-09-11 15:03 ` Nicolas Ferre 1 sibling, 2 replies; 14+ messages in thread From: Conor Dooley @ 2023-08-01 15:35 UTC (permalink / raw) To: Mihai Sain Cc: robh+dt, krzysztof.kozlowski+dt, conor+dt, nicolas.ferre, alexandre.belloni, claudiu.beznea, andre.przywara, andrei.simion, jerry.ray, devicetree, linux-arm-kernel, linux-kernel, cristian.birsan [-- Attachment #1: Type: text/plain, Size: 1360 bytes --] Hey, On Tue, Aug 01, 2023 at 02:11:50PM +0300, Mihai Sain wrote: > From: Andrei Simion <andrei.simion@microchip.com> > > Document device tree binding of SAMA5D29 Curiosity, from Microchip. > > Reviewed-by: Mihai Sain <mihai.sain@microchip.com> > Signed-off-by: Andrei Simion <andrei.simion@microchip.com> Your SoB is missing from here. > --- > Documentation/devicetree/bindings/arm/atmel-at91.yaml | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/Documentation/devicetree/bindings/arm/atmel-at91.yaml b/Documentation/devicetree/bindings/arm/atmel-at91.yaml > index dfb8fd089197..89d75fbb1de4 100644 > --- a/Documentation/devicetree/bindings/arm/atmel-at91.yaml > +++ b/Documentation/devicetree/bindings/arm/atmel-at91.yaml > @@ -79,6 +79,13 @@ properties: > - const: atmel,sama5d2 > - const: atmel,sama5 > > + - description: Microchip SAMA5D29 Curiosity > + items: > + - const: microchip,sama5d29-curiosity > + - const: atmel,sama5d29 > + - const: atmel,sama5d2 > + - const: atmel,sama5 What is the benefit of adding the two familial compatibles? Is there software that actually uses these? Thanks, Conor. > + > - items: > - const: atmel,sama5d27 > - const: atmel,sama5d2 > -- > 2.41.0 > [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] ^ permalink raw reply [flat|nested] 14+ messages in thread
* RE: [PATCH 1/2] dt-bindings: ARM: at91: Document Microchip SAMA5D29 Curiosity 2023-08-01 15:35 ` [PATCH 1/2] dt-bindings: ARM: at91: Document Microchip SAMA5D29 Curiosity Conor Dooley @ 2023-08-02 12:42 ` Mihai.Sain 2023-08-02 15:39 ` Conor Dooley 2023-09-11 15:03 ` Nicolas Ferre 1 sibling, 1 reply; 14+ messages in thread From: Mihai.Sain @ 2023-08-02 12:42 UTC (permalink / raw) To: conor Cc: robh+dt, krzysztof.kozlowski+dt, conor+dt, Nicolas.Ferre, alexandre.belloni, Claudiu.Beznea, andre.przywara, Andrei.Simion, Jerry.Ray, devicetree, linux-arm-kernel, linux-kernel, Cristian.Birsan Hi Conor, Please find bellow my answers. Hey, On Tue, Aug 01, 2023 at 02:11:50PM +0300, Mihai Sain wrote: > From: Andrei Simion <andrei.simion@microchip.com> > > Document device tree binding of SAMA5D29 Curiosity, from Microchip. > > Reviewed-by: Mihai Sain <mihai.sain@microchip.com> > Signed-off-by: Andrei Simion <andrei.simion@microchip.com> Your SoB is missing from here. Sure, I will add my SoB. > --- > Documentation/devicetree/bindings/arm/atmel-at91.yaml | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/Documentation/devicetree/bindings/arm/atmel-at91.yaml > b/Documentation/devicetree/bindings/arm/atmel-at91.yaml > index dfb8fd089197..89d75fbb1de4 100644 > --- a/Documentation/devicetree/bindings/arm/atmel-at91.yaml > +++ b/Documentation/devicetree/bindings/arm/atmel-at91.yaml > @@ -79,6 +79,13 @@ properties: > - const: atmel,sama5d2 > - const: atmel,sama5 > > + - description: Microchip SAMA5D29 Curiosity > + items: > + - const: microchip,sama5d29-curiosity > + - const: atmel,sama5d29 > + - const: atmel,sama5d2 > + - const: atmel,sama5 What is the benefit of adding the two familial compatibles? Is there software that actually uses these? Thanks, Conor. Currently there is no software. In this file we've just documented the previous boards from sam9, sama5, sama7 MPUs. Thanks, Mihai > + > - items: > - const: atmel,sama5d27 > - const: atmel,sama5d2 > -- > 2.41.0 > ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/2] dt-bindings: ARM: at91: Document Microchip SAMA5D29 Curiosity 2023-08-02 12:42 ` Mihai.Sain @ 2023-08-02 15:39 ` Conor Dooley 0 siblings, 0 replies; 14+ messages in thread From: Conor Dooley @ 2023-08-02 15:39 UTC (permalink / raw) To: Mihai.Sain Cc: robh+dt, krzysztof.kozlowski+dt, conor+dt, Nicolas.Ferre, alexandre.belloni, Claudiu.Beznea, andre.przywara, Andrei.Simion, Jerry.Ray, devicetree, linux-arm-kernel, linux-kernel, Cristian.Birsan [-- Attachment #1: Type: text/plain, Size: 1317 bytes --] On Wed, Aug 02, 2023 at 12:42:52PM +0000, Mihai.Sain@microchip.com wrote: > On Tue, Aug 01, 2023 at 02:11:50PM +0300, Mihai Sain wrote: > > --- > > Documentation/devicetree/bindings/arm/atmel-at91.yaml | 7 +++++++ > > 1 file changed, 7 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/arm/atmel-at91.yaml > > b/Documentation/devicetree/bindings/arm/atmel-at91.yaml > > index dfb8fd089197..89d75fbb1de4 100644 > > --- a/Documentation/devicetree/bindings/arm/atmel-at91.yaml > > +++ b/Documentation/devicetree/bindings/arm/atmel-at91.yaml > > @@ -79,6 +79,13 @@ properties: > > - const: atmel,sama5d2 > > - const: atmel,sama5 > > > > + - description: Microchip SAMA5D29 Curiosity > > + items: > > + - const: microchip,sama5d29-curiosity > > + - const: atmel,sama5d29 > > + - const: atmel,sama5d2 > > + - const: atmel,sama5 > > What is the benefit of adding the two familial compatibles? Is there software that actually uses these? > > Thanks, > Conor. > > Currently there is no software. > In this file we've just documented the previous boards from sam9, sama5, sama7 MPUs. Please fix your quoting. If no software cares about this, what is the benefit of adding familial compatibles? [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/2] dt-bindings: ARM: at91: Document Microchip SAMA5D29 Curiosity 2023-08-01 15:35 ` [PATCH 1/2] dt-bindings: ARM: at91: Document Microchip SAMA5D29 Curiosity Conor Dooley 2023-08-02 12:42 ` Mihai.Sain @ 2023-09-11 15:03 ` Nicolas Ferre 2023-09-11 15:57 ` Conor Dooley 1 sibling, 1 reply; 14+ messages in thread From: Nicolas Ferre @ 2023-09-11 15:03 UTC (permalink / raw) To: Conor Dooley, Mihai Sain Cc: robh+dt, krzysztof.kozlowski+dt, conor+dt, alexandre.belloni, claudiu.beznea, andre.przywara, andrei.simion, jerry.ray, devicetree, linux-arm-kernel, linux-kernel, cristian.birsan On 01/08/2023 at 17:35, Conor Dooley wrote: >> diff --git a/Documentation/devicetree/bindings/arm/atmel-at91.yaml b/Documentation/devicetree/bindings/arm/atmel-at91.yaml >> index dfb8fd089197..89d75fbb1de4 100644 >> --- a/Documentation/devicetree/bindings/arm/atmel-at91.yaml >> +++ b/Documentation/devicetree/bindings/arm/atmel-at91.yaml >> @@ -79,6 +79,13 @@ properties: >> - const: atmel,sama5d2 >> - const: atmel,sama5 >> >> + - description: Microchip SAMA5D29 Curiosity >> + items: >> + - const: microchip,sama5d29-curiosity >> + - const: atmel,sama5d29 >> + - const: atmel,sama5d2 >> + - const: atmel,sama5 > What is the benefit of adding the two familial compatibles? Is there > software that actually uses these? "atmel,sama5d2" is used by arch/arm/mach-at91/sama5.c file "atmel,sama5" is used by drivers/soc/atmel/soc.c file, so yes they are needed to instantiate a sama5d29 SoC. Best regards, Nicolas ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/2] dt-bindings: ARM: at91: Document Microchip SAMA5D29 Curiosity 2023-09-11 15:03 ` Nicolas Ferre @ 2023-09-11 15:57 ` Conor Dooley 0 siblings, 0 replies; 14+ messages in thread From: Conor Dooley @ 2023-09-11 15:57 UTC (permalink / raw) To: Nicolas Ferre Cc: Mihai Sain, robh+dt, krzysztof.kozlowski+dt, conor+dt, alexandre.belloni, claudiu.beznea, andre.przywara, andrei.simion, jerry.ray, devicetree, linux-arm-kernel, linux-kernel, cristian.birsan [-- Attachment #1: Type: text/plain, Size: 1177 bytes --] On Mon, Sep 11, 2023 at 05:03:07PM +0200, Nicolas Ferre wrote: > On 01/08/2023 at 17:35, Conor Dooley wrote: > > > diff --git a/Documentation/devicetree/bindings/arm/atmel-at91.yaml b/Documentation/devicetree/bindings/arm/atmel-at91.yaml > > > index dfb8fd089197..89d75fbb1de4 100644 > > > --- a/Documentation/devicetree/bindings/arm/atmel-at91.yaml > > > +++ b/Documentation/devicetree/bindings/arm/atmel-at91.yaml > > > @@ -79,6 +79,13 @@ properties: > > > - const: atmel,sama5d2 > > > - const: atmel,sama5 > > > + - description: Microchip SAMA5D29 Curiosity > > > + items: > > > + - const: microchip,sama5d29-curiosity > > > + - const: atmel,sama5d29 > > > + - const: atmel,sama5d2 > > > + - const: atmel,sama5 > > What is the benefit of adding the two familial compatibles? Is there > > software that actually uses these? > > "atmel,sama5d2" is used by arch/arm/mach-at91/sama5.c file > "atmel,sama5" is used by drivers/soc/atmel/soc.c file, so yes they are > needed to instantiate a sama5d29 SoC. Okay. Acked-by: Conor Dooley <conor.dooley@microchip.com> Thanks, Conor. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] ^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2023-09-12 11:01 UTC | newest] Thread overview: 14+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2023-08-01 11:11 [PATCH 1/2] dt-bindings: ARM: at91: Document Microchip SAMA5D29 Curiosity Mihai Sain 2023-08-01 11:11 ` [PATCH 2/2] ARM: dts: at91: sama5d29_curiosity: Add device tree for sama5d29_curiosity board Mihai Sain 2023-08-02 4:40 ` claudiu beznea 2023-08-02 12:19 ` Mihai.Sain 2023-08-03 9:09 ` claudiu beznea 2023-09-11 16:01 ` Nicolas Ferre 2023-09-12 11:01 ` claudiu beznea 2023-08-02 5:02 ` claudiu beznea 2023-08-02 11:59 ` Mihai.Sain 2023-08-01 15:35 ` [PATCH 1/2] dt-bindings: ARM: at91: Document Microchip SAMA5D29 Curiosity Conor Dooley 2023-08-02 12:42 ` Mihai.Sain 2023-08-02 15:39 ` Conor Dooley 2023-09-11 15:03 ` Nicolas Ferre 2023-09-11 15:57 ` Conor Dooley
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).