On Mon, Jun 20, 2016 at 01:39:13PM +0200, Hannes Reinecke wrote: > diff --git a/include/linux/libata.h b/include/linux/libata.h > index d15c19e..264414c 100644 > --- a/include/linux/libata.h > +++ b/include/linux/libata.h > @@ -152,6 +152,7 @@ enum { > ATA_PROT_FLAG_DATA = ATA_PROT_FLAG_PIO | ATA_PROT_FLAG_DMA, > ATA_PROT_FLAG_NCQ = (1 << 2), /* is NCQ */ > ATA_PROT_FLAG_ATAPI = (1 << 3), /* is ATAPI */ > + ATA_PROT_FLAG_FPDMA = ATA_PROT_FLAG_NCQ | ATA_PROT_FLAG_DMA, > > /* struct ata_device stuff */ > ATA_DFLAG_LBA = (1 << 0), /* device supports LBA */ > @@ -1093,6 +1094,11 @@ static inline int ata_is_data(u8 prot) > return ata_prot_flags(prot) & ATA_PROT_FLAG_DATA; > } > > +static inline int ata_is_fpdma(u8 prot) > +{ > + return ata_prot_flags(prot) & ATA_PROT_FLAG_FPDMA; > +} > + This is NCQ or DMA which isn't the same thing as some of the tests it's replacing. Is this intentional? Thanks. -- tejun -- 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