From: Sascha Hauer <s.hauer@pengutronix.de>
To: John Ogness <john.ogness@linutronix.de>
Cc: Baruch Siach <baruch@tkos.co.il>,
linux-mtd@lists.infradead.org,
linux-arm-kernel@lists.infradead.org,
Ivo Clarysse <ivo.clarysse@gmail.com>
Subject: Re: [PATCH 1/3] mxc_nand: set spare size and pages per block
Date: Tue, 17 Aug 2010 10:54:06 +0200 [thread overview]
Message-ID: <20100817085406.GW27749@pengutronix.de> (raw)
In-Reply-To: <80zkwm69uv.fsf@merkur.tec.linutronix.de>
On Mon, Aug 16, 2010 at 02:05:44PM +0200, John Ogness wrote:
> On 2010-08-16, Sascha Hauer <s.hauer@pengutronix.de> wrote:
> > Ok, here is an updated version of this patch
> >
> > Sascha
> >
> > commit 3a411e6c690654f78bea93dd8cdb14c27ba418a5
> > Author: Sascha Hauer <s.hauer@pengutronix.de>
> > Date: Mon Aug 9 14:21:00 2010 +0200
> >
> > mxc_nand: do not depend on disabling the irq in the interrupt handler
> >
> > This patch reverts the driver to enabling/disabling the NFC interrupt
> > mask rather than enabling/disabling the system interrupt. This cleans
> > up the driver so that it doesn't rely on interrupts being disabled
> > within the interrupt handler.
> > For i.MX21 we keep the current behaviour, that is calling
> > enable_irq/disable_irq_nosync to enable/disable interrupts.
> > This patch is based on earlier work by John Ogness.
>
> Acked-by: John Ogness <john.ogness@linutronix.de>
Can I consider this as an Acked-by for the whole series? And since you
tested it I would also add a tested-by. Here is a summary of the series
again:
The following changes since commit da5cabf80e2433131bf0ed8993abc0f7ea618c73:
Linux 2.6.36-rc1 (2010-08-15 17:41:37 -0700)
are available in the git repository at:
git://git.pengutronix.de/git/imx/linux-2.6.git mxc-nand-pu
John Ogness (2):
mxc_nand: Do not do byte accesses to the NFC buffer.
mxc_nand: remove unused variables.
Sascha Hauer (10):
mxc_nand: remove 0xe00 offset from registers
mxc_nand: rework get_dev_status
mxc_nand: make some internally used functions overwriteable
mxc_nand: factor out a check_int function
mxc_nand: add V1_V2 namespace to registers
mxc_nand: support 8bit ecc
mxc_nand: fix correct_data function
mxc_nand: Add v3 (i.MX51) Support
mxc_nand: do not depend on disabling the irq in the interrupt handler
mxc_nand: configure pages per block for v2 controller
drivers/mtd/nand/Kconfig | 2 +-
drivers/mtd/nand/mxc_nand.c | 671 +++++++++++++++++++++++++++++++++----------
2 files changed, 521 insertions(+), 152 deletions(-)
Sascha
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
WARNING: multiple messages have this Message-ID (diff)
From: s.hauer@pengutronix.de (Sascha Hauer)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/3] mxc_nand: set spare size and pages per block
Date: Tue, 17 Aug 2010 10:54:06 +0200 [thread overview]
Message-ID: <20100817085406.GW27749@pengutronix.de> (raw)
In-Reply-To: <80zkwm69uv.fsf@merkur.tec.linutronix.de>
On Mon, Aug 16, 2010 at 02:05:44PM +0200, John Ogness wrote:
> On 2010-08-16, Sascha Hauer <s.hauer@pengutronix.de> wrote:
> > Ok, here is an updated version of this patch
> >
> > Sascha
> >
> > commit 3a411e6c690654f78bea93dd8cdb14c27ba418a5
> > Author: Sascha Hauer <s.hauer@pengutronix.de>
> > Date: Mon Aug 9 14:21:00 2010 +0200
> >
> > mxc_nand: do not depend on disabling the irq in the interrupt handler
> >
> > This patch reverts the driver to enabling/disabling the NFC interrupt
> > mask rather than enabling/disabling the system interrupt. This cleans
> > up the driver so that it doesn't rely on interrupts being disabled
> > within the interrupt handler.
> > For i.MX21 we keep the current behaviour, that is calling
> > enable_irq/disable_irq_nosync to enable/disable interrupts.
> > This patch is based on earlier work by John Ogness.
>
> Acked-by: John Ogness <john.ogness@linutronix.de>
Can I consider this as an Acked-by for the whole series? And since you
tested it I would also add a tested-by. Here is a summary of the series
again:
The following changes since commit da5cabf80e2433131bf0ed8993abc0f7ea618c73:
Linux 2.6.36-rc1 (2010-08-15 17:41:37 -0700)
are available in the git repository at:
git://git.pengutronix.de/git/imx/linux-2.6.git mxc-nand-pu
John Ogness (2):
mxc_nand: Do not do byte accesses to the NFC buffer.
mxc_nand: remove unused variables.
Sascha Hauer (10):
mxc_nand: remove 0xe00 offset from registers
mxc_nand: rework get_dev_status
mxc_nand: make some internally used functions overwriteable
mxc_nand: factor out a check_int function
mxc_nand: add V1_V2 namespace to registers
mxc_nand: support 8bit ecc
mxc_nand: fix correct_data function
mxc_nand: Add v3 (i.MX51) Support
mxc_nand: do not depend on disabling the irq in the interrupt handler
mxc_nand: configure pages per block for v2 controller
drivers/mtd/nand/Kconfig | 2 +-
drivers/mtd/nand/mxc_nand.c | 671 +++++++++++++++++++++++++++++++++----------
2 files changed, 521 insertions(+), 152 deletions(-)
Sascha
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
next prev parent reply other threads:[~2010-08-17 8:54 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-08-10 11:34 [PATCH 1/3] mxc_nand: set spare size and pages per block John Ogness
2010-08-10 11:34 ` John Ogness
2010-08-10 11:35 ` [PATCH 2/3] mxc_nand: remove unused variables John Ogness
2010-08-10 11:35 ` John Ogness
2010-08-10 11:36 ` [PATCH 3/3] mxc_nand: mask instead of disabling (i.MX21 as exception) John Ogness
2010-08-10 11:36 ` John Ogness
2010-08-10 12:19 ` [PATCH 1/3] mxc_nand: set spare size and pages per block Sascha Hauer
2010-08-10 12:19 ` Sascha Hauer
2010-08-10 14:31 ` John Ogness
2010-08-10 14:31 ` John Ogness
2010-08-10 14:43 ` John Ogness
2010-08-10 14:43 ` John Ogness
2010-08-11 12:56 ` Sascha Hauer
2010-08-11 12:56 ` Sascha Hauer
2010-08-11 13:16 ` John Ogness
2010-08-11 13:16 ` John Ogness
2010-08-11 13:27 ` Sascha Hauer
2010-08-11 13:27 ` Sascha Hauer
2010-08-16 11:28 ` Sascha Hauer
2010-08-16 11:28 ` Sascha Hauer
2010-08-16 12:05 ` John Ogness
2010-08-16 12:05 ` John Ogness
2010-08-17 8:54 ` Sascha Hauer [this message]
2010-08-17 8:54 ` Sascha Hauer
2010-08-17 17:02 ` John Ogness
2010-08-17 17:02 ` John Ogness
2010-08-29 12:08 ` Artem Bityutskiy
2010-08-29 12:08 ` Artem Bityutskiy
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=20100817085406.GW27749@pengutronix.de \
--to=s.hauer@pengutronix.de \
--cc=baruch@tkos.co.il \
--cc=ivo.clarysse@gmail.com \
--cc=john.ogness@linutronix.de \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-mtd@lists.infradead.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.