diff options
| author |   Wei Yongjun <weiyongjun1@huawei.com> | 2020-05-20 11:41:16 +0800 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2020-05-21 17:14:18 -0700 |
| commit | 1401cf600d548c72f51e20b5841c330d5c11c9e2 (patch) | |
| tree | 341f692f29bf8bc3c19ab133fd3a09951d259653 /drivers | |
| parent | net: ethernet: ti: fix some return value check of cpsw_ale_create() (diff) | |
| download | linux-dev-1401cf600d548c72f51e20b5841c330d5c11c9e2.tar.xz linux-dev-1401cf600d548c72f51e20b5841c330d5c11c9e2.zip | |
net: ethernet: ti: am65-cpsw-nuss: fix error handling of am65_cpsw_nuss_probe
Convert to using IS_ERR() instead of NULL test for cpsw_ale_create()
error handling. Also fix to return negative error code from this error
handling case instead of 0 in.
Fixes: 93a76530316a ("net: ethernet: ti: introduce am65x/j721e gigabit eth subsystem driver")
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/net/ethernet/ti/am65-cpsw-nuss.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/net/ethernet/ti/am65-cpsw-nuss.c b/drivers/net/ethernet/ti/am65-cpsw-nuss.c index 2517ffba8178..88f52a2f85b3 100644 --- a/drivers/net/ethernet/ti/am65-cpsw-nuss.c +++ b/drivers/net/ethernet/ti/am65-cpsw-nuss.c @@ -1895,8 +1895,9 @@ static int am65_cpsw_nuss_probe(struct platform_device *pdev) ale_params.nu_switch_ale = true; common->ale = cpsw_ale_create(&ale_params); - if (!common->ale) { + if (IS_ERR(common->ale)) { dev_err(dev, "error initializing ale engine\n"); + ret = PTR_ERR(common->ale); goto err_of_clear; } |