summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTroy Kisky <troy.kisky@boundarydevices.com>2013-02-12 19:53:36 -0700
committerTroy Kisky <troy.kisky@boundarydevices.com>2013-02-13 12:55:39 -0700
commit35a7da60bdd1e6af2667be89a109ce47e92335dd (patch)
treead87a71235bd066648bf32903afb179d33c15643
parenta126c58d3b9001d53d5c2c9966d54a58fe8b019c (diff)
ipu_common: delay request_irq, check for handler not NULL3.0-boundary-imx6-ts3
-rw-r--r--drivers/mxc/ipu3/ipu_common.c39
1 files changed, 19 insertions, 20 deletions
diff --git a/drivers/mxc/ipu3/ipu_common.c b/drivers/mxc/ipu3/ipu_common.c
index 92703f34b6aa..3a760a1d9f5f 100644
--- a/drivers/mxc/ipu3/ipu_common.c
+++ b/drivers/mxc/ipu3/ipu_common.c
@@ -242,21 +242,6 @@ static int __devinit ipu_probe(struct platform_device *pdev)
goto failed_get_res;
}
- if (request_irq(ipu->irq_sync, ipu_irq_handler, 0, pdev->name, ipu) != 0) {
- dev_err(ipu->dev, "request SYNC interrupt failed\n");
- ret = -EBUSY;
- goto failed_req_irq_sync;
- }
- /* Some platforms have 2 IPU interrupts */
- if (ipu->irq_err >= 0) {
- if (request_irq
- (ipu->irq_err, ipu_irq_handler, 0, pdev->name, ipu) != 0) {
- dev_err(ipu->dev, "request ERR interrupt failed\n");
- ret = -EBUSY;
- goto failed_req_irq_err;
- }
- }
-
ipu_base = res->start;
/* base fixup */
if (g_ipu_hw_rev == 4) /* IPUv3H */
@@ -344,12 +329,29 @@ static int __devinit ipu_probe(struct platform_device *pdev)
if (!plat_data->bypass_reset)
clk_disable(ipu->ipu_clk);
+ if (request_irq(ipu->irq_sync, ipu_irq_handler, 0, pdev->name, ipu) != 0) {
+ dev_err(ipu->dev, "request SYNC interrupt failed\n");
+ ret = -EBUSY;
+ goto failed_req_irq_sync;
+ }
+ /* Some platforms have 2 IPU interrupts */
+ if (ipu->irq_err >= 0) {
+ if (request_irq
+ (ipu->irq_err, ipu_irq_handler, 0, pdev->name, ipu) != 0) {
+ dev_err(ipu->dev, "request ERR interrupt failed\n");
+ ret = -EBUSY;
+ goto failed_req_irq_err;
+ }
+ }
register_ipu_device(ipu, pdev->id);
ipu->online = true;
return ret;
+failed_req_irq_err:
+ free_irq(ipu->irq_sync, ipu);
+failed_req_irq_sync:
failed_clk_setup:
iounmap(ipu->cm_reg);
iounmap(ipu->ic_reg);
@@ -370,9 +372,6 @@ failed_clk_setup:
failed_ioremap:
if (ipu->irq_sync)
free_irq(ipu->irq_err, ipu);
-failed_req_irq_err:
- free_irq(ipu->irq_sync, ipu);
-failed_req_irq_sync:
failed_get_res:
return ret;
}
@@ -2415,8 +2414,8 @@ static irqreturn_t ipu_irq_handler(int irq, void *desc)
bit = --line;
int_stat &= ~(1UL << line);
line += (int_reg[i] - 1) * 32;
- result |=
- ipu->irq_list[line].handler(line,
+ if (ipu->irq_list[line].handler)
+ result |= ipu->irq_list[line].handler(line,
ipu->irq_list[line].
dev_id);
if (ipu->irq_list[line].flags & IPU_IRQF_ONESHOT) {