- 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 13:06:34 +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: <4F72F45D.7090800@mvista.com>
- List-id: <linux-ide.vger.kernel.org>
On Wed, 2012-03-28 at 15:22 +0400, Sergei Shtylyov wrote:
> Hello.
>
> On 28-03-2012 10:21, 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.
>
> MBR, Sergei
--
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
[Linux Filesystems]
[Linux SCSI]
[Linux RAID]
[Git]
[Kernel Newbies]
[Linux Newbie]
[Share Photos]
[Security]
[Netfilter]
[Bugtraq]
[Photo]
[Yosemite]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Samba]
[Video 4 Linux]
[Device Mapper]
[Linux Resources]