diff options
| author | Nicholas Krause <xerofoify@gmail.com> | 2014-12-02 17:09:02 -0500 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2014-12-04 12:58:50 -0500 |
| commit | 2ba520f0cd65c2e688f8beb495bb6634a61ee17b (patch) | |
| tree | 1c9b59df00724ec948e644fcb972d214df295cb8 | |
| parent | 598e9a9005ef3ad7540227c9161f7cfbdccd0dbe (diff) | |
libata: Remove FIXME comment in atapi_request_sense()
Remove the FIXME comment in atapi_request_sense() asking whether
memset of sense buffer is necessary. The buffer may be partially or
fully filled by the device. We want it to be cleared.
tj: Updated description.
Signed-off-by: Nicholas Krause <xerofoify@gmail.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
| -rw-r--r-- | drivers/ata/libata-scsi.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/ata/libata-scsi.c b/drivers/ata/libata-scsi.c index 1db6eab6dac3..6a5fcf14dbc3 100644 --- a/drivers/ata/libata-scsi.c +++ b/drivers/ata/libata-scsi.c | |||
| @@ -2595,7 +2595,6 @@ static void atapi_request_sense(struct ata_queued_cmd *qc) | |||
| 2595 | 2595 | ||
| 2596 | DPRINTK("ATAPI request sense\n"); | 2596 | DPRINTK("ATAPI request sense\n"); |
| 2597 | 2597 | ||
| 2598 | /* FIXME: is this needed? */ | ||
| 2599 | memset(cmd->sense_buffer, 0, SCSI_SENSE_BUFFERSIZE); | 2598 | memset(cmd->sense_buffer, 0, SCSI_SENSE_BUFFERSIZE); |
| 2600 | 2599 | ||
| 2601 | #ifdef CONFIG_ATA_SFF | 2600 | #ifdef CONFIG_ATA_SFF |