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:

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

Nope. Please go re-read what I wrote.

Then I don't understand why you wrote it -- I didn't urge you to fix the correct inlines.

source please -- I wouldn't have dropped the revision check if the rest of the inlines that check word 82/82 were using it.

In other words, as I said, you removed a correct check rather than
correcting others which may not be correct.

   Yes. But I have no time to fix everything for everybody.

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