[4085] in linux-scsi channel archive

home help back first fref pref prev next nref lref last post

Re: piss poor change in ncr53c8xx/linux-2.1.104r

daemon@ATHENA.MIT.EDU (Richard Waltham)
Mon Jun 8 22:16:32 1998

From: Richard Waltham <dormouse@farsrobt.demon.co.uk>
To: pavel@Elf.ucw.cz (Pavel Machek)
Date: 	Tue, 9 Jun 1998 01:38:58 +0100 (BST)
Cc: linux-kernel@vger.rutgers.edu, linux-scsi@vger.rutgers.edu
In-Reply-To: <19980607211649.18528@Elf.ucw.cz> from "Pavel Machek" at Jun 7, 98 09:16:49 pm

Pavel Machek wrote:
> 
> Hi!
> 
> > FOR YOUR INFORMATION, THIS DRIVER IS STILL MAINTAINED.
> 
> No need to be rude, that person did not have any bad intention. (I was
> not that person, FYI).
> 
> > FYI, this change will not be incorporated in my personal driver version, 
> > which is currently 2.6n and will become 3.0 soon with a patch against 
> > 2.0.34 followed by the announce that I stop maintaining ncr53c8xx 2.5
> > driver series.
> 
> Ook, what about following patch? (I consider having DELAY() function
> when there are generic mdelay() and udelay() functions available bad
> thing.)
> 
> 								Pavel
> 

mdelay certainly doesn't appear in my copy of 2.0.34.

Adding gratuitous changes to well maintained code is doing a great
disservice not only to the maintainers but to everyone else trying to keep
in sync with them.

It might look pretty but its no good if it doesn't work.

Bye for now
Richard

-- 
 Richard Waltham   |               Work: richard@digtalinterface.demon.co.uk
   At home in      |               Home:       dormouse@farsrobt.demon.co.uk
 Southampton UK    |                              100421.1276@compuserve.com

-
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@vger.rutgers.edu

home help back first fref pref prev next nref lref last post