On Sun, 2008-02-03 at 15:06 -0800, Randy.Dunlap wrote: > From: Randy Dunlap <randy.dunlap@xxxxxxxxxx> > > Add missing function parameter descriptions. > Make function short description fit on one line as required. > > Signed-off-by: Randy Dunlap <randy.dunlap@xxxxxxxxxx> > --- > drivers/scsi/scsi.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > --- linux-2.6.24-git12.orig/drivers/scsi/scsi.c > +++ linux-2.6.24-git12/drivers/scsi/scsi.c > @@ -969,9 +969,10 @@ void starget_for_each_device(struct scsi > EXPORT_SYMBOL(starget_for_each_device); > > /** > - * __starget_for_each_device - helper to walk all devices of a target > - * (UNLOCKED) > + * __starget_for_each_device - helper to walk all devices of a target (UNLOCKED) > * @starget: target whose devices we want to iterate over. > + * @data: parameter for callback @fn() > + * @fn: callback function that is invoked for each device > * > * This traverses over each device of @starget. It does _not_ > * take a reference on the scsi_device, so the whole loop must be ^^^^^ I have this, but it's a bit mysterious: something added an extra space to the non +/- lines making the patch reject. I pulled it out by hand and it applied fine. James - To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html