Re: [PATCH v2 0/4] Versatile Express DT support

2011-11-28 Thread Pawel Moll
On Mon, 2011-11-28 at 15:39 +, Rob Herring wrote: > Okay. I'll apply. > > But that's just a vendor. There's still binding docs needed for the > actual sil9022 device and some others. I think the sil one is the only undocumented device we use now, except for the primecells that Dave and I will

Re: [PATCH v2 0/4] Versatile Express DT support

2011-11-28 Thread Rob Herring
On 11/28/2011 09:20 AM, Pawel Moll wrote: > On Mon, 2011-11-28 at 14:25 +, Rob Herring wrote: >> Other than Dave's comments, looks good. For the series: >> >> Acked-by: Rob Herring > > Thanks! > >> There are a number of compatible strings for various peripherals defined >> without documentat

Re: [PATCH v2 0/4] Versatile Express DT support

2011-11-28 Thread Pawel Moll
On Mon, 2011-11-28 at 14:25 +, Rob Herring wrote: > Other than Dave's comments, looks good. For the series: > > Acked-by: Rob Herring Thanks! > There are a number of compatible strings for various peripherals defined > without documentation, but you're not really using them so I think it's

Re: [PATCH v2 0/4] Versatile Express DT support

2011-11-28 Thread Dave Martin
On Mon, Nov 28, 2011 at 03:09:07PM +, Pawel Moll wrote: > On Mon, 2011-11-28 at 14:57 +, Dave Martin wrote: > > Oh, btw, git am reports a few whitespace errors on the last full post of > > the series (v2). > > > > Can you shove the series through checkpatch.pl and/or git am and fix any > >

Re: [PATCH v2 0/4] Versatile Express DT support

2011-11-28 Thread Pawel Moll
On Mon, 2011-11-28 at 14:57 +, Dave Martin wrote: > Oh, btw, git am reports a few whitespace errors on the last full post of > the series (v2). > > Can you shove the series through checkpatch.pl and/or git am and fix any > minor issues, now that the series is relatively stable? I did, actuall

Re: [PATCH v2 0/4] Versatile Express DT support

2011-11-28 Thread Dave Martin
On Mon, Nov 28, 2011 at 08:25:12AM -0600, Rob Herring wrote: > On 11/23/2011 09:01 AM, Pawel Moll wrote: > > Hello again, > > > > This version of the series (hopefully) addresses all the suggestions > > made by Dave, Rob and Russell. > > > > The compatible values are specific for the tiles now an

Re: [PATCH v2 0/4] Versatile Express DT support

2011-11-28 Thread Dave Martin
On Mon, Nov 28, 2011 at 08:25:12AM -0600, Rob Herring wrote: > On 11/23/2011 09:01 AM, Pawel Moll wrote: > > Hello again, > > > > This version of the series (hopefully) addresses all the suggestions > > made by Dave, Rob and Russell. > > > > The compatible values are specific for the tiles now an

Re: [PATCH v2 0/4] Versatile Express DT support

2011-11-28 Thread Rob Herring
On 11/23/2011 09:01 AM, Pawel Moll wrote: > Hello again, > > This version of the series (hopefully) addresses all the suggestions > made by Dave, Rob and Russell. > > The compatible values are specific for the tiles now and the memory > map variant is defined as a custom property in the motherboa

[PATCH v2 0/4] Versatile Express DT support

2011-11-23 Thread Pawel Moll
Hello again, This version of the series (hopefully) addresses all the suggestions made by Dave, Rob and Russell. The compatible values are specific for the tiles now and the memory map variant is defined as a custom property in the motherboard node. Tested on V2P-CA9 coretile both with ATAGs and