Re: [PATCH 1/3] ARM: DTS: da850: add node for spi0

2016-04-17 Thread Sekhar Nori
On Friday 15 April 2016 09:47 PM, David Lechner wrote: > On 04/15/2016 05:24 AM, Sekhar Nori wrote: > >> >> This made me notice that num-cs is populated wrongly for spi1. It >> actually has 8 chip selects. This is fine though. > > I might as well fix it since I have to make changes anyway. Don't

Re: [PATCH 1/3] ARM: DTS: da850: add node for spi0

2016-04-15 Thread David Lechner
On 04/15/2016 03:16 PM, valdis.kletni...@vt.edu wrote: On Fri, 15 Apr 2016 11:17:55 -0500, David Lechner said: I omitted this on purpose. For my use case, I am using the SPI as write-only, So your SPI accesses are fire-and-forget, and nothing ever comes back? Yes. Seems a very dangerous w

Re: [PATCH 1/3] ARM: DTS: da850: add node for spi0

2016-04-15 Thread Valdis . Kletnieks
On Fri, 15 Apr 2016 11:17:55 -0500, David Lechner said: > I omitted this on purpose. For my use case, I am using the SPI as > write-only, So your SPI accesses are fire-and-forget, and nothing ever comes back? Seems a very dangerous way to design the use case, with no feedback if something suddenl

Re: [PATCH 1/3] ARM: DTS: da850: add node for spi0

2016-04-15 Thread David Lechner
On 04/15/2016 05:24 AM, Sekhar Nori wrote: This made me notice that num-cs is populated wrongly for spi1. It actually has 8 chip selects. This is fine though. I might as well fix it since I have to make changes anyway. Don't remember how I came up with 6. Also, it will be nice to add pin

Re: [PATCH 1/3] ARM: DTS: da850: add node for spi0

2016-04-15 Thread Sekhar Nori
On Thursday 14 April 2016 04:00 AM, David Lechner wrote: > Adds device definition for soc spi0 and also a aux data that is needed > for clock matching. > > Signed-off-by: David Lechner > --- > arch/arm/boot/dts/da850.dtsi | 10 ++ > arch/arm/mach-davinci/da8xx-dt.c | 1 + > 2 files