On 06/20/2016 05:40 PM, Tejun Heo wrote:
> On Mon, Jun 20, 2016 at 01:39:12PM +0200, Hannes Reinecke wrote:
>> Use accessors instead of the raw protocol value.
>>
>> Signed-off-by: Hannes Reinecke <hare@xxxxxxxx>
>> ---
>> drivers/scsi/libsas/sas_ata.c | 9 ++-------
>> 1 file changed, 2 insertions(+), 7 deletions(-)
>>
>> diff --git a/drivers/scsi/libsas/sas_ata.c b/drivers/scsi/libsas/sas_ata.c
>> index 935c430..e1da52c 100644
>> --- a/drivers/scsi/libsas/sas_ata.c
>> +++ b/drivers/scsi/libsas/sas_ata.c
>> @@ -233,15 +233,10 @@ static unsigned int sas_ata_qc_issue(struct ata_queued_cmd *qc)
>> task->task_state_flags = SAS_TASK_STATE_PENDING;
>> qc->lldd_task = task;
>>
>> - switch (qc->tf.protocol) {
>> - case ATA_PROT_NCQ:
>> + if (ata_is_ncq(qc->tf.protocol))
>> task->ata_task.use_ncq = 1;
>> - /* fall through */
>> - case ATAPI_PROT_DMA:
>> - case ATA_PROT_DMA:
>> + if (ata_is_dma(qc->tf.protocol))
>> task->ata_task.dma_xfer = 1;
>> - break;
>> - }
>
> As you're cleaning it up anyway, can you please make ata_is_*()
> functions return bool and do
>
> task->ata_task.use_ncq = ata_is_ncq()
>
> instead?
>
Sure, will do with the next round.
Cheers,
Hannes
--
Dr. Hannes Reinecke Teamlead Storage & Networking
hare@xxxxxxx +49 911 74053 688
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton
HRB 21284 (AG Nürnberg)
--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
