2003-03-03 16:07:35

by Cameron, Steve

[permalink] [raw]
Subject: RE: [PATCH] cciss: add passthrough ioctl




> On Mon, Mar 03, 2003 at 09:26:40AM +0600, Stephen Cameron wrote:
> > Because, in order to flash the array controller firmware,
> > it's got to be done that way...
>
> I don't buy this. Sorry. What's against creating a device for this
> controller itself ?
> (And yes, the kernel could use a formal ioctl number for "upgrade firmware")

Arg. Now I wish I didn't already port that code to 10 distributions.
Please shoot me.

How do you want it done?

-- steve


2003-03-03 16:34:42

by Alan

[permalink] [raw]
Subject: RE: [PATCH] cciss: add passthrough ioctl

On Mon, 2003-03-03 at 16:17, Cameron, Steve wrote:
>
> > On Mon, Mar 03, 2003 at 09:26:40AM +0600, Stephen Cameron wrote:
> > > Because, in order to flash the array controller firmware,
> > > it's got to be done that way...
> >
> > I don't buy this. Sorry. What's against creating a device for this
> > controller itself ?
> > (And yes, the kernel could use a formal ioctl number for "upgrade firmware")
>
> Arg. Now I wish I didn't already port that code to 10 distributions.
> Please shoot me.
>
> How do you want it done?

I really think it should stay as it is for now. Maybe in 2.7 we need to have
the firmware update argument, but we need to have it about *all* related
drivers and do it once and right.