On Wed, Nov 23, 2022 at 09:43, Dan Carpenter <error27@xxxxxxxxx> wrote:
> We shuffled the error handling around so this condition is dead code
> now. The "error" variable is always zero.
>
> Signed-off-by: Dan Carpenter <error27@xxxxxxxxx>
Reviewed-by: Mattijs Korpershoek <mkorpershoek@xxxxxxxxxxxx>
> ---
> No Fixes tag because this does not fix a bug but if we were to assign
> a tag it would be:
>
> Fixes: a85fbd649844 ("Input: elants_i2c - properly handle the reset GPIO when power is off")
>
> drivers/input/touchscreen/elants_i2c.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/input/touchscreen/elants_i2c.c b/drivers/input/touchscreen/elants_i2c.c
> index e1308e179dd6..af1dc43e988c 100644
> --- a/drivers/input/touchscreen/elants_i2c.c
> +++ b/drivers/input/touchscreen/elants_i2c.c
> @@ -1353,8 +1353,6 @@ static int elants_i2c_power_on(struct elants_data *ts)
> udelay(ELAN_POWERON_DELAY_USEC);
>
> gpiod_set_value_cansleep(ts->reset_gpio, 0);
> - if (error)
> - return error;
>
> msleep(ELAN_RESET_DELAY_MSEC);
>
> --
> 2.35.1
