From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bartlomiej Zolnierkiewicz Subject: Re: PATCH: 2.6.10 - IT8212 IDE Date: Wed, 29 Dec 2004 23:38:27 +0100 Message-ID: <58cb370e04122914384fba1973@mail.gmail.com> References: <1104246926.22366.97.camel@localhost.localdomain> <58cb370e041229092919c1b4a8@mail.gmail.com> <1104355483.31622.2.camel@localhost.localdomain> Reply-To: Bartlomiej Zolnierkiewicz Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: Received: from wproxy.gmail.com ([64.233.184.207]:26351 "EHLO wproxy.gmail.com") by vger.kernel.org with ESMTP id S261303AbUL2Wi2 (ORCPT ); Wed, 29 Dec 2004 17:38:28 -0500 Received: by wproxy.gmail.com with SMTP id 36so119441wra for ; Wed, 29 Dec 2004 14:38:27 -0800 (PST) In-Reply-To: <1104355483.31622.2.camel@localhost.localdomain> Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Alan Cox Cc: Linux Kernel Mailing List , linux-ide@vger.kernel.org On Wed, 29 Dec 2004 21:24:45 +0000, Alan Cox wrote: > On Mer, 2004-12-29 at 17:29, Bartlomiej Zolnierkiewicz wrote: > > ide_get_best_pio_mode(drive, 4, 5, NULL) always returns 4 > > [ quick fix == hardcode 4 for now and add FIXME ] > > Doesn't seem that way reading the code. u8 ide_get_best_pio_mode (ide_drive_t *drive, u8 mode_wanted, u8 max_mode, ide_pio_data_t *d) { /* mode_wanted == 4, max_mode == 5 */ ... if (mode_wanted != 255) { /* 4 != 255 */ pio_mode = mode_wanted; /* pio_mode == 4 */ } else if (!drive->id) { pio_mode = 0; } else if ((pio_mode = ide_scan_pio_blacklist(id->model)) != -1) { overridden = 1; blacklisted = 1; use_iordy = (pio_mode > 2); } else { ... } ... return pio_mode; }