Re: [PATCH] spi/imx: set the inactive state of the clock according to the clock polarity

2012-09-25 Thread Mark Brown
On Mon, Sep 24, 2012 at 01:31:22PM +0200, Dirk Behme wrote: > Let me know if you can find above thread on one of the both mailing > lists. If not, I would be happy to resend it again. Yes, of course - it's hard to apply patches from web archives. -

Re: [PATCH] spi/imx: set the inactive state of the clock according to the clock polarity

2012-09-24 Thread Dirk Behme
On 24.09.2012 12:02, Mark Brown wrote: > On Mon, Sep 24, 2012 at 08:34:27AM +0200, Dirk Behme wrote: > >> Being not so familiar with the flow of SPI patches, I was told that you >> are taking care of spi patches recently? > >> Do you like to have a look to this patch? > > Well, nobody appears

Re: [PATCH] spi/imx: set the inactive state of the clock according to the clock polarity

2012-09-24 Thread Mark Brown
On Mon, Sep 24, 2012 at 08:34:27AM +0200, Dirk Behme wrote: > Being not so familiar with the flow of SPI patches, I was told that you > are taking care of spi patches recently? > Do you like to have a look to this patch? Well, nobody appears to have sent me a copy of it... ---

Re: [PATCH] spi/imx: set the inactive state of the clock according to the clock polarity

2012-09-23 Thread Dirk Behme
Hi Mark, On 24.09.2012 07:06, Shawn Guo wrote: > On Fri, Aug 31, 2012 at 10:55:11AM +0200, Dirk Behme wrote: >> From: Knut Wohlrab >> >> There are SPI devices which need a SPI clock with active low polarity and >> high inactive state. >> >> Add the setting of the inactive state ECSPIx_CONFIGREG:S

Re: [PATCH] spi/imx: set the inactive state of the clock according to the clock polarity

2012-09-23 Thread Shawn Guo
On Fri, Aug 31, 2012 at 10:55:11AM +0200, Dirk Behme wrote: > From: Knut Wohlrab > > There are SPI devices which need a SPI clock with active low polarity and > high inactive state. > > Add the setting of the inactive state ECSPIx_CONFIGREG:SCLK CTL > according to the clock polarity ECSPIx_CONFI

Re: [PATCH] spi/imx: set the inactive state of the clock according to the clock polarity

2012-09-22 Thread Fabio Estevam
Hi Dirk, On Sat, Sep 22, 2012 at 3:10 AM, Dirk Behme wrote: > I'm not so familiar with the SPI world: Who would be the best person to ack > this, and in case it's ok finally apply it, then? Mark Brown has been taking care of spi patches recently. Regards, Fabio Estevam --

Re: [PATCH] spi/imx: set the inactive state of the clock according to the clock polarity

2012-09-21 Thread Dirk Behme
On 21.09.2012 09:54, Uwe Kleine-König wrote: > On Fri, Sep 21, 2012 at 09:36:51AM +0200, Dirk Behme wrote: >> On 31.08.2012 04:35, Shawn Guo wrote: >>> Copy Uwe ... > Copy our kernel mailing list ... > >> Ping ;) >> >> Any opinions on this? > Assuming it fixes access to a device I'm ok with your pa

Re: [PATCH] spi/imx: set the inactive state of the clock according to the clock polarity

2012-09-21 Thread Uwe Kleine-König
On Fri, Sep 21, 2012 at 09:36:51AM +0200, Dirk Behme wrote: > On 31.08.2012 04:35, Shawn Guo wrote: > >Copy Uwe ... Copy our kernel mailing list ... > Ping ;) > > Any opinions on this? Assuming it fixes access to a device I'm ok with your patch, but I don't really feel responsible for the driver

Re: [PATCH] spi/imx: set the inactive state of the clock according to the clock polarity

2012-09-21 Thread Dirk Behme
On 31.08.2012 04:35, Shawn Guo wrote: > Copy Uwe ... Ping ;) Any opinions on this? Thanks Dirk > On Fri, Aug 31, 2012 at 10:55:11AM +0200, Dirk Behme wrote: >> From: Knut Wohlrab >> >> There are SPI devices which need a SPI clock with active low polarity and >> high inactive state. >> >> Add

Re: [PATCH] spi/imx: set the inactive state of the clock according to the clock polarity

2012-08-31 Thread Shawn Guo
Copy Uwe ... On Fri, Aug 31, 2012 at 10:55:11AM +0200, Dirk Behme wrote: > From: Knut Wohlrab > > There are SPI devices which need a SPI clock with active low polarity and > high inactive state. > > Add the setting of the inactive state ECSPIx_CONFIGREG:SCLK CTL > according to the clock polarit

[PATCH] spi/imx: set the inactive state of the clock according to the clock polarity

2012-08-31 Thread Dirk Behme
From: Knut Wohlrab There are SPI devices which need a SPI clock with active low polarity and high inactive state. Add the setting of the inactive state ECSPIx_CONFIGREG:SCLK CTL according to the clock polarity ECSPIx_CONFIGREG:SCLK POL: DT without "spi-cpol" = 0 = clock active high polarity = i