Hi Magnus,
On Mon, Dec 19, 2016 at 7:28 PM, Magnus Lilja <lilja.magnus@xxxxxxxxx> wrote:
> Commit 3d8cc00073d6750ffe883685e49b2e4a0f596370 consolidated the two
The standard form to specify a commit is 3d8cc00073d6 ("dmaengine:
ipu: Consolidate duplicated irq handlers")
> interrupts routines into one, but the remaining interrupt routine only
> checks the status of the error interrupts, not the normal interrupts.
>
> This patch fixes that problem (tested on i.MX31 PDK board).
>
> Signed-off-by: Magnus Lilja <lilja.magnus@xxxxxxxxx>
Patch looks good, but I have some suggestions:
- Subject should start with dmaengine, so something like:
Subject: dmaengine: ipu: Make sure the ....
- You sent these two patches on a series, but as they touch different
subsystems they could be sent separately, each one to the proper
maintainer. For dmaengine the maintainer is Vinod. Hint: you can run
./scripts/get_maintainer.pl on your patch and it will list the
maintainer and lists the patches should be sent to.
You should also add a Fixes tag above your Signed-off-by like this:
Fixes: 3d8cc00073d6 ("dmaengine: ipu: Consolidate duplicated irq handlers")
Cc: <stable@xxxxxxxxxxxxxxx> # 4.3.x
--
To unsubscribe from this list: send the line "unsubscribe dmaengine" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
