Re: [PATCH] input synaptics-rmi4: Delete some obsolete code

[Dmitry Torokhov <dmitry.torokhov@gmail.com>]

On Fri, Dec 27, 2013 at 05:58:07PM -0800, Christopher Heiny wrote:
> The answer to the question "is this crap needed with F01 always present?" is
> "no, it's not".  Also delete obsolete [suspend|resume]_one_device.
> 
> Signed-off-by: Christopher Heiny <cheiny@synaptics.com>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Benjamin Tissoires <benjamin.tissoires@redhat.com>

Applied, thank you.

> 
> ---
> 
>  drivers/input/rmi4/rmi_driver.c | 48 -----------------------------------------
>  1 file changed, 48 deletions(-)
> 
> diff --git a/drivers/input/rmi4/rmi_driver.c b/drivers/input/rmi4/rmi_driver.c
> index 2ae9af9..a4e5236 100644
> --- a/drivers/input/rmi4/rmi_driver.c
> +++ b/drivers/input/rmi4/rmi_driver.c
> @@ -686,39 +686,6 @@ error_exit:
>  	return retval;
>  }
>  
> -#if 0
> -// XXX is this crap needed with F01 always present?
> -static int f01_notifier_call(struct notifier_block *nb,
> -				unsigned long action, void *data)
> -{
> -	struct device *dev = data;
> -	struct rmi_function *fn;
> -
> -	if (!rmi_is_function_device(dev))
> -		return 0;
> -
> -	fn = to_rmi_function(dev);
> -	if (fn->fd.function_number != 0x01)
> -		return 0;
> -
> -	switch (action) {
> -	case BUS_NOTIFY_BOUND_DRIVER:
> -		dev_dbg(dev, "%s: F01 driver bound.\n", __func__);
> -		enable_sensor(fn->rmi_dev);
> -		break;
> -	case BUS_NOTIFY_UNBIND_DRIVER:
> -		dev_dbg(dev, "%s: F01 driver going away.\n", __func__);
> -		disable_sensor(fn->rmi_dev);
> -		break;
> -	}
> -	return 0;
> -}
> -
> -static struct notifier_block rmi_bus_notifier = {
> -	.notifier_call = f01_notifier_call,
> -};
> -#endif
> -
>  #ifdef CONFIG_PM_SLEEP
>  static int rmi_driver_suspend(struct device *dev)
>  {
> @@ -738,13 +705,6 @@ static int rmi_driver_suspend(struct device *dev)
>  
>  	disable_sensor(rmi_dev);
>  
> -#if 0
> -	/** Do it backwards so F01 comes last. */
> -	list_for_each_entry_reverse(entry, &data->function_list, node)
> -		if (suspend_one_device(entry) < 0)
> -			goto exit;
> -#endif
> -
>  	if (data->post_suspend)
>  		retval = data->post_suspend(data->pm_data);
>  
> @@ -768,14 +728,6 @@ static int rmi_driver_resume(struct device *dev)
>  			goto exit;
>  	}
>  
> -#if 0
> -	/** Do it forwards, so F01 comes first. */
> -	list_for_each_entry(entry, &data->function_list, node) {
> -		if (resume_one_device(entry) < 0)
> -			goto exit;
> -	}
> -#endif
> -
>  	retval = enable_sensor(rmi_dev);
>  	if (retval)
>  		goto exit;

-- 
Dmitry