On Fri, Mar 25, 2005 at 03:39:52PM -0800, Greg KH wrote: > But can you take a look at drivers/scsi/scsi_transport_spi.c, line 265? > That is also going to need fixing up somehow. Gotta love that FIXME > comment...
Ok, the patch below seems to fix it, but I would like some validation I did the correct thing. Oh, looks like pci express now has problems too, I'll go hit that one next... thanks, greg k-h ------------- [scsi] use device_for_each_child() to properly access child devices. Signed-off-by: Greg Kroah-Hartman <[EMAIL PROTECTED]> diff -Nru a/drivers/scsi/scsi_transport_spi.c b/drivers/scsi/scsi_transport_spi.c --- a/drivers/scsi/scsi_transport_spi.c 2005-03-25 16:03:09 -08:00 +++ b/drivers/scsi/scsi_transport_spi.c 2005-03-25 16:03:09 -08:00 @@ -254,17 +254,21 @@ spi_transport_rd_attr(rti, "%d\n"); spi_transport_rd_attr(pcomp_en, "%d\n"); +/* we only care about the first child device so we return 1 */ +static int child_iter(struct device *dev, void *data) +{ + struct scsi_device *sdev = to_scsi_device(dev); + + spi_dv_device(sdev); + return 1; +} + static ssize_t store_spi_revalidate(struct class_device *cdev, const char *buf, size_t count) { struct scsi_target *starget = transport_class_to_starget(cdev); - /* FIXME: we're relying on an awful lot of device internals - * here. We really need a function to get the first available - * child */ - struct device *dev = container_of(starget->dev.children.next, struct device, node); - struct scsi_device *sdev = to_scsi_device(dev); - spi_dv_device(sdev); + device_for_each_child(&starget->dev, NULL, child_iter); return count; } static CLASS_DEVICE_ATTR(revalidate, S_IWUSR, NULL, store_spi_revalidate); - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/