lkml.org 
[lkml]   [2003]   [May]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [patch] Support for SiS 961/961B/962/963/630S/630ET/633/733 IDE

Hi,

Good job, but...

On Fri, 16 May 2003, Vojtech Pavlik wrote:
> And while doing the changes I did also some cleanups, mainly removing a
> bunch of debug code that doesn't seem very useful when lspci does the
> same job. And removing the config_drive_xfer_rate in favor of functions
> from ide-timing.h.

Debug code in fe. SiS IDE driver does the same as lspci given you get to
the point you can run lspci. Dumping of PCI conf regs should be moved to
generic IDE PCI code as it might be useful for other IDE PCI drivers.

Removing config_drive_xfer_rate() is bad,
fe. you don't check for bad drives now.

Regards,
--
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 13:35    [W:0.031 / U:0.540 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site