linux-spi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/5] spi-imx: some updates
@ 2009-12-08 21:12 Uwe Kleine-König
       [not found] ` <20091208211258.GA13706-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
  0 siblings, 1 reply; 12+ messages in thread
From: Uwe Kleine-König @ 2009-12-08 21:12 UTC (permalink / raw)
  To: spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f
  Cc: David Brownell, linux-kernel-u79uwXL29TY76Z2rM5mHXA, Daniel Mack,
	Andrew Morton, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r

Hello,

this series depends on "spi: drain MXC SPI transfer buffer when probing
device" which is currently in -mm and also included in my spi-imx branch
(see below).  IMHO at least my first patch should go into .33 as it
fixes an oops.

I'll post the patches for review as a reply to this mail.

Best regards
Uwe

The following changes since commit 6035ccd8e9e40bb654fbfdef325902ab531679a5:
  Linus Torvalds (1):
        Merge branch 'for-2.6.33' of git://git.kernel.dk/linux-2.6-block

are available in the git repository at:

  git://git.pengutronix.de/git/ukl/linux-2.6.git spi-imx

Daniel Mack (1):
      spi: drain MXC SPI transfer buffer when probing device

Sascha Hauer (1):
      spi-imx: Add mx25 support

Uwe Kleine-König (4):
      spi-imx: use platform_driver_probe as probe lives in .init.text
      spi-imx: correct check for platform_get_irq failing
      spi-imx: use positive logic to distinguish cpu variants
      spi-imx: don't access struct device directly but use dev_get_platdata

 drivers/spi/spi_imx.c |   23 +++++++++++++++--------
 1 files changed, 15 insertions(+), 8 deletions(-)
-- 
Pengutronix e.K.                              | Uwe Kleine-König            |
Industrial Linux Solutions                    | http://www.pengutronix.de/  |

------------------------------------------------------------------------------
Return on Information:
Google Enterprise Search pays you back
Get the facts.
http://p.sf.net/sfu/google-dev2dev

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH 1/5] spi-imx: use platform_driver_probe as probe lives in .init.text
       [not found] ` <20091208211258.GA13706-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
@ 2009-12-08 21:14   ` Uwe Kleine-König
       [not found]     ` <1260306850-14945-1-git-send-email-u.kleine-koenig-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
  2009-12-08 23:32   ` [PATCH 0/5] spi-imx: some updates Andrew Morton
  1 sibling, 1 reply; 12+ messages in thread
From: Uwe Kleine-König @ 2009-12-08 21:14 UTC (permalink / raw)
  To: spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f
  Cc: David Brownell, linux-kernel-u79uwXL29TY76Z2rM5mHXA, Daniel Mack,
	Andrew Morton, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r

Using platform_driver_register with a probe function defined using
__init is wrong.

This fixes an oops after:

	cd /sys/bus/platform/drivers/spi_imx
	echo -n spi_imx.0 > unbind
	echo -n spi_imx.0 > bind

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Cc: David Brownell <dbrownell@users.sourceforge.net>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Sascha Hauer <s.hauer@pengutronix.de>
---
 drivers/spi/spi_imx.c |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)

diff --git a/drivers/spi/spi_imx.c b/drivers/spi/spi_imx.c
index 1b17f44..c83ca05 100644
--- a/drivers/spi/spi_imx.c
+++ b/drivers/spi/spi_imx.c
@@ -666,13 +666,12 @@ static struct platform_driver spi_imx_driver = {
 		   .name = DRIVER_NAME,
 		   .owner = THIS_MODULE,
 		   },
-	.probe = spi_imx_probe,
 	.remove = __exit_p(spi_imx_remove),
 };
 
 static int __init spi_imx_init(void)
 {
-	return platform_driver_register(&spi_imx_driver);
+	return platform_driver_probe(&spi_imx_driver, spi_imx_probe);
 }
 
 static void __exit spi_imx_exit(void)
-- 
1.6.5.2


------------------------------------------------------------------------------
Return on Information:
Google Enterprise Search pays you back
Get the facts.
http://p.sf.net/sfu/google-dev2dev
_______________________________________________
spi-devel-general mailing list
spi-devel-general@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/spi-devel-general

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH 2/5] spi-imx: correct check for platform_get_irq failing
       [not found]     ` <1260306850-14945-1-git-send-email-u.kleine-koenig-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
