From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751930AbdF3McM (ORCPT ); Fri, 30 Jun 2017 08:32:12 -0400 Received: from esa3.microchip.iphmx.com ([68.232.153.233]:33892 "EHLO esa3.microchip.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751560AbdF3McF (ORCPT ); Fri, 30 Jun 2017 08:32:05 -0400 X-IronPort-AV: E=Sophos;i="5.40,286,1496127600"; d="scan'208";a="4284461" Subject: Re: [PATCH 5/5] ARM: dts: sama5d27_som1_ek: Add sama5d27 SoM1 EK support To: , , , , , , , , , References: <1498736674-9233-1-git-send-email-claudiu.beznea@microchip.com> <1498736674-9233-6-git-send-email-claudiu.beznea@microchip.com> <20170629150408.rcmwy74ozy4mtglo@rfolt0960.corp.atmel.com> From: m18063 Message-ID: <7e7874cc-11ad-e236-c7dd-086a0b997c2a@microchip.com> Date: Fri, 30 Jun 2017 15:32:00 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1 MIME-Version: 1.0 In-Reply-To: <20170629150408.rcmwy74ozy4mtglo@rfolt0960.corp.atmel.com> Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 >> Signed-off-by: Cristian Birsan >> --- >> 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 >> + * 2017 Cristian Birsan >> + * 2017 Claudiu Beznea >> + * >> + * 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 >> +#include >> + >> +/ { >> + 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 = ; >> + 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 = ; >> + 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 = ; >> + 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 = , >> + ; >> + bias-disable; >> + }; >> + >> + pinctrl_can1_default: can1_default { >> + pinmux = , >> + ; >> + bias-disable; >> + }; >> + >> + pinctrl_flx3_default: flx3_default { >> + pinmux = , >> + , >> + , >> + , >> + ; >> + bias-disable; >> + }; >> + >> + pinctrl_i2c1_default: i2c1_default { >> + pinmux = , >> + ; >> + bias-disable; >> + }; >> + >> + pinctrl_isc_base: isc_base { >> + pinmux = , >> + , >> + , >> + ; >> + bias-disable; >> + }; >> + >> + pinctrl_isc_data_8bit: isc_data_8bit { >> + pinmux = , >> + , >> + , >> + , >> + , >> + , >> + , >> + ; >> + bias-disable; >> + }; >> + >> + pinctrl_isc_data_9_10: isc_data_9_10 { >> + pinmux = , >> + ; >> + bias-disable; >> + }; >> + >> + pinctrl_isc_data_11_12: isc_data_11_12 { >> + pinmux = , >> + ; >> + bias-disable; >> + }; >> + >> + pinctrl_key_gpio_default: key_gpio_default { >> + pinmux = ; >> + bias-pull-up; >> + }; >> + >> + pinctrl_led_gpio_default: led_gpio_default { >> + pinmux = , >> + , >> + ; >> + bias-pull-up; >> + }; >> + >> + pinctrl_sdmmc0_default: sdmmc0_default { >> + cmd_data { >> + pinmux = , >> + , >> + , >> + , >> + , >> + , >> + , >> + , >> + ; >> + bias-pull-up; >> + }; >> + >> + ck_cd_vddsel { >> + pinmux = , >> + , >> + ; >> + bias-disable; >> + }; >> + }; >> + >> + pinctrl_sdmmc1_default: sdmmc1_default { >> + cmd_data { >> + pinmux = , >> + , >> + , >> + , >> + ; >> + bias-pull-up; >> + }; >> + >> + conf-ck_cd { >> + pinmux = , >> + ; >> + bias-disable; >> + }; >> + }; >> + >> + pinctrl_sensor_power: sensor_power { >> + pinmux = ; >> + bias-disable; >> + }; >> + >> + pinctrl_sensor_reset: sensor_reset { >> + pinmux = ; >> + bias-disable; >> + }; >> + >> + pinctrl_spi0_default: spi0_default { >> + pinmux = , >> + , >> + , >> + ; >> + bias-disable; >> + }; >> + >> + pinctrl_uart1_default: uart1_default { >> + pinmux = , >> + ; >> + bias-disable; >> + }; >> + >> + pinctrl_uart3_default: uart3_default { >> + pinmux = , >> + ; >> + bias-disable; >> + }; >> + >> + pinctrl_usb_default: usb_default { >> + pinmux = , >> + ; >> + bias-disable; >> + }; >> + >> + pinctrl_usba_vbus: usba_vbus { >> + pinmux = ; >> + bias-disable; >> + }; >> + >> + pinctrl_mikrobus1_an: mikrobus1_an { >> + pinmux = ; >> + bias-disable; >> + }; >> + >> + pinctrl_mikrobus2_an: mikrobus2_an { >> + pinmux = ; >> + bias-disable; >> + }; >> + >> + pinctrl_mikrobus1_rst: mikrobus1_rst { >> + pinmux = ; >> + bias-disable; >> + }; >> + >> + pinctrl_mikrobus2_rst: mikrobus2_rst { >> + pinmux = ; >> + bias-disable; >> + }; >> + >> + pinctrl_mikrobus1_spi_cs: mikrobus1_spi_cs { >> + pinmux = ; >> + bias-disable; >> + }; >> + >> + pinctrl_mikrobus2_spi_cs: mikrobus2_spi_cs { >> + pinmux = ; >> + bias-disable; >> + }; >> + >> + pinctrl_mikrobus_spi: mikrobus_spi { >> + pinmux = , >> + , >> + ; >> + bias-disable; >> + }; >> + >> + pinctrl_mikrobus1_pwm: mikrobus1_pwm { >> + pinmux = ; >> + bias-disable; >> + }; >> + >> + pinctrl_mikrobus2_pwm: mikrobus2_pwm { >> + pinmux = ; >> + bias-disable; >> + }; >> + >> + pinctrl_mikrobus1_int: mikrobus1_int { >> + pinmux = ; >> + bias-disable; >> + }; >> + >> + pinctrl_mikrobus2_int: mikrobus2_int { >> + pinmux = ; >> + bias-disable; >> + }; >> + >> + pinctrl_mikrobus1_uart: mikrobus1_uart { >> + pinmux = , >> + ; >> + bias-disable; >> + }; >> + >> + pinctrl_mikrobus2_uart: mikrobus2_uart { >> + pinmux = , >> + ; >> + bias-disable; >> + }; >> + >> + pinctrl_mikrobus_i2c: mikrobus1_i2c { >> + pinmux = , >> + ; >> + bias-disable; >> + }; >> + >> + pinctrl_flx4_default: flx4_uart_default { >> + pinmux = , >> + , >> + , >> + , >> + ; >> + 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@lists.infradead.org >> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel