From: jakob@viketoft.se (Jakob Viketoft)
To: linux-arm-kernel@lists.infradead.org
Subject: SPI trouble on Colibri 270 (PXA)...
Date: Fri, 23 Apr 2010 21:57:15 +0200 [thread overview]
Message-ID: <4BD1FB9B.9030902@viketoft.se> (raw)
In-Reply-To: <4BD1D0F8.60705@gmail.com>
Vernon Sauder wrote:
> Jakob Viketoft wrote, On 04/22/2010 10:36 AM:
<...>
>> The original code currently combines the two transfers in one message as
>> you say (leaving CS asserted unless cs_change is set). However, I don't
>> know if it's a problem with scheduling or that the interrupt takes too
>> long to serve, but on my platform the two transfers (in the same
>> message) doesn't go back-to-back as they should and CS gets deasserted
>> in the middle. Thus I have combined the two transfers into one and the
>> message only hold this single transfer. Now things work at my end, but
>> it's annoying that I don't have time to track down the real issue at the
>> moment...
>>
>> /Jakob
>>
>
> As part of this thread:
> http://permalink.gmane.org/gmane.linux.kernel.spi.devel/1517
> I added the following note to Documentation/spi/pxa2xx. I have not
> checked to see if the patch hit mainline. Is this part of the answer to
> your question?
>
>> +NOTE: the SPI driver cannot control the chip select if SSPFRM is used, so the
>> +chipselect is dropped after each spi_transfer. Most devices need chip select
>> +asserted around the complete message. Use SSPFRM as a GPIO (through cs_control)
>> +to accomodate these chips.
>
> That was my solution to using the pxa2xx SPI with M25Pxx flash.
>
Yes, I saw that note in the documentation. Unfortunately it wasn't what
was causing my problem. I think a code patch has been applied to control
the SSPFRM as a gpio through the .gpio_cs option, but this seem to be
missing from the documentation.
From the sound of it, I think I'm running into the same kind of trouble
as shown by the SPI controller on the EP93xx.
/Jakob
next prev parent reply other threads:[~2010-04-23 19:57 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-04-20 11:58 SPI trouble on Colibri 270 (PXA) Jakob Viketoft
2010-04-20 13:14 ` Eric Miao
2010-04-20 15:20 ` Jakob Viketoft
2010-04-21 9:44 ` Marek Vasut
2010-04-21 10:47 ` Eric Miao
2010-04-21 11:11 ` Martin Guy
2010-04-21 11:42 ` Eric Miao
2010-04-21 11:49 ` Martin Guy
2010-04-21 12:47 ` Jakob Viketoft
2010-04-21 14:53 ` Martin Guy
[not found] ` <g2xf17812d71004210607n4f3d82f2m4712d3bb97ef4924@mail.gmail.com>
2010-04-22 14:36 ` Jakob Viketoft
2010-04-23 16:55 ` Vernon Sauder
2010-04-23 19:57 ` Jakob Viketoft [this message]
2010-04-23 23:40 ` Eric Miao
2010-04-21 12:12 ` Marek Vasut
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4BD1FB9B.9030902@viketoft.se \
--to=jakob@viketoft.se \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).