On Friday 30 September 2016, Baoyou Xie wrote: > int megasas_transition_to_ready(struct megasas_instance *instance, int ocr); > -void megaraid_sas_kill_hba(struct megasas_instance *instance); > > extern u32 megasas_dbg_lvl; > -void megasas_sriov_heartbeat_handler(unsigned long instance_addr); > -int megasas_sriov_start_heartbeat(struct megasas_instance *instance, > - int initial); > -void megasas_start_timer(struct megasas_instance *instance, > - struct timer_list *timer, > - void *fn, unsigned long interval); > extern struct megasas_mgmt_info megasas_mgmt_info; > extern unsigned int resetwaittime; > extern unsigned int dual_qdepth_disable; I think you should remove all "extern" declarations from the .c file at the same time here, and move them to an appropriate header, or remove them in case the symbol is already gone and we only have the declaration left. Arnd -- 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