Re: PATCH: minor IDE clean up I noticed

From: Bartlomiej Zolnierkiewicz (bzolnier_at_elka.pw.edu.pl)
Date: 09/30/04

  • Next message: Bartlomiej Zolnierkiewicz: "Re: PATCH: (Test) it8212 driver for 2.6.9rc3"
    To: Alan Cox <alan@redhat.com>
    Date:	Thu, 30 Sep 2004 20:05:36 +0200
    
    

    fixed 4 days ago in ide-dev-2.6 :)

    On Thursday 30 September 2004 19:54, Alan Cox wrote:
    > 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/


  • Next message: Bartlomiej Zolnierkiewicz: "Re: PATCH: (Test) it8212 driver for 2.6.9rc3"

    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: sata related hang with linux-2.6
      ... IMHO there's something not quite right with the Silicon Image libata ... Perhaps the driver is enabling the hardware to generate interrupts ... before setting up the interrupt routine for it? ... send the line "unsubscribe linux-kernel" in ...
      (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)