[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1332997594.3793.118.camel@minggr>
Date: Thu, 29 Mar 2012 13:06:34 +0800
From: Lin Ming <ming.m.lin@...el.com>
To: Sergei Shtylyov <sshtylyov@...sta.com>
Cc: Zhang Rui <rui.zhang@...el.com>, Jeff Garzik <jgarzik@...ox.com>,
Alan Stern <stern@...land.harvard.edu>,
"Rafael J. Wysocki" <rjw@...k.pl>, Tejun Heo <tj@...nel.org>,
Aaron Lu <aaron.lu@....com>, linux-kernel@...r.kernel.org,
linux-ide@...r.kernel.org, linux-scsi@...r.kernel.org,
linux-pm@...r.kernel.org, linux-acpi <linux-acpi@...r.kernel.org>
Subject: Re: [PATCH v3 4/7] libata: detech Device Attention support
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@....com>
> > Signed-off-by: Lin Ming<ming.m.lin@...el.com>
> [...]
>
> > 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-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists