lkml.org 
[lkml]   [2004]   [Oct]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [BK PATCHES] ide-2.6 update
On Fri, 22 Oct 2004 19:07:15 +0100, Russell King
<rmk+lkml@arm.linux.org.uk> wrote:
> On Fri, Oct 22, 2004 at 07:38:44PM +0200, Bartlomiej Zolnierkiewicz wrote:
> > @@ -498,14 +483,6 @@
> > ICS_ARCIN_V6_INTRSTAT_1)) & 1;
> > }
> >
> > -static int icside_dma_verbose(ide_drive_t *drive)
> > -{
> > - printk(", %s (peak %dMB/s)",
> > - ide_xfer_verbose(drive->current_speed),
> > - 2000 / drive->drive_data);
> > - return 1;
> > -}
> > -
> > static int icside_dma_timeout(ide_drive_t *drive)
> > {
> > printk(KERN_ERR "%s: DMA timeout occurred: ", drive->name);
> > @@ -554,7 +531,6 @@
> > hwif->dma_start = icside_dma_start;
> > hwif->ide_dma_end = icside_dma_end;
> > hwif->ide_dma_test_irq = icside_dma_test_irq;
> > - hwif->ide_dma_verbose = icside_dma_verbose;
> > hwif->ide_dma_timeout = icside_dma_timeout;
> > hwif->ide_dma_lostirq = icside_dma_lostirq;
>
> Why are you denying me the ability to report detailed drive transfer
> speed information? Would you prefer me to printk a separate line
> and further clutter up the message log instead?

From the changelog:
> * icside.c version repeated info given by ->ide_dma_check()

If DMA is going to be used:
* ->ide_dma_check is always called
* icside_dma_check() always calls icside_set_speed()
* icside_set_speed() always does

printk("%s: %s selected (peak %dMB/s)\n", drive->name,
ide_xfer_verbose(xfer_mode), 2000 / drive->drive_data);

so what am I missing?

Bartlomiej
-
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/

\
 
 \ /
  Last update: 2005-03-22 14:07    [W:0.024 / U:0.432 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site