diff options
author | Jens Axboe <axboe@kernel.dk> | 2017-06-23 17:18:54 +0200 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2017-06-23 17:18:54 +0200 |
commit | 8c66ac6a28a460273e1ad263bb05056dc0e68760 (patch) | |
tree | 0778d025344fed439505041f203c94a0813be86b /drivers/dma/sun6i-dma.c | |
parent | Merge commit '8e8320c9315c' into for-4.13/block (diff) | |
download | linux-8c66ac6a28a460273e1ad263bb05056dc0e68760.tar.xz linux-8c66ac6a28a460273e1ad263bb05056dc0e68760.zip |
mtip32xx: fix up the checking for internal command failure
This fixes up two commits that have touched this driver. The
command status field is now a blk_status_t, so we can't check
for < 0 and we definitely can't assume it's holding -Exxxx error
values. All we care about here is whether ->status is zero or not.
Check for that, and remove the various attempts at smart error
reporting. Just log to dmesg what command failed, and the
blk_status_t value.
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Fixes: 2a842acab109 ("block: introduce new block status code type")
Fixes: 3f5e6a35774c ("mtip32xx: convert internal command issue to block IO path")
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'drivers/dma/sun6i-dma.c')
0 files changed, 0 insertions, 0 deletions