You could remove the __bfa_trc32() routine Thanks, Acked-by: Anil Gurumurthy <anil.gurumur...@qlogic.com>
-----Original Message----- From: Martin K. Petersen [mailto:martin.peter...@oracle.com] Sent: 10 February 2016 22:59 To: Denys Vlasenko <dvlas...@redhat.com> Cc: James Bottomley <jbottom...@parallels.com>; Fabian Frederick <f...@skynet.be>; Anil Gurumurthy <anil.gurumur...@qlogic.com>; Christoph Hellwig <h...@lst.de>; Guenter Roeck <li...@roeck-us.net>; Ben Hutchings <b...@decadent.org.uk>; linux-kernel <linux-ker...@vger.kernel.org>; linux-scsi <linux-scsi@vger.kernel.org> Subject: Re: [PATCH RESEND] bfa: deinline __bfa_trc() and __bfa_trc32() >>>>> "Denys" == Denys Vlasenko <dvlas...@redhat.com> writes: Denys> __bfa_trc32() is very similar, so it is uninlined too. However, Denys> it appears to be unused, therefore this patch ifdefs it out. Why don't we just drop it? -- Martin K. Petersen Oracle Linux Engineering -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html