Re: [PATCH 2/3] DT: EVM: add LEDs

2016-09-28 Thread H. Nikolaus Schaller
> Am 29.09.2016 um 07:14 schrieb Rob Herring : > > On Tue, Sep 27, 2016 at 12:26 AM, H. Nikolaus Schaller > wrote: > > The subject gives no indication this is for OMAP. Please follow > standard patterns. > >> Signed-off-by: H. Nikolaus Schaller

Re: [PATCH 2/3] DT: EVM: add LEDs

2016-09-28 Thread H. Nikolaus Schaller
> Am 29.09.2016 um 07:14 schrieb Rob Herring : > > On Tue, Sep 27, 2016 at 12:26 AM, H. Nikolaus Schaller > wrote: > > The subject gives no indication this is for OMAP. Please follow > standard patterns. Already fixed. Please see v2. > >>

Re: [PATCH 2/3] DT: EVM: add LEDs

2016-09-28 Thread Rob Herring
On Tue, Sep 27, 2016 at 12:26 AM, H. Nikolaus Schaller wrote: The subject gives no indication this is for OMAP. Please follow standard patterns. > Signed-off-by: H. Nikolaus Schaller > --- > arch/arm/boot/dts/omap5-uevm.dts | 60 >

Re: [PATCH 2/3] DT: EVM: add LEDs

2016-09-28 Thread Tony Lindgren
* H. Nikolaus Schaller [160928 08:29]: > > > Am 28.09.2016 um 17:12 schrieb Tony Lindgren : > > > > * H. Nikolaus Schaller [160927 23:50]: > >> Hi, > >> > >>> Am 28.09.2016 um 05:37 schrieb Tony Lindgren : > >>> >

Re: [PATCH 2/3] DT: EVM: add LEDs

2016-09-28 Thread H. Nikolaus Schaller
> Am 28.09.2016 um 17:12 schrieb Tony Lindgren : > > * H. Nikolaus Schaller [160927 23:50]: >> Hi, >> >>> Am 28.09.2016 um 05:37 schrieb Tony Lindgren : >>> >>> * H. Nikolaus Schaller [160927 13:11]: > Am

Re: [PATCH 2/3] DT: EVM: add LEDs

2016-09-28 Thread Tony Lindgren
* H. Nikolaus Schaller [160927 23:50]: > Hi, > > > Am 28.09.2016 um 05:37 schrieb Tony Lindgren : > > > > * H. Nikolaus Schaller [160927 13:11]: > >>> Am 27.09.2016 um 21:49 schrieb Tony Lindgren : > >>> How about this

Re: [PATCH 2/3] DT: EVM: add LEDs

2016-09-28 Thread H. Nikolaus Schaller
Hi, > Am 28.09.2016 um 05:37 schrieb Tony Lindgren : > > * H. Nikolaus Schaller [160927 13:11]: >>> Am 27.09.2016 um 21:49 schrieb Tony Lindgren : >>> How about this for defaults: >>> >>> - heartbeat for led3 >>> - cpu0 for led4 >>> -

Re: [PATCH 2/3] DT: EVM: add LEDs

2016-09-27 Thread Tony Lindgren
* H. Nikolaus Schaller [160927 13:11]: > > Am 27.09.2016 um 21:49 schrieb Tony Lindgren : > > How about this for defaults: > > > > - heartbeat for led3 > > - cpu0 for led4 > > - cpu1 for led5 > > Good idea. Will try. > > What I don't exactly know is if

Re: [PATCH 2/3] DT: EVM: add LEDs

2016-09-27 Thread H. Nikolaus Schaller
Hi Tony, > Am 27.09.2016 um 21:49 schrieb Tony Lindgren : > > * H. Nikolaus Schaller [160926 22:27]: >> +led@6 { >> +label = "omap5:green:led3"; >> +gpios = < 4 GPIO_ACTIVE_HIGH>; >> +

Re: [PATCH 2/3] DT: EVM: add LEDs

2016-09-27 Thread Tony Lindgren
* H. Nikolaus Schaller [160926 22:27]: > + led@6 { > + label = "omap5:green:led3"; > + gpios = < 4 GPIO_ACTIVE_HIGH>; > + linux,default-trigger = "heartbeat"; > + default-state = "off";

[PATCH 2/3] DT: EVM: add LEDs

2016-09-26 Thread H. Nikolaus Schaller
Signed-off-by: H. Nikolaus Schaller --- arch/arm/boot/dts/omap5-uevm.dts | 60 1 file changed, 60 insertions(+) diff --git a/arch/arm/boot/dts/omap5-uevm.dts b/arch/arm/boot/dts/omap5-uevm.dts index be659e8..19f5f0a 100644 ---