-by: Michael Hennerich michael.henner...@analog.com
Signed-off-by: Sonic Zhang sonic.zh...@analog.com
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/spi/spi_bfin5xx.c b/drivers/spi
There's no need for these queue funcs to be inlined, so drop the
markings. This shaves off a few hundred duplicated bytes.
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c |8
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/spi
: unsupported bits_per_word
So in the transfer logic, have a bits_per_word setting of 0 fall into the
8 bit transfer logic.
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/spi/spi_bfin5xx.c b
This array isn't written anywhere, so constify it.
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/spi/spi_bfin5xx.c b/drivers/spi/spi_bfin5xx.c
index 8822bcc..32d900d 100644
Rather than hardcoding the register sizes/offsets in this file, use the
existing struct in the spi header for reading/writing the hardware.
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c | 208 ++---
1 files changed, 84
: unsupported bits_per_word
So in the transfer logic, have a bits_per_word setting of 0 fall into the
8 bit transfer logic.
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
v2
- wasn't quite as simple as i'd hope ...
drivers/spi/spi_bfin5xx.c |7 ---
1 files changed, 4 insertions
-by: Bob Liu lliu...@gmail.com
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c |8
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/spi/spi_bfin5xx.c b/drivers/spi/spi_bfin5xx.c
index f706dba..ae30fa3 100644
--- a/drivers/spi/spi_bfin5xx.c
On Mon, Jun 6, 2011 at 03:18, Grant Likely wrote:
Sort the SPI makefile and enforce the naming convention spi_*.c for
spi drivers.
spi_topcliff_pch.c uses KBUILD_MODNAME with a few funcs, but perhaps
those are only for /proc/ status tracking ? worth double checking ...
-mike
On Tue, Jun 7, 2011 at 00:48, Feng Tang wrote:
On Tue, 7 Jun 2011 12:09:52 +0800 Grant Likely wrote:
On Mon, Jun 6, 2011 at 8:28 PM, Feng Tang wrote:
On Tue, 7 Jun 2011 00:49:20 +0800 Dirk Brandewie wrote:
My changes are to the designware driver :-) I will get my patches
re-based onto the
On Mon, Mar 28, 2011 at 04:57, Mike Frysinger wrote:
The Blackfin SPORT peripheral is a pretty flexible device. With enough
coaching, we can make it generate SPI compatible waveforms. This is
desirable as the SPORT can run at much higher clock frequencies than the
dedicated on-chip SPI
On Wed, Apr 6, 2011 at 10:49, Vasily Khoruzhick wrote:
drivers/spi/spi_bfin5xx.c | 2 +-
seems to past my smoke test.
Acked-by: Mike Frysinger vap...@gentoo.org
-mike
--
Xperia(TM) PLAY
It's a major breakthrough
On Tue, Feb 15, 2011 at 16:20, Grant Likely wrote:
On Tue, Jan 11, 2011 at 02:22:05PM -0500, Mike Frysinger wrote:
diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig
index 1906840..95ba5fe 100644
--- a/drivers/spi/Kconfig
+++ b/drivers/spi/Kconfig
@@ -66,6 +66,15 @@ config SPI_BFIN
michael.henner...@analog.com
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
v3
- fix up random feedback from Grant
drivers/spi/Kconfig |9 +
drivers/spi/Makefile |1 +
drivers/spi/spi_bfin_sport.c | 952 ++
3 files
On Mon, Jan 31, 2011 at 03:46, Aaron.Wu wrote:
--- a/include/linux/spi/spi.h
+++ b/include/linux/spi/spi.h
@@ -240,9 +246,9 @@ static inline void spi_unregister_driver(struct
spi_driver *sdrv)
*/
struct spi_master {
struct device dev;
-
struct list_head list;
+
From: Bob Liu lliu...@gmail.com
We can do multiples of 8bit transfers when using the hardware CS and a
little bit of magic, so make it work.
Signed-off-by: Bob Liu lliu...@gmail.com
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c | 103
-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/drivers/spi/spi_bfin5xx.c b/drivers/spi/spi_bfin5xx.c
index e8d68b7..a284624 100644
--- a/drivers/spi/spi_bfin5xx.c
+++ b/drivers/spi/spi_bfin5xx.c
@@ -653,6
On Mon, Jan 10, 2011 at 10:22, Grant Likely wrote:
On Mon, Jan 10, 2011 at 09:39:40AM -0500, Mike Frysinger wrote:
On Mon, Nov 1, 2010 at 01:56, Grant Likely wrote:
On Fri, Oct 22, 2010 at 02:53:35AM -0400, Mike Frysinger wrote:
+ /* SPI framework hookup */
+ struct spi_master
michael.henner...@analog.com
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/Kconfig |9 +
drivers/spi/Makefile |1 +
drivers/spi/bfin_sport_spi.c | 949 ++
3 files changed, 959 insertions(+), 0 deletions(-)
create
these known errors in the data interrupt once we've successfully
finished processing all of the pending data.
Signed-off-by: Michael Hennerich michael.henner...@analog.com
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c |9 +
1 files changed, 9 insertions(+), 0
on the fly (like change bit sizes), the setup
function incorrectly fails.
Signed-off-by: Michael Hennerich michael.henner...@analog.com
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c | 13 -
1 files changed, 8 insertions(+), 5 deletions(-)
diff --git a/drivers
On Fri, Oct 22, 2010 at 02:30, Grant Likely wrote:
On Fri, Oct 22, 2010 at 02:01:47AM -0400, Mike Frysinger wrote:
From: Michael Hennerich michael.henner...@analog.com
The gpiolib code does not allow people to do gpio_request() on a GPIO
once it has already been requested. So make sure we
On Fri, Oct 22, 2010 at 03:16, Grant Likely wrote:
On Fri, Oct 22, 2010 at 02:53:35AM -0400, Mike Frysinger wrote:
From: Cliff Cai cliff@analog.com
The Blackfin SPORT peripheral is a pretty flexible device. With enough
coaching, we can make it generate SPI compatible waveforms
On Fri, Oct 22, 2010 at 03:25, Grant Likely wrote:
On Fri, Oct 22, 2010 at 02:41:12AM -0400, Mike Frysinger wrote:
On Fri, Oct 22, 2010 at 02:30, Grant Likely wrote:
On Fri, Oct 22, 2010 at 02:01:47AM -0400, Mike Frysinger wrote:
From: Michael Hennerich michael.henner...@analog.com
On Mon, Oct 18, 2010 at 02:02, Grant Likely wrote:
On Sun, Oct 17, 2010 at 06:59:24PM -0400, Mike Frysinger wrote:
-struct driver_data {
+struct master_data {
Or better yet, to protect against global collisions, bfin5xx_master perhaps?
i'd rather not. there is too much bfin5xx stuff already
On Mon, Oct 18, 2010 at 02:04, Grant Likely wrote:
On Sun, Oct 17, 2010 at 06:59:27PM -0400, Mike Frysinger wrote:
Expand the BIT_CTL defines to use the naming convention of the hardware,
and expand the masks to cover all documented bits.
Signed-off-by: Mike Frysinger vap...@gentoo.org
On Mon, Oct 18, 2010 at 02:12, Grant Likely wrote:
On Sun, Oct 17, 2010 at 06:59:41PM -0400, Mike Frysinger wrote:
From: Michael Hennerich michael.henner...@analog.com
Some systems using this bus sometimes have very basic devices on them
such as regulators. So we need to be loaded even
On Mon, Oct 18, 2010 at 02:20, Grant Likely wrote:
On Mon, Oct 18, 2010 at 02:10:17AM -0400, Mike Frysinger wrote:
On Mon, Oct 18, 2010 at 02:02, Grant Likely wrote:
On Sun, Oct 17, 2010 at 06:59:24PM -0400, Mike Frysinger wrote:
-struct driver_data {
+struct master_data {
Or better
On Mon, Oct 18, 2010 at 02:28, Grant Likely wrote:
On Sun, Oct 17, 2010 at 06:59:13PM -0400, Mike Frysinger wrote:
Quite a bit here, but these patches have been around for a while. The
reason they haven't been pushed up before was that some of the first
changes introduced known bugs
On Mon, Oct 18, 2010 at 02:34, Mike Frysinger wrote:
On Mon, Oct 18, 2010 at 02:28, Grant Likely wrote:
I *might* pick these up for .36, but it is very late for picking up
stuff that hasn't had any linux-next exposure yet.
these issues have been around for so long that missing another 2.6.x
On Mon, Oct 18, 2010 at 02:34, Mike Frysinger wrote:
On Mon, Oct 18, 2010 at 02:28, Grant Likely wrote:
On Sun, Oct 17, 2010 at 06:59:13PM -0400, Mike Frysinger wrote:
Quite a bit here, but these patches have been around for a while. The
reason they haven't been pushed up before was that some
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c |4
1 files changed, 0 insertions(+), 4 deletions(-)
diff --git a/drivers/spi/spi_bfin5xx.c b/drivers/spi/spi_bfin5xx.c
index 4f20b92..61c2ebf 100644
--- a/drivers/spi/spi_bfin5xx.c
+++ b/drivers/spi
it makes no sense to toggle the Flag Enable bit dynamically when we
actually want to control the Flag Value, do this when setting up the device
and then only handle toggling of the CS value during runtime.
Signed-off-by: Barry Song barry.s...@analog.com
Signed-off-by: Mike Frysinger vap...@gentoo.org
: http://blackfin.uclinux.org/gf/tracker/5630
Signed-off-by: Wolfgang Muees wolfgang.m...@auerswald.de
Signed-off-by: Michael Hennerich michael.henner...@analog.com
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c |6 ++
1 files changed, 6 insertions(+), 0
Reported-by: David Brownell dbrown...@users.sourceforge.net
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c |8 ++--
1 files changed, 2 insertions(+), 6 deletions(-)
diff --git a/drivers/spi/spi_bfin5xx.c b/drivers/spi/spi_bfin5xx.c
index 5e07f49..045c82a
Liu (1):
Blackfin SPI: check per-transfer bits_per_word
Daniel Mack (1):
Blackfin SPI: fix resources leakage
Michael Hennerich (1):
Blackfin SPI: init early
Mike Frysinger (13):
Blackfin SPI: drop custom cs_change_per_word support
Blackfin SPI: punt useless null read/write funcs
Rather than having to look up the same 3 sets of functions at the same
time, just use an ops structure so we only need to set one pointer.
Reported-by: David Brownell dbrown...@users.sourceforge.net
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c | 58
The chip ops should always be initialized, so having null fallback
functions are useless.
Reported-by: David Brownell dbrown...@users.sourceforge.net
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c | 47
1 files changed
yi...@analog.com
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c | 235 +++--
1 files changed, 185 insertions(+), 50 deletions(-)
diff --git a/drivers/spi/spi_bfin5xx.c b/drivers/spi/spi_bfin5xx.c
index b835254..3736c35 100644
The current structure names are a bit confusing as to what they represent,
so use better names.
Reported-by: David Brownell dbrown...@users.sourceforge.net
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c | 100 ++--
1 files
The driver that we based ours on uses a little extra memory behind the
normal driver state, but we don't. So drop this useless bit of memory.
Reported-by: David Brownell dbrown...@users.sourceforge.net
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c |4 ++--
1
This lets us push the short SPI MMR bit names out of the global namespace.
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c | 21 +++--
1 files changed, 11 insertions(+), 10 deletions(-)
diff --git a/drivers/spi/spi_bfin5xx.c b/drivers/spi
...@analog.com
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c | 120 -
1 files changed, 75 insertions(+), 45 deletions(-)
diff --git a/drivers/spi/spi_bfin5xx.c b/drivers/spi/spi_bfin5xx.c
index 10a6dc3..4f20b92 100644
Expand the BIT_CTL defines to use the naming convention of the hardware,
and expand the masks to cover all documented bits.
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
arch/blackfin/include/asm/bfin5xx_spi.h | 11 +++
1 files changed, 7 insertions(+), 4 deletions(-)
diff --git
No point in creating our own version of true/false defines when there is
already a standard stdbool available to us.
Reported-by: David Brownell dbrown...@users.sourceforge.net
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c | 18 --
1 files
From: Barry Song barry.s...@analog.com
We can't rely on the SPI_CTL/SPI_FLG registers retaining their state when
suspending, so save/restore their entire values.
Signed-off-by: Barry Song barry.s...@analog.com
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c | 17
the hardware CS. If it's
above, we treat it as a GPIO CS. This keeps the CS unique as seen by
the common code and prevents conflicts.
Signed-off-by: Barry Song barry.s...@analog.com
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
arch/blackfin/include/asm/bfin5xx_spi.h |3 +-
drivers/spi
.
Signed-off-by: Barry Song barry.s...@analog.com
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/drivers/spi/spi_bfin5xx.c b/drivers/spi/spi_bfin5xx.c
index c23ac3b..54f9dbb 100644
--- a/drivers/spi
From: Yi Li yi...@analog.com
Using disable_irq() on the IRQ whose handler we are currently executing in
can easily lead to a hang. So use the nosync variant here.
Signed-off-by: Yi Li yi...@analog.com
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c |2 +-
1
This reduces duplication between the setup/transfer functions and keeps
values cached during setup from overriding values changed on a transfer
basis (like bits_per_word).
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c | 57
From: Barry Song barry.s...@analog.com
Using disable_irq() on the IRQ whose handler we are currently executing in
can easily lead to a hang. So use the nosync variant here.
Signed-off-by: Barry Song barry.s...@analog.com
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi
...@students.olin.edu
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c | 12 +---
1 files changed, 9 insertions(+), 3 deletions(-)
diff --git a/drivers/spi/spi_bfin5xx.c b/drivers/spi/spi_bfin5xx.c
index 830c705..376f2f0 100644
--- a/drivers/spi/spi_bfin5xx.c
+++ b/drivers
Who knows what people will try!
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c |4
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/drivers/spi/spi_bfin5xx.c b/drivers/spi/spi_bfin5xx.c
index 18b313a..a91e556 100644
--- a/drivers/spi
From: Rob Maris maris@vdi.de
Signed-off-by: Rob Maris maris@vdi.de
Signed-off-by: Barry Song barry.s...@analog.com
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/spi
From: Bob Liu lliu...@gmail.com
Currently, if the bits_per_word when doing a transfer is not 8bits, we
always treat it as 16bits when we should actually be returning an error.
Signed-off-by: Bob Liu lliu...@gmail.com
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c
-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c |9 +++--
1 files changed, 7 insertions(+), 2 deletions(-)
diff --git a/drivers/spi/spi_bfin5xx.c b/drivers/spi/spi_bfin5xx.c
index 7882afc..21de860 100644
--- a/drivers/spi/spi_bfin5xx.c
+++ b/drivers/spi
While combining things, also switch to the proper SPI bit define names.
This lets us punt the rarely used SPI defines.
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
arch/blackfin/include/asm/bfin5xx_spi.h | 68 ---
drivers/spi/spi_bfin5xx.c
From: Rob Maris maris@vdi.de
Signed-off-by: Rob Maris maris@vdi.de
Signed-off-by: Barry Song barry.s...@analog.com
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c |3 +--
1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/drivers/spi
On Wed, Feb 17, 2010 at 13:17, Ernst Schwab wrote:
The current runtime API looks like this:
spi_async(struct spi_device*, struct spi_message*);
spi_sync(struct spi_device*, struct spi_message*);
The API needs to be extended to this:
spi_async(struct spi_device*, struct spi_message*)
On Wed, Feb 17, 2010 at 15:06, Grant Likely wrote:
On Wed, Feb 17, 2010 at 12:03 PM, Mike Frysinger wrote:
On Wed, Feb 17, 2010 at 13:17, Ernst Schwab wrote:
The current runtime API looks like this:
spi_async(struct spi_device*, struct spi_message*);
spi_sync(struct spi_device*, struct
the SPI bus with SPI MMC card.
This patch introduces 2 APIs for SPI bus locking operation.
Signed-off-by: Yi Li yi...@analog.com
Signed-off-by: Bryan Wu coolo...@kernel.org
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
Andrew: we've posted these in the past with no response. could you pick
On Tue, Feb 16, 2010 at 19:21, Ernst Schwab wrote:
Mike Frysinger wrote:
this is what i proposed last time, but we havent gotten around to
implementing it:
- add a new spi_master flag to spi.h like SPI_MASTER_HALF_DUPLEX --
SPI_MASTER_LOCK_BUS
- have the mmc_spi code check that bit
On Tue, Feb 16, 2010 at 22:48, Grant Likely wrote:
On Tue, Feb 16, 2010 at 5:07 PM, Mike Frysinger wrote:
On Tue, Feb 16, 2010 at 15:43, Grant Likely wrote:
On Tue, Feb 16, 2010 at 12:57 PM, Ernst Schwab wrote:
From: Yi Li yi...@analog.com
For some MMC cards over SPI bus, it needs to lock
On Sat, Feb 13, 2010 at 11:51, Jonathan Cameron wrote:
http://blackfin.uclinux.org/gf/project/linux-kernel/scmsvn/
people who like git can always use:
http://blackfin.uclinux.org/git/?p=readonly-mirrors/linux-kernel.git;a=summary
-mike
On Tue, Jan 19, 2010 at 11:12, Grant Likely wrote:
On Fri, Dec 18, 2009 at 10:34 AM, Mike Frysinger wrote:
On Fri, Dec 18, 2009 at 08:14, Mike Frysinger wrote:
On Fri, Dec 18, 2009 at 02:29, James Getzendanner wrote:
From: Andy Getzendanner james.getzendan...@students.olin.edu
The patch also
On Wed, Dec 16, 2009 at 17:49, H Hartley Sweeten wrote:
On Thursday, September 17, 2009 3:03 PM, Mike Frysinger wrote:
From: Yi Li yi...@analog.com
For some MMC cards over SPI bus, it needs to lock the SPI bus for its own
use. The SPI transfer must not be interrupted by other SPI devices
On Thu, Dec 10, 2009 at 21:25, David Brownell wrote:
On Monday 12 October 2009, Mike Frysinger wrote:
i was browsing the change history of spidev when i noticed the change:
aaacf4bb51b243875b203e6ff73b5047636b4efa spi: avoid spidev crash when
device is removed
which seems to be due
2009/11/24 Uwe Kleine-König
The function spidev_remove is used only wrapped by __devexit_p so define
it using __devexit.
this doesnt go far enough to address actual section mismatch issues in
the driver. i posted a more complete one here:
http://lkml.org/lkml/2009/10/13/76
and akpm has picked
i was browsing the change history of spidev when i noticed the change:
aaacf4bb51b243875b203e6ff73b5047636b4efa spi: avoid spidev crash when
device is removed
which seems to be due to this change:
3d81252ddb10f63ae4db713d9b32faabe641b850 device create: spi: convert
device_create to
The driver already uses __devexit_p() in the structure, but looks like
actual __dev{init,exit} markings were forgotten.
The spidev_spi driver also needs renaming to include a _driver suffix
to avoid section mismatch warnings.
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi
On Thu, Sep 17, 2009 at 19:47, Mike Frysinger vap...@gentoo.org wrote:
From: Yi Li yi...@analog.com
Make sure we don't leak peripheral/gpio resources when an error occurs
during setup, and make sure we don't call kfree() twice on the same
object. Also add/fix some error messages
On Thu, Sep 17, 2009 at 18:03, Mike Frysinger wrote:
--- a/drivers/spi/spi.c
+++ b/drivers/spi/spi.c
@@ -653,6 +653,54 @@ static void spi_complete(void *arg)
+int spi_lock_bus(struct spi_device *spi)
+{
+ if (spi-master-lock_bus)
+ return spi-master-lock_bus(spi
On Mon, Sep 21, 2009 at 02:33, Li Yi wrote:
On Fri, 2009-09-18 at 14:29 -0700, Andrew Morton wrote:
On Thu, 17 Sep 2009 18:03:16 -0400
Mike Frysinger vap...@gentoo.org wrote:
From: Yi Li yi...@analog.com
For some MMC cards over SPI bus, it needs to lock the SPI bus for its own
use
On Fri, Sep 18, 2009 at 19:00, H Hartley Sweeten wrote:
On Thursday, September 17, 2009 3:54 PM, Mike Frysinger wrote:
I assume the spi master driver must supply the {lock/unlock}_bus methods
to properly support the locking.
currently, yes. a common solution would be nice. ideas/patches
yi...@analog.com
Signed-off-by: Bryan Wu coolo...@kernel.org
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
Andrew: we've posted these in the past with no response. could you pick
them up please ?
drivers/spi/spi.c | 48 +++
include
On Thu, Sep 17, 2009 at 18:45, H Hartley Sweeten wrote:
On Thursday, September 17, 2009 3:03 PM, Mike Frysinger wrote:
From: Yi Li yi...@analog.com
For some MMC cards over SPI bus, it needs to lock the SPI bus for its own
use. The SPI transfer must not be interrupted by other SPI devices
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c |4
1 files changed, 0 insertions(+), 4 deletions(-)
diff --git a/drivers/spi/spi_bfin5xx.c b/drivers/spi/spi_bfin5xx.c
index 8ec4969..5591bce 100644
--- a/drivers/spi/spi_bfin5xx.c
+++ b/drivers/spi
From: Yi Li yi...@analog.com
Make sure we don't leak peripheral/gpio resources when an error occurs
during setup, and make sure we don't call kfree() twice on the same
object. Also add/fix some error messages in the process.
Signed-off-by: Yi Li yi...@analog.com
Signed-off-by: Mike Frysinger
yi...@analog.com
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c | 235 +++--
1 files changed, 185 insertions(+), 50 deletions(-)
diff --git a/drivers/spi/spi_bfin5xx.c b/drivers/spi/spi_bfin5xx.c
index 3249c5d..a8fc922 100644
From: Daniel Mack dan...@caiaq.de
Re-order setup() a bit so we don't leak memory/dma/gpio resources upon
errors.
Signed-off-by: Daniel Mack dan...@caiaq.de
Signed-off-by: Bryan Wu coolo...@kernel.org
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
drivers/spi/spi_bfin5xx.c | 99
From: Wolfgang Muees wolfgang.m...@auerswald.de
We should make sure the SPI controller is in a sane state in case the
boot loader left it in a crappy state. Such as DMA pending which causes
interrupts to fire on us.
Signed-off-by: Wolfgang Muees wolfgang.m...@auerswald.de
Signed-off-by: Mike
functions to provide the
required exclusivity.
Signed-off-by: Yi Li yi...@analog.com
Signed-off-by: Mike Frysinger vap...@gentoo.org
---
v2
- drop now unused maybe_count_child as pointed out by H Hartley Sweeten
drivers/mmc/host/mmc_spi.c | 41 ++---
1
On Wed, Sep 2, 2009 at 05:50, Robin Getz wrote:
On Tue 1 Sep 2009 23:17, Song, Barry pondered:
No. I can't agree other coding style issues you said. I will keep my
original codes for these issues except adding a BUS_SPI.
style is different than performance
Did you think about Mike's
On Mon, Aug 31, 2009 at 23:55, Barry Song wrote:
David,
The patch[1/2] adds a file with platform_data definition for AD714x in
include/linux/spi,
which describes the components layout on special target boards and need be
filled in
arch/mach.
i dont think this needs to split merged between
On Mon, Aug 31, 2009 at 23:55, Barry Song21cn...@gmail.com wrote:
--- /dev/null
+++ b/include/linux/spi/ad714x.h
@@ -0,0 +1,60 @@
+/*
+ * include/linux/ad714x.h
+ *
+ * AD714x is very flexible, it can be used as buttons, scrollwheel,
+ * slider, touchpad at the same time. That depends on
On Mon, Aug 31, 2009 at 23:55, Barry Song wrote:
ad7142/ad7147 are Programmable Controllers for Capacitance Touch Sensors.
The chips don't restrict the specific usage, and it can be used as button/
slider/scrollwheel/touchpad etc. depending on the hardware connection.
One special target board
From: Daniel Mack dan...@caiaq.de
Re-order setup() a bit so we don't leak memory/dma/gpio resources upon
errors.
Signed-off-by: Daniel Mack dan...@caiaq.de
Signed-off-by: Mike Frysinger vap...@gentoo.org
Signed-off-by: Bryan Wu coolo...@kernel.org
---
drivers/spi/spi_bfin5xx.c | 101
On Thu, Feb 5, 2009 at 17:12, Andrew Morton wrote:
On Thu, 5 Feb 2009 18:07:31 +0800 Bryan Wu wrote:
From: Mike Frysinger vapier@gmail.com
fix NULL pointer crash when cleaning up from invalid platform resources
Signed-off-by: Mike Frysinger vapier@gmail.com
Signed-off-by: Bryan Wu
On Tue, Feb 26, 2008 at 11:46 PM, David Brownell [EMAIL PROTECTED] wrote:
On Tuesday 26 February 2008, Bryan Wu wrote:
On Tue, Feb 26, 2008 at 11:16 PM, Hans Eklund [EMAIL PROTECTED] wrote:
--
/* NOTES:
*
* ...
*
* - MMC depends on a different chipselect management
On 10/31/07, David Brownell [EMAIL PROTECTED] wrote:
(ISTR the M25P16 in $SUBJECT has two read commands, one of
which is only usable at clock rates below 33 MHz or so, but
most other commands can work above that speed just fine.)
pretty much all SPI flashes have two read commands (slow and
On 10/30/07, David Brownell [EMAIL PROTECTED] wrote:
And also, pay closer attention to when you may be making
changes that make device drivers work differently over
your spi_master controller driver than anyone elses ...
such platform-specific behaviors are undesirable, and when
they go
90 matches
Mail list logo