PATCH: minor IDE clean up I noticed
From: Alan Cox (alan_at_redhat.com)
Date: 09/30/04
- Previous message: Alan Cox: "PATCH: resend: VIA Velocity belongs under gigabit ethernet"
- Next in thread: Bartlomiej Zolnierkiewicz: "Re: PATCH: minor IDE clean up I noticed"
- Reply: Bartlomiej Zolnierkiewicz: "Re: PATCH: minor IDE clean up I noticed"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Date: Thu, 30 Sep 2004 13:54:30 -0400 To: linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org
Nowdays a drive always has a driver
diff -u --new-file --recursive --exclude-from /usr/src/exclude linux.vanilla-2.6.9rc3/drivers/ide/ide-proc.c linux-2.6.9rc3/drivers/ide/ide-proc.c
--- linux.vanilla-2.6.9rc3/drivers/ide/ide-proc.c 2004-09-30 15:35:48.169975152 +0100
+++ linux-2.6.9rc3/drivers/ide/ide-proc.c 2004-09-30 16:33:46.423200288 +0100
@@ -365,20 +365,7 @@
{
unsigned short *val = (unsigned short *) page;
- /*
- * The current code can't handle a driverless
- * identify query taskfile. Now the right fix is
- * to add a 'default' driver but that is a bit
- * more work.
- *
- * FIXME: this has to be fixed for hotswap devices
- */
-
- if(DRIVER(drive))
- err = taskfile_lib_get_identify(drive, page);
- else /* This relies on the ID changes */
- val = (unsigned short *)drive->id;
-
+ err = taskfile_lib_get_identify(drive, page);
if(!err)
{
char *out = ((char *)page) + (SECTOR_WORDS * 4);
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
- Previous message: Alan Cox: "PATCH: resend: VIA Velocity belongs under gigabit ethernet"
- Next in thread: Bartlomiej Zolnierkiewicz: "Re: PATCH: minor IDE clean up I noticed"
- Reply: Bartlomiej Zolnierkiewicz: "Re: PATCH: minor IDE clean up I noticed"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Relevant Pages
- Re: My thoughts on the "new development model"
... |>>> 2.6 tree is great for gentoo users who like gcc consuming all CPU ...
|>> that after a month or so of fixes etc it will be a very stable kernel ... driver
for 2.6.7 for an ADSL card; a development driver for 2.6.5 for a ... send the line "unsubscribe
linux-kernel" in ... (Linux-Kernel) - RE: Error mounting root fs on 72:01 using Promise FastTrak TX2000 (PDC20271)
... The Promise ataraid driver never gets loaded. ... send the line "unsubscribe
linux-kernel" in ... Please read the FAQ at http://www.tux.org/lkml/ ... More
majordomo info at http://vger.kernel.org/majordomo-info.html ... (Linux-Kernel) - (no subject)
... > driver layer is needed to handle these operations cleanly. ... Device
addition and removal work just fine with that scheme. ... send the line "unsubscribe linux-kernel"
in ... Please read the FAQ at http://www.tux.org/lkml/ ... (Linux-Kernel) - Re: SL811 problem on mach-pxa
... It was tested with _both_ workarounds for IRQ issues; ... unlike the predecessors
to this driver). ... I've had reports that one of the ... send the line "unsubscribe
linux-kernel" in ... (Linux-Kernel) - RE: 3ware driver (3w-xxxx) in 2.6.10: procfs entry
... Did you try "Download 'In Engineering Phase' Software" link? ... Subject: 3ware
driver in 2.6.10: procfs entry ... send the line "unsubscribe linux-kernel" in ...
Please read the FAQ at http://www.tux.org/lkml/ ... (Linux-Kernel)