linux-spi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Ned Forrester <nforrester@whoi.edu>
To: Marek Vasut <marex@denx.de>
Cc: spi-devel-general@lists.sourceforge.net,
	"Bastelli Carlo \(yahoo\)" <carlo_bastelli@yahoo.it>,
	Fabio Estevam <festevam@gmail.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: SLAVE Side SPI kernel driver development
Date: Fri, 24 Aug 2012 12:54:55 -0400	[thread overview]
Message-ID: <5037B1DF.40503@whoi.edu> (raw)
In-Reply-To: <201208241738.45601.marex@denx.de>

On 08/24/2012 11:38 AM, Marek Vasut wrote:
> Dear Fabio Estevam,
> 
>> On Fri, Aug 24, 2012 at 11:57 AM, Bastelli Carlo (yahoo)
> 
> CCing SPI list
> 
>> <carlo_bastelli@yahoo.it> wrote:
>>> Hello, I'm Carlo, I have a difficult task at work, my boss asked me to
>>> develop
>>>
>>>  a driver SPI slave side on embedded ARM processor running Linux.
>>>
>>> Precisely linux occur on the SPI bus as a slave, not master as required
>>> by the hierarchical structure of the current kernel. The new driver
>>> will
>>>
>>>  receive via interrupt and respond when selected by its chip select.
> 
> So a soft-spi ? It's gonna be slow as hell and unstable too.
> 
>>>  Can
>>>  anyone suggest the best way to develop this driver?
> 
> Get a CPU which has SPI-slave capable IP, start from there. If you were to 
> implement soft-spi this way, you'd have to use FIQ on ARM, to be able to handle 
> stuff with reasonable latency ... check how the ps/2-gpio driver is done in 
> Linux ... I tried it, it barely worked, you don't want to go down that road.
> 
>>>  Personally, I
>>>
>>> have identified two main roads, the first if possible to exploit the
>>> structure SPI master driver
> 
> Not happening. You'd have to split the parts from the struct spi_master into 
> common and master-only parts. That's for starters.
> 
>>> with a few tricks that now I can not
>>> define. The second most radical, write a driver for a UART like, but
>>> managing the SPI device.
>>> Thank you for any suggestions or examples.
> 
> First of all, the SPI slave mode is COMPLETELY different from the master mode. 
> You need to react on incoming CS-low ASAP, which might turn out to be a problem 
> on it's own. Then, even if you do, you do not know how long the transfer is 
> going to be, so you need to allocate enough buffers as the transfer progresses 
> onwards.
> 
> In case of your soft-spi, how do you plan to sample the CLK line to sync clock?
> 
>> Marek got spi slave running on a mx28. See his patch:
>> http://www.spinics.net/lists/arm-kernel/msg190860.html
> 
> This was tested (and developed) on this board:
> http://www.denx-cs.de/?q=M28
> 
> The userland code is really easy too, simply pump it into spidev or read from 
> it. But all that is only possible because of the SPI IP in the mx28 is nice.
> 
>> Regards,
>>
>> Fabio Estevam
> 
> Best regards,
> Marek Vasut

I generally agree with the above comments.  A generalized SPI slave must
be able to respond instantly, often within one clock cycle.  This can
only work in hardware, or in an OS like Linux when the activity on the
bus is completely predictable by the Linux slave.  The only cases I know
of are streaming of data in a predictable way between a single master
and single slave (no other devices on the bus).

This topic comes up periodically, but I don't think there has been any
change in the situation for all the time I've followed the discussion.
I recommend that you read the archives of this mailing list to
understand the limitations of a potential slave mode.  The archives can
be found at:

http://sourceforge.net/mailarchive/forum.php?forum_name=spi-devel-general

then look for threads with "slave" in the subject in the following
months (I'd give thread start dates, but parts of the threads are
disconnected in the archives, and this is the easiest way to see all the
parts, day ranges are in parentheses):

2007/04	(13-18)
2007/09/27 and 2007/10 (13-17)
2009/03 (2-3)
2009/12/18 and 2010/1/19 and 2010/02 (14-16)
2010/07 (26) and 2010/08 (13)
2011/01 (28-31) and 2011/02 (1-10)
2011/10 (18)
2012/06 (29-30 and 2012/07 (1-17)

-- 
Ned Forrester                                       nforrester@whoi.edu
Oceanographic Systems Lab                           508-289-2226 Office
Applied Ocean Physics and Engineering Dept.
Woods Hole Oceanographic Institution          Woods Hole, MA 02543, USA
http://www.whoi.edu/
http://www.whoi.edu/page.do?pid=29856
http://www.whoi.edu/hpb/Site.do?id=1532

  reply	other threads:[~2012-08-24 16:54 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1345820228.87750.YahooMailNeo@web29604.mail.ird.yahoo.com>
     [not found] ` <CAOMZO5Ds6NKsTCptQXDGZDJasb119hYWqXMzX8nG5WFArysL7A@mail.gmail.com>
2012-08-24 15:38   ` SLAVE Side SPI kernel driver development Marek Vasut
2012-08-24 16:54     ` Ned Forrester [this message]
2012-08-25  7:46       ` Bastelli Carlo
2012-08-25 11:44         ` Marek Vasut
2012-08-25 14:06           ` Bastelli Carlo

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=5037B1DF.40503@whoi.edu \
    --to=nforrester@whoi.edu \
    --cc=carlo_bastelli@yahoo.it \
    --cc=festevam@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=marex@denx.de \
    --cc=spi-devel-general@lists.sourceforge.net \
    /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).