Commit 0e7d8d48 authored by Jan Evert van Grootheest's avatar Jan Evert van Grootheest Committed by Bartlomiej Zolnierkiewicz

ht6560b: force prefetch for some devices

Prefetch needs to be set for some ide devices to work when connected to
a ht6560b interface. This was not always done properly, causing a system
with a HD and CD on the primary interface to not work properly. Or, in
effect, hang hard.

This patch forces prefetch on devices before checking whether it
is necessary to change the settings in the interface

This patch should also be applied to 2.4. I don't currently have a
2.4 tree around.

(also change my email address)
Signed-off-by: default avatarJan Evert van Grootheest <janevert@caiway.nl>
Cc: Sergei Shtylyov <sshtylyov@ru.mvista.com>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>
Signed-off-by: default avatarBartlomiej Zolnierkiewicz <bzolnier@gmail.com>
parent 1a1990f5
...@@ -21,18 +21,21 @@ ...@@ -21,18 +21,21 @@
* "Prefetch" mode bit OFF for ide disks and * "Prefetch" mode bit OFF for ide disks and
* ON for anything else. * ON for anything else.
* *
* Version 0.08 Need to force prefetch for CDs and other non-disk
* devices. (not sure which devices exactly need
* prefetch)
* *
* HT-6560B EIDE-controller support * HT-6560B EIDE-controller support
* To activate controller support use kernel parameter "ide0=ht6560b". * To activate controller support use kernel parameter "ide0=ht6560b".
* Use hdparm utility to enable PIO mode support. * Use hdparm utility to enable PIO mode support.
* *
* Author: Mikko Ala-Fossi <maf@iki.fi> * Author: Mikko Ala-Fossi <maf@iki.fi>
* Jan Evert van Grootheest <janevert@iae.nl> * Jan Evert van Grootheest <janevert@caiway.nl>
* *
* Try: http://www.maf.iki.fi/~maf/ht6560b/ * Try: http://www.maf.iki.fi/~maf/ht6560b/
*/ */
#define HT6560B_VERSION "v0.07" #define HT6560B_VERSION "v0.08"
#include <linux/module.h> #include <linux/module.h>
#include <linux/types.h> #include <linux/types.h>
...@@ -134,11 +137,16 @@ static void ht6560b_selectproc (ide_drive_t *drive) ...@@ -134,11 +137,16 @@ static void ht6560b_selectproc (ide_drive_t *drive)
select = HT_CONFIG(drive); select = HT_CONFIG(drive);
timing = HT_TIMING(drive); timing = HT_TIMING(drive);
/*
* Need to enforce prefetch sometimes because otherwise
* it'll hang (hard).
*/
if (drive->media != ide_disk || !drive->present)
select |= HT_PREFETCH_MODE;
if (select != current_select || timing != current_timing) { if (select != current_select || timing != current_timing) {
current_select = select; current_select = select;
current_timing = timing; current_timing = timing;
if (drive->media != ide_disk || !drive->present)
select |= HT_PREFETCH_MODE;
(void)inb(HT_CONFIG_PORT); (void)inb(HT_CONFIG_PORT);
(void)inb(HT_CONFIG_PORT); (void)inb(HT_CONFIG_PORT);
(void)inb(HT_CONFIG_PORT); (void)inb(HT_CONFIG_PORT);
...@@ -188,11 +196,12 @@ static int __init try_to_init_ht6560b(void) ...@@ -188,11 +196,12 @@ static int __init try_to_init_ht6560b(void)
outb(HT_TIMING_DEFAULT, 0x1f6); /* IDE_SELECT_REG */ outb(HT_TIMING_DEFAULT, 0x1f6); /* IDE_SELECT_REG */
(void) inb(0x1f7); /* IDE_STATUS_REG */ (void) inb(0x1f7); /* IDE_STATUS_REG */
printk("\nht6560b " HT6560B_VERSION printk("ht6560b " HT6560B_VERSION
": chipset detected and initialized" ": chipset detected and initialized"
#ifdef DEBUG #ifdef DEBUG
" with debug enabled" " with debug enabled"
#endif #endif
"\n"
); );
return 1; return 1;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment