Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=4660897e6c2daa198fc8e3f47ae2a4aef69c80b0
Commit:     4660897e6c2daa198fc8e3f47ae2a4aef69c80b0
Parent:     4c6c914e4c2e0f91775ce4051b5a800c55175462
Author:     Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]>
AuthorDate: Sun Jul 8 15:21:58 2007 +0200
Committer:  Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]>
CommitDate: Sun Jul 8 15:21:58 2007 +0200

    qd65xx: fix PIO mode selection
    
    PIO4 is a maximum PIO mode supported by a driver.  Using "255" as a max_mode
    argument to ide_get_best_pio_mode() could result in wrong timings being used
    by a driver (for "pio" equal to 5) or OOPS (for "pio" values > 5 && < 255).
    
    Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]>
    Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]>
    Reviewed-by: Alan Cox <[EMAIL PROTECTED]>
---
 drivers/ide/legacy/qd65xx.c |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)

diff --git a/drivers/ide/legacy/qd65xx.c b/drivers/ide/legacy/qd65xx.c
index d1414a7..7783745 100644
--- a/drivers/ide/legacy/qd65xx.c
+++ b/drivers/ide/legacy/qd65xx.c
@@ -258,8 +258,7 @@ static void qd6580_tune_drive (ide_drive_t *drive, u8 pio)
        int recovery_time = 415; /* worst case values from the dos driver */
 
        if (drive->id && !qd_find_disk_type(drive, &active_time, 
&recovery_time)) {
-               pio = ide_get_best_pio_mode(drive, pio, 255, &d);
-               pio = min_t(u8, pio, 4);
+               pio = ide_get_best_pio_mode(drive, pio, 4, &d);
 
                switch (pio) {
                        case 0: break;
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to