diff options
author | Mike Christie <mchristi@redhat.com> | 2016-06-05 15:32:19 -0400 |
---|---|---|
committer | Jens Axboe <axboe@fb.com> | 2016-06-07 15:41:38 -0400 |
commit | 8e45c6f8808357704519c217dc633bc43afc2a61 (patch) | |
tree | 4d9172e474c0478eafc3895cafeec433fce88cbc | |
parent | 1b9a9ab78b0ab79dc1f0ddd5fbed7833ec7de3a4 (diff) |
ide cd: do not set REQ_WRITE on requests.
The block layer will set the correct READ/WRITE operation flags/fields
when creating a request, so there is not need for drivers to set the
REQ_WRITE flag.
Signed-off-by: Mike Christie <mchristi@redhat.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Hannes Reinecke <hare@suse.com>
Signed-off-by: Jens Axboe <axboe@fb.com>
-rw-r--r-- | drivers/ide/ide-cd_ioctl.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/ide/ide-cd_ioctl.c b/drivers/ide/ide-cd_ioctl.c index 474173eb31bb..5887a7a09e37 100644 --- a/drivers/ide/ide-cd_ioctl.c +++ b/drivers/ide/ide-cd_ioctl.c | |||
@@ -459,9 +459,6 @@ int ide_cdrom_packet(struct cdrom_device_info *cdi, | |||
459 | layer. the packet must be complete, as we do not | 459 | layer. the packet must be complete, as we do not |
460 | touch it at all. */ | 460 | touch it at all. */ |
461 | 461 | ||
462 | if (cgc->data_direction == CGC_DATA_WRITE) | ||
463 | flags |= REQ_WRITE; | ||
464 | |||
465 | if (cgc->sense) | 462 | if (cgc->sense) |
466 | memset(cgc->sense, 0, sizeof(struct request_sense)); | 463 | memset(cgc->sense, 0, sizeof(struct request_sense)); |
467 | 464 | ||