platform/x86: silicom-platform: clean up a check
authorDan Carpenter <dan.carpenter@linaro.org>
Fri, 12 Jan 2024 06:56:25 +0000 (09:56 +0300)
committerIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
Wed, 24 Jan 2024 10:34:45 +0000 (12:34 +0200)
The error message in this code can't be reached because value is either
zero or non-zero.  There isn't a third option.  Really, it's nicer to
write this as a one liner.

Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Link: https://lore.kernel.org/r/a58bffb7-0a8b-4195-b273-f65a188ace7b@moroto.mountain
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
drivers/platform/x86/silicom-platform.c

index 6ce43ccb3112eb542256ac697108af261acf8b93..c0910af16a3ac2719cd330eb6617232411e4f7ed 100644 (file)
@@ -256,12 +256,7 @@ static void silicom_gpio_set(struct gpio_chip *gc,
        if (direction == GPIO_LINE_DIRECTION_IN)
                return;
 
-       if (value)
-               silicom_mec_port_set(channel, 0);
-       else if (value == 0)
-               silicom_mec_port_set(channel, 1);
-       else
-               pr_err("Wrong argument value: %d\n", value);
+       silicom_mec_port_set(channel, !value);
 }
 
 static int silicom_gpio_direction_output(struct gpio_chip *gc,