From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergei Shtylyov Subject: Re: [PATCH] ide/libata: fix ata_id_is_cfa() Date: Fri, 23 Jan 2009 19:30:59 +0300 Message-ID: <4979F0C3.50501@ru.mvista.com> References: <200901231615.38011.sshtylyov@ru.mvista.com> <20090123133352.60add80e@lxorguk.ukuu.org.uk> <4979CBC1.3040106@ru.mvista.com> <20090123154340.7adf5a9e@lxorguk.ukuu.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from h155.mvista.com ([63.81.120.155]:65185 "EHLO imap.sh.mvista.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1754265AbZAWQa3 (ORCPT ); Fri, 23 Jan 2009 11:30:29 -0500 In-Reply-To: <20090123154340.7adf5a9e@lxorguk.ukuu.org.uk> Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Alan Cox Cc: bzolnier@gmail.com, jgarzik@pobox.com, linux-ide@vger.kernel.org, gdu@mns.spb.ru 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