On Sun, 3 Feb 2008, James Bottomley wrote: > 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. Thanks. I think that I had a pine config error that caused the problem. Should be fixed now. -- ~Randy - 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
