diff options
| author | Connor Hansen <cmdkhh@gmail.com> | 2011-06-12 00:06:48 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2011-06-12 00:06:48 +0200 |
| commit | 3307d0d83b7bf636dc6dd2aa4a584d0f52cc185b (patch) | |
| tree | 69a8886e0cd21e6369fd196e237972a5fbfcaefc | |
| parent | linux/seqlock.h should #include asm/processor.h for cpu_relax() (diff) | |
| download | linux-3307d0d83b7bf636dc6dd2aa4a584d0f52cc185b.tar.xz linux-3307d0d83b7bf636dc6dd2aa4a584d0f52cc185b.zip | |
ide-cd: signedness warning fix again
One of the legit warnings 'make W=3 drivers/ide/ide-cd.c'
generates is:
drivers/ide/ide-cd.c: In function ide_cd_do_request
drivers/ide/ide-cd.c:828:2: warning: conversion to int from \
unsigned int may change the sign of the result
drivers/ide/ide-cd.c:833:2: warning: conversion to int from \
unsigned int may change the sign of the result
nsectors is declared int, should be unsigned int.
blk_rq_sectors() returns unsigned int, and ide_complete_rq
expects unsigned int as well. Fixes both warnings.
Signed-off-by: Connor Hansen <cmdkhh@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | drivers/ide/ide-cd.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/ide/ide-cd.c b/drivers/ide/ide-cd.c index 144d27261e43..04b09564bfa9 100644 --- a/drivers/ide/ide-cd.c +++ b/drivers/ide/ide-cd.c @@ -778,7 +778,8 @@ static ide_startstop_t ide_cd_do_request(ide_drive_t *drive, struct request *rq, sector_t block) { struct ide_cmd cmd; - int uptodate = 0, nsectors; + int uptodate = 0; + unsigned int nsectors; ide_debug_log(IDE_DBG_RQ, "cmd: 0x%x, block: %llu", rq->cmd[0], (unsigned long long)block); |