diff options
author | Fabio Estevam <fabio.estevam@freescale.com> | 2014-02-13 22:04:44 -0500 |
---|---|---|
committer | Herbert Xu <herbert@gondor.apana.org.au> | 2014-02-26 16:56:54 -0500 |
commit | d9588f874482c0163caf19c2acbe5ce6260d42bf (patch) | |
tree | 4160f85e1e0046e6761de3aec3a010e01574c74d /drivers/crypto | |
parent | b3bd5869fd65f509d96a9fcb47cdea81163a811d (diff) |
crypto: mxs-dcp - Fix platform_get_irq() error handling
We should test the error case for each platform_get_irq() assignment and
propagate the error accordingly.
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
Acked-by: Marek Vasut <marex@denx.de>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to 'drivers/crypto')
-rw-r--r-- | drivers/crypto/mxs-dcp.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/drivers/crypto/mxs-dcp.c b/drivers/crypto/mxs-dcp.c index 37e070670702..08761d61d4f5 100644 --- a/drivers/crypto/mxs-dcp.c +++ b/drivers/crypto/mxs-dcp.c | |||
@@ -908,9 +908,14 @@ static int mxs_dcp_probe(struct platform_device *pdev) | |||
908 | 908 | ||
909 | iores = platform_get_resource(pdev, IORESOURCE_MEM, 0); | 909 | iores = platform_get_resource(pdev, IORESOURCE_MEM, 0); |
910 | dcp_vmi_irq = platform_get_irq(pdev, 0); | 910 | dcp_vmi_irq = platform_get_irq(pdev, 0); |
911 | if (dcp_vmi_irq < 0) { | ||
912 | ret = dcp_vmi_irq; | ||
913 | goto err_mutex; | ||
914 | } | ||
915 | |||
911 | dcp_irq = platform_get_irq(pdev, 1); | 916 | dcp_irq = platform_get_irq(pdev, 1); |
912 | if (dcp_vmi_irq < 0 || dcp_irq < 0) { | 917 | if (dcp_irq < 0) { |
913 | ret = -EINVAL; | 918 | ret = dcp_irq; |
914 | goto err_mutex; | 919 | goto err_mutex; |
915 | } | 920 | } |
916 | 921 | ||