On Mon, Jan 17, 2005 at 03:07:48PM +0100, Cajus Pollmeier wrote:
> Sorry for that, but there's no time on my side to work on a complete old/new 
> merger - and I think it doesn't make any sense this way either.
> 
> I've looked at your code and there are two little things that do not work for 
> some drivers but are easily added without any overhead:
> 
> SCSI-drivers that don't provide informations in /proc/scsi/DRIVER ...
> ... are not detected because the driver name is derived from the
>     directory name
> ... the SCSI_HOSTNUMBER cannot be retrieved
> 
> After fixing the issues from above, I added support for the "ipr" driver
> in the scsi driver case/esac clause.
> 
> Maybe the SCSI_CHANNEL could be defined outside the sysfs based
> if/fi clause. There are no dependencies to external binaries.

this patch small ipr patch looks ok at a glance.  I have no way to
test it though.

-- 
Ethan Benson
http://www.alaska.net/~erbenson/

Attachment: pgpf7deaxuk4I.pgp
Description: PGP signature

Reply via email to