Re: [PATCH] ide/libata: fix ata_id_is_cfa()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello.

Alan Cox wrote:

The word 82 validity bit is not sufficient as that bit is itself not

   There's no word 82 validity bits, its validity bits are in word 83.

Word 83 sorry - I seem to have 82 and 83 bit flipped in my brain, not
good for ATA work ;)

defined in ATA < 3. Otherwise the change looks correct.

Well, then we need to fix every case of using *only* the validity bits in ata.h.

No reason to go removing ones that are correct

You've just effectively claimed them to be incorrect with your claim what validity bits are not sufficient and ATA revision must be checked. Check the source please -- I wouldn't have dropped the revision check if the rest of the inlines that check word 82/82 were using 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

[Index of Archives]     [Linux Filesystems]     [Linux SCSI]     [Linux RAID]     [Git]     [Kernel Newbies]     [Linux Newbie]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Samba]     [Device Mapper]

  Powered by Linux