archive mirror
 help / color / mirror / Atom feed
From: jassi brar <>
To: Linus Walleij <>
Cc: spi mailing list
	Ken Mills <>
Subject: Re: [PATCH] Add support for slave controllers plus sysfs entries for power management
Date: Wed, 17 Feb 2010 10:43:12 +0900	[thread overview]
Message-ID: <> (raw)
In-Reply-To: <>

On Wed, Feb 17, 2010 at 4:33 AM, Linus Walleij
<> wrote:
> 2010/2/15 jassi brar <>:
>> I don't think adding SPI_SLAVE support is just a matter of providing
>> additional callbacks and structures, as is pointed out in this thread....
> You mean that the responsiveness / control of latencies is the other thing
> that's needed? Yep so it is. But getting the infrastructure in place doesn't
> hurt because this is something many people (including self) need and Ken
> over at Intel is the only one actually doing something about it.
> Getting SPI slaves to actually work by spawning their worker threads as
> realtime under that patchset is of course a larger issue. One does not
> exclude the other tho.
I tend to resonate with Ned Forrester's concerns and Grant Likely
already has clarified my point.

As a further note, it *seems* someone just needs a way to test his SPI
controller's SLAVE mode(I used to do that with much lesser changes
to the SPI stack). Or I will be interested to know (to the extent
your privilege to disclose extends) which device running a full fledged
non-real time OS(Linux) needs to work as a SPI slave?
I have been supporting Samsung's SPI drivers internally for quite
some time now and am yet to come across one usage of it's SLAVE
mode. Also, it's only intuitive that 'chips'(with minimal f/w, fixed
functionality and bounded delays) run as SPI SLAVE and
devices(with multiple processes and indeterminate latencies) as

So, I am not excited about (*coughs*) seeing SPI SLAVE support
implemented without (a) real need for it (b) properly thought out
architecture and not just a workaround.


SOLARIS 10 is the OS for Data Centers - provides features such as DTrace,
Predictive Self Healing and Award Winning ZFS. Get Solaris 10 NOW

  parent reply	other threads:[~2010-02-17  1:43 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-12-18 21:06 [PATCH] Add support for slave controllers plus sysfs entries for power management Ken Mills
2010-01-19 15:59 ` Grant Likely
     [not found]   ` <>
2010-02-14 23:20     ` Linus Walleij
     [not found]       ` <>
2010-02-15  1:37         ` jassi brar
     [not found]           ` <>
2010-02-16 19:33             ` Linus Walleij
     [not found]               ` <>
2010-02-16 20:48                 ` Ned Forrester
     [not found]                   ` <4B7B048C.8080205-/>
2010-02-17  3:19                     ` Grant Likely
2010-02-16 22:07                 ` Grant Likely
2010-02-17  1:43                 ` jassi brar [this message]
2010-02-16 22:01             ` Grant Likely

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:

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \ \ \ \ \ \

* 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).