From: Milton Miller <miltonm@bga.com>
To: Kumar Gala <galak@kernel.crashing.org>
Cc: linuxppc-dev@ozlabs.org
Subject: Re: SPI devices and OF
Date: Thu, 5 Apr 2007 10:00:23 -0500 [thread overview]
Message-ID: <9cceb498d8a04450a84ca99cde917a3b@bga.com> (raw)
In-Reply-To: <1175758338.30879.123.camel@localhost.localdomain>
Ben Herrenschmidt wrote:
> Kumar Gala wrote:
>> From explicitly board code like we do today. I mean the mechanism
>> can very so greatly that trying to decided and come up with all
>> possible cases and somehow encoding that in the device tree isn't
>> worth the effort. Additionally you'll still need code to handle the
>> actual chip select and I don't see how you make that generic at all.
At this point I don't think we are trying to make a generic master. At
most we are trying to expose the spi device in a manner that can be
reused by board-specific master drivers.
>> A board designer could use I2C, GPIO, or something off an FPGA. I
>> just dont see trying to 'encode' this in the device tree as providing
>> any real value.
>
> Or board designers can use board specific device-tree bits and board
> specific code to udnerstand them :-) That works too and can be handy if
> you have for example several versions of a board with small differences
> that you want to expose that way in the device-tree.
>
> That is, the devive-tree -can- be used to put proprietary stuff, though
> if you do so, you should try to use prefixes on your properties, like
> mycompany,xxxx
I agree with Ben here.
Asserting the chip select is the responsibility of the master driver,
and that is selected by the compatable property in the device tree
to a specific driver.
The reg property in the device node can be useful to the master driver
wihtout expressing how to assert it in the master node.
milton
next prev parent reply other threads:[~2007-04-05 15:00 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-04-04 11:09 SPI devices and OF Sascha Hauer
2007-04-04 11:51 ` Vitaly Wool
2007-04-04 16:34 ` Kumar Gala
2007-04-04 17:03 ` Arnd Bergmann
2007-04-04 17:12 ` Kumar Gala
2007-04-05 7:32 ` Benjamin Herrenschmidt
2007-04-05 15:00 ` Milton Miller [this message]
2007-04-05 8:33 ` Sascha Hauer
2007-04-04 15:57 ` Milton Miller
2007-04-05 8:50 ` Sascha Hauer
2007-04-05 14:44 ` Milton Miller
2007-04-05 19:11 ` Sascha Hauer
2007-04-05 19:52 ` Arnd Bergmann
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=9cceb498d8a04450a84ca99cde917a3b@bga.com \
--to=miltonm@bga.com \
--cc=galak@kernel.crashing.org \
--cc=linuxppc-dev@ozlabs.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).