diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2018-05-31 09:44:49 +0300 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2018-05-31 16:12:00 -0400 |
| commit | 8005b09d99fac78e6f5fb9da30b5ae94840af03b (patch) | |
| tree | 792ab5311bd824a95523f7489ee3344fa07bed6d /tools | |
| parent | 0f51f3582f22e543b78c4e113220ed1c35acbd97 (diff) | |
| download | linux-8005b09d99fac78e6f5fb9da30b5ae94840af03b.tar.gz linux-8005b09d99fac78e6f5fb9da30b5ae94840af03b.tar.bz2 linux-8005b09d99fac78e6f5fb9da30b5ae94840af03b.zip | |
net: ethernet: davinci_emac: fix error handling in probe()
The current error handling code has an issue where it does:
if (priv->txchan)
cpdma_chan_destroy(priv->txchan);
The problem is that ->txchan is either valid or an error pointer (which
would lead to an Oops). I've changed it to use multiple error labels so
that the test can be removed.
Also there were some missing calls to netif_napi_del().
Fixes: 3ef0fdb2342c ("net: davinci_emac: switch to new cpdma layer")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'tools')
0 files changed, 0 insertions, 0 deletions