sector_t is a synonym for u64 and all architectures define u64 as unsigned long long. Hence, it is not necessary to cast type sector_t to unsigned long long. Remove a superfluous cast to improve compile-time type checking. Reviewed-by: Yu Kuai <yukuai3@xxxxxxxxxx> Reviewed-by: Chaitanya Kulkarni <kch@xxxxxxxxxx> Signed-off-by: Bart Van Assche <bvanassche@xxxxxxx> --- block/bfq-iosched.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/block/bfq-iosched.c b/block/bfq-iosched.c index 50e51047e1fe..d24143f55fd2 100644 --- a/block/bfq-iosched.c +++ b/block/bfq-iosched.c @@ -760,8 +760,7 @@ bfq_rq_pos_tree_lookup(struct bfq_data *bfqd, struct rb_root *root, if (rb_link) *rb_link = p; - bfq_log(bfqd, "rq_pos_tree_lookup %llu: returning %d", - (unsigned long long)sector, + bfq_log(bfqd, "rq_pos_tree_lookup %llu: returning %d", sector, bfqq ? bfqq->pid : 0); return bfqq;
