Hi Lee,
>From: Lee Jones [mailto:lee.jo...@linaro.org]
>Obtain IRQ number and request IRQ resource via the usual methods. We're
>also registering an IRQ handler to inform us of any completed tasks.
>Notice that we're starting to make use of the device struct that we
>defined before. In keeping with the subject of the patch, we're also
>adding the related local enable_irq() and disable_irq() methods. Again,
>these will be utilised in a greater capacity in latter commits.
>
>Signed-off-by: Lee Jones <lee.jo...@linaro.org>
>---
> drivers/mtd/nand/stm_nand_bch.c | 62 +++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 62 insertions(+)
>
>diff --git a/drivers/mtd/nand/stm_nand_bch.c b/drivers/mtd/nand/stm_nand_bch.c
>index bd11070..76a0d02 100644
>--- a/drivers/mtd/nand/stm_nand_bch.c
>+++ b/drivers/mtd/nand/stm_nand_bch.c
>@@ -15,11 +15,14 @@
> #include <linux/kernel.h>
> #include <linux/module.h>
> #include <linux/io.h>
>+#include <linux/interrupt.h>
> #include <linux/device.h>
> #include <linux/platform_device.h>
> #include <linux/completion.h>
> #include <linux/mtd/stm_nand.h>
>
>+#include "stm_nand_regs.h"
>+
> /* NANDi Controller (Hamming/BCH) */
> struct nandi_controller {
> void __iomem *base; /* Controller base*/
>@@ -54,6 +57,51 @@ struct nandi_controller {
> /* 'page_buf' */
> };
>
>+/*
>+ * NANDi Interrupts (shared by Hamming and BCH controllers)
>+ */
>+static irqreturn_t nandi_irq_handler(int irq, void *dev)
>+{
>+ struct nandi_controller *nandi = dev;
>+ unsigned int status;
>+
>+ status = readl(nandi->base + NANDBCH_INT_STA);
>+
>+ if (status & NANDBCH_INT_SEQNODESOVER) {
You should also check, if BCH ECC mode is actually set to
struct nandi_controller *nandi = dev;
if (nandi->bch_ecc_mode && (status & NANDBCH_INT_SEQNODESOVER)) {
>+ /* BCH */
>+ writel(NANDBCH_INT_CLR_SEQNODESOVER,
>+ nandi->base + NANDBCH_INT_CLR);
>+ complete(&nandi->seq_completed);
At a given time only one of the two controllers (HAM/BCH) would be active.
right ?
So, do don't need to check for both interrupts. you can return if either one is
successful.
return IRQ_HANDLED; /* BCH ECC IRQ handles successfully */
>+ }
>+ if (status & NAND_INT_RBN) {
>+ /* Hamming */
>+ writel(NAND_INT_CLR_RBN, nandi->base + NANDHAM_INT_CLR);
>+ complete(&nandi->rbn_completed);
-- same for this --
return IRQ_HANDLED; /* HAM ECC IRQ handled successfully */
>+ }
>+
>+ return IRQ_HANDLED;
And if no valid source is found then
return IRQ_NONE; /* spurious interrupt */
>+}
>+
with regards, pekon
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
