From: Mark Brown <broonie@kernel.org>
To: Hans de Goede <hdegoede@redhat.com>
Cc: Michal Suchanek <hramrach@gmail.com>,
Maxime Ripard <maxime.ripard@free-electrons.com>,
Martin Sperl <kernel@martin.sperl.org>,
linux-sunxi <linux-sunxi@googlegroups.com>,
Jonathan Corbet <corbet@lwn.net>,
linux-spi <linux-spi@vger.kernel.org>,
linux-doc <linux-doc@vger.kernel.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [linux-sunxi] [PATCH 2/3] spidev: Add DT binding example.
Date: Mon, 27 Apr 2015 12:25:39 +0100 [thread overview]
Message-ID: <20150427112539.GR22845@sirena.org.uk> (raw)
In-Reply-To: <553E099C.4070208@redhat.com>
[-- Attachment #1: Type: text/plain, Size: 1404 bytes --]
On Mon, Apr 27, 2015 at 12:04:12PM +0200, Hans de Goede wrote:
> Have you seen my mail about the raspberry pi use-case? Using dt-overlays
> simply is not an acceptable answer there. There are legitimate use-cases
> for a "generic spi bus" concept with the bus only being accessible via
> spidev.
> Blocking this use-case because you do not believe it is a valid use-case
> is not going to help, this will just lead to the custom distros these
> boards are shipping doing some ugly hack, which is not what we want
> IMHO.
I don't think you've fully considered your use case here. As I said in
my reply to your earlier e-mail I think what you're looking for here is
something like better UI around overlays. Registering a SPI bus without
knowing what's connected to it doesn't allow generic maker style usage
of the board, it's just as likely to hinder a user as help them. For
example, if someone wants to use the SPI pins for another function such
as GPIO they'll have to handle that (and may have problems with pin
conflicts causing electrical issues if they do load up the DT with
spidev in it). If someone has a SPI device they want to bind an in
kernel driver to they'll have to handle that, if they want to use a GPIO
to provide an additional chip select they'll have to handle that too.
Note that the discussion here isn't about userspace drivers, it's about
how the hardware is described.
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]
next prev parent reply other threads:[~2015-04-27 11:26 UTC|newest]
Thread overview: 69+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <cover.1430034797.git.hramrach@gmail.com>
[not found] ` <f6e34cb3f0b72fcb0ae18ccf96962fb8a1c477f7.1430034797.git.hramrach@gmail.com>
2015-04-26 8:39 ` [PATCH 1/3] ARM: dts: sunxi: A10s Olinuxino add missing SPI and simplefb Maxime Ripard
2015-04-26 9:21 ` Michal Suchanek
2015-04-26 12:52 ` Maxime Ripard
[not found] ` <4c27d44b2bdd759424ce4a4b2e8f6abf5d5a6735.1430034797.git.hramrach@gmail.com>
2015-04-26 8:42 ` [PATCH 3/3] ARM: sunxi: spi: use proper errno when message is too long Maxime Ripard
2015-04-26 11:42 ` Michal Suchanek
2015-04-26 12:54 ` Maxime Ripard
[not found] ` <bb069283a5c2ccfbc05177f1ed41cabb1485796e.1430034797.git.hramrach@gmail.com>
2015-04-26 10:32 ` [PATCH 2/3] spidev: Add DT binding example Mark Brown
2015-04-26 10:54 ` Michal Suchanek
2015-04-26 11:01 ` Mark Brown
2015-04-26 11:23 ` [linux-sunxi] " Hans de Goede
2015-04-26 11:56 ` [linux-sunxi] " Martin Sperl
2015-04-26 12:38 ` Michal Suchanek
2015-04-26 12:51 ` Maxime Ripard
2015-04-26 14:14 ` Michal Suchanek
2015-04-26 14:33 ` Maxime Ripard
2015-04-26 14:40 ` Hans de Goede
2015-04-26 15:47 ` Maxime Ripard
2015-04-27 6:51 ` Michal Suchanek
2015-04-27 10:05 ` Maxime Ripard
2015-04-27 10:10 ` Mark Brown
2015-04-27 14:28 ` Michal Suchanek
2015-04-27 15:13 ` Geert Uytterhoeven
2015-04-27 15:44 ` Michal Suchanek
2015-04-26 15:33 ` Michal Suchanek
2015-04-26 15:54 ` Maxime Ripard
2015-04-26 18:53 ` Michal Suchanek
2015-04-27 10:04 ` Maxime Ripard
2015-04-27 11:18 ` Michal Suchanek
2015-04-27 10:46 ` Mark Brown
2015-04-27 9:36 ` Mark Brown
2015-04-27 9:39 ` Michal Suchanek
2015-04-27 10:59 ` Mark Brown
2015-04-27 10:04 ` Hans de Goede
2015-04-27 10:09 ` Hans de Goede
2015-04-27 11:25 ` Mark Brown [this message]
2015-04-27 14:14 ` Martin Sperl
2015-04-27 15:27 ` Mark Brown
2015-04-27 16:25 ` Martin Sperl
2015-04-27 17:59 ` Mark Brown
2015-04-27 10:16 ` Mark Brown
2015-04-27 17:30 ` Maxime Ripard
2015-04-27 18:07 ` Mark Brown
[not found] ` <cd282abf-898a-4f01-90a6-8bf2db160b8e@googlegroups.com>
2015-04-28 12:52 ` Michal Suchanek
[not found] ` <28a25eda-bba0-4a2e-9890-b3d3bef7ac7e@googlegroups.com>
2015-04-28 14:11 ` Michal Suchanek
2015-04-28 14:12 ` Maxime Ripard
2015-04-28 14:35 ` Michal Suchanek
2015-04-28 14:16 ` Mark Brown
2015-04-28 14:22 ` Michal Suchanek
2015-04-28 17:17 ` Mark Brown
2015-04-28 20:43 ` Michal Suchanek
2015-04-29 17:40 ` Mark Brown
2015-04-29 17:44 ` Michal Suchanek
2015-04-29 18:06 ` Mark Brown
2015-04-29 18:37 ` Michal Suchanek
2015-04-29 18:56 ` Geert Uytterhoeven
2015-04-29 19:24 ` Michal Suchanek
2015-04-30 19:58 ` Mark Brown
2015-05-03 9:01 ` Martin Sperl
2015-05-03 9:59 ` Mark Brown
2015-05-03 21:00 ` Martin Sperl
2015-05-04 8:36 ` Michal Suchanek
2015-05-04 10:12 ` Mark Brown
2015-05-04 10:42 ` Michal Suchanek
2015-05-04 13:17 ` Mark Brown
2015-05-03 10:02 ` Geert Uytterhoeven
2015-05-12 14:27 ` Maxime Ripard
2015-05-12 14:52 ` Michal Suchanek
2015-05-12 16:06 ` Mark Brown
2015-04-26 11:26 ` Michal Suchanek
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=20150427112539.GR22845@sirena.org.uk \
--to=broonie@kernel.org \
--cc=corbet@lwn.net \
--cc=hdegoede@redhat.com \
--cc=hramrach@gmail.com \
--cc=kernel@martin.sperl.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-spi@vger.kernel.org \
--cc=linux-sunxi@googlegroups.com \
--cc=maxime.ripard@free-electrons.com \
/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
all inboxes | Powered by JetHome®