@ 2009-12-08 21:14       ` Uwe Kleine-König
       [not found]         ` <1260306850-14945-2-git-send-email-u.kleine-koenig-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
  0 siblings, 1 reply; 12+ messages in thread
From: Uwe Kleine-König @ 2009-12-08 21:14 UTC (permalink / raw)
  To: spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f
  Cc: David Brownell, linux-kernel-u79uwXL29TY76Z2rM5mHXA, Daniel Mack,
	Andrew Morton, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r

platform_get_irq returns -ENXIO if there is no entry.  So don't check
return value being nonzero but greater or equal to zero.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Cc: David Brownell <dbrownell@users.sourceforge.net>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Sascha Hauer <s.hauer@pengutronix.de>
---
 drivers/spi/spi_imx.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/spi/spi_imx.c b/drivers/spi/spi_imx.c
index c83ca05..d8d9a35 100644
--- a/drivers/spi/spi_imx.c
+++ b/drivers/spi/spi_imx.c
@@ -554,7 +554,7 @@ static int __init spi_imx_probe(struct platform_device *pdev)
 	}
 
 	spi_imx->irq = platform_get_irq(pdev, 0);
-	if (!spi_imx->irq) {
+	if (spi_imx->irq < 0) {
 		ret = -EINVAL;
 		goto out_iounmap;
 	}
-- 
1.6.5.2


------------------------------------------------------------------------------
Return on Information:
Google Enterprise Search pays you back
Get the facts.
http://p.sf.net/sfu/google-dev2dev
_______________________________________________
spi-devel-general mailing list
spi-devel-general@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/spi-devel-general

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH 3/5] spi-imx: use positive logic to distinguish cpu variants
       [not found]         ` <1260306850-14945-2-git-send-email-u.kleine-koenig-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
@ 2009-12-08 21:14           ` Uwe Kleine-König
       [not found]             ` <1260306850-14945-3-git-send-email-u.kleine-koenig-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
  0 siblings, 1 reply; 12+ messages in thread
From: Uwe Kleine-König @ 2009-12-08 21:14 UTC (permalink / raw)
  To: spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f
  Cc: David Brownell, linux-kernel-u79uwXL29TY76Z2rM5mHXA, Daniel Mack,
	Andrew Morton, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r

This is much safer when support for new variants is added.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Cc: David Brownell <dbrownell@users.sourceforge.net>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Sascha Hauer <s.hauer@pengutronix.de>
---
 drivers/spi/spi_imx.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/spi/spi_imx.c b/drivers/spi/spi_imx.c
index d8d9a35..0ccbd1c 100644
--- a/drivers/spi/spi_imx.c
+++ b/drivers/spi/spi_imx.c
@@ -593,7 +593,7 @@ static int __init spi_imx_probe(struct platform_device *pdev)
 	clk_enable(spi_imx->clk);
 	spi_imx->spi_clk = clk_get_rate(spi_imx->clk);
 
-	if (!cpu_is_mx31() || !cpu_is_mx35())
+	if (cpu_is_mx1() || cpu_is_mx21() || cpu_is_mx27())
 		writel(1, spi_imx->base + MXC_RESET);
 
 	/* drain receive buffer */
-- 
1.6.5.2


------------------------------------------------------------------------------
Return on Information:
Google Enterprise Search pays you back
Get the facts.
http://p.sf.net/sfu/google-dev2dev
_______________________________________________
spi-devel-general mailing list
spi-devel-general@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/spi-devel-general

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH 4/5] spi-imx: Add mx25 support
       [not found]             ` <1260306850-14945-3-git-send-email-u.kleine-koenig-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
@ 2009-12-08 21:14               ` Uwe Kleine-König
       [not found]                 ` <1260306850-14945-4-git-send-email-u.kleine-koenig-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
  0 siblings, 1 reply; 12+ messages in thread
From: Uwe Kleine-König @ 2009-12-08 21:14 UTC (permalink / raw)
  To: spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f
  Cc: David Brownell, linux-kernel-u79uwXL29TY76Z2rM5mHXA, Daniel Mack,
	Andrew Morton, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r

