forked from mirrors/linux
		
	net: ipa: kill ipa_modem_init()
A recent commit made disabling the SMP2P "setup ready" interrupt unrelated to ipa_modem_stop(). Given that, it seems fitting to get rid of ipa_modem_init() and ipa_modem_exit() (which are trivial wrapper functions), and call ipa_smp2p_init() and ipa_smp2p_exit() directly instead. Signed-off-by: Alex Elder <elder@linaro.org> Signed-off-by: Jakub Kicinski <kuba@kernel.org>
This commit is contained in:
		
							parent
							
								
									8abe197038
								
							
						
					
					
						commit
						76b5fbcd6b
					
				
					 3 changed files with 4 additions and 16 deletions
				
			
		|  | @ -29,6 +29,7 @@ | ||||||
| #include "ipa_mem.h" | #include "ipa_mem.h" | ||||||
| #include "ipa_table.h" | #include "ipa_table.h" | ||||||
| #include "ipa_modem.h" | #include "ipa_modem.h" | ||||||
|  | #include "ipa_smp2p.h" | ||||||
| #include "ipa_uc.h" | #include "ipa_uc.h" | ||||||
| #include "ipa_interrupt.h" | #include "ipa_interrupt.h" | ||||||
| #include "gsi_trans.h" | #include "gsi_trans.h" | ||||||
|  | @ -733,7 +734,7 @@ static int ipa_probe(struct platform_device *pdev) | ||||||
| 	if (ret) | 	if (ret) | ||||||
| 		goto err_endpoint_exit; | 		goto err_endpoint_exit; | ||||||
| 
 | 
 | ||||||
| 	ret = ipa_modem_init(ipa, modem_init); | 	ret = ipa_smp2p_init(ipa, modem_init); | ||||||
| 	if (ret) | 	if (ret) | ||||||
| 		goto err_table_exit; | 		goto err_table_exit; | ||||||
| 
 | 
 | ||||||
|  | @ -775,7 +776,7 @@ static int ipa_probe(struct platform_device *pdev) | ||||||
| 	ipa_deconfig(ipa); | 	ipa_deconfig(ipa); | ||||||
| err_power_put: | err_power_put: | ||||||
| 	pm_runtime_put_noidle(dev); | 	pm_runtime_put_noidle(dev); | ||||||
| 	ipa_modem_exit(ipa); | 	ipa_smp2p_exit(ipa); | ||||||
| err_table_exit: | err_table_exit: | ||||||
| 	ipa_table_exit(ipa); | 	ipa_table_exit(ipa); | ||||||
| err_endpoint_exit: | err_endpoint_exit: | ||||||
|  | @ -821,7 +822,7 @@ static int ipa_remove(struct platform_device *pdev) | ||||||
| 	ipa_deconfig(ipa); | 	ipa_deconfig(ipa); | ||||||
| out_power_put: | out_power_put: | ||||||
| 	pm_runtime_put_noidle(dev); | 	pm_runtime_put_noidle(dev); | ||||||
| 	ipa_modem_exit(ipa); | 	ipa_smp2p_exit(ipa); | ||||||
| 	ipa_table_exit(ipa); | 	ipa_table_exit(ipa); | ||||||
| 	ipa_endpoint_exit(ipa); | 	ipa_endpoint_exit(ipa); | ||||||
| 	gsi_exit(&ipa->gsi); | 	gsi_exit(&ipa->gsi); | ||||||
|  |  | ||||||
|  | @ -442,16 +442,6 @@ static int ipa_modem_notify(struct notifier_block *nb, unsigned long action, | ||||||
| 	return NOTIFY_OK; | 	return NOTIFY_OK; | ||||||
| } | } | ||||||
| 
 | 
 | ||||||
| int ipa_modem_init(struct ipa *ipa, bool modem_init) |  | ||||||
| { |  | ||||||
| 	return ipa_smp2p_init(ipa, modem_init); |  | ||||||
| } |  | ||||||
| 
 |  | ||||||
| void ipa_modem_exit(struct ipa *ipa) |  | ||||||
| { |  | ||||||
| 	ipa_smp2p_exit(ipa); |  | ||||||
| } |  | ||||||
| 
 |  | ||||||
| int ipa_modem_config(struct ipa *ipa) | int ipa_modem_config(struct ipa *ipa) | ||||||
| { | { | ||||||
| 	void *notifier; | 	void *notifier; | ||||||
|  |  | ||||||
|  | @ -18,9 +18,6 @@ void ipa_modem_skb_rx(struct net_device *netdev, struct sk_buff *skb); | ||||||
| void ipa_modem_suspend(struct net_device *netdev); | void ipa_modem_suspend(struct net_device *netdev); | ||||||
| void ipa_modem_resume(struct net_device *netdev); | void ipa_modem_resume(struct net_device *netdev); | ||||||
| 
 | 
 | ||||||
| int ipa_modem_init(struct ipa *ipa, bool modem_init); |  | ||||||
| void ipa_modem_exit(struct ipa *ipa); |  | ||||||
| 
 |  | ||||||
| int ipa_modem_config(struct ipa *ipa); | int ipa_modem_config(struct ipa *ipa); | ||||||
| void ipa_modem_deconfig(struct ipa *ipa); | void ipa_modem_deconfig(struct ipa *ipa); | ||||||
| 
 | 
 | ||||||
|  |  | ||||||
		Loading…
	
		Reference in a new issue
	
	 Alex Elder
						Alex Elder