----- Original Message -----
From: "Kevin Hilman" <khil...@deeprootsystems.com>
To: "Hemanth V" <heman...@ti.com>
Cc: <linux-omap@vger.kernel.org>; "Tony Lindgren" <t...@atomide.com>; "David
Brownell" <davi...@pacbell.net>
Sent: Thursday, June 11, 2009 8:17 PM
Subject: Re: [PATCH 1/2] McSPI Slave and DMA,FIFO support
"Hemanth V" <heman...@ti.com> writes:
----- Original Message -----
From: "Hemanth V" <heman...@ti.com>
To: "Tony Lindgren" <t...@atomide.com>
Cc: <linux-omap@vger.kernel.org>
Sent: Friday, June 05, 2009 3:28 PM
Subject: Re: [PATCH 1/2] McSPI Slave and DMA,FIFO support
----- Original Message -----
From: "Tony Lindgren" <t...@atomide.com>
To: "Hemanth V" <heman...@ti.com>
Cc: <linux-omap@vger.kernel.org>
Sent: Tuesday, June 02, 2009 11:36 PM
Subject: Re: [PATCH 1/2] McSPI Slave and DMA,FIFO support
Hi,
Sorry for the delay in replying, few comments below.
* Hemanth V <heman...@ti.com> [090519 22:57]:
This patch adds support for McSPI slave and FIFO. DMA and FIFO
could be enabled together for better throughput. Platform config
parameters have been added to enable these features on any particular
McSPI controller.
FIFO can be enabled by defining fifo_depth parameter. fifo_depth needs
to be a multiple of buffer size that is used for read/write.
These features are useful when you have high throughput devices
like WLAN or Modem connected over SPI.
Signed-off-by: Hemanth V <heman...@ti.com>
arch/arm/mach-omap2/devices.c | 5
arch/arm/plat-omap/include/mach/mcspi.h | 16 +
drivers/spi/omap2_mcspi.c | 343
++++++++++++++++++++++++++++----
3 files changed, 325 insertions(+), 39 deletions(-)
As this is mostly drivers/spi/omap2_mcspi.c, this patch should get
merged via:
$ grep -A7 "SPI SUBSYSTEM" MAINTAINERS
SPI SUBSYSTEM
P: David Brownell
M: dbrown...@users.sourceforge.net
L: spi-devel-gene...@lists.sourceforge.net
S: Maintained
F: Documentation/spi/
F: drivers/spi/
F: include/linux/spi/
Please keep linux-omap list Cc'd too so everybody can follow
the progress.
Tony, is this list active. The archives seem to be flooded with spam
mails
http://sourceforge.net/mailarchive/forum.php?forum_name=spi-devel-general
Kevin, Can u suggest what is to be done in this situation. spi-devel
list doesnot seem
to be active and Tony is not willing to merge this patch. Should I
send this to LKML
First, you haven't addressed any of the comments made on the list
about your series.
Tony isn't merging this patch because most of it should go via the SPI
subsystem.
A Santosh suggested, you need to break this up into parts that are
OMAP specific (arch/arm/*) and parts that go via the SPI subsystem
(drivers/spi/*.) If doing this breaks your series, then fix it
But then that would cause a problem, we will not be able to
use arch/arm/plat-omap/include/mach/mcspi.h and will need to redefine those
variables in omap2_mcspi.c which obviously is not the preferred way.
because if it breaks compile this way, then upstream maintainers will
surely hit the same errors and complain.
David Brownell is the SPI maintainer. I suggest you send to him,
CC'ing linux-omap.
Kevin
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
