OK, so I'll give you a B for effort on this one, but: On Wed, 2012-12-12 at 02:26 +0530, sreekanth.reddy@xxxxxxx wrote: > The variable 'chain_flags' and 'phy_number' are initialized but never > used otherwise, > So remove those unused variables. There has to be a From: Wei Yongjun <yongjun_wei@xxxxxxxxxxxxxxxxx> At the beginning of this patch so git gets the authorship correct > > Signed-off-by: Wei Yongjun <yongjun_wei@xxxxxxxxxxxxxxxxx> > Acked-by: Sreekanth Reddy <Sreekanth.Reddy@xxxxxxx> And this should be Signed-off-by: you not acked-by, because you sent the patch to me (signoffs follow the trail of transmission). I've fixed all of this up, for this one time. 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