Re: [PATCH 2/2] megaraid_sas: clean function declarations in megaraid_sas_fusion.c up

From: Arnd Bergmann
Date: Fri Sep 30 2016 - 10:35:14 EST


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