Re: PATCH 2.5.2-pre9 scsi cleanup

Martin Dalecki (dalecki@evision-ventures.com)
Tue, 08 Jan 2002 19:43:08 +0100


Cameron, Steve wrote:

>Martin Dalecki [mailto:dalecki@evision-ventures.com] wrote,
>regarding removal of scsi_device_types[] from drivers/scsi/scsi.c
>
>>Cameron, Steve wrote:
>>
>[...]
>
>>>Hmmm, I was using that.... (In, for example,
>>>the cciss patch here: http://www.geocities.com/smcameron
>>>It's not any big deal, though.)
>>>
>>Precisely this "not any big deal" is the point: It was the wrong
>>approach to a trivial problem ;-).
>>
>
>So what's the right approach? I can invent my own easily enough,
>but each driver doing its own thing doesn't seem right. I assumed
>that it was in scsi.c foi common usage, so each driver that wanted
>to say, use these device type strings in diagnostic messages or
>some such wouldn't have to reinvent this wheel, and so all the
>drivers would consistently use the same names. Will it be
>replaced with something else?
>
>Just want to know so I don't waste (even more :-) time
>doing something dumb.
>

Please just case in the ->type enum. And if you wan't to provide special
messages, well
then please do it yourself, the removal showed, that nearly no one
driver used the generic
stuff, so it wasn't trully generic at all. (It should be handled by some
userlevel stuff anyway...)

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