Re: [PATCH 1/2] Revert "mfd: cros_ec: Remove unused __remove function"

[Lee Jones <lee.jones@linaro.org>]

On Wed, 20 Jun 2018, Dmitry Torokhov wrote:

> This reverts commit 556c242045f0c1613aac2e64dc5b2ff0e4bc89e1.
> 
> The patch that this change is purported to fix is broken and should be
> reverted; thus we reverting this one as well.

You need to provide more information.

How does the original patch break the build/code execution?
What is this patch doing to rectify the issue?
Any other information you think is relevant.

> ---
> 
> Lee,
> 
> I see the broken code is already in Linus' tree, so please merge send
> it on at your earliest convenience.
> 
> Thanks,
> Dmitry
> 
> 
>  drivers/mfd/cros_ec_dev.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/mfd/cros_ec_dev.c b/drivers/mfd/cros_ec_dev.c
> index 306e1fd109bd..4b1dbe81fcb6 100644
> --- a/drivers/mfd/cros_ec_dev.c
> +++ b/drivers/mfd/cros_ec_dev.c
> @@ -262,6 +262,8 @@ static const struct file_operations fops = {
>  #endif
>  };
>  
> +static void __remove(struct device *dev) { }
> +
>  static void cros_ec_sensors_register(struct cros_ec_dev *ec)
>  {
>  	/*

-- 
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog