diff options
author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2015-01-13 20:10:56 -0500 |
---|---|---|
committer | Jassi Brar <jaswinder.singh@linaro.org> | 2015-02-06 00:27:56 -0500 |
commit | 356d5d28f2cd45ee1956ee868de6f5d37bb9f7dd (patch) | |
tree | 8dd6fce04de27006759c61428701c3f1fc392043 /drivers/mailbox | |
parent | e36f014edff70fc02b3d3d79cead1d58f289332e (diff) |
Mailbox: Fix return value check in pcc_init()
In case of error, the function platform_create_bundle() returns
ERR_PTR() and never returns NULL. The NULL test in the return value
check should be replaced with IS_ERR().
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Diffstat (limited to 'drivers/mailbox')
-rw-r--r-- | drivers/mailbox/pcc.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/mailbox/pcc.c b/drivers/mailbox/pcc.c index 6dbf6fcbdfaf..70576fb707a5 100644 --- a/drivers/mailbox/pcc.c +++ b/drivers/mailbox/pcc.c | |||
@@ -393,9 +393,9 @@ static int __init pcc_init(void) | |||
393 | pcc_pdev = platform_create_bundle(&pcc_mbox_driver, | 393 | pcc_pdev = platform_create_bundle(&pcc_mbox_driver, |
394 | pcc_mbox_probe, NULL, 0, NULL, 0); | 394 | pcc_mbox_probe, NULL, 0, NULL, 0); |
395 | 395 | ||
396 | if (!pcc_pdev) { | 396 | if (IS_ERR(pcc_pdev)) { |
397 | pr_err("Err creating PCC platform bundle\n"); | 397 | pr_err("Err creating PCC platform bundle\n"); |
398 | return -ENODEV; | 398 | return PTR_ERR(pcc_pdev); |
399 | } | 399 | } |
400 | 400 | ||
401 | return 0; | 401 | return 0; |