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
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 knut.wohl...@de.bosch.com There are SPI devices which need a SPI clock with active low polarity and high inactive state. Add the setting of the inactive state

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 Shawn Guo
On Fri, Aug 31, 2012 at 10:55:11AM +0200, Dirk Behme wrote: From: Knut Wohlrab knut.wohl...@de.bosch.com 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

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

2012-09-22 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 patch, but I

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 dirk.be...@gmail.com 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 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 knut.wohl...@de.bosch.com There are SPI devices which need a SPI clock with active low polarity and high inactive

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 (i.e.

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

2012-08-31 Thread Dirk Behme
From: Knut Wohlrab knut.wohl...@de.bosch.com 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

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 knut.wohl...@de.bosch.com 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