From: Maxime Ripard <maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
To: Alexandru Gagniuc <mr.nuke.me-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: linux-spi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
pawel.moll-5wv7dgnIgG8@public.gmane.org,
mark.rutland-5wv7dgnIgG8@public.gmane.org,
ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org,
galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org,
linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH 0/3] ARM: sun4i: SPI improvements
Date: Thu, 11 Dec 2014 17:56:02 +0100 [thread overview]
Message-ID: <20141211165602.GD8739@lukather> (raw)
In-Reply-To: <1418033642-29731-1-git-send-email-mr.nuke.me-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
[-- Attachment #1: Type: text/plain, Size: 941 bytes --]
On Mon, Dec 08, 2014 at 04:13:59AM -0600, Alexandru Gagniuc wrote:
> This is a series of patches that gets SPI working in a better state on the
> Cubieboard. The main motivation was to use flashrom to to program SPI ROMs.
> There is one patch missing, which adds an spidev node on the Cubieboard. Since
> Maxime did not like the approach in that patch, I've left it out for now.
Just to make things straight, it's not me alone that didn't like the
approach.
It's both wrong on a theorical point of view to put that in the DT,
since it has no hardware meaning at all (ie it doesn't describe the
hardware, or the state the bootloader left the hardware in), and this
has been NAKed a number of times already by Mark Brown.
See http://permalink.gmane.org/gmane.linux.kernel.spi.devel/11747 for
example.
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
next prev parent reply other threads:[~2014-12-11 16:56 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-12-08 10:13 [PATCH 0/3] ARM: sun4i: SPI improvements Alexandru Gagniuc
[not found] ` <1418033642-29731-1-git-send-email-mr.nuke.me-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2014-12-08 10:14 ` [PATCH 1/3] ARM: sun4i: spi: Allow transfers larger than FIFO size Alexandru Gagniuc
[not found] ` <1418033642-29731-2-git-send-email-mr.nuke.me-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2015-01-11 20:06 ` mrnuke
2014-12-08 10:14 ` [PATCH 2/3] ARM: sun4i: dt: Add pin muxing options for SPI Alexandru Gagniuc
[not found] ` <1418033642-29731-3-git-send-email-mr.nuke.me-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2014-12-11 16:57 ` Maxime Ripard
2014-12-08 10:14 ` [PATCH 3/3] ARM: sun4i: dt: cubieboard: Enable SPI0 Alexandru Gagniuc
[not found] ` <1418033642-29731-4-git-send-email-mr.nuke.me-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2014-12-11 16:57 ` Maxime Ripard
2014-12-11 16:56 ` Maxime Ripard [this message]
2014-12-15 16:59 ` [PATCH 0/3] ARM: sun4i: SPI improvements Mark Brown
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=20141211165602.GD8739@lukather \
--to=maxime.ripard-wi1+55scjutkeb57/3fjtnbpr1lh4cv8@public.gmane.org \
--cc=broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org \
--cc=ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org \
--cc=linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org \
--cc=linux-spi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=mark.rutland-5wv7dgnIgG8@public.gmane.org \
--cc=mr.nuke.me-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=pawel.moll-5wv7dgnIgG8@public.gmane.org \
--cc=robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.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).