On Fri, 2009-05-22 at 16:34 +0530, Kashyap, Desai wrote: > Fuctions which are not used are removed by this patch. > --- > > Signed-off-by: Kashyap Desai <kashyap.desai@xxxxxxx> > --- > diff --git a/drivers/message/fusion/mptsas.c b/drivers/message/fusion/mptsas.c > index 10abd62..29a8fe2 100644 > --- a/drivers/message/fusion/mptsas.c > +++ b/drivers/message/fusion/mptsas.c > @@ -3035,30 +3035,6 @@ mptsas_find_phyinfo_by_sas_address(MPT_ADAPTER *ioc, u64 sas_address) > return phy_info; > } > > -static struct mptsas_phyinfo * > -mptsas_find_phyinfo_by_target(MPT_ADAPTER *ioc, u8 channel, u8 id) Please combine this with the patch that actually removes the last use otherwise, for all static functions, gcc is going to warn about defined but unused functions. The 10 lines you save by separating this aren't really significant given that some of the other patches are up to a thousand lines. 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