- To: Sergei Shtylyov <sshtylyov@xxxxxxxxxx>
- Subject: Re: [PATCH v3 4/7] libata: detech Device Attention support
- From: Lin Ming <ming.m.lin@xxxxxxxxx>
- Date: Thu, 29 Mar 2012 19:18:19 +0800
- Cc: Zhang Rui <rui.zhang@xxxxxxxxx>, Jeff Garzik <jgarzik@xxxxxxxxx>, Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>, "Rafael J. Wysocki" <rjw@xxxxxxx>, Tejun Heo <tj@xxxxxxxxxx>, Aaron Lu <aaron.lu@xxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx, linux-ide@xxxxxxxxxxxxxxx, linux-scsi@xxxxxxxxxxxxxxx, linux-pm@xxxxxxxxxxxxxxx, linux-acpi <linux-acpi@xxxxxxxxxxxxxxx>
- In-reply-to: <4F744412.4090806@mvista.com>
On Thu, Mar 29, 2012 at 7:14 PM, Sergei Shtylyov <sshtylyov@xxxxxxxxxx> wrote:
> Hello.
>
>
> On 29-03-2012 9:06, Lin Ming wrote:
>
>>>> Add a new flag ATA_DFLAG_DA to indicate that device supports "Device
>>>> Attention".
>
>
>>>> Acked-by: Aaron Lu<aaron.lu@xxxxxxx>
>>>> Signed-off-by: Lin Ming<ming.m.lin@xxxxxxxxx>
>>>
>>> [...]
>
>
>>>> diff --git a/include/linux/ata.h b/include/linux/ata.h
>>>> index 32df2b6..6ee41cc 100644
>>>> --- a/include/linux/ata.h
>>>> +++ b/include/linux/ata.h
>>>> @@ -578,6 +578,7 @@ static inline int ata_is_data(u8 prot)
>>>> ((u64) (id)[(n) + 0]) )
>>>>
>>>> #define ata_id_cdb_intr(id) (((id)[ATA_ID_CONFIG]& 0x60) == 0x20)
>>>> +#define atapi_id_has_da(id) ((id)[77]& (1<< 4))
>
>
>>> We prefix these macros with 'ata_' despite them being applied to
>>> IDENTIFY
>>> DEVICE data or IDENTIFY PAKCET DEVICE data. ata_id_cdb_intr(), for
>>> example,
>>> applies only to IDENTIFY PAKCET DEVICE data.
>
>
>> OK, will change it.
>
>
> Also, what does the word "detech" mean in the subject?
Typo! It should be "detect"
>
>
> MBR, Sergei
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
[SCSI Target Devel]
[Linux SCSI Target Infrastructure]
[Kernel Newbies]
[Share Photos]
[IDE]
[Security]
[Git]
[Netfilter]
[Bugtraq]
[Photos]
[Yosemite]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Linux ATA RAID]
[Linux IIO]
[Samba]
[Video 4 Linux]
[Device Mapper]
[Linux Resources]