From: Sascha Hauer <s.hauer@pengutronix.de>

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Cc: David Brownell <dbrownell@users.sourceforge.net>
Cc: Andrew Morton <akpm@linux-foundation.org>
---
 drivers/spi/spi_imx.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/spi/spi_imx.c b/drivers/spi/spi_imx.c
index 0ccbd1c..59b3a30 100644
--- a/drivers/spi/spi_imx.c
+++ b/drivers/spi/spi_imx.c
@@ -208,7 +208,7 @@ static int mx31_config(struct spi_imx_data *spi_imx,
 
 	if (cpu_is_mx31())
 		reg |= (config->bpw - 1) << MX31_CSPICTRL_BC_SHIFT;
-	else if (cpu_is_mx35()) {
+	else if (cpu_is_mx25() || cpu_is_mx35()) {
 		reg |= (config->bpw - 1) << MX35_CSPICTRL_BL_SHIFT;
 		reg |= MX31_CSPICTRL_SSCTL;
 	}
@@ -222,7 +222,7 @@ static int mx31_config(struct spi_imx_data *spi_imx,
 	if (config->cs < 0) {
 		if (cpu_is_mx31())
 			reg |= (config->cs + 32) << MX31_CSPICTRL_CS_SHIFT;
-		else if (cpu_is_mx35())
+		else if (cpu_is_mx25() || cpu_is_mx35())
 			reg |= (config->cs + 32) << MX35_CSPICTRL_CS_SHIFT;
 	}
 
@@ -565,7 +565,7 @@ static int __init spi_imx_probe(struct platform_device *pdev)
 		goto out_iounmap;
 	}
 
-	if (cpu_is_mx31() || cpu_is_mx35()) {
+	if (cpu_is_mx25() || cpu_is_mx31() || cpu_is_mx35()) {
 		spi_imx->intctrl = mx31_intctrl;
 		spi_imx->config = mx31_config;
 		spi_imx->trigger = mx31_trigger;
@@ -597,7 +597,7 @@ static int __init spi_imx_probe(struct platform_device *pdev)
 		writel(1, spi_imx->base + MXC_RESET);
 
 	/* drain receive buffer */
-	if (cpu_is_mx31() || cpu_is_mx35())
+	if (cpu_is_mx25() || cpu_is_mx31() || cpu_is_mx35())
 		while (readl(spi_imx->base + MX3_CSPISTAT) & MX3_CSPISTAT_RR)
 			readl(spi_imx->base + MXC_CSPIRXDATA);
 
-- 
1.6.5.2


------------------------------------------------------------------------------
Return on Information:
Google Enterprise Search pays you back
Get the facts.
http://p.sf.net/sfu/google-dev2dev
_______________________________________________
spi-devel-general mailing list
spi-devel-general@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/spi-devel-general

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH 5/5] spi-imx: don't access struct device directly but use dev_get_platdata
       [not found]                 ` <1260306850-14945-4-git-send-email-u.kleine-koenig-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
@ 2009-12-08 21:14                   ` Uwe Kleine-König
  0 siblings, 0 replies; 12+ messages in thread
From: Uwe Kleine-König @ 2009-12-08 21:14 UTC (permalink / raw)
  To: spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f
  Cc: David Brownell, linux-kernel-u79uwXL29TY76Z2rM5mHXA, Daniel Mack,
	Andrew Morton, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r

Also there is no casting needed to assign a void pointer.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Cc: David Brownell <dbrownell@users.sourceforge.net>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Sascha Hauer <s.hauer@pengutronix.de>
---
 drivers/spi/spi_imx.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/spi/spi_imx.c b/drivers/spi/spi_imx.c
index 59b3a30..ea6d701 100644
--- a/drivers/spi/spi_imx.c
+++ b/drivers/spi/spi_imx.c
@@ -492,7 +492,7 @@ static int __init spi_imx_probe(struct platform_device *pdev)
 	struct resource *res;
 	int i, ret;
 
-	mxc_platform_info = (struct spi_imx_master *)pdev->dev.platform_data;
+	mxc_platform_info = dev_get_platdata(&pdev->dev);
 	if (!mxc_platform_info) {
 		dev_err(&pdev->dev, "can't get the platform data\n");
 		return -EINVAL;
-- 
1.6.5.2


------------------------------------------------------------------------------
Return on Information:
Google Enterprise Search pays you back
Get the facts.
http://p.sf.net/sfu/google-dev2dev
_______________________________________________
spi-devel-general mailing list
spi-devel-general@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/spi-devel-general

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 0/5] spi-imx: some updates
       [not found] ` <20091208211258.GA13706-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
  2009-12-08 21:14   ` [PATCH 1/5] spi-imx: use platform_driver_probe as probe lives in .init.text Uwe Kleine-König
@ 2009-12-08 23:32   ` Andrew Morton
  2009-12-09  0:25     ` Grant Likely
  1 sibling, 1 reply; 12+ messages in thread
From: Andrew Morton @ 2009-12-08 23:32 UTC (permalink / raw)
  To: Uwe Kleine-König
  Cc: David Brownell, linux-kernel-u79uwXL29TY76Z2rM5mHXA, Daniel Mack,
	spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r

On Tue, 8 Dec 2009 22:12:58 +0100
Uwe Kleine-K__nig <u.kleine-koenig-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org> wrote:

> this series depends on "spi: drain MXC SPI transfer buffer when probing
> device" which is currently in -mm and also included in my spi-imx branch
> (see below).  IMHO at least my first patch should go into .33 as it
> fixes an oops.

I queued all five for 2.6.33.


Once I've merged up the current -mm patch queue:

atmel_spi-fix-dma-addr-calculation-for-len-buffer_size.patch
spi-controller-driver-for-designware-spi-core.patch
#
spi-add-pci-interface-driver-for-designware-spi-core.patch
spi-add-pci-interface-driver-for-designware-spi-core-fix.patch
spi-controller-driver-for-designware-spi-core-fix.patch
#
spidev-add-proper-section-markers.patch
spidev-use-declare_bitmap-instead-of-declaring-the-array.patch
spidev-use-declare_bitmap-instead-of-declaring-the-array-checkpatch-fixes.patch
spi-drain-mxc-spi-transfer-buffer-when-probing-device.patch
spi_s3c24xx-add-fiq-pseudo-dma-support.patch
spi-superh-msiof-spi-master-driver.patch
spi-superh-msiof-spi-master-driver-update.patch
spi-nuc900-add-arm-driver-support-for-nuc900.patch
spi-fix-spidev-compilation-failure-when-verbose-is-defined.patch
#
spi-imx-use-platform_driver_probe-as-probe-lives-in-inittext.patch
spi-imx-correct-check-for-platform_get_irq-failing.patch
spi-imx-use-positive-logic-to-distinguish-cpu-variants.patch
spi-imx-add-mx25-support.patch
spi-imx-dont-access-struct-device-directly-but-use-dev_get_platdata.patch


I shall stop sending SPI patches to Linus.  I'll send any which I
accumulate to Grant instead.



------------------------------------------------------------------------------
Return on Information:
Google Enterprise Search pays you back
Get the facts.
http://p.sf.net/sfu/google-dev2dev

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 0/5] spi-imx: some updates
  2009-12-08 23:32   ` [PATCH 0/5] spi-imx: some updates Andrew Morton
@ 2009-12-09  0:25     ` Grant Likely
       [not found]       ` <fa686aa40912081625y3750a7fegbac38926182e38c6-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
  2009-12-09  0:47       ` [spi-devel-general] " jassi brar
  0 siblings, 2 replies; 12+ messages in thread
From: Grant Likely @ 2009-12-09  0:25 UTC (permalink / raw)
  To: Andrew Morton
  Cc: Uwe Kleine-König, spi-devel-general, David Brownell,
	Sascha Hauer, Daniel Mack, linux-arm-kernel, linux-kernel

2009/12/8 Andrew Morton <akpm@linux-foundation.org>:
> On Tue, 8 Dec 2009 22:12:58 +0100
> Uwe Kleine-K__nig <u.kleine-koenig@pengutronix.de> wrote:
>
>> this series depends on "spi: drain MXC SPI transfer buffer when probing
>> device" which is currently in -mm and also included in my spi-imx branch
>> (see below).  IMHO at least my first patch should go into .33 as it
>> fixes an oops.
>
> I queued all five for 2.6.33.

I had also picked up "drain MXC SPI transfer..." into my next-spi tree
earlier today.  After a few days I'll send Linus a pull request.  I
just need to make sure that Ben's tree gets pulled first because I've
got other stuff in there depending on it.

> Once I've merged up the current -mm patch queue:
[... list of patches snipped ...]
> I shall stop sending SPI patches to Linus.  I'll send any which I
> accumulate to Grant instead.

I'm new to the job, so I'm being cautions about which things I'm pick
up.  However, I'll happily merge anything that Andrew says is okay.
Andrew, when in the merge window do you typically send out patch
queues?

g.

-- 
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 0/5] spi-imx: some updates
       [not found]       ` <fa686aa40912081625y3750a7fegbac38926182e38c6-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
@ 2009-12-09  0:35         ` Andrew Morton
  2009-12-09  6:15           ` Grant Likely
  0 siblings, 1 reply; 12+ messages in thread
From: Andrew Morton @ 2009-12-09  0:35 UTC (permalink / raw)
  To: Grant Likely
  Cc: David Brownell, linux-kernel-u79uwXL29TY76Z2rM5mHXA, Daniel Mack,
	Uwe Kleine-König,
	spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r

On Tue, 8 Dec 2009 17:25:59 -0700
Grant Likely <grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org> wrote:

> I'm new to the job, so I'm being cautions about which things I'm pick
> up.  However, I'll happily merge anything that Andrew says is okay.

oop.  If I'm going to send a patch to a subsystem maintainer I'll usually give
it less attention than the ones I plan to merge myself.

Oh well, at least this way people will get their patches merged easily ;)

> Andrew, when in the merge window do you typically send out patch
> queues?

Late.  I stage the -mm patches after linux-next so I spend most of the
merge window waiting for people to get their stuff into mainline.

Often there are stragglers who I need to route around.  It's usually
not a significant problem though.

------------------------------------------------------------------------------
Return on Information:
Google Enterprise Search pays you back
Get the facts.
http://p.sf.net/sfu/google-dev2dev

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [spi-devel-general] [PATCH 0/5] spi-imx: some updates
  2009-12-09  0:25     ` Grant Likely
       [not found]       ` <fa686aa40912081625y3750a7fegbac38926182e38c6-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
@ 2009-12-09  0:47       ` jassi brar
  2009-12-09  1:50         ` Grant Likely
  1 sibling, 1 reply; 12+ messages in thread
From: jassi brar @ 2009-12-09  0:47 UTC (permalink / raw)
  To: Grant Likely
  Cc: Andrew Morton, David Brownell, linux-kernel, Daniel Mack,
	Uwe Kleine-König, spi-devel-general, linux-arm-kernel

On Wed, Dec 9, 2009 at 9:25 AM, Grant Likely <grant.likely@secretlab.ca> wrote:
> 2009/12/8 Andrew Morton <akpm@linux-foundation.org>:
>> On Tue, 8 Dec 2009 22:12:58 +0100
>> Uwe Kleine-K__nig <u.kleine-koenig@pengutronix.de> wrote:
>>
>>> this series depends on "spi: drain MXC SPI transfer buffer when probing
>>> device" which is currently in -mm and also included in my spi-imx branch
>>> (see below).  IMHO at least my first patch should go into .33 as it
>>> fixes an oops.
>>
>> I queued all five for 2.6.33.
>
> I had also picked up "drain MXC SPI transfer..." into my next-spi tree
> earlier today.  After a few days I'll send Linus a pull request.  I
> just need to make sure that Ben's tree gets pulled first because I've
> got other stuff in there depending on it.
>
>> Once I've merged up the current -mm patch queue:
> [... list of patches snipped ...]
>> I shall stop sending SPI patches to Linus.  I'll send any which I
>> accumulate to Grant instead.
>
> I'm new to the job, so I'm being cautions about which things I'm pick
> up.  However, I'll happily merge anything that Andrew says is okay.
> Andrew, when in the merge window do you typically send out patch
> queues?
>

Grant, may I know of your tree, if it's public?
thanks

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [spi-devel-general] [PATCH 0/5] spi-imx: some updates
  2009-12-09  0:47       ` [spi-devel-general] " jassi brar
@ 2009-12-09  1:50         ` Grant Likely
  0 siblings, 0 replies; 12+ messages in thread
From: Grant Likely @ 2009-12-09  1:50 UTC (permalink / raw)
  To: jassi brar
  Cc: Andrew Morton, David Brownell, linux-kernel, Daniel Mack,
	Uwe Kleine-König, spi-devel-general, linux-arm-kernel

On Tue, Dec 8, 2009 at 5:47 PM, jassi brar <jassisinghbrar@gmail.com> wrote:
> On Wed, Dec 9, 2009 at 9:25 AM, Grant Likely <grant.likely@secretlab.ca> wrote:
>> I'm new to the job, so I'm being cautions about which things I'm pick
>> up.  However, I'll happily merge anything that Andrew says is okay.
>> Andrew, when in the merge window do you typically send out patch
>> queues?
>>
>
> Grant, may I know of your tree, if it's public?

It's the next-spi branch of:

git://git.secretlab.ca/git/linux-2.6.git

g.


-- 
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 0/5] spi-imx: some updates
  2009-12-09  0:35         ` Andrew Morton
@ 2009-12-09  6:15           ` Grant Likely
  0 siblings, 0 replies; 12+ messages in thread
From: Grant Likely @ 2009-12-09  6:15 UTC (permalink / raw)
  To: Andrew Morton
  Cc: David Brownell, Sascha Hauer, linux-kernel, Daniel Mack,
	Uwe Kleine-König, spi-devel-general, linux-arm-kernel

On Tue, Dec 8, 2009 at 5:35 PM, Andrew Morton <akpm@linux-foundation.org> wrote:
> On Tue, 8 Dec 2009 17:25:59 -0700
> Grant Likely <grant.likely@secretlab.ca> wrote:
>
>> I'm new to the job, so I'm being cautions about which things I'm pick
>> up.  However, I'll happily merge anything that Andrew says is okay.
>
> oop.  If I'm going to send a patch to a subsystem maintainer I'll usually give
> it less attention than the ones I plan to merge myself.
>
> Oh well, at least this way people will get their patches merged easily ;)

Hmmm... maybe I'll be more selective then.

I've also got JH to set up a patchwork page for SPI so that I won't
drop anything on the floor.  I could certainly use some help with
triaging & acking the stuff in there.  Anyone want to volunteer?

>> Andrew, when in the merge window do you typically send out patch
>> queues?
>
> Late.  I stage the -mm patches after linux-next so I spend most of the
> merge window waiting for people to get their stuff into mainline.

Okay, thanks.

> Often there are stragglers who I need to route around.  It's usually
> not a significant problem though.

g.

-- 
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2009-12-09  6:15 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-12-08 21:12 [PATCH 0/5] spi-imx: some updates Uwe Kleine-König
     [not found] ` <20091208211258.GA13706-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2009-12-08 21:14   ` [PATCH 1/5] spi-imx: use platform_driver_probe as probe lives in .init.text Uwe Kleine-König
     [not found]     ` <1260306850-14945-1-git-send-email-u.kleine-koenig-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2009-12-08 21:14       ` [PATCH 2/5] spi-imx: correct check for platform_get_irq failing Uwe Kleine-König
     [not found]         ` <1260306850-14945-2-git-send-email-u.kleine-koenig-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2009-12-08 21:14           ` [PATCH 3/5] spi-imx: use positive logic to distinguish cpu variants Uwe Kleine-König
     [not found]             ` <1260306850-14945-3-git-send-email-u.kleine-koenig-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2009-12-08 21:14               ` [PATCH 4/5] spi-imx: Add mx25 support Uwe Kleine-König
     [not found]                 ` <1260306850-14945-4-git-send-email-u.kleine-koenig-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2009-12-08 21:14                   ` [PATCH 5/5] spi-imx: don't access struct device directly but use dev_get_platdata Uwe Kleine-König
2009-12-08 23:32   ` [PATCH 0/5] spi-imx: some updates Andrew Morton
2009-12-09  0:25     ` Grant Likely
     [not found]       ` <fa686aa40912081625y3750a7fegbac38926182e38c6-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2009-12-09  0:35         ` Andrew Morton
2009-12-09  6:15           ` Grant Likely
2009-12-09  0:47       ` [spi-devel-general] " jassi brar
2009-12-09  1:50         ` Grant Likely

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).