2015-03-09 15:39 GMT+01:00 Linus Walleij <linus.walleij@xxxxxxxxxx>: > On Fri, Feb 20, 2015 at 7:01 PM, Maxime Coquelin > <mcoquelin.stm32@xxxxxxxxx> wrote: > >> The STMicrolectornics's STM32F419 MCU has the following main features: >> - Cortex-M4 core running up to @180MHz >> - 2MB internal flash, 256KBytes internal RAM >> - FMC controller to connect SDRAM, NOR and NAND memories >> - SD/MMC/SDIO support >> - Ethernet controller >> - USB OTFG FS & HS controllers >> - I2C, SPI, CAN busses support >> - Several 16 & 32 bits general purpose timers >> - Serial Audio interface >> - LCD controller >> >> Signed-off-by: Maxime Coquelin <mcoquelin.stm32@xxxxxxxxx> > (...) >> + soc { >> + usart1: usart@40011000 { >> + status = "okay"; >> + }; >> + >> + leds { >> + compatible = "gpio-leds"; >> + red { >> + #gpio-cells = <2>; >> + label = "Front Panel LED"; >> + gpios = <&gpiog 14 0>; >> + linux,default-trigger = "heartbeat"; >> + }; >> + green { >> + #gpio-cells = <2>; >> + gpios = <&gpiog 13 0>; >> + default-state = "off"; >> + }; >> + }; >> + }; > > The LEDs are mounted on the board not the SoC, right? > > So move them outside of the soc node, to the top level > of the DTS file. Yes, you are right. Andreas already made the same remark, and the top-level move is already here in the upcoming v3. Thanks, Maxime > > Yours, > Linus Walleij -- 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