Re: [PATCH 5/5] ARM: dts: sama5d27_som1_ek: Add sama5d27 SoM1 EK support

From: m18063
Date: Fri Jun 30 2017 - 08:33:32 EST


Hi Ludovic,

On 29.06.2017 18:04, Ludovic Desroches wrote:
> On Thu, Jun 29, 2017 at 02:44:34PM +0300, Claudiu Beznea wrote:
>> Add specifig DTS files and bindings for sama5d27 SoM1 EK board.
>>
>> Signed-off-by: Claudiu Beznea <claudiu.beznea@xxxxxxxxxxxxx>
>> Signed-off-by: Cristian Birsan <cristian.birsan@xxxxxxxxxxxxx>
>> ---
>> arch/arm/boot/dts/Makefile | 1 +
>> arch/arm/boot/dts/at91-sama5d27_som1_ek.dts | 551 ++++++++++++++++++++++++++++
>> 2 files changed, 552 insertions(+)
>> create mode 100644 arch/arm/boot/dts/at91-sama5d27_som1_ek.dts
>>
>> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
>> index 9449493..3396f73 100644
>> --- a/arch/arm/boot/dts/Makefile
>> +++ b/arch/arm/boot/dts/Makefile
>> @@ -46,6 +46,7 @@ dtb-$(CONFIG_SOC_AT91SAM9) += \
>> at91sam9x35ek.dtb
>> dtb-$(CONFIG_SOC_SAM_V7) += \
>> at91-kizbox2.dtb \
>> + at91-sama5d27_som1_ek.dtb \
>> at91-sama5d2_xplained.dtb \
>> at91-sama5d3_xplained.dtb \
>> at91-tse850-3.dtb \
>> diff --git a/arch/arm/boot/dts/at91-sama5d27_som1_ek.dts b/arch/arm/boot/dts/at91-sama5d27_som1_ek.dts
>> new file mode 100644
>> index 0000000..a87125e
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/at91-sama5d27_som1_ek.dts
>> @@ -0,0 +1,551 @@
>> +/*
>> + * at91-sama5d27_som1_ek.dts - Device Tree file for SAMA5D27-SOM1-EK board
>> + *
>> + * Copyright (c) 2017, Microchip Technology Inc.
>> + * 2016 Nicolas Ferre <nicolas.ferre@xxxxxxxxx>
>> + * 2017 Cristian Birsan <cristian.birsan@xxxxxxxxxxxxx>
>> + * 2017 Claudiu Beznea <claudiu.beznea@xxxxxxxxxxxxx>
>> + *
>> + * This file is dual-licensed: you can use it either under the terms
>> + * of the GPL or the X11 license, at your option. Note that this dual
>> + * licensing only applies to this file, and not this project as a
>> + * whole.
>> + *
>> + * a) This file is free software; you can redistribute it and/or
>> + * modify it under the terms of the GNU General Public License as
>> + * published by the Free Software Foundation; either version 2 of the
>> + * License, or (at your option) any later version.
>> + *
>> + * This file is distributed in the hope that it will be useful,
>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
>> + * GNU General Public License for more details.
>> + *
>> + * Or, alternatively,
>> + *
>> + * b) Permission is hereby granted, free of charge, to any person
>> + * obtaining a copy of this software and associated documentation
>> + * files (the "Software"), to deal in the Software without
>> + * restriction, including without limitation the rights to use,
>> + * copy, modify, merge, publish, distribute, sublicense, and/or
>> + * sell copies of the Software, and to permit persons to whom the
>> + * Software is furnished to do so, subject to the following
>> + * conditions:
>> + *
>> + * The above copyright notice and this permission notice shall be
>> + * included in all copies or substantial portions of the Software.
>> + *
>> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
>> + * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
>> + * OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
>> + * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
>> + * HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
>> + * WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
>> + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
>> + * OTHER DEALINGS IN THE SOFTWARE.
>> + */
>> +/dts-v1/;
>> +#include "at91-sama5d27_som1.dtsi"
>> +#include <dt-bindings/mfd/atmel-flexcom.h>
>> +#include <dt-bindings/gpio/gpio.h>
>> +
>> +/ {
>> + model = "Atmel SAMA5D27 SOM1 EK";
>> + compatible = "atmel,sama5d27-som1-ek", "atmel,sama5d27-som1", "atmel,sama5d27", "atmel,sama5d2", "atmel,sama5";
>> +
>> + chosen {
>> + stdout-path = "serial0:115200n8";
>> + };
>> +
>> + memory {
>> + reg = <0x20000000 0x20000>;
>> + };
>> +
>> + ahb {
>> + usb0: gadget@00300000 {
>> + atmel,vbus-gpio = <&pioA 116 GPIO_ACTIVE_HIGH>;
>
> Even if it is not done in other dts file (I plan to send patches to fix
> it), prefer macros to avoid errors: PIN_PD20.
Indeed, it is better to use directly the pin macros. I will do the necessary
changes in v2.

Thank you,
Claudiu
>
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_usba_vbus>;
>> + status = "okay";
>> + };
>> +
>> + usb1: ohci@00400000 {
>> + num-ports = <3>;
>> + atmel,vbus-gpio = <&pioA 10 GPIO_ACTIVE_HIGH>;
>
> Ditto
Ditto
>
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_usb_default>;
>> + status = "okay";
>> + };
>> +
>> + usb2: ehci@00500000 {
>> + status = "okay";
>> + };
>> +
>> + sdmmc0: sdio-host@a0000000 {
>> + bus-width = <8>;
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_sdmmc0_default>;
>> + status = "okay";
>> + };
>> +
>> + sdmmc1: sdio-host@b0000000 {
>> + bus-width = <4>;
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_sdmmc1_default>;
>> + status = "okay";
>> + };
>> +
>> + apb {
>> + isc: isc@f0008000 {
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_isc_base &pinctrl_isc_data_8bit &pinctrl_isc_data_9_10 &pinctrl_isc_data_11_12>;
>> + status = "okay";
>> + };
>> +
>> + spi0: spi@f8000000 {
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_spi0_default>;
>> + status = "okay";
>> + };
>> +
>> + macb0: ethernet@f8008000 {
>> + status = "okay";
>> + };
>> +
>> + uart1: serial@f8020000 {
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_uart1_default>;
>> + atmel,use-dma-rx;
>> + atmel,use-dma-tx;
>> + status = "okay";
>> + };
>> +
>> + uart2: serial@f8024000 {
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_mikrobus2_uart>;
>> + atmel,use-dma-rx;
>> + atmel-use-dma-tx;
>> + status = "okay";
>> + };
>> +
>> + pwm0: pwm@f802c000 {
>> + status = "okay";
>> + };
>> +
>> + flx1: flexcom@f8038000 {
>> + atmel,flexcom-mode = <ATMEL_FLEXCOM_MODE_TWI>;
>> + status = "disabled";
>> +
>> + i2c2: i2c@600 {
>> + compatible = "atmel,sama5d2-i2c";
>> + reg = <0x600 0x200>;
>> + interrupts = <20 IRQ_TYPE_LEVEL_HIGH 7>;
>> + dmas = <0>, <0>;
>> + dma-names = "tx", "rx";
>> + #address-cells = <1>;
>> + #size-cells = <0>;
>> + clocks = <&flx1_clk>;
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_mikrobus_i2c>;
>> + atmel,fifo-size = <16>;
>> + status = "disabled";
>> + };
>> + };
>> +
>> + shdwc@f8048010 {
>> + atmel,shdwc-debouncer = <976>;
>> + atmel,wakeup-rtc-timer;
>> +
>> + input@0 {
>> + reg = <0>;
>> + atmel,wakeup-type = "low";
>> + };
>> + };
>> +
>> + watchdog@f8048040 {
>> + status = "okay";
>> + };
>> +
>> + can0: can@f8054000 {
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_can0_default>;
>> + };
>> +
>> + uart3: serial@fc008000 {
>> + atmel,use-dma-rx;
>> + atmel,use-dma-tx;
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_uart3_default>;
>> + status = "disabled";
>> + };
>> +
>> + uart4: serial@fc00c000 {
>> + atmel,use-dma-rx;
>> + atmel,use-dma-tx;
>> + pinctrl-name = "default";
>> + pinctrl-0 = <&pinctrl_mikrobus1_uart>;
>> + status = "okay";
>> + };
>> +
>> + flx3: flexcom@fc014000 {
>> + atmel,flexcom-mode = <ATMEL_FLEXCOM_MODE_SPI>;
>> + status = "disabled";
>> +
>> + uart7: serial@200 {
>> + compatible = "atmel,at91sam9260-usart";
>> + reg = <0x200 0x200>;
>> + interrupts = <22 IRQ_TYPE_LEVEL_HIGH 7>;
>> + clocks = <&flx3_clk>;
>> + clock-names = "usart";
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_flx3_default>;
>> + atmel,fifo-size = <32>;
>> + status = "disabled";
>> + };
>> +
>> + spi2: spi@400 {
>> + compatible = "atmel,at91rm9200-spi";
>> + reg = <0x400 0x200>;
>> + interrupts = <22 IRQ_TYPE_LEVEL_HIGH 7>;
>> + clocks = <&flx3_clk>;
>> + clock-names = "spi_clk";
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_flx3_default>;
>> + atmel,fifo-size = <16>;
>> + status = "disabled";
>> + };
>> + };
>> +
>> + flx4: flexcom@fc018000 {
>> + atmel,flexcom-mode = <ATMEL_FLEXCOM_MODE_SPI>;
>> + status = "okay";
>> +
>> + uart6: serial@200 {
>> + compatible = "atmel,at91sam9260-usart";
>> + reg = <0x200 0x200>;
>> + interrupts = <23 IRQ_TYPE_LEVEL_HIGH 7>;
>> + clocks = <&flx4_clk>;
>> + clock-names = "usart";
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_flx4_default>;
>> + atmel,fifo-size = <32>;
>> + status = "disabled";
>> + };
>> +
>> + spi3: spi@400 {
>> + compatible = "atmel,at91rm9200-spi";
>> + reg = <0x400 0x200>;
>> + interrupts = <23 IRQ_TYPE_LEVEL_HIGH 7>;
>> + clocks = <&flx4_clk>;
>> + clock-names = "spi_clk";
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_mikrobus_spi &pinctrl_mikrobus1_spi_cs &pinctrl_mikrobus2_spi_cs>;
>> + atmel,fifo-size = <16>;
>> + status = "okay";
>> + };
>> +
>> + i2c3: i2c@600 {
>> + compatible = "atmel,sama5d2-i2c";
>> + reg = <0x600 0x200>;
>> + interrupts = <23 IRQ_TYPE_LEVEL_HIGH 7>;
>> + dmas = <0>, <0>;
>> + dma-names = "tx", "rx";
>> + #address-cells = <1>;
>> + #size-cells = <0>;
>> + clocks = <&flx4_clk>;
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_flx4_default>;
>> + atmel,fifo-size = <16>;
>> + status = "disabled";
>> + };
>> + };
>> +
>> + i2c1: i2c@fc028000 {
>> + dmas = <0>, <0>;
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_i2c1_default>;
>> + status = "okay";
>> + };
>> +
>> + pinctrl@fc038000 {
>> +
>> + 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_flx3_default: flx3_default {
>> + pinmux = <PIN_PC20__FLEXCOM3_IO0>,
>> + <PIN_PC19__FLEXCOM3_IO1>,
>> + <PIN_PC18__FLEXCOM3_IO2>,
>> + <PIN_PC21__FLEXCOM3_IO3>,
>> + <PIN_PC22__FLEXCOM3_IO4>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_i2c1_default: i2c1_default {
>> + pinmux = <PIN_PD4__TWD1>,
>> + <PIN_PD5__TWCK1>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_isc_base: isc_base {
>> + pinmux = <PIN_PC21__ISC_PCK>,
>> + <PIN_PC22__ISC_VSYNC>,
>> + <PIN_PC23__ISC_HSYNC>,
>> + <PIN_PC24__ISC_MCK>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_isc_data_8bit: isc_data_8bit {
>> + pinmux = <PIN_PC20__ISC_D11>,
>> + <PIN_PC19__ISC_D10>,
>> + <PIN_PC18__ISC_D9>,
>> + <PIN_PC17__ISC_D8>,
>> + <PIN_PC16__ISC_D7>,
>> + <PIN_PC15__ISC_D6>,
>> + <PIN_PC14__ISC_D5>,
>> + <PIN_PC13__ISC_D4>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_isc_data_9_10: isc_data_9_10 {
>> + pinmux = <PIN_PC12__ISC_D3>,
>> + <PIN_PC11__ISC_D2>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_isc_data_11_12: isc_data_11_12 {
>> + pinmux = <PIN_PC10__ISC_D1>,
>> + <PIN_PC9__ISC_D0>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_key_gpio_default: key_gpio_default {
>> + pinmux = <PIN_PA29__GPIO>;
>> + bias-pull-up;
>> + };
>> +
>> + pinctrl_led_gpio_default: led_gpio_default {
>> + pinmux = <PIN_PA27__GPIO>,
>> + <PIN_PB1__GPIO>,
>> + <PIN_PA31__GPIO>;
>> + bias-pull-up;
>> + };
>> +
>> + pinctrl_sdmmc0_default: sdmmc0_default {
>> + cmd_data {
>> + pinmux = <PIN_PA1__SDMMC0_CMD>,
>> + <PIN_PA2__SDMMC0_DAT0>,
>> + <PIN_PA3__SDMMC0_DAT1>,
>> + <PIN_PA4__SDMMC0_DAT2>,
>> + <PIN_PA5__SDMMC0_DAT3>,
>> + <PIN_PA6__SDMMC0_DAT4>,
>> + <PIN_PA7__SDMMC0_DAT5>,
>> + <PIN_PA8__SDMMC0_DAT6>,
>> + <PIN_PA9__SDMMC0_DAT7>;
>> + bias-pull-up;
>> + };
>> +
>> + ck_cd_vddsel {
>> + pinmux = <PIN_PA0__SDMMC0_CK>,
>> + <PIN_PA11__SDMMC0_VDDSEL>,
>> + <PIN_PA13__SDMMC0_CD>;
>> + bias-disable;
>> + };
>> + };
>> +
>> + pinctrl_sdmmc1_default: sdmmc1_default {
>> + cmd_data {
>> + pinmux = <PIN_PA28__SDMMC1_CMD>,
>> + <PIN_PA18__SDMMC1_DAT0>,
>> + <PIN_PA19__SDMMC1_DAT1>,
>> + <PIN_PA20__SDMMC1_DAT2>,
>> + <PIN_PA21__SDMMC1_DAT3>;
>> + bias-pull-up;
>> + };
>> +
>> + conf-ck_cd {
>> + pinmux = <PIN_PA22__SDMMC1_CK>,
>> + <PIN_PA30__SDMMC1_CD>;
>> + bias-disable;
>> + };
>> + };
>> +
>> + pinctrl_sensor_power: sensor_power {
>> + pinmux = <PIN_PB12__GPIO>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_sensor_reset: sensor_reset {
>> + pinmux = <PIN_PB11__GPIO>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_spi0_default: spi0_default {
>> + pinmux = <PIN_PA14__SPI0_SPCK>,
>> + <PIN_PA15__SPI0_MOSI>,
>> + <PIN_PA16__SPI0_MISO>,
>> + <PIN_PA17__SPI0_NPCS0>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_uart1_default: uart1_default {
>> + pinmux = <PIN_PD2__URXD1>,
>> + <PIN_PD3__UTXD1>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_uart3_default: uart3_default {
>> + pinmux = <PIN_PC12__URXD3>,
>> + <PIN_PC13__UTXD3>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_usb_default: usb_default {
>> + pinmux = <PIN_PA10__GPIO>,
>> + <PIN_PD19__GPIO>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_usba_vbus: usba_vbus {
>> + pinmux = <PIN_PD20__GPIO>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_mikrobus1_an: mikrobus1_an {
>> + pinmux = <PIN_PD25__GPIO>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_mikrobus2_an: mikrobus2_an {
>> + pinmux = <PIN_PD26__GPIO>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_mikrobus1_rst: mikrobus1_rst {
>> + pinmux = <PIN_PB2__GPIO>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_mikrobus2_rst: mikrobus2_rst {
>> + pinmux = <PIN_PA26__GPIO>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_mikrobus1_spi_cs: mikrobus1_spi_cs {
>> + pinmux = <PIN_PD0__FLEXCOM4_IO4>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_mikrobus2_spi_cs: mikrobus2_spi_cs {
>> + pinmux = <PIN_PC31__FLEXCOM4_IO3>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_mikrobus_spi: mikrobus_spi {
>> + pinmux = <PIN_PC28__FLEXCOM4_IO0>,
>> + <PIN_PC29__FLEXCOM4_IO1>,
>> + <PIN_PC30__FLEXCOM4_IO2>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_mikrobus1_pwm: mikrobus1_pwm {
>> + pinmux = <PIN_PB1__PWML1>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_mikrobus2_pwm: mikrobus2_pwm {
>> + pinmux = <PIN_PA31__PWML0>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_mikrobus1_int: mikrobus1_int {
>> + pinmux = <PIN_PB0__GPIO>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_mikrobus2_int: mikrobus2_int {
>> + pinmux = <PIN_PA25__GPIO>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_mikrobus1_uart: mikrobus1_uart {
>> + pinmux = <PIN_PB3__URXD4>,
>> + <PIN_PB4__UTXD4>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_mikrobus2_uart: mikrobus2_uart {
>> + pinmux = <PIN_PD23__URXD2>,
>> + <PIN_PD24__UTXD2>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_mikrobus_i2c: mikrobus1_i2c {
>> + pinmux = <PIN_PA24__FLEXCOM1_IO0>,
>> + <PIN_PA23__FLEXCOM1_IO1>;
>> + bias-disable;
>> + };
>> +
>> + pinctrl_flx4_default: flx4_uart_default {
>> + pinmux = <PIN_PC28__FLEXCOM4_IO0>,
>> + <PIN_PC29__FLEXCOM4_IO1>,
>> + <PIN_PC30__FLEXCOM4_IO2>,
>> + <PIN_PC31__FLEXCOM4_IO3>,
>> + <PIN_PD0__FLEXCOM4_IO4>;
>> + bias-disable;
>> + };
>> + };
>> +
>> + can1: can@fc050000 {
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_can1_default>;
>> + status = "okay";
>> + };
>> + };
>> + };
>> +
>> + gpio_keys {
>> + compatible = "gpio-keys";
>> +
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_key_gpio_default>;
>> +
>> + bp1 {
>> + label = "USER";
>> + gpios = <&pioA 29 GPIO_ACTIVE_LOW>;
>
> Ditto
Ditto
>
>> + linux,code = <0x104>;
>> + };
>> + };
>> +
>> + leds {
>> + compatible = "gpio-leds";
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_led_gpio_default>;
>> + status = "okay";
>> +
>> + red {
>> + label = "red";
>> + gpios = <&pioA 27 GPIO_ACTIVE_LOW>;
>
> Ditto
Ditto
>
>> + };
>> +
>> + green {
>> + label = "green";
>> + gpios = <&pioA 33 GPIO_ACTIVE_LOW>;
>
> Ditto
Ditto
>
>> + };
>> +
>> + blue {
>> + label = "blue";
>> + gpios = <&pioA 31 GPIO_ACTIVE_LOW>;
>
> Ditto
Ditto
>
> Regards
>
> Ludovic
>
>> + linux,default-trigger = "heartbeat";
>> + };
>> + };
>> +};
>> --
>> 2.7.4
>>
>>
>> _______________________________________________
>> linux-arm-kernel mailing list
>> linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
>> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel