On Monday 21 November 2016 11:06 PM, Rob Herring wrote: > On Mon, Nov 21, 2016 at 11:28:01AM +0530, Lokesh Vutla wrote: >> From: Schuyler Patton <spatton@xxxxxx> >> >> The AM571x-IDK board is a board based on TI's AM5718 SOC >> which has a single core 1.5GHz A15 processor. This board is a >> development platform for the Industrial market with: >> - 1GB of DDR3L >> - Dual 1Gbps Ethernet >> - HDMI, >> - PRU-ICSS >> - uSD >> - 16GB eMMC >> - CAN >> - RS-485 >> - PCIe >> - USB3.0 >> - Video Input Port >> - Industrial IO port and expansion connector >> >> The link to the data sheet and TRM can be found here: >> >> http://www.ti.com/product/AM5718 >> >> Initial support is only for basic peripherals. >> >> Signed-off-by: Schuyler Patton <spatton@xxxxxx> >> Signed-off-by: Nishanth Menon <nm@xxxxxx> >> Signed-off-by: Dave Gerlach <d-gerlach@xxxxxx> >> Signed-off-by: Lokesh Vutla <lokeshvutla@xxxxxx> >> --- >> >> Logs: http://pastebin.ubuntu.com/23510390/ >> >> .../devicetree/bindings/arm/omap/omap.txt | 3 + >> arch/arm/boot/dts/Makefile | 1 + >> arch/arm/boot/dts/am571x-idk.dts | 82 ++++++++++++++++++++++ >> 3 files changed, 86 insertions(+) >> create mode 100644 arch/arm/boot/dts/am571x-idk.dts >> >> diff --git a/Documentation/devicetree/bindings/arm/omap/omap.txt b/Documentation/devicetree/bindings/arm/omap/omap.txt >> index f53e2ee..647ffd3 100644 >> --- a/Documentation/devicetree/bindings/arm/omap/omap.txt >> +++ b/Documentation/devicetree/bindings/arm/omap/omap.txt >> @@ -175,6 +175,9 @@ Boards: >> - AM5728 IDK >> compatible = "ti,am5728-idk", "ti,am5728", "ti,dra742", "ti,dra74", "ti,dra7" >> >> +- AM5718 IDK >> + compatible = "ti,am5718-idk", "ti,am5728", "ti,dra722", "ti,dra72", "ti,dra7" > > I've said this before I think, but 5 compat string is a bit much. Some > of these genericish ones should be dropped. Doesn't really hurt though. Yeah dra722, dra72 can be dropped from this. Will do it. > > A couple of nits below, otherwise: > > Acked-by: Rob Herring <robh@xxxxxxxxxx> > >> + >> - DRA742 EVM: Software Development Board for DRA742 >> compatible = "ti,dra7-evm", "ti,dra742", "ti,dra74", "ti,dra7" >> >> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile >> index befcd26..c298078 100644 >> --- a/arch/arm/boot/dts/Makefile >> +++ b/arch/arm/boot/dts/Makefile >> @@ -588,6 +588,7 @@ dtb-$(CONFIG_SOC_DRA7XX) += \ >> am57xx-cl-som-am57x.dtb \ >> am57xx-sbc-am57x.dtb \ >> am572x-idk.dtb \ >> + am571x-idk.dtb \ >> dra7-evm.dtb \ >> dra72-evm.dtb \ >> dra72-evm-revc.dtb >> diff --git a/arch/arm/boot/dts/am571x-idk.dts b/arch/arm/boot/dts/am571x-idk.dts >> new file mode 100644 >> index 0000000..a6a743e >> --- /dev/null >> +++ b/arch/arm/boot/dts/am571x-idk.dts >> @@ -0,0 +1,82 @@ >> +/* >> + * Copyright (C) 2015-2016 Texas Instruments Incorporated - http://www.ti.com/ >> + * >> + * This program is free software; you can redistribute it and/or modify >> + * it under the terms of the GNU General Public License version 2 as >> + * published by the Free Software Foundation. >> + */ >> +/dts-v1/; >> + >> +#include "dra72x.dtsi" >> +#include <dt-bindings/gpio/gpio.h> >> +#include <dt-bindings/interrupt-controller/irq.h> >> +#include "am57xx-idk-common.dtsi" >> + >> +/ { >> + model = "TI AM5718 IDK"; >> + compatible = "ti,am5718-idk", "ti,am5718", "ti,dra722", >> + "ti,dra72", "ti,dra7"; >> + >> + memory@0 { > > unit address is wrong. okay. > >> + device_type = "memory"; >> + reg = <0x0 0x80000000 0x0 0x40000000>; >> + }; >> + >> + status-leds { > > Just "leds" okay. Will update it and send a v2. Thanks and regards, Lokesh > >> + compatible = "gpio-leds"; >> + cpu0-led { >> + label = "status0:red:cpu0"; >> + gpios = <&gpio2 25 GPIO_ACTIVE_HIGH>; >> + default-state = "off"; >> + linux,default-trigger = "cpu0"; >> + }; >> + >> + usr0-led { >> + label = "status0:green:usr"; >> + gpios = <&gpio2 26 GPIO_ACTIVE_HIGH>; >> + default-state = "off"; >> + }; >> + >> + heartbeat-led { >> + label = "status0:blue:heartbeat"; >> + gpios = <&gpio2 27 GPIO_ACTIVE_HIGH>; >> + default-state = "off"; >> + linux,default-trigger = "heartbeat"; >> + }; >> + >> + usr1-led { >> + label = "status1:red:usr"; >> + gpios = <&gpio2 28 GPIO_ACTIVE_HIGH>; >> + default-state = "off"; >> + }; >> + >> + usr2-led { >> + label = "status1:green:usr"; >> + gpios = <&gpio2 21 GPIO_ACTIVE_HIGH>; >> + default-state = "off"; >> + }; >> + >> + mmc0-led { >> + label = "status1:blue:mmc0"; >> + gpios = <&gpio2 19 GPIO_ACTIVE_HIGH>; >> + default-state = "off"; >> + linux,default-trigger = "mmc0"; >> + }; >> + }; >> + >> + extcon_usb2: extcon_usb2 { >> + compatible = "linux,extcon-usb-gpio"; >> + id-gpio = <&gpio5 7 GPIO_ACTIVE_HIGH>; >> + }; >> +}; >> + >> +&mmc1 { >> + status = "okay"; >> + vmmc-supply = <&ldo1_reg>; >> + bus-width = <4>; >> + cd-gpios = <&gpio6 27 0>; /* gpio 219 */ >> +}; >> + >> +&omap_dwc3_2 { >> + extcon = <&extcon_usb2>; >> +}; >> -- >> 2.10.1 >> -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html