Re: cciss CSMI via sysfs for 2.6

From: Greg KH
Date: Fri Feb 18 2005 - 16:34:56 EST


On Fri, Feb 18, 2005 at 07:46:28PM +0000, Christoph Hellwig wrote:
> > /*
> > + * sysfs stuff
> > + * this should be moved to it's own file, maybe cciss_sysfs.h
> > + */
> > +
> > +static ssize_t cciss_firmver_show(struct device *dev, char *buf)
> > +{
> > + ctlr_info_t *h = dev->driver_data;
> > + return sprintf(buf,"%c%c%c%c\n", h->firm_ver[0], h->firm_ver[1],
> > + h->firm_ver[2], h->firm_ver[3]);
> > +}
>
> I really wish we had a common firmver release attribut in the driver
> core, as mentioned in the fc transport class thread. Greg?

For a device? It seems a huge overkill to add this attribute for
_every_ device in the system, when only a small minority can actually
use it. Just put it as a default scsi or transport class attribute
instead.

thanks,

greg k-h
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/